Skip to content

Conversation

tomchy
Copy link
Contributor

@tomchy tomchy commented Sep 25, 2025

Use secondary_app_partition as code partition in Direct XIP without partition manager.
This allows to define slot0/1 partitions independently from the application's code partition.

Ref: NCSDK-35612

@NordicBuilder NordicBuilder added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Sep 25, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Sep 25, 2025

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@047df0c nrfconnect/sdk-zephyr@40af3c9 (main) nrfconnect/[email protected]

All manifest checks OK

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Sep 25, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 1c62196ea86e54c44cd8425255d98a6cf03e727e
zephyr: PR head: 40af3c982b9a743cea2feabe07d3db66337fb68e

more details

sdk-nrf:

PR head: 1c62196ea86e54c44cd8425255d98a6cf03e727e
merge base: e5d6866a602267d5a2c20047be5f47fdbc195851
target head (main): e5d6866a602267d5a2c20047be5f47fdbc195851
Diff

zephyr:

PR head: 40af3c982b9a743cea2feabe07d3db66337fb68e
merge base: 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 (6)
applications
│  ├── nrf_desktop
│  │  ├── configuration
│  │  │  ├── nrf54h20dk_nrf54h20_cpuapp
│  │  │  │  │ app_common.dtsi
│  │  │  ├── nrf54h20dk_nrf54h20_cpurad
│  │  │  │  ├── images
│  │  │  │  │  ├── ipc_radio
│  │  │  │  │  │  │ app.overlay
subsys
│  ├── mcuboot
│  │  │ mcuboot_secondary_app.overlay
west.yml
zephyr
│  ├── boards
│  │  ├── nordic
│  │  │  ├── nrf54h20dk
│  │  │  │  ├── nrf54h20dk_nrf54h20_cpuapp.dts
│  │  │  │  │ nrf54h20dk_nrf54h20_cpurad.dts

Outputs:

Toolchain

Version:
Build docker image:

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

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests
    • ◻️ desktop52_verification
    • ◻️ 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
    • 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-proprietary_esb
    • 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

Copy link

You can find the documentation preview for this PR here.

@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-35612_Dedicated_secondary_app_partition branch from be04de3 to f3c29cb Compare September 26, 2025 09:09
@tomchy tomchy changed the title subsys: mcuboot: Use secondary_app_partition mcuboot: Use secondary_app_partition Sep 26, 2025
@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-35612_Dedicated_secondary_app_partition branch from f3c29cb to e800500 Compare September 26, 2025 15:48
@tomchy tomchy marked this pull request as ready for review September 29, 2025 10:31
@tomchy tomchy requested a review from a team September 29, 2025 10:31
@tomchy tomchy requested review from a team as code owners September 29, 2025 10:31
@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-35612_Dedicated_secondary_app_partition branch 2 times, most recently from 5a22374 to 13e4976 Compare September 30, 2025 08:50
@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-35612_Dedicated_secondary_app_partition branch from 13e4976 to 167986e Compare October 2, 2025 08:54
@NordicBuilder
Copy link
Contributor

Memory footprint analysis revealed the following potential issues

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.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)

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

Use secondary_app_partition as code partition in Direct XIP without
partition manager.
This allows to define slot0/1 partitions independently from the
application's code partition.

Ref: NCSDK-35612

Signed-off-by: Tomasz Chyrowicz <[email protected]>
@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-35612_Dedicated_secondary_app_partition branch from 167986e to 1c62196 Compare October 2, 2025 10:56
@NordicBuilder NordicBuilder removed the DNM label Oct 2, 2025
@tomchy tomchy merged commit 0628208 into nrfconnect:main Oct 2, 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

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.

6 participants