Skip to content

Conversation

@rugeGerritsen
Copy link
Contributor

This commit reduces the time it takes to elevate the priority of a flash operation from 30 ms to 10 ms.

In applications where the radio is heavily used, this improves the responsiveness. Examples of such application types:

  • Bluetooth applications using relatively low connection intervals
  • Bluetooth applications perfoming background scanning
  • 802.15.4 applications
  • Bluetooth LE audio applications

For applications where this is not the case, it is unlikely that any behavior change will be seen.

This change also prepares the driver for fixing a bug in the scheduler where the timeslot request timed out before the specified timeout.

This commit reduces the time it takes to elevate the priority
of a flash operation from 30 ms to 10 ms.

In applications where the radio is heavily used, this improves
the responsiveness. Examples of such application types:
 - Bluetooth applications using relatively low connection intervals
 - Bluetooth applications perfoming background scanning
 - 802.15.4 applications
 - Bluetooth LE audio applications

For applications where this is not the case, it is unlikely that
any behavior change will be seen.

This change also prepares the driver for fixing a bug in the scheduler
where the timeslot request timed out before the specified timeout.

Signed-off-by: Rubin Gerritsen <[email protected]>
Copilot AI review requested due to automatic review settings November 13, 2025 14:32
@rugeGerritsen rugeGerritsen requested review from a team as code owners November 13, 2025 14:32
@NordicBuilder NordicBuilder added the doc-required PR must not be merged without tech writer approval. label Nov 13, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR reduces the default timeout for normal priority MPSL flash operations from 30ms to 10ms to improve responsiveness in radio-intensive applications. The change affects applications using low connection intervals, background scanning, 802.15.4, or Bluetooth LE audio by reducing the time before flash operations are elevated to higher priority.

Key Changes

  • Reduced default timeout from 30000µs to 10000µs in the flash sync MPSL Kconfig
  • Updated release notes to document the default timeout change

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
drivers/mpsl/flash_sync/Kconfig Changed default timeout value from 30ms to 10ms
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst Added release note documenting the timeout reduction

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@NordicBuilder NordicBuilder requested review from a team November 13, 2025 14:33
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 13, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 1bddc4a899472791a23bbd6e3db04214b36f9e63

more details

sdk-nrf:

PR head: 1bddc4a899472791a23bbd6e3db04214b36f9e63
merge base: d8494e3b06d938e6eeac65fb1508c27691512908
target head (main): 892fa5af165bfd21430ac04d91fdec731fa893d2
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)
doc
│  ├── nrf
│  │  ├── releases_and_maturity
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst
drivers
│  ├── mpsl
│  │  ├── flash_sync
│  │  │  │ Kconfig

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
    • sdk-nrf test count: 1087
  • ✅ Integration tests
    • ✅ test_ble_nrf_config
    • ✅ test-fw-nrfconnect-ble_samples
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-fw-nrfconnect-rs
    • ✅ test-fw-nrfconnect-fem
    • ✅ test-fw-nrfconnect-thread-main
    • ✅ test-sdk-find-my
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • 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-tfm
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • 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

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-25569/nrf/releases_and_maturity/releases/release-notes-changelog.html

@Tschet1 Tschet1 self-requested a review November 18, 2025 09:28
@rugeGerritsen rugeGerritsen removed the DNM label Nov 18, 2025
@rugeGerritsen rugeGerritsen merged commit aa9cd36 into nrfconnect:main Nov 18, 2025
25 of 26 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.

6 participants