Skip to content

manifest: sdk-mcuboot: [nrf noup] bootutil: Use correct set of KMU key slots #24511

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 3 commits into
base: main
Choose a base branch
from
Open

manifest: sdk-mcuboot: [nrf noup] bootutil: Use correct set of KMU key slots #24511

wants to merge 3 commits into from
+41 −6

Conversation

NordicBuilder
Copy link
Contributor

@NordicBuilder NordicBuilder commented Sep 12, 2025 •
edited by gchwier
Loading

Automatically created by action-manifest-pr GH action from PR: nrfconnect/sdk-mcuboot#531

test_sdk_mcuboot: grch-correct-kmu-slot

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

NordicBuilder commented Sep 12, 2025 •
edited
Loading

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

Name Old Revision New Revision Diff
mcuboot nrfconnect/sdk-mcuboot@f34b20b nrfconnect/sdk-mcuboot#531 nrfconnect/sdk-mcuboot#531/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 Sep 12, 2025 •
edited
Loading

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 5cea07c176bb0d59dd11295c23cd899285e03edb
mcuboot: PR head: cce391ffecd85b6886388ac3658997d4aaa0b71d

more details

sdk-nrf:

PR head: 5cea07c176bb0d59dd11295c23cd899285e03edb
merge base: 81daeb705c4cfadfd8c890c802a9d30ce15323b7
target head (main): abf4bbb51c4c73622cb7cf7e6da2959cd5f21ab2
Diff

mcuboot:

PR head: cce391ffecd85b6886388ac3658997d4aaa0b71d
merge base: f34b20b2b159e4f1dc30dcf20efefb31701c7b78
target head (main): 26e93ba6a1317a42b946757cafc761e4900ff495
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 (8) 
bootloader
│  ├── mcuboot
│  │  ├── boot
│  │  │  ├── bootutil
│  │  │  │  ├── src
│  │  │  │  │  │ ed25519_psa.c
│  │  │  ├── zephyr
│  │  │  │  │ Kconfig
cmake
│  ├── sysbuild
│  │  │ generate_default_keyfile.cmake
sysbuild
│  ├── CMakeLists.txt
│  │ Kconfig.mcuboot
tests
│  ├── subsys
│  │  ├── kmu
│  │  │  ├── pytest
│  │  │  │  ├── common.py
│  │  │  │  │ test_kmu_provision.py
west.yml

Outputs:

Toolchain

Version:
Build docker image:

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

  • 🟠 Toolchain
  • 🟠 Build twister
  • 🟠 Integration tests
    • 🟠 test-sdk-audio
    • 🟠 desktop52_verification
    • 🟠 test-fw-nrfconnect-apps
    • 🟠 test_ble_nrf_config
    • 🟠 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_serial_lte_modem
    • 🟠 test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • 🟠 test-fw-nrfconnect-nrf-iot_samples
    • 🟠 test-fw-nrfconnect-nrf-iot_lwm2m
    • 🟠 test-fw-nrfconnect-nrf-iot_thingy91
    • 🟠 test-fw-nrfconnect-nrf_crypto
    • 🟠 test-fw-nrfconnect-proprietary_esb
    • 🟠 test-fw-nrfconnect-rpc
    • 🟠 test-fw-nrfconnect-rs
    • 🟠 test-fw-nrfconnect-fem
    • 🟠 test-fw-nrfconnect-tfm
    • 🟠 test-fw-nrfconnect-thread-main
    • 🟠 test-sdk-find-my
    • 🟠 test-fw-nrfconnect-nrf_lrcs_mosh
    • 🟠 test-fw-nrfconnect-nrf_lrcs_positioning
    • 🟠 test-sdk-wifi
    • 🟠 test-low-level
    • 🟠 test-sdk-pmic-samples
    • 🟠 test-sdk-mcuboot
    • 🟠 test-sdk-dfu
    • 🟠 test-fw-nrfconnect-ps-main
    • 🟠 test-secdom-samples-public

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

@NordicBuilder NordicBuilder force-pushed the auto-manifest-sdk-mcuboot-531 branch from 4f0165b to 2b810a7 Compare September 12, 2025 10:16
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 12, 2025 •
edited
Loading

You can find the documentation preview for this PR here.

@NordicBuilder NordicBuilder mentioned this pull request Sep 24, 2025
Open
@NordicBuilder NordicBuilder force-pushed the auto-manifest-sdk-mcuboot-531 branch from 2b810a7 to 162b79f Compare October 1, 2025 08:14
@nordicjm nordicjm requested a review from a team as a code owner October 1, 2025 08:16
@nordicjm nordicjm requested a review from frkv October 1, 2025 08:37
@NordicBuilder
Copy link
Contributor Author

NordicBuilder commented Oct 1, 2025 •
edited
Loading

Memory footprint analysis revealed the following potential issues

applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 9102[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 5858[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.nrf_desktop.zrelease[[email protected]/nrf54h20/cpuapp]: RAM size increased by 23657[B] in comparison to the main[6df6566] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[[email protected]/nrf54h20/cpuapp]: ROM size increased by 12044[B] in comparison to the main[6df6566] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 12744[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 9492[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.nrf_desktop.zdebug.uart[[email protected]/nrf54h20/cpuapp]: RAM size increased by 23661[B] in comparison to the main[6df6566] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug.uart[[email protected]/nrf54h20/cpuapp]: ROM size increased by 13752[B] in comparison to the main[6df6566] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)

Note: This message is automatically posted and updated by the CI (latest/sdk-nrf/PR-24511/5)

@gchwier gchwier requested a review from a team as a code owner October 1, 2025 14:07
nvlsianpu
nvlsianpu reviewed Oct 6, 2025
View reviewed changes
Copy link
Contributor

@nvlsianpu nvlsianpu left a comment

Choose a reason for hiding this comment

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

LGTM

@nordicjm
Copy link
Contributor

nordicjm commented Oct 7, 2025

LGTM

Can you approve it?

@NordicBuilder
manifest: Update sdk-mcuboot revision (auto-manifest PR)
8a8adea 
Automatically created by Github Action

Signed-off-by: Nordic Builder <[email protected]>
@NordicBuilder NordicBuilder force-pushed the auto-manifest-sdk-mcuboot-531 branch from 95558d6 to 983df4e Compare October 7, 2025 06:20
nordicjm and others added 2 commits October 7, 2025 08:40
@nordicjm
sysbuild: Add option to use original KMU keyslots for MCUboot
c439423 
Adds a sysbuild Kconfig option for this, sets it in the target
MCUboot image and updates the keyfile generation file to output
for the correct slot

Signed-off-by: Jamie McCrae <[email protected]>
@gchwier @nordicjm
tests: Use correct KMU keyslot during provisioning
5cea07c 
Updated tests to use proper keyname for MCUboot

Signed-off-by: Grzegorz Chwierut <[email protected]>
@nordicjm nordicjm force-pushed the auto-manifest-sdk-mcuboot-531 branch from 983df4e to 5cea07c Compare October 7, 2025 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nvlsianpu nvlsianpu nvlsianpu left review comments

@de-nordic de-nordic de-nordic approved these changes

@frkv frkv Awaiting requested review from frkv

At least 2 approving reviews are required 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-mcuboot
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@NordicBuilder @nordicjm @nvlsianpu @de-nordic @gchwier