max/embassy
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[can] Update macrotable comment

Browse Source 
The chip description are corrected in:
https://github.com/embassy-rs/stm32-data/pull/123
This commit is contained in:
Timo Kröger 2022-02-28 22:43:21 +01:00
parent 4c30543938
commit 6929350552
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
embassy-stm32/src/can/bxcan.rs
View File

@ -92,15 +92,13 @@ foreach_peripheral!(
const NUM_FILTER_BANKS: u8 = 14; const NUM_FILTER_BANKS: u8 = 14;
} }
}; };
// Only correct when CAN2 also exists… Fix on yaml level? // CAN1 and CAN2 is a combination of master and slave instance.
// There are only 14 filter banks when CAN2 is not available. // CAN1 owns the filter bank and needs to be enabled in order
// for CAN2 to receive messages.
(can, CAN1) => { (can, CAN1) => {
unsafe impl bxcan::FilterOwner for peripherals::CAN1 { unsafe impl bxcan::FilterOwner for peripherals::CAN1 {
const NUM_FILTER_BANKS: u8 = 28; const NUM_FILTER_BANKS: u8 = 28;
} }
};
(can, CAN2) => {
// CAN2 is always a slave instance where CAN1 is the master instance
unsafe impl bxcan::MasterInstance for peripherals::CAN1 {} unsafe impl bxcan::MasterInstance for peripherals::CAN1 {}
}; };
(can, CAN3) => { (can, CAN3) => {