Bluetooth: Controller: Fix periodic advertising sync window

cvinayak · carlescufi · commit 05c85ddbcf8e · 2023-10-24T15:12:18.000+02:00
Fix periodic advertising sync window calculation to include
the scheduling resolution margin, i.e. be double as with
the event jitter value.

Signed-off-by: Vinayak Kariappa Chettimada &lt;vich@nordicsemi.no&gt;
diff --git a/subsys/bluetooth/controller/ll_sw/ull_scan_aux.c b/subsys/bluetooth/controller/ll_sw/ull_scan_aux.c
@@ -646,8 +646,8 @@ void ull_scan_aux_setup(memq_link_t *link, struct node_rx_hdr *rx)
 		window_widening_us = SCA_DRIFT_500_PPM_US(aux_offset_us);
 	}
 
-	lll_aux->window_size_us += (EVENT_TICKER_RES_MARGIN_US +
-				    ((EVENT_JITTER_US + window_widening_us) << 1));
+	lll_aux->window_size_us += ((EVENT_TICKER_RES_MARGIN_US + EVENT_JITTER_US +
+				     window_widening_us) << 1);
 
 	ready_delay_us = lll_radio_rx_ready_delay_get(lll_aux->phy,
 						      PHY_FLAGS_S8);

Original file line number	Diff line number	Diff line change
`@@ -646,8 +646,8 @@ void ull_scan_aux_setup(memq_link_t link, struct node_rx_hdr rx)`
`646`	`646`	`window_widening_us = SCA_DRIFT_500_PPM_US(aux_offset_us);`
`647`	`647`	`}`
`648`	`648`
`649`		`- lll_aux->window_size_us += (EVENT_TICKER_RES_MARGIN_US +`
`650`		`- ((EVENT_JITTER_US + window_widening_us) << 1));`
	`649`	`+ lll_aux->window_size_us += ((EVENT_TICKER_RES_MARGIN_US + EVENT_JITTER_US +`
	`650`	`+ window_widening_us) << 1);`
`651`	`651`
`652`	`652`	`ready_delay_us = lll_radio_rx_ready_delay_get(lll_aux->phy,`
`653`	`653`	`PHY_FLAGS_S8);`