NaxdyOrg/embassy
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

rp/watchdog: fix overflow if period is longer than 4294 seconds.

Browse source
This commit is contained in:
Dario Nieuwenhuis 2023-05-13 00:33:00 +02:00
parent e179e7cf85
commit 3edd81a94e
1 changed files with 5 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -89,14 +89,12 @@ impl Watchdog {
pub fn start(&mut self, period: Duration) { pub fn start(&mut self, period: Duration) {
const MAX_PERIOD: u32 = 0xFFFFFF; const MAX_PERIOD: u32 = 0xFFFFFF;
let delay_us = period.as_micros() as u32; let delay_us = period.as_micros();
if delay_us > MAX_PERIOD / 2 { if delay_us > (MAX_PERIOD / 2) as u64 {
panic!( panic!("Period cannot exceed {} microseconds", MAX_PERIOD / 2);
"Period cannot exceed maximum load value of {} ({} microseconds))",
MAX_PERIOD,
MAX_PERIOD / 2
);
} }
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
self.load_value = delay_us * 2; self.load_value = delay_us * 2;