From a07702ba7bde8410b224e80afb6aa7a93f1fb375 Mon Sep 17 00:00:00 2001
From: Ulf Lilleengen <ulf.lilleengen@gmail.com>
Date: Wed, 3 Jul 2024 09:03:25 +0200
Subject: [PATCH] fix cancellation hanging on event never occuring
- The end event is not triggered by stop and might therefor never occur
- Clear the end_event after operation is done instead of disabled
---
embassy-nrf/src/radio/ble.rs | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/embassy-nrf/src/radio/ble.rs b/embassy-nrf/src/radio/ble.rs
index 93003fb19..0fae54f52 100644
--- a/embassy-nrf/src/radio/ble.rs
+++ b/embassy-nrf/src/radio/ble.rs
@@ -347,12 +347,10 @@ impl<'d, T: Instance> Radio<'d, T> {
trace!("radio drop: stopping");
r.intenclr.write(|w| w.end().clear());
- r.events_end.reset();
r.tasks_stop.write(|w| unsafe { w.bits(1) });
- // The docs don't explicitly mention any event to acknowledge the stop task
- while r.events_end.read().bits() == 0 {}
+ r.events_end.reset();
trace!("radio drop: stopped");
});
@@ -382,7 +380,7 @@ impl<'d, T: Instance> Radio<'d, T> {
.await;
compiler_fence(Ordering::SeqCst);
- r.events_disabled.reset(); // ACK
+ r.events_end.reset(); // ACK
// Everthing ends fine, so it disable the drop
drop.defuse();