Bluetooth: Controller: nRF54Lx: Use NRF_GRTC for radio scheduling #146247
cvinayakcompliance.yml
on: pull_request
Run compliance checks on patch series (PR)
2m 28s
Annotations
1 error, 1 warning, and 10 notices
|
Run compliance checks on patch series (PR)
Process completed with exit code 1.
|
|
Run compliance checks on patch series (PR):
ClangFormat.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#clang-format for more details.
You may want to run clang-format on this change:
-#else /* CONFIG_BT_CTLR_NRF_GRTC */
+#else /* CONFIG_BT_CTLR_NRF_GRTC */
File:subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c
Line:24
You may want to run clang-format on this change:
- NRF_GRTC->MODE = (GRTC_MODE_SYSCOUNTEREN_Disabled <<
- GRTC_MODE_SYSCOUNTEREN_Pos) &
+ NRF_GRTC->MODE = (GRTC_MODE_SYSCOUNTEREN_Disabled << GRTC_MODE_SYSCOUNTEREN_Pos) &
GRTC_MODE_SYSCOUNTEREN_Msk;
nrf_grtc_task_trigger(NRF_GRTC, NRF_GRTC_TASK_CLEAR);
#if defined(CONFIG_BT_CTLR_NRF_GRTC_KEEPRUNNING)
NRF_GRTC->KEEPRUNNING =
- (GRTC_KEEPRUNNING_REQUEST1_Active <<
- GRTC_KEEPRUNNING_REQUEST1_Pos) &
+ (GRTC_KEEPRUNNING_REQUEST1_Active << GRTC_KEEPRUNNING_REQUEST1_Pos) &
GRTC_KEEPRUNNING_REQUEST1_Msk;
NRF_GRTC->TIMEOUT = 0U;
NRF_GRTC->INTERVAL = 0U;
NRF_GRTC->WAKETIME = 4U;
#endif /* CONFIG_BT_CTLR_NRF_GRTC_KEEPRUNNING */
- NRF_GRTC->CLKCFG = ((GRTC_CLKCFG_CLKSEL_LFXO <<
- GRTC_CLKCFG_CLKSEL_Pos) &
- GRTC_CLKCFG_CLKSEL_Msk) |
- ((GRTC_CLKCFG_CLKFASTDIV_Min <<
- GRTC_CLKCFG_CLKFASTDIV_Pos) &
- GRTC_CLKCFG_CLKFASTDIV_Msk);
+ NRF_GRTC->CLKCFG =
+ ((GRTC_CLKCFG_CLKSEL_LFXO << GRTC_CLKCFG_CLKSEL_Pos) & GRTC_CLKCFG_CLKSEL_Msk) |
+ ((GRTC_CLKCFG_CLKFASTDIV_Min << GRTC_CLKCFG_CLKFASTDIV_Pos) &
+ GRTC_CLKCFG_CLKFASTDIV_Msk);
File:subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c
Line:55
You may want to run clang-format on this change:
- NRF_GRTC->MODE = ((GRTC_MODE_SYSCOUNTEREN_Enabled <<
- GRTC_MODE_SYSCOUNTEREN_Pos) &
- GRTC_MODE_SYSCOUNTEREN_Msk) |
+ NRF_GRTC->MODE =
+ ((GRTC_MODE_SYSCOUNTEREN_Enabled << GRTC_MODE_SYSCOUNTEREN_Pos) &
+ GRTC_MODE_SYSCOUNTEREN_Msk) |
#if defined(CONFIG_BT_CTLR_NRF_GRTC_AUTOEN_CPUACTIVE)
- ((GRTC_MODE_AUTOEN_CpuActive <<
- GRTC_MODE_AUTOEN_Pos) &
- GRTC_MODE_AUTOEN_Msk) |
+ ((GRTC_MODE_AUTOEN_CpuActive << GRTC_MODE_AUTOEN_Pos) & GRTC_MODE_AUTOEN_Msk) |
#endif /* CONFIG_BT_CTLR_NRF_GRTC_AUTOEN_CPUACTIVE */
#if defined(CONFIG_BT_CTLR_NRF_GRTC_AUTOEN_DEFAULT)
- ((GRTC_MODE_AUTOEN_Default <<
- GRTC_MODE_AUTOEN_Pos) &
- GRTC_MODE_AUTOEN_Msk) |
+ ((GRTC_MODE_AUTOEN_Default << GRTC_MODE_AUTOEN_Pos) & GRTC_MODE_AUTOEN_Msk) |
#endif /* CONFIG_BT_CTLR_NRF_GRTC_AUTOEN_DEFAULT */
- 0U;
+ 0U;
nrf_grtc_task_trigger(NRF_GRTC, NRF_GRTC_TASK_START);
#endif /* CONFIG_BT_CTLR_NRF_GRTC_START */
-#else /* !CONFIG_BT_CTLR_NRF_GRTC */
+#else /* !CONFIG_BT_CTLR_NRF_GRTC */
File:subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c
Line:90
You may want to run clang-format on this change:
-#else /* !CONFIG_BT_CTLR_NRF_GRTC */
+#else /* !CONFIG_BT_CTLR_NRF_GRTC */
File:subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c
Line:105
You may want to run clang-format on this change:
-#else /* !CONFIG_BT_CTLR_NRF_GRTC */
+#else /* !CONFIG_BT_CTLR_NRF_GRTC */
File:subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c
Line:122
You may want to run clang-format on this change:
-#else /* !CONFIG_BT_CTLR_NRF_GRTC */
+#else /* !CONFIG_BT_CTLR_NRF_GRTC */
File:subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c
Line:147
You may want to run clang-format on this change:
-#else /* !CONFIG_BT_CTLR_NRF_GRTC */
+#else /* !CONFIG_BT_CTLR_NRF_GRTC */
File:subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c
Line:202
You may want to run clang-format on this change:
-#define HAL_CNTR_GRTC_EVENT_COMPARE_TICKER \
+#define HAL_CNTR_GRTC_EVENT_COMPARE_TICKER \
_CONCAT(NRF_GRTC_EVENT_COMPARE_, HAL_CNTR_GRTC_CC_IDX_TICKER)
-#define HAL_CNTR_GRTC_INTENSET_COMPARE_TICKER_Msk \
+#define HAL_CNTR_GRTC_INTENSET_COMPARE_TICKER_Msk \
_CONCAT(_CONCAT(GRTC_INTENSET1_COMPARE, HAL_CNTR_GRTC_CC_IDX_TICKER), _Msk)
-#define HAL_CNTR_GRTC_CC_IDX_RADIO 11
-#define HAL_CNTR_GRTC_EVENT_COMPARE_RADIO \
+#define HAL_CNTR_GRTC_CC_IDX_RADIO 11
+#define HAL_CN
|
|
You may want to run clang-format on this change:
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c#L24
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c:24
-#else /* CONFIG_BT_CTLR_NRF_GRTC */
+#else /* CONFIG_BT_CTLR_NRF_GRTC */
|
|
You may want to run clang-format on this change:
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c#L55
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c:55
- NRF_GRTC->MODE = (GRTC_MODE_SYSCOUNTEREN_Disabled <<
- GRTC_MODE_SYSCOUNTEREN_Pos) &
+ NRF_GRTC->MODE = (GRTC_MODE_SYSCOUNTEREN_Disabled << GRTC_MODE_SYSCOUNTEREN_Pos) &
GRTC_MODE_SYSCOUNTEREN_Msk;
nrf_grtc_task_trigger(NRF_GRTC, NRF_GRTC_TASK_CLEAR);
#if defined(CONFIG_BT_CTLR_NRF_GRTC_KEEPRUNNING)
NRF_GRTC->KEEPRUNNING =
- (GRTC_KEEPRUNNING_REQUEST1_Active <<
- GRTC_KEEPRUNNING_REQUEST1_Pos) &
+ (GRTC_KEEPRUNNING_REQUEST1_Active << GRTC_KEEPRUNNING_REQUEST1_Pos) &
GRTC_KEEPRUNNING_REQUEST1_Msk;
NRF_GRTC->TIMEOUT = 0U;
NRF_GRTC->INTERVAL = 0U;
NRF_GRTC->WAKETIME = 4U;
#endif /* CONFIG_BT_CTLR_NRF_GRTC_KEEPRUNNING */
- NRF_GRTC->CLKCFG = ((GRTC_CLKCFG_CLKSEL_LFXO <<
- GRTC_CLKCFG_CLKSEL_Pos) &
- GRTC_CLKCFG_CLKSEL_Msk) |
- ((GRTC_CLKCFG_CLKFASTDIV_Min <<
- GRTC_CLKCFG_CLKFASTDIV_Pos) &
- GRTC_CLKCFG_CLKFASTDIV_Msk);
+ NRF_GRTC->CLKCFG =
+ ((GRTC_CLKCFG_CLKSEL_LFXO << GRTC_CLKCFG_CLKSEL_Pos) & GRTC_CLKCFG_CLKSEL_Msk) |
+ ((GRTC_CLKCFG_CLKFASTDIV_Min << GRTC_CLKCFG_CLKFASTDIV_Pos) &
+ GRTC_CLKCFG_CLKFASTDIV_Msk);
|
|
You may want to run clang-format on this change:
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c#L90
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c:90
- NRF_GRTC->MODE = ((GRTC_MODE_SYSCOUNTEREN_Enabled <<
- GRTC_MODE_SYSCOUNTEREN_Pos) &
- GRTC_MODE_SYSCOUNTEREN_Msk) |
+ NRF_GRTC->MODE =
+ ((GRTC_MODE_SYSCOUNTEREN_Enabled << GRTC_MODE_SYSCOUNTEREN_Pos) &
+ GRTC_MODE_SYSCOUNTEREN_Msk) |
#if defined(CONFIG_BT_CTLR_NRF_GRTC_AUTOEN_CPUACTIVE)
- ((GRTC_MODE_AUTOEN_CpuActive <<
- GRTC_MODE_AUTOEN_Pos) &
- GRTC_MODE_AUTOEN_Msk) |
+ ((GRTC_MODE_AUTOEN_CpuActive << GRTC_MODE_AUTOEN_Pos) & GRTC_MODE_AUTOEN_Msk) |
#endif /* CONFIG_BT_CTLR_NRF_GRTC_AUTOEN_CPUACTIVE */
#if defined(CONFIG_BT_CTLR_NRF_GRTC_AUTOEN_DEFAULT)
- ((GRTC_MODE_AUTOEN_Default <<
- GRTC_MODE_AUTOEN_Pos) &
- GRTC_MODE_AUTOEN_Msk) |
+ ((GRTC_MODE_AUTOEN_Default << GRTC_MODE_AUTOEN_Pos) & GRTC_MODE_AUTOEN_Msk) |
#endif /* CONFIG_BT_CTLR_NRF_GRTC_AUTOEN_DEFAULT */
- 0U;
+ 0U;
nrf_grtc_task_trigger(NRF_GRTC, NRF_GRTC_TASK_START);
#endif /* CONFIG_BT_CTLR_NRF_GRTC_START */
-#else /* !CONFIG_BT_CTLR_NRF_GRTC */
+#else /* !CONFIG_BT_CTLR_NRF_GRTC */
|
|
You may want to run clang-format on this change:
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c#L105
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c:105
-#else /* !CONFIG_BT_CTLR_NRF_GRTC */
+#else /* !CONFIG_BT_CTLR_NRF_GRTC */
|
|
You may want to run clang-format on this change:
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c#L122
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c:122
-#else /* !CONFIG_BT_CTLR_NRF_GRTC */
+#else /* !CONFIG_BT_CTLR_NRF_GRTC */
|
|
You may want to run clang-format on this change:
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c#L147
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c:147
-#else /* !CONFIG_BT_CTLR_NRF_GRTC */
+#else /* !CONFIG_BT_CTLR_NRF_GRTC */
|
|
You may want to run clang-format on this change:
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c#L202
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.c:202
-#else /* !CONFIG_BT_CTLR_NRF_GRTC */
+#else /* !CONFIG_BT_CTLR_NRF_GRTC */
|
|
You may want to run clang-format on this change:
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.h#L15
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/cntr.h:15
-#define HAL_CNTR_GRTC_EVENT_COMPARE_TICKER \
+#define HAL_CNTR_GRTC_EVENT_COMPARE_TICKER \
_CONCAT(NRF_GRTC_EVENT_COMPARE_, HAL_CNTR_GRTC_CC_IDX_TICKER)
-#define HAL_CNTR_GRTC_INTENSET_COMPARE_TICKER_Msk \
+#define HAL_CNTR_GRTC_INTENSET_COMPARE_TICKER_Msk \
_CONCAT(_CONCAT(GRTC_INTENSET1_COMPARE, HAL_CNTR_GRTC_CC_IDX_TICKER), _Msk)
-#define HAL_CNTR_GRTC_CC_IDX_RADIO 11
-#define HAL_CNTR_GRTC_EVENT_COMPARE_RADIO \
+#define HAL_CNTR_GRTC_CC_IDX_RADIO 11
+#define HAL_CNTR_GRTC_EVENT_COMPARE_RADIO \
|
|
You may want to run clang-format on this change:
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/radio/radio.c#L253
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/radio/radio.c:253
- NRF_RADIO->TIMING = (RADIO_TIMING_RU_Legacy << RADIO_TIMING_RU_Pos) &
- RADIO_TIMING_RU_Msk;
-#else /* !CONFIG_BT_CTLR_TIFS_HW */
- NRF_RADIO->TIMING = (RADIO_TIMING_RU_Fast << RADIO_TIMING_RU_Pos) &
- RADIO_TIMING_RU_Msk;
+ NRF_RADIO->TIMING = (RADIO_TIMING_RU_Legacy << RADIO_TIMING_RU_Pos) & RADIO_TIMING_RU_Msk;
+#else /* !CONFIG_BT_CTLR_TIFS_HW */
+ NRF_RADIO->TIMING = (RADIO_TIMING_RU_Fast << RADIO_TIMING_RU_Pos) & RADIO_TIMING_RU_Msk;
|
|
You may want to run clang-format on this change:
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/radio/radio.c#L417
subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/radio/radio.c:417
-#else /* !RADIO_DATAWHITEIV_DATAWHITEIV_Msk */
+#else /* !RADIO_DATAWHITEIV_DATAWHITEIV_Msk */
NRF_RADIO->DATAWHITE = HAL_RADIO_RESET_VALUE_DATAWHITE | iv;
#endif /* !RADIO_DATAWHITEIV_DATAWHITEIV_Msk */
-#else /* !CONFIG_SOC_COMPATIBLE_NRF54LX */
+#else /* !CONFIG_SOC_COMPATIBLE_NRF54LX */
|
Artifacts
Produced during runtime
| Name | Size | |
|---|---|---|
|
compliance.xml
|
4.05 KB |
|