Add CAN bus safety
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- rust
- Ambito
- embedded-iot, networking
Direzione di ricerca
Start with boards/esp/src/can.rs:127 for the hard-coded TWAI bitrate, then inspect the [[hardware.esp.can_buses]] configuration and the MCP2515 can_speed setting. The remaining work spans bitrate validation, CAN-FD read-only behavior, exposing RX health in system state, and BLE/debug-only read-only control; done means those safety requirements are implemented and verified across the affected platform and app paths.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The firmware should ensure the CAN bus is healthy at all time. For development purposes, there should be a way to control the CAN read-only mode. And on each boot, before sending any CAN messages, we need to make sure the car is the right one for the firmware. So each platform implementations need a way to detect what cars they are plugged into (VIN is probably a good way).
Remaining from the audit
Done on main: listen-only while read-only, bus-off restart, bounded TX, per-bus
TX queues.
- Configurable bitrate. TWAI is hard-coded to 500 kbps
(boards/esp/src/can.rs:127). Addbitrateto[[hardware.esp.can_buses]]
(MCP2515 already hascan_speed). The vehicle crate (or meta.toml) should
declare the expected bitrate per bus so xtask can check it. - CAN-FD buses. Several E-GMP buses are CAN-FD. The TWAI isn't FD-tolerant:
in listen-only it only sees errors, but in Normal mode it will error-frame
every FD frame. Refuse to leave read-only on a bus declared FD, or document it. - Expose bus health (the new RX stats: errors, overruns, bus-offs) in system
state so the app can show it, not just on the serial log. - Dev control of read-only mode: when added, BLE + debug builds only,
never MQTT.
- Lingua principale
- Rust
- Stelle
- 1
- Fork
- 0
- Merge medio
- 10h 56m
- PR unite (30g)
- 15
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di OpenCarControls/firmware
-
Clean up E-GMP leftoversForse già presa @Tichael l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
OpenCarControls/firmware#34 ·
I maintainer di solito rispondono entro 1 giorno
-
BLE: send the current state when the app subscribesForse già presa @Tichael l’ha presa oggi. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
OpenCarControls/firmware#58 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
OpenCarControls/firmware#55 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
OpenCarControls/firmware#54 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
OpenCarControls/firmware#53 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di OpenCarControls/firmware
Issue simili
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 86/100
I maintainer di solito rispondono entro 1 giorno
-
area:sim good first issue kind:feature
Difficoltà 2/5 1-3 ore Idoneità per principianti 87/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
bitcoindevkit/bdk#2432 ·
I maintainer di solito rispondono entro 3 giorni
-
The demo's interval button and chart legend keep the previous interval after an interval changeApertabug ui
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
userFRM/egui-charts#670 ·
I maintainer di solito rispondono entro 1 giorno
-
docs good first issue
Difficoltà 1/5 1-3 ore Idoneità per principianti 84/100
Usenova-Labs/usenova-backend#35 ·
I maintainer di solito rispondono entro 1 giorno