itsscb/embassy
1
0
Fork 0
mirror of https://github.com/embassy-rs/embassy.git synced 2025-10-03 07:04:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

stm32: Fix log storm when no CAN is connected

Browse Source 
Running the bxcan driver without having it connected
to a CAN bus causes the `info` logs to bombard.
This removes the logging statements as they looked
like remnants from the development of the driver.
This commit is contained in:
Emil Fresk 2024-08-25 07:14:19 +02:00
parent 545eb39819
commit 41e162541a
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
embassy-stm32/src/can/bxcan/mod.rs
View File

@ -68,7 +68,6 @@ pub struct SceInterruptHandler<T: Instance> {
impl<T: Instance> interrupt::typelevel::Handler<T::SCEInterrupt> for SceInterruptHandler<T> { impl<T: Instance> interrupt::typelevel::Handler<T::SCEInterrupt> for SceInterruptHandler<T> {
unsafe fn on_interrupt() { unsafe fn on_interrupt() {
info!("sce irq");
let msr = T::regs().msr(); let msr = T::regs().msr();
let msr_val = msr.read(); let msr_val = msr.read();
@ -76,9 +75,8 @@ impl<T: Instance> interrupt::typelevel::Handler<T::SCEInterrupt> for SceInterrup
msr.modify(|m| m.set_slaki(true)); msr.modify(|m| m.set_slaki(true));
T::state().err_waker.wake(); T::state().err_waker.wake();
} else if msr_val.erri() { } else if msr_val.erri() {
info!("Error interrupt");
// Disable the interrupt, but don't acknowledge the error, so that it can be // Disable the interrupt, but don't acknowledge the error, so that it can be
// forwarded off the the bus message consumer. If we don't provide some way for // forwarded off the bus message consumer. If we don't provide some way for
// downstream code to determine that it has already provided this bus error instance // downstream code to determine that it has already provided this bus error instance
// to the bus message consumer, we are doomed to re-provide a single error instance for // to the bus message consumer, we are doomed to re-provide a single error instance for
// an indefinite amount of time. // an indefinite amount of time.