Skip to content

samples/dfu/firmware_loader: common partitioning of nrf54lm20dongle#28154

Merged
nvlsianpu merged 1 commit into
nrfconnect:mainfrom
nvlsianpu:nrf54lm20dongle_comon_part
Apr 24, 2026
Merged

samples/dfu/firmware_loader: common partitioning of nrf54lm20dongle#28154
nvlsianpu merged 1 commit into
nrfconnect:mainfrom
nvlsianpu:nrf54lm20dongle_comon_part

Conversation

@nvlsianpu
Copy link
Copy Markdown
Contributor

@nvlsianpu nvlsianpu commented Apr 14, 2026

Added DTS overlay for partitions on nRF54LM20dongle.
This is for firmware loader - slot0_parttion spans till end of RRAM.
MCUboot Is (and must use same partitioning).
An application build must keep the same start address but
can downsize the partition.

@NordicBuilder
Copy link
Copy Markdown
Contributor

NordicBuilder commented Apr 14, 2026

CI Information

To view the history of this post, click the 'edited' button above
Build number: 5

Inputs:

Sources:

sdk-nrf: PR head: d46f27a764efe0e3327a220dba0d23b4d3928bba

more details

sdk-nrf:

PR head: d46f27a764efe0e3327a220dba0d23b4d3928bba
merge base: 2f84e6146f57ef6227f9edc068ecab6867c23273
target head (main): c8ec6dd3723bb32ed04747fb4dff697577742179
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)
samples
│  ├── dfu
│  │  ├── fw_loader
│  │  │  ├── ble_mcumgr
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf54lm20dongle_nrf54lm20b_cpuapp.overlay
│  │  │  ├── usb_mcumgr
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf54lm20dongle_nrf54lm20b_cpuapp.overlay

Outputs:

Toolchain

Version: 53ac68dd62
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:53ac68dd62_5ea73affbf

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 4
  • ✅ Integration tests
    • ✅ test-sdk-mcuboot
    • ✅ test-sdk-dfu
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • 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-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • 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-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread-main
    • test-low-level
    • test-sdk-find-my
    • test-sdk-wifi

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

Comment on lines +14 to +15
/delete-node/ &boot_partition;
/delete-node/ &slot0_partition;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
/delete-node/ &boot_partition;
/delete-node/ &slot0_partition;

Comment on lines +20 to +22
reg = <0x0 DT_SIZE_K(2036)>;
ranges = <0x0 0x0 DT_SIZE_K(2036)>;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
reg = <0x0 DT_SIZE_K(2036)>;
ranges = <0x0 0x0 DT_SIZE_K(2036)>;

Comment on lines +24 to +40
compatible = "fixed-partitions";
#address-cells = <1>;
#size-cells = <1>;

/* Matches board layout */
boot_partition: partition@0 {
label = "mcuboot";
reg = <0x0 DT_SIZE_K(24)>;
};

/* Matches board layout */
slot1_partition: partition@6000 {
label = "image-1";
reg = <0x6000 DT_SIZE_K(116)>;
};

/* Start matches board layout */
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
compatible = "fixed-partitions";
#address-cells = <1>;
#size-cells = <1>;
/* Matches board layout */
boot_partition: partition@0 {
label = "mcuboot";
reg = <0x0 DT_SIZE_K(24)>;
};
/* Matches board layout */
slot1_partition: partition@6000 {
label = "image-1";
reg = <0x6000 DT_SIZE_K(116)>;
};
/* Start matches board layout */

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/delete-node/ &boot_partition;
/delete-node/ &slot0_partition;
/delete-node/ &slot1_partition;
/delete-node/ &storage_partition;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this file needs to be in the boards folder of both samples, samples must be self contained and allow copying out for vscode usability

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ok, duplicated and moved accordingly.

Copy link
Copy Markdown
Contributor

@Damian-Nordic Damian-Nordic left a comment

Choose a reason for hiding this comment

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

Found a few minor spelling and grammar issues but LGTM overall.

Comment on lines +23 to +27
/* This is nRF54lm20dongle_nrf54lm20b_cpuapp peculiar partitioning for MCUboot and firmware loader
* slot0_partition spans RRAM with can be update by the firmware loader and booted by MCUboot.
* beginning of the slot0_partition must match with the beginning of the slot0_partition of the user
* application. Althought Application can be build for smaler partition and live some space for
* other purposes like a storage partition.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
/* This is nRF54lm20dongle_nrf54lm20b_cpuapp peculiar partitioning for MCUboot and firmware loader
* slot0_partition spans RRAM with can be update by the firmware loader and booted by MCUboot.
* beginning of the slot0_partition must match with the beginning of the slot0_partition of the user
* application. Althought Application can be build for smaler partition and live some space for
* other purposes like a storage partition.
/* This is nRF54lm20dongle_nrf54lm20b_cpuapp-specific partitioning for MCUboot and firmware loader.
* slot0_partition defines RRAM region which can be updated by the firmware loader and booted by MCUboot.
* Beginning of the slot0_partition must match the beginning of the slot0_partition of the user
* application, but the application can use a smaller partition and leave some space for
* other purposes like a storage partition.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

OK

@nvlsianpu nvlsianpu force-pushed the nrf54lm20dongle_comon_part branch from 4a4fdfa to b6204f7 Compare April 24, 2026 09:30
@nvlsianpu nvlsianpu force-pushed the nrf54lm20dongle_comon_part branch from b6204f7 to d9bf481 Compare April 24, 2026 09:34
Added DTS overlay for partitions on nRF54LM20dongle.
This is for firmware loader - slot0_parttion spans till end of RRAM.
MCUboot Is (and must use same partitioning).
An application build must keep the same start address but
can downsize the partition.

Signed-off-by: Andrzej Puzdrowski <andrzej.puzdrowski@nordicsemi.no>
@nvlsianpu nvlsianpu force-pushed the nrf54lm20dongle_comon_part branch from d9bf481 to d46f27a Compare April 24, 2026 09:39
@nvlsianpu nvlsianpu merged commit ca38e00 into nrfconnect:main Apr 24, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants