Skip to content
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

mpsl: Update dependencies for 54L bsim model #1452

Closed
wants to merge 1 commit into from

Conversation

hermabe
Copy link
Member

@hermabe hermabe commented Sep 2, 2024

To allow building for the 54L bsim board

To allow building for the 54L bsim board

Signed-off-by: Herman Berget <[email protected]>
@@ -11,7 +11,7 @@ config MPSL
select NRF_802154_MULTIPROTOCOL_SUPPORT if NRF_802154_RADIO_DRIVER
select MULTITHREADING_LOCK
depends on (SOC_SERIES_BSIM_NRFXX || SOC_SERIES_NRF52X || SOC_NRF5340_CPUNET ||\
SOC_NRF54H20_CPURAD || SOC_SERIES_NRF54LX)
SOC_NRF54H20_CPURAD || SOC_COMPATIBLE_NRF54LX)
Copy link
Contributor

@rugeGerritsen rugeGerritsen Sep 2, 2024

Choose a reason for hiding this comment

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

This change requires zephyrproject-rtos/zephyr@fb774fe

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, didnt realize this wasnt here yet. Will have to fromtree that, given that the upmerge is delayed.

Some more changes are needed in the bsim board to run sdc anyway (Related to clock, psa-rng, and bt-hci dts).

Copy link
Member Author

Choose a reason for hiding this comment

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

Seems all the dts issues have already been fixed upstream

@hermabe hermabe added the DNM Do not merge label Sep 2, 2024
@hermabe
Copy link
Member Author

hermabe commented Sep 3, 2024

SOC_SERIES_BSIM_NRFXX is already there, no need for this change.

@hermabe hermabe closed this Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DNM Do not merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants