-
Notifications
You must be signed in to change notification settings - Fork 8.2k
boards: Add nrf53 bsim #30734
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
boards: Add nrf53 bsim #30734
Conversation
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.
You simplify some conditionals similar to:
defined(CONFIG_SOC_SERIES_NRF53X) || defined(CONFIG_SOC_COMPATIBLE_NRF53X_CPUNET)
by adding select SOC_COMPATIBLE_NRF53X_CPUNET to SOC_NRF5340_CPUNET inzephyr/soc/arm/nordic_nrf/nrf53/Kconfig.soc
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.
You may consider adding the remaining IRQ names as well, see nrf5340_network.h.
- IPC
- EGU
|
Is the intent to replace nrf52_bsim? |
@galak. I would assume nrf53_bsim can be used as an addition to nrf52_bsim. The HAL is slightly different between nrf52 and nrf53 series devices. @cvinayak and @carlescufi may comment. |
it would be good to explain what the difference / value is between the two. This is more about a question from a testing/ci point of view. |
Update nrf_hw_models version to include nrf53 support. Signed-off-by: Aurora Sletnes Bjørlo <[email protected]>
This folder will contain common content for both nrf52_bsim and nrf53_cpunet_bsim. Signed-off-by: Aurora Sletnes Bjørlo <[email protected]>
The board models the nrf53840 cpunet. Signed-off-by: Aurora Sletnes Bjørlo <[email protected]>
Ensure correct logging of irq with nrf53. Signed-off-by: Aurora Sletnes Bjørlo <[email protected]>
Select COMPATIBLE_NRF53X_CPUNET in SOC_NRF5340_CPUNET. Also add defined(CONFIG_SOC_SERIES_NRF53X) where we have defined(NRF53_SERIES). Signed-off-by: Aurora Sletnes Bjørlo <[email protected]>
Rename nrf52_bsim to nrf5x_bsim to match with change due to added nrf53_bsim. Signed-off-by: Aurora Sletnes Bjørlo <[email protected]>
…_CPUNET So that the nrfx drivers can be compiled for a simulated nrf53 cpunet target. Signed-off-by: Aurora Sletnes Bjørlo <[email protected]>
Add to the SOC_COMPATIBLE options a common option to choose compatibility with a simulated BabbleSim target. Also choose this option for the simulated targets nrf52_bsim and nrf53_cpunet_bsim. Signed-off-by: Aurora Sletnes Bjørlo <[email protected]>
bb8edac to
e201fb1
Compare
|
This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time. |
This PR adds support for simulating the NRF53 network core. To be able to use bluetooth, you will need #30732.
Note: Bluetooth does not fully work, will need some help debugging this.