From 521ea5561a7974cbd260f453de9eb4c281b81f74 Mon Sep 17 00:00:00 2001 From: Vinayak Kariappa Chettimada Date: Fri, 19 Jul 2024 03:09:04 +0200 Subject: [PATCH] Bluetooth: Controller: Add assertion checks for delayed radio start Add assertion checks for delayed radio start. Signed-off-by: Vinayak Kariappa Chettimada --- subsys/bluetooth/controller/ll_sw/nordic/lll/lll_scan_aux.c | 1 + subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync.c | 1 + subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c | 5 +---- 3 files changed, 3 insertions(+), 4 deletions(-) diff --git a/subsys/bluetooth/controller/ll_sw/nordic/lll/lll_scan_aux.c b/subsys/bluetooth/controller/ll_sw/nordic/lll/lll_scan_aux.c index b1e95bb0632..d28d25dc557 100644 --- a/subsys/bluetooth/controller/ll_sw/nordic/lll/lll_scan_aux.c +++ b/subsys/bluetooth/controller/ll_sw/nordic/lll/lll_scan_aux.c @@ -338,6 +338,7 @@ void lll_scan_aux_isr_aux_setup(void *param) aux_start_us -= EVENT_JITTER_US; start_us = radio_tmr_start_us(0, aux_start_us); + LL_ASSERT(start_us == (aux_start_us + 1U)); /* Setup header complete timeout */ hcto = start_us; diff --git a/subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync.c b/subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync.c index d0285ac9ea4..63b6e9a53b5 100644 --- a/subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync.c +++ b/subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync.c @@ -738,6 +738,7 @@ static void isr_aux_setup(void *param) aux_start_us -= EVENT_JITTER_US; start_us = radio_tmr_start_us(0, aux_start_us); + LL_ASSERT(start_us == (aux_start_us + 1U)); /* Setup header complete timeout */ hcto = start_us; diff --git a/subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c b/subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c index cb6023b4775..2f4db61e796 100644 --- a/subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c +++ b/subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c @@ -1046,10 +1046,7 @@ isr_rx_next_subevent: start_us = hcto; hcto = radio_tmr_start_us(0U, start_us); - /* FIXME: Assertion check disabled until investigation as to - * why there is high ISR latency causing assertion here. - */ - /* LL_ASSERT(hcto == (start_us + 1U)); */ + LL_ASSERT(hcto == (start_us + 1U)); /* Add 8 us * subevents so far, as radio was setup to listen * 4 us early and subevents could have a 4 us drift each until