manifest: sdk-zephyr: [fromlist] poweroff: nordic: streamline RAM retention strategy #19140

NordicBuilder · 2024-11-28T14:41:00Z

Automatically created by action-manifest-pr GH action from PR: nrfconnect/sdk-zephyr#2313

NordicBuilder · 2024-11-28T14:41:35Z

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

Name	Old Revision	New Revision	Diff
zephyr	nrfconnect/sdk-zephyr@`6baae08`	nrfconnect/sdk-zephyr#2313	nrfconnect/sdk-zephyr#2313/files

⛔ DNM label due to: 1 project with PR revision

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

NordicBuilder · 2024-11-28T14:57:07Z

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 25

Inputs:

Sources:

sdk-nrf: PR head: c53676dfebfc5b053cb39b093582526a1be2d94b
zephyr: PR head: 354a7c9a5661b41efe1fb0e81ebe39200fbae423

more details

sdk-nrf:

PR head: c53676dfebfc5b053cb39b093582526a1be2d94b
merge base: a5e4efe25d070995701534193bc786779f34d9fd
target head (main): 584e1c85cd2f058ee69b802bfd913b823e315b2c
Diff

zephyr:

PR head: 354a7c9a5661b41efe1fb0e81ebe39200fbae423
merge base: 581e63819936a0e8ae6040a6cb23ec091f5b02d7
target head (main): c21cc96eb32aa8ea89144f02decfe51104fe401e
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 (29)

samples
│  ├── bluetooth
│  │  ├── peripheral_power_profiling
│  │  │  ├── src
│  │  │  │  │ main.c
│  ├── nfc
│  │  ├── system_off
│  │  │  ├── src
│  │  │  │  │ main.c
tests
│  ├── benchmarks
│  │  ├── current_consumption
│  │  │  ├── system_off
│  │  │  │  ├── boards
│  │  │  │  │  ├── nrf52840dk_nrf52840.overlay
│  │  │  │  │  ├── nrf52dk_nrf52832.overlay
│  │  │  │  │  ├── nrf54l15dk_nrf54l05_cpuapp.overlay
│  │  │  │  │  ├── nrf54l15dk_nrf54l10_cpuapp.overlay
│  │  │  │  │  │ nrf54l15dk_nrf54l15_cpuapp.overlay
│  │  │  │  │ testcase.yaml
west.yml
zephyr
│  ├── drivers
│  │  ├── retained_mem
│  │  │  ├── Kconfig.nrf
│  │  │  │ retained_mem_nrf_ram_ctrl.c
│  ├── include
│  │  ├── zephyr
│  │  │  ├── drivers
│  │  │  │  ├── retained_mem
│  │  │  │  │  │ nrf_retained_mem.h
│  ├── modules
│  │  ├── hal_nordic
│  │  │  ├── nrfx
│  │  │  │  │ CMakeLists.txt
│  ├── samples
│  │  ├── boards
│  │  │  ├── nordic
│  │  │  │  ├── system_off
│  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  ├── Kconfig
│  │  │  │  │  ├── README.rst
│  │  │  │  │  ├── boards
│  │  │  │  │  │  ├── nrf52840dk_nrf52840.overlay
│  │  │  │  │  │  ├── nrf52dk_nrf52832.overlay
│  │  │  │  │  │  ├── nrf54l15dk_nrf54l05_cpuapp.overlay
│  │  │  │  │  │  ├── nrf54l15dk_nrf54l10_cpuapp.overlay
│  │  │  │  │  │  │ nrf54l15dk_nrf54l15_cpuapp.overlay
│  │  │  │  │  ├── sample.yaml
│  │  │  │  │  ├── src
│  │  │  │  │  │  ├── main.c
│  │  │  │  │  │  │ retained.c
│  ├── soc
│  │  ├── nordic
│  │  │  ├── common
│  │  │  │  ├── Kconfig
│  │  │  │  │ poweroff.c
│  │  │  ├── nrf52
│  │  │  │  │ Kconfig
│  │  │  ├── nrf54h
│  │  │  │  │ Kconfig
│  │  │  ├── nrf54l
│  │  │  │  │ Kconfig

Outputs:

Toolchain

Version: b77d8c1312
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:b77d8c1312_912848a074

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

◻️ Toolchain - Skipped: existing toolchain is used
✅ Build twister
- sdk-nrf test count: 1255
- sdk-zephyr test count: 5449
❌ Integration tests
- ✅ test-sdk-audio
- ✅ test-fw-nrfconnect-ble_samples
- ✅ test-fw-nrfconnect-chip
- ✅ test-fw-nrfconnect-nfc
- ✅ test-fw-nrfconnect-nrf-iot_thingy91
- ❌ test-fw-nrfconnect-rs
- ❌ test-fw-nrfconnect-fem
- ✅ test-fw-nrfconnect-thread
- ✅ test-sdk-find-my
- ❌ test-sdk-sidewalk
- ✅ test-low-level
- ❌ test-sdk-dfu
- ⚠️ test-fw-nrfconnect-nrf-iot_cloud

Disabled integration tests

- desktop52_verification
- doc-internal
- test_ble_nrf_config
- test-fw-nrfconnect-apps
- test-fw-nrfconnect-ble_mesh
- test-fw-nrfconnect-boot
- test-fw-nrfconnect-nrf-iot_libmodem-nrf
- test-fw-nrfconnect-nrf-iot_lwm2m
- test-fw-nrfconnect-nrf-iot_mosh
- test-fw-nrfconnect-nrf-iot_nrf_provisioning
- test-fw-nrfconnect-nrf-iot_positioning
- test-fw-nrfconnect-nrf-iot_samples
- test-fw-nrfconnect-nrf-iot_serial_lte_modem
- test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
- test-fw-nrfconnect-nrf_crypto
- test-fw-nrfconnect-ps
- test-fw-nrfconnect-rpc
- test-fw-nrfconnect-tfm
- test-fw-nrfconnect-zigbee
- test-sdk-mcuboot
- test-sdk-pmic-samples
- test-sdk-wifi
- test-secdom-samples-public

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

NordicBuilder · 2024-11-28T14:58:36Z

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publish GitHub Action.

nika-nordic · 2024-11-28T15:17:41Z

@maje-emb is there a CI check for peripheral_power_profiling power usage, to make sure no regression is introduced?

maje-emb · 2024-12-02T10:28:48Z

@maje-emb is there a CI check for peripheral_power_profiling power usage, to make sure no regression is introduced?

^ @nordic-piks, @nordic-bami Are there such tests?

nordic-segl · 2024-12-02T10:44:41Z

@maje-emb is there a CI check for peripheral_power_profiling power usage, to make sure no regression is introduced?

^ @nordic-piks, @nordic-bami Are there such tests?

I see two failures in nightly run
https://jenkins-ncs.nordicsemi.no/job/latest/job/test-fw-twister-ncs-zephyr/job/twister-power-consumption-testing-customizable/view/change-requests/job/PR-1633/2/
both are on nrf5340dk.

nordic-segl · 2024-12-03T06:07:39Z

Updated results (all pass):
https://jenkins-ncs.nordicsemi.no/job/latest/job/test-fw-twister-ncs-zephyr/job/twister-power-consumption-testing-customizable/job/main/159/

maje-emb

The Peripheral_power_profiling and system_off samples look good.
Thank you for the alignment.

Automatically created by Github Action Signed-off-by: Nordic Builder <[email protected]>

RAM retention is now managed within sys_poweroff() call. Signed-off-by: Nikodem Kastelik <[email protected]>

Zephyr system_off sample changed how RAM retention is handled. Signed-off-by: Nikodem Kastelik <[email protected]>

nordic-segl · 2024-12-04T16:14:30Z

tests/benchmarks/current_consumption/system_off/testcase.yaml

    platform_allow:
      - nrf52840dk/nrf52840
      - nrf52dk/nrf52832
-      - nrf5340dk/nrf5340/cpuapp


nrf5340dk was removed from the platform_allow list. Is this intentional?

yes. nrf_retained "mode" of system_off sample is now deprecated. retained_mem Zephyr driver shall be used instead. Unfortunately, due to bug in nrfx, retained_mem driver is not compatible with nrf5340 and nrf9160. Because of this fact I have removed nrf5340dk from retention variant of this test. It will be restored in NCS 3.0, when bug in nrfx is resolved

NordicBuilder · 2024-12-04T22:23:15Z

Automatically closed by action-manifest-pr GH action

NordicBuilder requested a review from a team as a code owner November 28, 2024 14:41

github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Nov 28, 2024

NordicBuilder added manifest-zephyr DNM labels Nov 28, 2024

nika-nordic requested a review from a team as a code owner November 28, 2024 15:16

nordicjm approved these changes Nov 29, 2024

View reviewed changes

nika-nordic force-pushed the auto-manifest-sdk-zephyr-2313 branch from 31cba16 to 76c370b Compare November 29, 2024 12:06

nika-nordic requested a review from a team as a code owner November 29, 2024 12:06

NordicBuilder force-pushed the auto-manifest-sdk-zephyr-2313 branch from 76c370b to f034de6 Compare November 29, 2024 12:07

nika-nordic force-pushed the auto-manifest-sdk-zephyr-2313 branch from f034de6 to 3d6f294 Compare November 29, 2024 16:02

NordicBuilder force-pushed the auto-manifest-sdk-zephyr-2313 branch from 3d6f294 to eb7f178 Compare November 29, 2024 16:02

nika-nordic force-pushed the auto-manifest-sdk-zephyr-2313 branch from eb7f178 to da52efa Compare December 2, 2024 14:34

NordicBuilder force-pushed the auto-manifest-sdk-zephyr-2313 branch 2 times, most recently from f6de207 to 1fa85f5 Compare December 2, 2024 14:51

