-
Notifications
You must be signed in to change notification settings - Fork 614
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: nrf54h20: refactor MRAM board level code #2115
base: main
Are you sure you want to change the base?
Conversation
… DK" This reverts commit 38f2104. Signed-off-by: Gerard Marull-Paretas <[email protected]>
@@ -38,8 +38,6 @@ config NRFS | |||
bool "nRF Services Support" | |||
select NRFS_LOCAL_DOMAIN if (SOC_NRF54H20_CPUAPP || SOC_NRF54H20_CPURAD || SOC_NRF9280_CPUAPP || SOC_NRF9280_CPURAD) | |||
depends on HAS_NRFS | |||
depends on !MISRA_SANE |
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.
Do you know if this is addressed? #1856 (comment)
(if not, maybe it should've been taken upstream)
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.
If that's the case then this needs to be upstreamed.
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.
I've tested with the option and I'm able to compile successfully, maybe it's not a problem anymore?
Turn off suspending MRAM for nRF54H20 SoC. This change is required so sections of code depending on critical timings will not have unacceptable latency. Signed-off-by: Jan Zyczkowski <[email protected]> Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 58284ff) (cherry picked from commit 9b6cae8) (cherry picked from commit 2c2f60d)
f356b66
to
2ec1e6f
Compare
Move code to soc level, also refactor it a bit. This makes sure this setting is applied regardless of the board.