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

RP235x watchdog doesn't have the double count bug

Browse Source
This commit is contained in:
Caleb Jamison 2025-03-28 17:45:41 -04:00
parent 954d1554d4
commit 49badcff1a
No known key found for this signature in database
GPG Key ID: 4B73E51E6C18B035
1 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
embassy-rp/src/watchdog.rs
View File

@ -89,17 +89,25 @@ impl Watchdog {
/// Start the watchdog timer /// Start the watchdog timer
pub fn start(&mut self, period: Duration) { pub fn start(&mut self, period: Duration) {
#[cfg(feature = "rp2040")]
const MAX_PERIOD: u32 = 0xFFFFFF / 2;
#[cfg(feature = "_rp235x")]
const MAX_PERIOD: u32 = 0xFFFFFF; const MAX_PERIOD: u32 = 0xFFFFFF;
let delay_us = period.as_micros(); let delay_us = period.as_micros();
if delay_us > (MAX_PERIOD / 2) as u64 { if delay_us > (MAX_PERIOD) as u64 {
panic!("Period cannot exceed {} microseconds", MAX_PERIOD / 2); panic!("Period cannot exceed {} microseconds", MAX_PERIOD);
} }
let delay_us = delay_us as u32; let delay_us = delay_us as u32;
// Due to a logic error, the watchdog decrements by 2 and // Due to a logic error, the watchdog decrements by 2 and
// the load value must be compensated; see RP2040-E1 // the load value must be compensated; see RP2040-E1
// This errata is fixed in the RP235x
if cfg!(feature = "rp2040") {
self.load_value = delay_us * 2; self.load_value = delay_us * 2;
} else {
self.load_value = delay_us;
}
self.enable(false); self.enable(false);
self.configure_wdog_reset_triggers(); self.configure_wdog_reset_triggers();