Skip to content

Conversation

@Damian-Nordic
Copy link
Contributor

  1. Enable LTO for MCUBoot on Thingy:53 to fix flash overflow and align with other samples and applications.
  2. Switch to new USB stack to fix warnings about using deprecated functions and configuration.
  3. Revert commit that removes Thingy:53 support.

@Damian-Nordic Damian-Nordic requested a review from a team as a code owner November 20, 2025 22:28
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Nov 20, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 20, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: fed1f58ad1fc3da8abbd732b1b46ffa4f66d798b

more details

sdk-nrf:

PR head: fed1f58ad1fc3da8abbd732b1b46ffa4f66d798b
merge base: 00c893ac44d8d40e762eb117a37994186d6cdf00
target head (main): b708336d5f3b9cc9ba91176e97d975c4295e01ba
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 (5)
doc
│  ├── nrf
│  │  ├── releases_and_maturity
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst
samples
│  ├── bluetooth
│  │  ├── peripheral_status
│  │  │  ├── boards
│  │  │  │  ├── thingy53_nrf5340_cpuapp.conf
│  │  │  │  │ thingy53_nrf5340_cpuapp_ns.conf
│  │  │  ├── sample.yaml
│  │  │  ├── sysbuild
│  │  │  │  ├── mcuboot
│  │  │  │  │  ├── boards
│  │  │  │  │  │  │ thingy53_nrf5340_cpuapp.conf

Outputs:

Toolchain

Version: df3cc9d822
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:df3cc9d822_e595b21c39

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: 3
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-ble_samples
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • 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

github-actions bot commented Nov 20, 2025

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-25709/nrf/releases_and_maturity/releases/release-notes-changelog.html

Copy link
Contributor

@grochu grochu left a comment

Choose a reason for hiding this comment

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

Enable LTO for MCUBoot to fix flash overflow and align with
other samples and applications.

Signed-off-by: Damian Krolik <[email protected]>
Switch to new USB stack to fix warnings about using
deprecated functions and configuration.

Signed-off-by: Damian Krolik <[email protected]>
Thingy:53 is again supported for peripheral_{status,lbs,uart}.

Signed-off-by: Damian Krolik <[email protected]>
@Damian-Nordic Damian-Nordic requested a review from a team as a code owner November 24, 2025 14:23
@Damian-Nordic Damian-Nordic requested a review from grochu November 24, 2025 14:23
@NordicBuilder NordicBuilder added doc-required PR must not be merged without tech writer approval. and removed changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Nov 24, 2025
@rlubos rlubos merged commit 2ab0a19 into nrfconnect:main Nov 25, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-required PR must not be merged without tech writer approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants