-
Notifications
You must be signed in to change notification settings - Fork 8.4k
soc: nordic: nrf54l: Remove selecting a DTS-type option in Kconfig #100518
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.
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
base: main
Are you sure you want to change the base?
Conversation
Remove selecting `CONFIG_HAS_HW_NRF_RADIO_IEEE802154` that should be generated from DTS based on the SoC selection. Signed-off-by: Michał Grochala <michal.grochala@nordicsemi.no>
| select CPU_CORTEX_M33 | ||
| select CPU_CORTEX_M_HAS_DWT | ||
| select CPU_HAS_ICACHE | ||
| select HAS_HW_NRF_RADIO_IEEE802154 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't specific nRF54L SoCs symbols, e.g. SOC_NRF54L05_CPUAPP, select it then? Or is it going to be completely removed from any nRF54L? Or do there targets already have the required nodes in DTS for the symbol to be generated from?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It shouldn't be selected from Kconfig directly as it depends on a DTS property. The dependency is defined here: https://github.com/zephyrproject-rtos/zephyr/blob/main/soc/nordic/common/Kconfig.peripherals#L145
|



Remove selecting
CONFIG_HAS_HW_NRF_RADIO_IEEE802154that should be generated from DTS based on the SoC selection.