Skip to content

applications: nrf_desktop: Fix B0 partition labels in DTS memory maps - #28987

Merged
rlubos merged 1 commit into
nrfconnect:mainfrom
maje-emb:desktop_fix_b0_labels
May 26, 2026
Merged

applications: nrf_desktop: Fix B0 partition labels in DTS memory maps#28987
rlubos merged 1 commit into
nrfconnect:mainfrom
maje-emb:desktop_fix_b0_labels

Conversation

@maje-emb

Copy link
Copy Markdown
Contributor

After migrating B0 memory maps from PM to DTS, the B0 image partitions only used s0_partition and s1_partition labels.
That broke compatibility with configurations that expect the PM.

@maje-emb maje-emb added this to the 3.4.0 milestone May 25, 2026
@maje-emb
maje-emb requested a review from a team as a code owner May 25, 2026 10:49
@NordicBuilder

NordicBuilder commented May 25, 2026

Copy link
Copy Markdown
Contributor

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: e73b7cb7181d32ee318783ece73f9c2fd6a0599a

more details

sdk-nrf:

PR head: e73b7cb7181d32ee318783ece73f9c2fd6a0599a
merge base: 25a4b6580bea5d28ba15447d02b2778a84b9cfcc
target head (main): e795138a6e5a0743cc27bb1c47e378b8882476f6
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)
applications
│  ├── nrf_desktop
│  │  ├── configuration
│  │  │  ├── nrf52840dk_nrf52840
│  │  │  │  │ memory_map_b0.dtsi
│  │  │  ├── nrf5340dk_nrf5340_cpuapp
│  │  │  │  │ memory_map_b0.dtsi

Outputs:

Toolchain

Version: f0aa129f09
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:f0aa129f09_5ea73affbf

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister - Skipped: Skipping Build & Test as it succeeded in a previous run: 5
  • ✅ Integration tests
    • ✅ desktop52_verification - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-apps_nrfdesktop - Skipped: Job was skipped as it succeeded in a previous run
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • 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-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-wifi
    • test-secdom-samples-public

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

@github-actions

Copy link
Copy Markdown

You can find the documentation preview for this PR here.

@maje-emb
maje-emb force-pushed the desktop_fix_b0_labels branch 2 times, most recently from 4bd7e66 to 342476b Compare May 25, 2026 14:58
Add slot0_partition and slot1_partition as additional labels on
the s0_partition / s1_partition so that the map works with PM.

Ref: NCSDK-39474

Signed-off-by: Marcin Jelinski <marcin.jelinski@nordicsemi.no>
@maje-emb
maje-emb force-pushed the desktop_fix_b0_labels branch from 342476b to e73b7cb Compare May 26, 2026 12:02
@rlubos rlubos closed this May 26, 2026
@rlubos rlubos reopened this May 26, 2026
@rlubos
rlubos merged commit e8df060 into nrfconnect:main May 26, 2026
23 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.

6 participants