MCP2515 hardware masks are too strict
I maintainer di solito rispondono entro 1 giorno
Una pull request collegata è già stata integrata.
- #61 di @Tichael — integrata
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 72/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- rust
- Ambito
- embedded-iot
Direzione di ricerca
Read compute_mcp_masks in boards/esp/src/can.rs around line 617 and the two_filters_ored_into_mask0 test. Check how masks are combined and written for standard IDs, then update the calculation and test for the stated extended-ID fallback. Done means the test covers the corrected behavior and hardware masks no longer reject frames accepted by the software filter.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
compute_mcp_masks (boards/esp/src/can.rs:617) still ORs the per-filter
masks; its doc comment describes AND. OR makes the hardware reject frames the
software filter accepts: 0x100/0x700 + 0x200/0x0FF → 0x7FF, so only two
exact IDs pass. The test two_filters_ored_into_mask0 locks in the bug.
Extended IDs aren't handled either (masks are written as standard IDs).
Fix: AND the masks (a bit must match only if every filter in that buffer cares
about it). Fall back to accept-all hardware masks when any filter is extended
(the software filter still applies). Update the test.
- Lingua principale
- Rust
- Stelle
- 1
- Fork
- 0
- Merge medio
- 11h 26m
- PR unite (30g)
- 18
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
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 51/100
OpenCarControls/firmware#68 ·
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
-
First-setup pairing window is too short and stays open after the first bondForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertaenhancement
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
OpenCarControls/firmware#53 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
OpenCarControls/firmware#51 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di OpenCarControls/firmware
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
farion1231/cc-switch#8829 ·
I maintainer di solito rispondono entro 1 giorno
-
bug ci good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
longbridge/gpui-kit#3677 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 68/100
ilya-zlobintsev/LACT#1235 ·
I maintainer di solito rispondono entro 1 giorno
-
[Bug]: Shared Sink flush/close can complete without submitting buffered tasksForse già presa @RivTian l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
apalis-dev/apalis#797 ·
I maintainer di solito rispondono entro 7 giorni