Skip to content

Enable SOC_NRF54H20_CPURAD_ENABLE and SOC_NRF53_CPUNET_ENABLE by defa… #3027

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Enable SOC_NRF54H20_CPURAD_ENABLE and SOC_NRF53_CPUNET_ENABLE by defa… #3027

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion soc/nordic/nrf53/Kconfig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -208,7 +208,7 @@ config SOC_NRF53_CPUNET_MGMT

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

@nordicjm nordicjm Jul 7, 2025

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

)

Copy link
Contributor

@rugeGerritsen rugeGerritsen Jul 7, 2025

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

@Mirlenko Mirlenko Jul 8, 2025

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

@SebastianBoe SebastianBoe Jul 8, 2025

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 :)

select SOC_NRF53_CPUNET_MGMT
help
This option enables releasing the Network 'force off' signal, which
Expand Down
2 changes: 1 addition & 1 deletion soc/nordic/nrf54h/Kconfig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ config SOC_NRF54H20_TDD_ENABLE

config SOC_NRF54H20_CPURAD_ENABLE
bool "Boot the nRF54H20 Radio core"
default y if NRF_802154_SER_HOST
default y if NRF_802154_SER_HOST || BT
depends on SOC_NRF54H20_CPUAPP
select NRF_IRONSIDE_CPUCONF_SERVICE
select SOC_LATE_INIT_HOOK
Expand Down
Loading