modules: hal_nordic: nrf_802154: CCA threshold config in dBm + nRF54L15 clock latency #90795
Conversation
|
The following west manifest projects have changed revision in this Pull Request:
✅ All manifest checks OK Note: This message is automatically posted and updated by the Manifest GitHub Action. |
github-actions
bot
added
manifest
manifest-hal_nordic
DNM (manifest)
e-rk
changed the title
github-actions
bot
requested review from
jaz1-nordic,
kl-cruz,
magp-nordic,
masz-nordic and
nika-nordic
| @@ -18,7 +18,11 @@ static bool hfclk_is_running; | |||
|
|
|||
| void nrf_802154_clock_init(void) | |||
| { | |||
| /* Intentionally empty. */ | |||
| #ifdef NRF54L_SERIES | |||
| uint32_t clock_latency_us = z_nrf_clock_bt_ctlr_hf_get_startup_time_us(); | |||
There was a problem hiding this comment.
I'm just curious. Is there a reason to not do
#if DT_NODE_EXISTS(DT_NODELABEL(hfxo))
uint32_t clock_latency_us = DT_PROP(DT_NODELABEL(hfxo), startup_time_us);
nrf_802154_clock_hfclk_latency_set(clock_latency_us);
#endif?
If you prefer to call z_nrf_clock_bt_ctlr_hf_get_startup_time_us, note it is wrapped with
#if DT_NODE_EXISTS(DT_NODELABEL(hfxo))
https://github.com/zephyrproject-rtos/zephyr/blob/main/drivers/clock_control/clock_control_nrf.c#L440
There was a problem hiding this comment.
The property is marked as required on nordic,nrf54l-hfxo binding. If there is no such property, the build will fail due to missing property.
github-actions
bot
removed
the
DNM (manifest)
|
Blocked by: #92009 |
The nrf-802154 component of hal_nordic is updated. The CCA threshold configuration is now done in dBm units, instead of hardware-dependent scale. Fixed a bug, where the CONFIG_NRF_802154_CCA_ED_THRESHOLD units were different from the NRF_802154_CCA_ED_THRESHOLD units. Now all configuration is done in dBm. Signed-off-by: Rafał Kuźnia <[email protected]>
The HFXO used on nRF54L15 DK has the typical startup latency of 854us. Signed-off-by: Rafał Kuźnia <[email protected]>
The clock startup latency is now obtained from the clock driver and the nrf-802154 is now set to a value that matches hardware capabilities. Signed-off-by: Rafał Kuźnia <[email protected]>
|
|
Rebased after the littlefs twister test fix was merged to main. |
The nrf-802154 component of hal_nordic is updated. The CCA threshold configuration is now done in dBm units, instead of hardware-dependent scale.
Fixed a bug, where the CONFIG_NRF_802154_CCA_ED_THRESHOLD units were different from the NRF_802154_CCA_ED_THRESHOLD units. Now all configuration is done in dBm.
Addidtionally, added clock latency configuration and a more optimized HFXO startup time for nRF54L15 DK.