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

boards: nordic: nrf54h20: disable sec ipc #87460

Merged

Conversation

bjarki-andreasen
Copy link
Collaborator

The secure domain IPC node statuses are being used to generate UICRs which tell the secure domain to attempt connecting to the app and radio cores. secure domain should only be enabled if used, so the nodes should default to disabled.

Without this change, the H20 is unable to go to sleep since secure domain stays awake, persistanly trying to connect to both cores.

The secure domain IPC node statuses are being used to generate UICRs
which tell the secure domain to attempt connecting to the app and
radio cores. secure domain should only be enabled if used, so the
nodes should default to disabled.

Without this change, the H20 is unable to go to sleep since secure
domain stays awake, persistanly trying to connect to both cores.

Signed-off-by: Bjarki Arge Andreasen <bjarki.andreasen@nordicsemi.no>
@zephyrbot zephyrbot added the platform: nRF Nordic nRFx label Mar 21, 2025
@bjarki-andreasen bjarki-andreasen added Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. and removed Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. labels Mar 21, 2025
Copy link
Collaborator

@ppryga-nordic ppryga-nordic left a comment

Choose a reason for hiding this comment

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

LGTM

@kartben kartben merged commit 5386a64 into zephyrproject-rtos:main Mar 27, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: nRF Nordic nRFx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants