Skip to content

manifest: sdk-zephyr: [nrf fromlist] dts: nordic: nrf54h20: Update pm policy values #19087

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
Closed
wants to merge 1 commit into from
Closed

manifest: sdk-zephyr: [nrf fromlist] dts: nordic: nrf54h20: Update pm policy values #19087

wants to merge 1 commit into from

Conversation

@NordicBuilder
Copy link
Contributor

@NordicBuilder NordicBuilder commented Nov 27, 2024

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

@NordicBuilder NordicBuilder requested a review from a team as a code owner November 27, 2024 08:50
@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Nov 27, 2024
@NordicBuilder
Copy link
Contributor Author

NordicBuilder commented Nov 27, 2024 •
edited
Loading

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

Name Old Revision New Revision Diff
zephyr nrfconnect/[email protected] nrfconnect/sdk-zephyr#2302 nrfconnect/sdk-zephyr#2302/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 Author

NordicBuilder commented Nov 27, 2024 •
edited
Loading

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 25d6cd1e91a218dc9d5730889eac4d90d27e09f9
zephyr: PR head: 1312bd8d7c11a8ccb8b15d994ce4aaeffea536a4

more details

sdk-nrf:

PR head: 25d6cd1e91a218dc9d5730889eac4d90d27e09f9
merge base: fe34349d5cbbdd6e415cf72ac7eb72f3847a10a0
target head (main): fe34349d5cbbdd6e415cf72ac7eb72f3847a10a0
Diff

zephyr:

PR head: 1312bd8d7c11a8ccb8b15d994ce4aaeffea536a4
merge base: 0a2e205750dc7eff8767afa543b2fd23a1fbb332
target head (main): 0a2e205750dc7eff8767afa543b2fd23a1fbb332
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 (2) 
west.yml
zephyr
│  ├── dts
│  │  ├── common
│  │  │  ├── nordic
│  │  │  │  │ nrf54h20.dtsi

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: 160
    • sdk-zephyr test count: 5451
  • ✅ Integration tests
    • ✅ test-low-level
    • ✅ test-sdk-dfu
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • 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_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-sdk-audio
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi
    • test-secdom-samples-public

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

@NordicBuilder
Copy link
Contributor Author

NordicBuilder commented Nov 27, 2024

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.

@nordic-segl
Copy link
Contributor

nordic-segl commented Nov 27, 2024 •
edited
Loading

I see that power consumption tests are passing here
https://jenkins-ncs.nordicsemi.no/job/latest/job/test-fw-twister-ncs-zephyr/job/twister-power-consumption-testing-customizable/job/main/151/
However, test variant "idle" is the same as "s2ram" now. Observed current in "idle" is ~0.003 mA:

15:04:51  DEBUG   - PYTEST: DEBUG: Deserializing data (PPK format) from file: /jenkins_cache/test-zephyr-sanitycheck/sdk-nrf/ncs/twister-out/nrf54h20dk_nrf54h20_cpuapp/idle_spim_loopback/benchmarks.multicore.idle_spim_loopback.4_bytes.idle/_results.ppk
15:04:51  DEBUG   - PYTEST: DEBUG: Initialized PostProcess with measurement: Measurement from PPK2: 999936 samples, 2024-11-27 15:04:51.175551
(...)
15:04:51  DEBUG   - PYTEST: INFO: Record #0: record.start_index=0, record.stop_index=5, record.average_current=0.003217, record.below_threshold=True
15:04:51  DEBUG   - PYTEST: INFO: Record #1: record.start_index=6, record.stop_index=56, record.average_current=1.932741, record.below_threshold=False
15:04:51  DEBUG   - PYTEST: INFO: Record #2: record.start_index=57, record.stop_index=80, record.average_current=0.003072, record.below_threshold=True
15:04:51  DEBUG   - PYTEST: INFO: Record #3: record.start_index=81, record.stop_index=131, record.average_current=1.927476, record.below_threshold=False
15:04:51  DEBUG   - PYTEST: INFO: Record #4: record.start_index=132, record.stop_index=155, record.average_current=0.002993, record.below_threshold=True
15:04:51  DEBUG   - PYTEST: INFO: Record #5: record.start_index=156, record.stop_index=206, record.average_current=1.927408, record.below_threshold=False
15:04:51  DEBUG   - PYTEST: INFO: Record #6: record.start_index=207, record.stop_index=230, record.average_current=0.003032, record.below_threshold=True
15:04:51  DEBUG   - PYTEST: INFO: Record #7: record.start_index=231, record.stop_index=281, record.average_current=1.928795, record.below_threshold=False
15:04:51  DEBUG   - PYTEST: INFO: Record #8: record.start_index=282, record.stop_index=305, record.average_current=0.003033, record.below_threshold=True
15:04:51  DEBUG   - PYTEST: INFO: Record #9: record.start_index=306, record.stop_index=356, record.average_current=1.927796, record.below_threshold=False
15:04:51  DEBUG   - PYTEST: INFO: Record #10: record.start_index=357, record.stop_index=380, record.average_current=0.003035, record.below_threshold=True
15:04:51  DEBUG   - PYTEST: INFO: Record #11: record.start_index=381, record.stop_index=431, record.average_current=1.927685, record.below_threshold=False
15:04:51  DEBUG   - PYTEST: INFO: Record #12: record.start_index=432, record.stop_index=455, record.average_current=0.003034, record.below_threshold=True
15:04:51  DEBUG   - PYTEST: INFO: Record #13: record.start_index=456, record.stop_index=498, record.average_current=1.949538, record.below_threshold=False

@nordic-segl
Copy link
Contributor

nordic-segl commented Nov 27, 2024

I believe, power consumption tests could revert Adam's change with

/ {
	cpus {
		power-states {
			s2ram: s2ram {
				min-residency-us = <800000>;
			};
		};
	};
};

However, such code has to be added to >20 tests (I'm too lazy to count them 😄 )

@NordicBuilder @adamkondraciuk
manifest: Update sdk-zephyr revision (auto-manifest PR)
25d6cd1 
Automatically created by Github Action

Signed-off-by: Nordic Builder <[email protected]>
@NordicBuilder NordicBuilder force-pushed the auto-manifest-sdk-zephyr-2302 branch from d901ccc to 598c01a Compare December 6, 2024 08:27
@adamkondraciuk adamkondraciuk force-pushed the auto-manifest-sdk-zephyr-2302 branch from 598c01a to 25d6cd1 Compare December 6, 2024 08:27
@github-actions
Copy link

github-actions bot commented Jan 7, 2025

This pull request has been marked as stale because it has been open (more than) 30 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 7 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Jan 7, 2025
@github-actions github-actions bot closed this Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nordicjm nordicjm nordicjm approved these changes

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 Stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@NordicBuilder @nordic-segl @nordicjm