low power for h5

3 files changed, 37 insertions, 52 deletions

diff --git a/embassy-stm32/src/rtc/datetime.rs b/embassy-stm32/src/rtc/datetime.rs
index bab8cf4a3..77d89293d 100644
--- a/embassy-stm32/src/rtc/datetime.rs
+++ b/embassy-stm32/src/rtc/datetime.rs
@@ -148,9 +148,9 @@ impl DateTime {
     ) -> Result<Self, Error> {
         if year > 4095 {
             Err(Error::InvalidYear)
-        } else if month < 1 || month > 12 {
+        } else if !(1..=12).contains(&month) {
             Err(Error::InvalidMonth)
-        } else if day < 1 || day > 31 {
+        } else if !(1..=31).contains(&day) {
             Err(Error::InvalidDay)
         } else if hour > 23 {
             Err(Error::InvalidHour)
diff --git a/embassy-stm32/src/rtc/mod.rs b/embassy-stm32/src/rtc/mod.rs
index 00abe9356..b12a0db66 100644
--- a/embassy-stm32/src/rtc/mod.rs
+++ b/embassy-stm32/src/rtc/mod.rs
@@ -42,7 +42,7 @@ pub(crate) enum WakeupPrescaler {
     Div16 = 16,
 }
 
-#[cfg(any(stm32wb, stm32f4, stm32l0, stm32g4, stm32l5, stm32g0))]
+#[cfg(any(stm32f4, stm32l0, stm32g4, stm32l5, stm32wb, stm32h5, stm32g0))]
 impl From<WakeupPrescaler> for crate::pac::rtc::vals::Wucksel {
     fn from(val: WakeupPrescaler) -> Self {
         use crate::pac::rtc::vals::Wucksel;
@@ -56,7 +56,7 @@ impl From<WakeupPrescaler> for crate::pac::rtc::vals::Wucksel {
     }
 }
 
-#[cfg(any(stm32wb, stm32f4, stm32l0, stm32g4, stm32l5, stm32g0))]
+#[cfg(any(stm32f4, stm32l0, stm32g4, stm32l5, stm32wb, stm32h5, stm32g0))]
 impl From<crate::pac::rtc::vals::Wucksel> for WakeupPrescaler {
     fn from(val: crate::pac::rtc::vals::Wucksel) -> Self {
         use crate::pac::rtc::vals::Wucksel;
@@ -81,8 +81,7 @@ impl WakeupPrescaler {
             WakeupPrescaler::Div16,
         ]
         .iter()
-        .skip_while(|psc| **psc as u32 <= val)
-        .next()
+        .find(|psc| **psc as u32 > val)
         .unwrap_or(&WakeupPrescaler::Div16)
     }
 }
@@ -159,7 +158,7 @@ impl RtcTimeProvider {
             }
         }
 
-        return Err(RtcError::ReadFailure);
+        Err(RtcError::ReadFailure)
     }
 }
 
@@ -190,7 +189,7 @@ impl Default for RtcConfig {
 }
 
 /// Calibration cycle period.
-#[derive(Copy, Clone, Debug, PartialEq)]
+#[derive(Default, Copy, Clone, Debug, PartialEq)]
 #[repr(u8)]
 pub enum RtcCalibrationCyclePeriod {
     /// 8-second calibration period
@@ -198,15 +197,10 @@ pub enum RtcCalibrationCyclePeriod {
     /// 16-second calibration period
     Seconds16,
     /// 32-second calibration period
+    #[default]
     Seconds32,
 }
 
-impl Default for RtcCalibrationCyclePeriod {
-    fn default() -> Self {
-        RtcCalibrationCyclePeriod::Seconds32
-    }
-}
-
 impl Rtc {
     /// Create a new RTC instance.
     pub fn new(_rtc: impl Peripheral<P = RTC>, rtc_config: RtcConfig) -> Self {
@@ -254,13 +248,13 @@ impl Rtc {
     /// Will return `RtcError::InvalidDateTime` if the datetime is not a valid range.
     pub fn set_datetime(&mut self, t: DateTime) -> Result<(), RtcError> {
         self.write(true, |rtc| {
-            let (ht, hu) = byte_to_bcd2(t.hour() as u8);
-            let (mnt, mnu) = byte_to_bcd2(t.minute() as u8);
-            let (st, su) = byte_to_bcd2(t.second() as u8);
+            let (ht, hu) = byte_to_bcd2(t.hour());
+            let (mnt, mnu) = byte_to_bcd2(t.minute());
+            let (st, su) = byte_to_bcd2(t.second());
 
-            let (dt, du) = byte_to_bcd2(t.day() as u8);
-            let (mt, mu) = byte_to_bcd2(t.month() as u8);
-            let yr = t.year() as u16;
+            let (dt, du) = byte_to_bcd2(t.day());
+            let (mt, mu) = byte_to_bcd2(t.month());
+            let yr = t.year();
             let yr_offset = (yr - 2000_u16) as u8;
             let (yt, yu) = byte_to_bcd2(yr_offset);
 
@@ -338,7 +332,7 @@ impl Rtc {
     }
 
     #[cfg(feature = "low-power")]
-    /// start the wakeup alarm and wtih a duration that is as close to but less than
+    /// start the wakeup alarm and with a duration that is as close to but less than
     /// the requested duration, and record the instant the wakeup alarm was started
     pub(crate) fn start_wakeup_alarm(
         &self,
@@ -422,20 +416,15 @@ impl Rtc {
                 #[cfg(any(rtc_v3, rtc_v3u5, rtc_v3l5))]
                 regs.scr().write(|w| w.set_cwutf(Calrf::CLEAR));
 
-                #[cfg(all(stm32g0))]
-                crate::pac::EXTI
-                    .rpr(0)
-                    .modify(|w| w.set_line(RTC::EXTI_WAKEUP_LINE, true));
-                #[cfg(all(not(stm32g0), not(stm32l5)))]
+                // Check RM for EXTI and/or NVIC section, "Event event input mapping" or "EXTI interrupt/event mapping" or something similar,
+                // there is a table for every "Event input" / "EXTI Line".
+                // If you find the EXTI line related to "RTC wakeup" marks as "Configurable" (not "Direct"),
+                // then write 1 to related field of Pending Register, to clean it's pending state.
+                #[cfg(any(exti_v1, stm32h7, stm32wb))]
                 crate::pac::EXTI
                     .pr(0)
                     .modify(|w| w.set_line(RTC::EXTI_WAKEUP_LINE, true));
 
-                #[cfg(stm32l5)]
-                crate::pac::EXTI
-                    .fpr(0)
-                    .modify(|w| w.set_line(RTC::EXTI_WAKEUP_LINE, true));
-
                 <RTC as crate::rtc::SealedInstance>::WakeupInterrupt::unpend();
             });
         }
@@ -465,7 +454,7 @@ pub(crate) fn byte_to_bcd2(byte: u8) -> (u8, u8) {
         value -= 10;
     }
 
-    (bcd_high, ((bcd_high << 4) | value) as u8)
+    (bcd_high, ((bcd_high << 4) | value))
 }
 
 pub(crate) fn bcd2_to_byte(bcd: (u8, u8)) -> u8 {
diff --git a/embassy-stm32/src/rtc/v3.rs b/embassy-stm32/src/rtc/v3.rs
index 8a78d16e1..e51e09e7c 100644
--- a/embassy-stm32/src/rtc/v3.rs
+++ b/embassy-stm32/src/rtc/v3.rs
@@ -50,7 +50,7 @@ impl super::Rtc {
             clock_drift = Self::RTC_CALR_MAX_PPM;
         }
 
-        clock_drift = clock_drift / Self::RTC_CALR_RESOLUTION_PPM;
+        clock_drift /= Self::RTC_CALR_RESOLUTION_PPM;
 
         self.write(false, |rtc| {
             rtc.calr().write(|w| {
@@ -129,29 +129,25 @@ impl super::Rtc {
 impl SealedInstance for crate::peripherals::RTC {
     const BACKUP_REGISTER_COUNT: usize = 32;
 
-    #[cfg(all(feature = "low-power", stm32g4))]
-    const EXTI_WAKEUP_LINE: usize = 20;
-
-    #[cfg(all(feature = "low-power", stm32g0))]
-    const EXTI_WAKEUP_LINE: usize = 19;
-
-    #[cfg(all(feature = "low-power", stm32g0))]
-    type WakeupInterrupt = crate::interrupt::typelevel::RTC_TAMP;
-
-    #[cfg(all(feature = "low-power", stm32g4))]
-    type WakeupInterrupt = crate::interrupt::typelevel::RTC_WKUP;
-
-    #[cfg(all(feature = "low-power", stm32l5))]
-    const EXTI_WAKEUP_LINE: usize = 17;
-
-    #[cfg(all(feature = "low-power", stm32l5))]
-    type WakeupInterrupt = crate::interrupt::typelevel::RTC;
+    #[cfg(feature = "low-power")]
+    cfg_if::cfg_if!(
+        if #[cfg(stm32g4)] {
+            const EXTI_WAKEUP_LINE: usize = 20;
+            type WakeupInterrupt = crate::interrupt::typelevel::RTC_WKUP;
+        } else if #[cfg(stm32g0)] {
+            const EXTI_WAKEUP_LINE: usize = 19;
+            type WakeupInterrupt = crate::interrupt::typelevel::RTC_TAMP;
+        } else if #[cfg(any(stm32l5, stm32h5))] {
+            const EXTI_WAKEUP_LINE: usize = 17;
+            type WakeupInterrupt = crate::interrupt::typelevel::RTC;
+        }
+    );
 
     fn read_backup_register(_rtc: &Rtc, register: usize) -> Option<u32> {
         #[allow(clippy::if_same_then_else)]
         if register < Self::BACKUP_REGISTER_COUNT {
             //Some(rtc.bkpr()[register].read().bits())
-            None // RTC3 backup registers come from the TAMP peripe=heral, not RTC. Not() even in the L412 PAC
+            None // RTC3 backup registers come from the TAMP peripheral, not RTC. Not() even in the L412 PAC
         } else {
             None
         }
@@ -159,7 +155,7 @@ impl SealedInstance for crate::peripherals::RTC {
 
     fn write_backup_register(_rtc: &Rtc, register: usize, _value: u32) {
         if register < Self::BACKUP_REGISTER_COUNT {
-            // RTC3 backup registers come from the TAMP peripe=heral, not RTC. Not() even in the L412 PAC
+            // RTC3 backup registers come from the TAMP peripheral, not RTC. Not() even in the L412 PAC
             //self.rtc.bkpr()[register].write(|w| w.bits(value))
         }
     }