nika-nordic force-pushed the auto-manifest-sdk-zephyr-2313 branch from 1fa85f5 to 0d21333 Compare December 2, 2024 17:27

NordicBuilder force-pushed the auto-manifest-sdk-zephyr-2313 branch from 0d21333 to f7517f1 Compare December 2, 2024 17:28

NordicBuilder force-pushed the auto-manifest-sdk-zephyr-2313 branch from f7517f1 to 1d48eae Compare December 3, 2024 07:33

nika-nordic force-pushed the auto-manifest-sdk-zephyr-2313 branch from 1d48eae to 7a6ad49 Compare December 3, 2024 09:43

NordicBuilder force-pushed the auto-manifest-sdk-zephyr-2313 branch from 7a6ad49 to c1fc881 Compare December 3, 2024 09:43

nika-nordic force-pushed the auto-manifest-sdk-zephyr-2313 branch from c1fc881 to d3a3fee Compare December 3, 2024 13:33

NordicBuilder force-pushed the auto-manifest-sdk-zephyr-2313 branch from d3a3fee to d3da76b Compare December 3, 2024 13:33

maje-emb approved these changes Dec 4, 2024

View reviewed changes

nika-nordic force-pushed the auto-manifest-sdk-zephyr-2313 branch from d3da76b to f054022 Compare December 4, 2024 10:04

NordicBuilder force-pushed the auto-manifest-sdk-zephyr-2313 branch from f054022 to fbdabd5 Compare December 4, 2024 10:05

NordicBuilder and others added 2 commits December 4, 2024 13:29

manifest: Update sdk-zephyr revision (auto-manifest PR)

d7a7847

Automatically created by Github Action Signed-off-by: Nordic Builder <[email protected]>

samples: nrf54l: remove direct RAM retention management

51b8722

RAM retention is now managed within sys_poweroff() call. Signed-off-by: Nikodem Kastelik <[email protected]>

NordicBuilder force-pushed the auto-manifest-sdk-zephyr-2313 branch from fbdabd5 to cd2f699 Compare December 4, 2024 13:29

benchmarks: system_off: align to new RAM retention in system_off sample

c53676d

Zephyr system_off sample changed how RAM retention is handled. Signed-off-by: Nikodem Kastelik <[email protected]>

NordicBuilder force-pushed the auto-manifest-sdk-zephyr-2313 branch from cd2f699 to c53676d Compare December 4, 2024 13:33

nordic-segl reviewed Dec 4, 2024

View reviewed changes

NordicBuilder closed this Dec 4, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

manifest: sdk-zephyr: [fromlist] poweroff: nordic: streamline RAM retention strategy #19140

manifest: sdk-zephyr: [fromlist] poweroff: nordic: streamline RAM retention strategy #19140

Uh oh!

NordicBuilder commented Nov 28, 2024

Uh oh!

NordicBuilder commented Nov 28, 2024 •

edited

Loading

Uh oh!

NordicBuilder commented Nov 28, 2024 •

edited

Loading

Uh oh!

NordicBuilder commented Nov 28, 2024

Uh oh!

nika-nordic commented Nov 28, 2024

Uh oh!

maje-emb commented Dec 2, 2024 •

edited

Loading

Uh oh!

nordic-segl commented Dec 2, 2024 •

edited

Loading

Uh oh!

nordic-segl commented Dec 3, 2024 •

edited

Loading

Uh oh!

maje-emb left a comment

Uh oh!

nordic-segl Dec 4, 2024

Uh oh!

nika-nordic Dec 4, 2024

Uh oh!

NordicBuilder commented Dec 4, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

manifest: sdk-zephyr: [fromlist] poweroff: nordic: streamline RAM retention strategy #19140

manifest: sdk-zephyr: [fromlist] poweroff: nordic: streamline RAM retention strategy #19140

Uh oh!

Conversation

NordicBuilder commented Nov 28, 2024

Uh oh!

NordicBuilder commented Nov 28, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

NordicBuilder commented Nov 28, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CI Information

Inputs:

Sources:

Github labels

Outputs:

Toolchain

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

Uh oh!

NordicBuilder commented Nov 28, 2024

Uh oh!

nika-nordic commented Nov 28, 2024

Uh oh!

maje-emb commented Dec 2, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nordic-segl commented Dec 2, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nordic-segl commented Dec 3, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

maje-emb left a comment

Choose a reason for hiding this comment

Uh oh!

nordic-segl Dec 4, 2024

Choose a reason for hiding this comment

Uh oh!

nika-nordic Dec 4, 2024

Choose a reason for hiding this comment

Uh oh!

NordicBuilder commented Dec 4, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

NordicBuilder commented Nov 28, 2024 •

edited

Loading

NordicBuilder commented Nov 28, 2024 •

edited

Loading

maje-emb commented Dec 2, 2024 •

edited

Loading

nordic-segl commented Dec 2, 2024 •

edited

Loading

nordic-segl commented Dec 3, 2024 •

edited

Loading