Fix `nrf::rtc` errata workaround (#858)

* Deprecate `should_dequeue_check`

* Fix errata by delaying the wakeup point

* Add changelog

* Fix changelog typos

1 files changed, 9 insertions, 5 deletions

diff --git a/rtic-time/src/monotonic.rs b/rtic-time/src/monotonic.rs
index 0c0d6f3..0e8d2b8 100644
--- a/rtic-time/src/monotonic.rs
+++ b/rtic-time/src/monotonic.rs
@@ -36,11 +36,15 @@ pub trait Monotonic {
     /// queue in RTIC checks this.
     fn set_compare(instant: Self::Instant);
 
-    /// Override for the dequeue check, override with timers that have bugs.
-    ///
-    /// E.g. nRF52 RTCs needs to be dequeued if the time is within 4 ticks.
-    fn should_dequeue_check(release_at: Self::Instant) -> bool {
-        <Self as Monotonic>::now() >= release_at
+    /// This method used to be required by an errata workaround
+    /// for the nrf52 family, but it has been disabled as the
+    /// workaround was erroneous.
+    #[deprecated(
+        since = "1.2.0",
+        note = "this method is erroneous and has been disabled"
+    )]
+    fn should_dequeue_check(_: Self::Instant) -> bool {
+        panic!("This method should not be used as it is erroneous.")
     }
 
     /// Clear the compare interrupt flag.