Skip to content

Conversation

@adamkondraciuk
Copy link
Contributor

Some of the tests failed due to the addition of the PM_DEVICE module to the GPIO driver.

@adamkondraciuk adamkondraciuk requested a review from a team as a code owner July 11, 2025 11:50
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jul 11, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jul 11, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 29b72c8bd8e1b1fe4a87ce70069180680ee4b7c4
zephyr: PR head: 63febf7715f82807e3b70eb33b3365a7372412a8

more details

sdk-nrf:

PR head: 29b72c8bd8e1b1fe4a87ce70069180680ee4b7c4
merge base: 6401cfb0cce6e2049fda34bd92fa482864a821e5
target head (main): f43fe358f7ef5dde85dc2c735b35283e88a81d0d
Diff

zephyr:

PR head: 63febf7715f82807e3b70eb33b3365a7372412a8
merge base: f39f7647e2526ffc081096e8b764693d620a93c9
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 (9)
tests
│  ├── benchmarks
│  │  ├── multicore
│  │  │  ├── idle_exmif
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf54h20dk_nrf54h20_cpuapp.overlay
│  │  │  ├── idle_pwm_loopback
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf54h20dk_nrf54h20_cpuapp.overlay
│  │  │  │  ├── remote
│  │  │  │  │  ├── boards
│  │  │  │  │  │  │ nrf54h20dk_nrf54h20_cpurad.overlay
│  │  ├── power_consumption
│  │  │  ├── flash
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf54h20dk_nrf54h20_cpuapp.overlay
west.yml
zephyr
│  ├── boards
│  │  ├── nordic
│  │  │  ├── nrf54h20dk
│  │  │  │  │ nrf54h20dk_nrf54h20_cpuapp.dts
│  ├── samples
│  │  ├── drivers
│  │  │  ├── jesd216
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf54h20dk_nrf54h20_cpuapp.overlay
│  │  │  ├── spi_flash
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf54h20dk_nrf54h20_cpuapp.overlay
│  ├── tests
│  │  ├── drivers
│  │  │  ├── flash
│  │  │  │  ├── common
│  │  │  │  │  ├── boards
│  │  │  │  │  │  │ nrf54h20dk_nrf54h20_cpuapp.overlay

Outputs:

Toolchain

Version:
Build docker image:

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

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests
    • ◻️ test-fw-nrfconnect-rs
    • ◻️ test-fw-nrfconnect-fem
    • ◻️ 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
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • 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-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread-main
    • test-sdk-audio
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

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

@adamkondraciuk adamkondraciuk force-pushed the NRFX-7900-Fix-currennt-consumption-issue branch from bed360f to 7633662 Compare July 11, 2025 11:58
@adamkondraciuk adamkondraciuk force-pushed the NRFX-7900-Fix-currennt-consumption-issue branch from 7633662 to 2f391b6 Compare July 15, 2025 07:47
@zycz
Copy link
Contributor

zycz commented Jul 25, 2025

Shouldn't we change it at board lvl?

@nordic-piks
Copy link
Contributor

Shouldn't we change it at board lvl?

Good point.
@adamkondraciuk Can it be done at board?
All power consumption testing are "unfortunately" done only at sdk-nrf repo, thus zephyr/sdk-zephyr will no be fixed, thus other people/customers may struggle to get current measurements low.

@nordic-piks
Copy link
Contributor

Shouldn't we change it at board lvl?

Good point. @adamkondraciuk Can it be done at board? All power consumption testing are "unfortunately" done only at sdk-nrf repo, thus zephyr/sdk-zephyr will no be fixed, thus other people/customers may struggle to get current measurements low.

So exmif and gpio6 are disabled by default and only tests/samples which actually need them, enables them?

@adamkondraciuk adamkondraciuk force-pushed the NRFX-7900-Fix-currennt-consumption-issue branch from 2f391b6 to 1a74348 Compare August 6, 2025 17:37
@adamkondraciuk adamkondraciuk requested review from a team as code owners August 6, 2025 17:37
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 6, 2025

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@f39f764 nrfconnect/sdk-zephyr@63febf7 (main) nrfconnect/[email protected]

All manifest checks OK

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

@github-actions
Copy link

github-actions bot commented Aug 6, 2025

You can find the documentation preview for this PR here.

@adamkondraciuk adamkondraciuk force-pushed the NRFX-7900-Fix-currennt-consumption-issue branch from 1a74348 to b66dd7b Compare August 6, 2025 20:26
Some of the tests failed due to the addition of the PM_DEVICE module
to the GPIO driver.

Signed-off-by: Adam Kondraciuk <[email protected]>
@adamkondraciuk adamkondraciuk force-pushed the NRFX-7900-Fix-currennt-consumption-issue branch from b66dd7b to 834c477 Compare August 6, 2025 20:30
Update sdk-zephyr to have EXMIF disabled by default.

Signed-off-by: Adam Kondraciuk <[email protected]>
@adamkondraciuk adamkondraciuk force-pushed the NRFX-7900-Fix-currennt-consumption-issue branch from 834c477 to 29b72c8 Compare August 7, 2025 08:25
@NordicBuilder NordicBuilder removed the DNM label Aug 7, 2025
@rlubos
Copy link
Contributor

rlubos commented Aug 7, 2025

@adamkondraciuk This needs to be backported manually, automatic backports don't work with manifest PRs.

@rlubos rlubos merged commit 7a10bf7 into nrfconnect:main Aug 7, 2025
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport v3.1-branch changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants