Skip to content
This repository has been archived by the owner on Oct 2, 2023. It is now read-only.

Ecu rework on c8 #24

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Ecu rework on c8 #24

wants to merge 16 commits into from

Conversation

francisfu-embedded
Copy link

ECU firmware v2

@a2k-hanlon a2k-hanlon force-pushed the ecu_rework_on_c8 branch 2 times, most recently from b361c79 to cd1a63e Compare June 21, 2022 08:32
@a2k-hanlon
Copy link

Fixed an issue preventing fault lights from blinking during the fault state, adjusted the blink frequency, removed the STM32CubeIDE files from the project completely in favour of PlatformIO, moved the HLIM and LLIM logic out of the interrupt module and into their own module, and did a bit of other housekeeping.

The debouncing in the HLIM and LLIM logic also needed some correction, so please try exercising it by forcing the Elithion BMS HLIM and LLIM signals using the PC interface program for the Elithion, and make sure it still works as expected.

If everything else still works, we should merge this in to master now.

@NicolasRic
Copy link

During testing removing debouncing caused random faults... In the current state the ecu is faulting too late. We need to fix this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants