Skip to content

Conversation

@martintv
Copy link
Contributor

Since this isn't makes powerconsumption be very high, we can let it only be done when "advanced" bluetotoh features are used.

@martintv martintv requested review from a team as code owners October 31, 2025 12:37
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 31, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 31, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 5de186623ca56e3ef2585e2926f49534ad6fe6ea

more details

sdk-nrf:

PR head: 5de186623ca56e3ef2585e2926f49534ad6fe6ea
merge base: ca6603db120f1098438bbcb4ff5e17aca37b5665
target head (main): 340b62559c443699988e9f76ca0208232197b802
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)
subsys
│  ├── mpsl
│  │  ├── init
│  │  │  ├── Kconfig
│  │  │  │ mpsl_init.c

Outputs:

Toolchain

Version:
Build docker image:

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

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ 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_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-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.

@NordicBuilder NordicBuilder requested a review from a team November 1, 2025 07:35
@martintv martintv force-pushed the kcondig_around_rram_in_nonsecure branch 3 times, most recently from b5b015e to 04c0040 Compare November 2, 2025 12:23
Comment on lines +125 to +129
default y
help
This option forces RRAM to stay on.
This is not needed when only basic bluetooth features are used.
Only needed when for example LLPM, Frame space update, ISO or Channel Sounding is used.
Copy link
Contributor

Choose a reason for hiding this comment

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

why not default y when those feature Kconfigs are enabled? instead of just always on?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

right now TRUSTED_EXECUTION_NONSECURE is only experimentally supported.
what I want to achive with this KConfig is that I can tell customers who complain that powerconsumption is too high, that they can set this config to =n as long as they know they are not using any of those features. I didnt find nice kconfigs for all those features to use for those features, because some of them get set automaticaly as long as the HW supports them. So then I would have to tell customers how to disable those configs.

Right now I think just doing it this way is the easiest. but when we move TFM support out of experimental, I think we should improve this yes, either by having it as kconfig like this which defaults depending on the features. But we might also do it even fancier, at runtime and only force RRAM on, on links when any of these features are enabled.

Bottom line, I want to merge this now so I can tell customer in NCS 3.2 that uses NS, that they can enable this if they want low power when using bluetooth as long as they dont use any of these features.

Since this isn't makes powerconsumption be very high,
we can let it only be done when "advanced" bluetotoh features are used.

Signed-off-by: Martin Tverdal <[email protected]>
@martintv martintv force-pushed the kcondig_around_rram_in_nonsecure branch from 04c0040 to 5de1866 Compare November 3, 2025 10:14
@NordicBuilder NordicBuilder requested review from a team November 3, 2025 10:14
@nordicjm nordicjm changed the title mpsl: Add KConfig around fordign RRAM when running from NONSECURE mpsl: Add KConfig around forcign RRAM when running from NONSECURE Nov 3, 2025
@nordicjm nordicjm changed the title mpsl: Add KConfig around forcign RRAM when running from NONSECURE mpsl: Add KConfig around forcing RRAM when running from NONSECURE Nov 3, 2025
@hermabe hermabe merged commit d31e3c0 into nrfconnect:main Nov 3, 2025
19 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants