FOME-Tech / FOME-Tech/fome-fw

ECU succeptible to misconfiguration through UART/Add UART read only functionality

Open
#72 2 comments 0 reactions 0 assignees View on GitHub
🐞 bug 🐞 enhancement
Dominant language
C
Stars
138
Forks
69
Avg merge
5d 20h
Merged PRs (30d)
4

Description

This has been an issue I've had on RE but it's recently popped up on FOME too where things connected to the UART have the ability to misconfigure the ECU. In both of these cases it has been using EFI Analytics software.
Here's some examples:
- Connected to a HC-06 BT module and using shadowdash MS on RE: The car would occasionally loose count of the timing and stall at idle only when connected to this, the car would also occasionally go into a 2 step rev limiter mode at 4000rpm when that feature was disabled (e.g. on highway, opened app and it instantly wouldn't let me rev past 4000rpm).
- Pidash connected to UART and FOME: Sometimes the ECU will read parameters from the tune file in TS Dash software running on the Pi which has resulted in the gear detection only working when I burn the tune from a laptop or the ECU erroring out from certain pins being configured as invalid.

In both cases I believe by adding the ability for the ECU to only be read from over the UART would stop these issues from occuring. This would stop there being a reliance on whatever is attached to the bus not writing junk data to the ECU in the case of a bug or a corruption. I acknoledge that my Pidash running TS dash could corrupt occasionally but preventing it from being able to change the ECU settings would make it more idiot proof

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.