Skip to content

Conversation

@Mirlenko
Copy link

@Mirlenko Mirlenko commented Jul 7, 2025

…ult for BLE samples

At the moment these configs should be selected explicitly for each sample. Apply them by default as these configs enable the clock for radio (network) core during application boot time and requests power for radio (network) core. Once powered, radio (network) core begins executing instructions.

…ult for BLE samples

At the moment these configs should be selected explicitly for each sample.
Apply them by default as these configs enable the clock for radio (network) core
during application boot time and requests power for radio (network) core.
Once powered, radio (network) core begins executing instructions.

Signed-off-by: Mirlenko, Aleksandr <[email protected]>
config SOC_NRF53_CPUNET_ENABLE
bool "NRF53 Network MCU is enabled at boot time"
default y if NRF_802154_SER_HOST
default y if NRF_802154_SER_HOST || BT
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rather than no-up here, put these in nrf/soc/nordic/nrf5* folders (can just make a Kconfig.defconfig and put e.g.

config SOC_NRF53_CPUNET_ENABLE
	default y if BT

)

config SOC_NRF53_CPUNET_ENABLE
bool "NRF53 Network MCU is enabled at boot time"
default y if NRF_802154_SER_HOST
default y if NRF_802154_SER_HOST || BT
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would assume this change is not needed? We previously didn't need to set SOC_NRF53_CPUNET_ENABLE for Bluetooth applications

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had a talk with @SebastianBoe , and he suggested to keep SOC_NRF53_CPUNET_ENABLE and SOC_NRF54H20_CPURAD_ENABLE consistent.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we are observing that 54H needs to set this for BT applications, but 53 for some reason does not, then
I would also support having 54H copy whatever 53 is doing at the moment.

As long as there is no unnecessary divergence between 53 and 54 I am happy :)

@Mirlenko Mirlenko closed this Jul 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants