Skip to content

Nrfx 8402 run power consumption in idle cache retained #24761

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Nrfx 8402 run power consumption in idle cache retained #24761

wants to merge 4 commits into from

Conversation

nordic-segl
Copy link
Contributor

No description provided.

@nordic-segl nordic-segl requested review from a team as code owners September 29, 2025 12:55
@NordicBuilder NordicBuilder added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Sep 29, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Sep 29, 2025 •
edited
Loading

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@047df0c (main) nrfconnect/sdk-zephyr#3249 nrfconnect/sdk-zephyr#3249/files

⛔ DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Sep 29, 2025 •
edited
Loading

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 11

Inputs:

Sources:

sdk-nrf: PR head: d93ef412d7028d3af5167417caa8a8378d9adbff
zephyr: PR head: 158be952dde1ab937718872cced8710ebb34cf12

more details

sdk-nrf:

PR head: d93ef412d7028d3af5167417caa8a8378d9adbff
merge base: a14dc406b3f303f64c3d8c4ace21049469c0d8a5
target head (main): a14dc406b3f303f64c3d8c4ace21049469c0d8a5
Diff

zephyr:

PR head: 158be952dde1ab937718872cced8710ebb34cf12
merge base: 318b87179093864727c181d80c06565138a4d071
target head (main): 047df0c5a1ba5a20a2f72ed8bf81dfdb99fcaaa2
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (27) 
tests
│  ├── benchmarks
│  │  ├── multicore
│  │  │  ├── common
│  │  │  │  │ workaround_idle_retained.overlay
│  │  │  ├── idle
│  │  │  │  │ testcase.yaml
│  │  │  ├── idle_flpr
│  │  │  │  ├── prj.conf
│  │  │  │  ├── remote
│  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  ├── boards
│  │  │  │  │  │  │ nrf54h20dk_nrf54h20_cpurad.overlay
│  │  │  │  │  ├── prj.conf
│  │  │  │  │  ├── prj_s2ram.conf
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ main.c
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  ├── sysbuild.cmake
│  │  │  │  │ testcase.yaml
│  │  │  ├── idle_ppr
│  │  │  │  │ testcase.yaml
│  │  │  ├── idle_pwm_led
│  │  │  │  │ testcase.yaml
│  │  │  ├── idle_pwm_loopback
│  │  │  │  │ testcase.yaml
│  │  │  ├── idle_spim_loopback
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── idle_wdt
│  │  │  │  │ testcase.yaml
west.yml
zephyr
│  ├── boards
│  │  ├── nordic
│  │  │  ├── nrf54h20dk
│  │  │  │  ├── nrf54h20dk_nrf54h20_cpuapp.dts
│  │  │  │  │ nrf54h20dk_nrf54h20_cpurad.dts
│  ├── dts
│  │  ├── vendor
│  │  │  ├── nordic
│  │  │  │  │ nrf54h20.dtsi
│  ├── soc
│  │  ├── nordic
│  │  │  ├── nrf54h
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig.defconfig.nrf54h20_cpuapp
│  │  │  │  ├── Kconfig.defconfig.nrf54h20_cpurad
│  │  │  │  ├── power.c
│  │  │  │  ├── soc.c
│  │  │  │  │ soc.h

Outputs:

Toolchain

Version: a7529a11f4
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:a7529a11f4_bba2ea5f2e

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ❌ Build twister
    • sdk-nrf test count: 22
    • sdk-zephyr test count: 837
  • ❌ Integration tests
    • ❌ test-sdk-audio
    • ❌ test-fw-nrfconnect-chip
    • ❌ test-fw-nrfconnect-nfc
    • ❌ test-fw-nrfconnect-nrf-iot_cloud
    • ❌ test-fw-nrfconnect-rs
    • ❌ test-fw-nrfconnect-fem
    • ❌ test-fw-nrfconnect-thread-main
    • ❌ test-sdk-find-my
    • ❌ test-low-level
    • ❌ test-sdk-mcuboot
    • ❌ test-sdk-dfu
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-tfm
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@github-actions GitHub Actions
Copy link

You can find the documentation preview for this PR here.

@nordic-segl nordic-segl force-pushed the NRFX-8402_Run-power-consumption-in-idle_cache_retained branch from 7e19251 to 576338d Compare September 30, 2025 07:07
@nordic-segl nordic-segl force-pushed the NRFX-8402_Run-power-consumption-in-idle_cache_retained branch from 576338d to cca170c Compare September 30, 2025 08:55
@nordic-segl nordic-segl requested a review from a team as a code owner September 30, 2025 08:55
@nordic-segl nordic-segl force-pushed the NRFX-8402_Run-power-consumption-in-idle_cache_retained branch from cca170c to 84910af Compare September 30, 2025 10:13
nordicjm
nordicjm requested changes Sep 30, 2025
View reviewed changes
tests/benchmarks/multicore/common/workaround_idle_retained.overlay
idle_cache_disabled: idle_cache_disabled {
min-residency-us = <790000>;
};
s2ram: s2ram {
Copy link
Contributor

Choose a reason for hiding this comment

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

newline missing line 17

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed, thanks.

@nordic-segl nordic-segl marked this pull request as draft September 30, 2025 11:53
@nordic-segl nordic-segl force-pushed the NRFX-8402_Run-power-consumption-in-idle_cache_retained branch 5 times, most recently from 4e48b35 to 89a27cf Compare October 2, 2025 06:46
@nordic-segl nordic-segl marked this pull request as ready for review October 2, 2025 06:50
@nordic-segl
manifest: update sdk-zephyr
3b4b043 
Include:
soc: nordic: nrf54h: Implement idle with cache retained state
sdk-zepjyr PR #: 3249
zephyr upstream PR #: 95474

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl
tests: benchmarks: multicore: Run tests on IDLE with cache retained
d6a5186 
Extend tests with configuration that checks driver operation
together with Low Power CPU state IDLE with cache retained.

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl
tests: benchmarks: multicore: idle_flpr: Update the test
8c426cb 
Recently, there were improvements in the idle_ppr test.
Apply similar changes to the idle_flpr test.

Since, FLPR doesn't support GPIO, skip code that indicates
cpu activity.

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl
tests: benchmarks: multicore: idle_spim_loopback: Add boot delay
d93ef41 
Pytest requires the first captured state to be low power state.
Due to internal test framework delays, Pytest collects last ~27 ms
of the idle state. Then MCU wakes up. As a result some Idle tests
started to randomly fail.

Add arbitrary delay at the beginning of main so the first wake up
happens later.

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl nordic-segl force-pushed the NRFX-8402_Run-power-consumption-in-idle_cache_retained branch from 89a27cf to d93ef41 Compare October 2, 2025 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nordicjm nordicjm nordicjm requested changes

@nordic-piks nordic-piks nordic-piks approved these changes

@adamkondraciuk adamkondraciuk Awaiting requested review from adamkondraciuk adamkondraciuk is a code owner

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. DNM manifest manifest-zephyr
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nordic-segl @NordicBuilder @nordicjm @nordic-piks