Enable CI clippy checks nrf52832 and fix issues

Author: plaes

ci.sh

@@ -15,7 +15,7 @@ cargo clippy -p nrf-mpsl --features nrf52805
 cargo clippy -p nrf-mpsl --features nrf52810
 cargo clippy -p nrf-mpsl --features nrf52811
 cargo clippy -p nrf-mpsl --features nrf52820
-# cargo clippy -p nrf-mpsl --features nrf52832
+cargo clippy -p nrf-mpsl --features nrf52832
 cargo clippy -p nrf-mpsl --features nrf52833
 cargo clippy -p nrf-mpsl --features nrf52840
 cargo clippy -p nrf-mpsl --features nrf52840,defmt
@@ -33,7 +33,7 @@ cargo clippy -p nrf-sdc --features nrf52805,peripheral,central
 cargo clippy -p nrf-sdc --features nrf52810,peripheral,central
 cargo clippy -p nrf-sdc --features nrf52811,peripheral,central
 cargo clippy -p nrf-sdc --features nrf52820,peripheral,central
-# cargo clippy -p nrf-sdc --features nrf52832,peripheral,central
+cargo clippy -p nrf-sdc --features nrf52832,peripheral,central
 cargo clippy -p nrf-sdc --features nrf52833,peripheral,central
 cargo clippy -p nrf-sdc --features nrf52840,peripheral
 cargo clippy -p nrf-sdc --features nrf52840,central

nrf-mpsl/src/flash.rs

@@ -404,12 +404,11 @@ impl FlashOp {
     }
 
     // No partial erase for this chip, just do one page at a time
-    #[allow(unused_variables)]
     #[cfg(feature = "nrf52832")]
     fn erase<F: Fn() -> u32>(
-        get_time: F,
-        slot_duration_us: u32,
-        elapsed: &mut u32,
+        _get_time: F,
+        _slot_duration_us: u32,
+        _elapsed: &mut u32,
         address: &mut u32,
         to: u32,
     ) -> core::ops::ControlFlow<()> {
@@ -421,7 +420,7 @@ impl FlashOp {
         p.config.write(|w| w.wen().ren());
         *address += PAGE_SIZE as u32;
         if *address >= to {
-            return ControlFlow::Break(());
+            ControlFlow::Break(())
         } else {
             ControlFlow::Continue(())
         }