Skip to content

Conversation

eivindj-nordic
Copy link
Contributor

Update nrf-bm-internal revision.
Some changes required to Kconfig etc.

@eivindj-nordic eivindj-nordic self-assigned this Aug 18, 2025
@eivindj-nordic eivindj-nordic requested review from a team as code owners August 18, 2025 12:09
@NordicBuilder
Copy link

NordicBuilder commented Aug 18, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
sdk-nrf-bm-internal https://github.com/nrfconnect/sdk-nrf-bm-internal/commit/2086b4bb5ee2cc68b824fe5ecd4e2461987e4c5c https://github.com/nrfconnect/sdk-nrf-bm-internal/pull/6 nrfconnect/sdk-nrf-bm-internal#6/files

DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

Comment on lines 15 to 23
config HAS_SOFTDEVICE_S115
bool
depends on BOARD_BM_NRF54L15DK_NRF54L05_CPUAPP_S115_SOFTDEVICE || \
BOARD_BM_NRF54L15DK_NRF54L05_CPUAPP_S115_SOFTDEVICE_MCUBOOT || \
BOARD_BM_NRF54L15DK_NRF54L10_CPUAPP_S115_SOFTDEVICE || \
BOARD_BM_NRF54L15DK_NRF54L10_CPUAPP_S115_SOFTDEVICE_MCUBOOT || \
BOARD_BM_NRF54L15DK_NRF54L15_CPUAPP_S115_SOFTDEVICE || \
BOARD_BM_NRF54L15DK_NRF54L15_CPUAPP_S115_SOFTDEVICE_MCUBOOT
default y
Copy link
Contributor

Choose a reason for hiding this comment

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

this does not belong here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think this can be removed in total. Should be enough to depend on the SoC.

@@ -10,7 +10,7 @@ zephyr_library_sources(
irq_connect.c
)

if (CONFIG_SOFTDEVICE_S115)
if (CONFIG_SOFTDEVICE AND CONFIG_SOC_SERIES_NRF54LX)
Copy link
Contributor

Choose a reason for hiding this comment

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

no space between bracket and if in cmake

@eivindj-nordic eivindj-nordic requested a review from a team as a code owner August 25, 2025 13:21
Copy link

You can find the documentation preview for this PR here.

Update nrf-bm-internal revision.
Some changes required to Kconfig etc.

Signed-off-by: Eivind Jølsgard <[email protected]>
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.

3 participants