Skip to content

Conversation

nordic-segl
Copy link
Contributor

Add missing property to spi130 node.
Add zephyr,pm-device-runtime-auto since the sample enables PM.

Add missing property to spi130 node.
Add `zephyr,pm-device-runtime-auto` since the sample enables PM.

Signed-off-by: Sebastian Głąb <[email protected]>
Copy link

sonarqubecloud bot commented Oct 6, 2025

@nordic-segl
Copy link
Contributor Author

Sample works in internal CI

12:21:49  INFO    - 1/1 [email protected]/nrf54h20/cpuapp samples/boards/nordic/spis_wakeup/sample.drivers.spis.wakeup PASSED (device: 1051111246, 9.491s <zephyr>)

@bjarki-andreasen
Copy link
Contributor

bjarki-andreasen commented Oct 7, 2025

config PM_DEVICE_RUNTIME_DEFAULT_ENABLE
default y if PM_DEVICE_RUNTIME
enables runtime for every peripheral on every soc by default, so it should not make any difference to add the devicetree flag.

@nordic-segl
Copy link
Contributor Author

config PM_DEVICE_RUNTIME_DEFAULT_ENABLE
default y if PM_DEVICE_RUNTIME

enables runtime for every peripheral on every soc by default, so it should not make any difference to add the devicetree flag.

I see it was added in this PR
#95668

However, it's not present in sdk-zephyr
https://github.com/nrfconnect/sdk-zephyr/blob/main/soc/nordic/common/Kconfig.defconfig

Do You know when this change will be added in NCS?

@nordic-segl nordic-segl added the DNM This PR should not be merged (Do Not Merge) label Oct 7, 2025
@bjarki-andreasen
Copy link
Contributor

bjarki-andreasen commented Oct 7, 2025

That is being worked on here nrfconnect/sdk-zephyr#3314 so hopefully soon, though until then this PR has no negative impact so we can merge this now no problem

@nordic-segl nordic-segl marked this pull request as draft October 7, 2025 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Samples Samples DNM This PR should not be merged (Do Not Merge) platform: nRF Nordic nRFx size: XS A PR changing only a single line of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants