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

soc: nordic: nrf54h: Change PM_DEVICE_RUNTIME default #87807

Merged
merged 2 commits into from
Apr 3, 2025

Conversation

nordic-krch
Copy link
Collaborator

@nordic-krch nordic-krch commented Mar 28, 2025

nrf54h20 device requires device runtime PM to be enabled when device PM is in use.

When runtime PM is not enabled then CONFIG_PM_DEVICE_SYSTEM_MANAGED is set. It calls suspend/resume actions from idle state with scheduler off and some actions do not support that (spi12x, uart12x).

gmarull
gmarull previously approved these changes Mar 28, 2025
@nordic-krch nordic-krch dismissed stale reviews from gmarull and bjarki-andreasen via 5c3025d April 1, 2025 08:38
nrf54h20 device requires device runtime PM to be enabled when
device PM is in use.

Signed-off-by: Krzysztof Chruściński <krzysztof.chruscinski@nordicsemi.no>
Explicitly disable device runtime PM since this test is expecting
it to be disabled and there are SoC now that turns on it by default.

Signed-off-by: Krzysztof Chruściński <krzysztof.chruscinski@nordicsemi.no>
@nordic-krch
Copy link
Collaborator Author

@bjarki-andreasen @gmarull can you re-approve. I had to disable runtime PM in one test that require non-runtime PM.

@github-actions github-actions bot added the area: UART Universal Asynchronous Receiver-Transmitter label Apr 3, 2025
@github-actions github-actions bot requested a review from dcpleung April 3, 2025 06:24
@kartben kartben merged commit 24c01a9 into zephyrproject-rtos:main Apr 3, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: UART Universal Asynchronous Receiver-Transmitter platform: nRF Nordic nRFx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants