Skip to content

[nrf fromlist] samples: cleanup sysbuild/hello_world sample #3264

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
tejlmand wants to merge 746 commits into
base: main
Choose a base branch
from
Open

[nrf fromlist] samples: cleanup sysbuild/hello_world sample #3264

tejlmand wants to merge 746 commits into from

Conversation

@tejlmand
Copy link
Contributor

@tejlmand tejlmand commented Sep 10, 2025

Remove the prompt for REMOTE_BOARD.
REMOTE_BOARD should not be having a prompt or be configurable on command
line.
Instead REMOTE_BOARD should be defined based on the SoC as this will
allow creation of new boards using the same SoC, and thereby be able
to build the sample out-of-the-box for any supported SoC.

This allows us to remove several single line config files.

For SoCs with more than two CPU clusters, such as the nRF54h20, then a
choice is provided to select specific core.

Remove the REMOTE_BOARD restriction, as this sample will build and run
even for single core SoCs, and may be useful for testing other sysbuild
multi-image features even for single cores SoCs.

Upstream PR #: 95745

nordicjm and others added 30 commits August 1, 2025 11:34
@nordicjm @rlubos
[nrf noup] boards: nordic: thingy53: Add sysbuild Kconfig file
fa07e94 
Adds a sysbuild Kconfig file which enables external flash when
partition manager is enabled

Signed-off-by: Jamie McCrae <[email protected]>
(cherry picked from commit 41431fb)
@nordicjm @rlubos
[nrf noup] scripts: west: build: Use sysbuild by default if in NCS dir
5bbb2c8 
Uses sysbuild by default when building an application which resides in
an allowed NCS-based directory when the sysbuild config key is not
set. Do not use sysbuild if the mps2 board is being used to avoid
problems with CI testing.

Signed-off-by: Jamie McCrae <[email protected]>
(cherry picked from commit 5cfd8be)
@nordicjm @rlubos
[nrf noup] board: nordic: thingy53: Enable default images for sysbuild
f730f82 
Enables MCUboot, empty network core and network core updates by
default when building for the thingy53

Signed-off-by: Jamie McCrae <[email protected]>
(cherry picked from commit b712330)
@rlubos
[nrf noup] include: net: add NCS extensions
d7a6277 
Add some socket options and address family extensions to Zephyr
headers, which will be useful for nRF Connect SDK.

Add secure socket options:
* Add CID socket options to NCS specific options.
* Add TLS/DTLS tls ciphersuite chosen socket option to NCS specific
  options.
* Add TLS/DTLS connection save/load socket options to NCS specific
  options.
* Add TLS/DTLS handshake status socket option to NCS specific options.

* Add SO_KEEPOPEN socket option.
* Add SO_RAI socket options.
* Add SO_IPV6_DELAYED_ADDR_REFRESH socket option.

The "author" of this commit is a contact person; various people with
s-o-b lines following here have contributed to the maintenance of this
patch.

Signed-off-by: Andreas Moltumyr <[email protected]>
Signed-off-by: Andrzej Głąbek <[email protected]>
Signed-off-by: Christopher Métrailler <[email protected]>
Signed-off-by: Emanuele Di Santo <[email protected]>
Signed-off-by: Glenn Ruben Bakke <[email protected]>
Signed-off-by: Håkon Alseth <[email protected]>
Signed-off-by: Ioannis Glaropoulos <[email protected]>
Signed-off-by: Jan Tore Guggedal <[email protected]>
Signed-off-by: Joakim Andersson <[email protected]>
Signed-off-by: Martí Bolívar <[email protected]>
Signed-off-by: Mirko Covizzi <[email protected]>
Signed-off-by: Petri Honkala <[email protected]>
Signed-off-by: Robert Lubos <[email protected]>
Signed-off-by: Tommi Mammela <[email protected]>
Signed-off-by: Trond Einar Snekvik <[email protected]>
Signed-off-by: Torsten Rasmussen <[email protected]>
Signed-off-by: Eivind Jølsgard <[email protected]>
Signed-off-by: Dominik Ermel <[email protected]>
(cherry picked from commit b9a0c11)
@sigvartmh @rlubos
[nrf noup] tree-wide: support NCS Partition Manager (PM) definitions
5c2eecf 
Partition Manager (PM) is a component of the nRF Connect SDK (NCS)
which uses yaml files to resolve flash partition placement with a
holistic view of the entire device, including each firmware image
present on the flash device, and various subsystems, such as settings
and NFFS.

When this NCS extension is used, various source files which would use
partition information from devicetree in "vanilla" zephyr instead use
defines generated by PM instead.

This commit removes support for HEX_FILES_TO_MERGE, as it conflicts
with PM.

The settings subsystem pm.yml defines a partition 'settings_storage'.
The nffs subsystem pm.yml defines 'nffs_storage'.

Leverage label translation to avoid patching partition names.

Refer to the NCS documentation page for this feature for more details.

This is a long-running out of tree patch which has been worked on by
several people. The following sign-offs are in alphabetical order by
first name.

Signed-off-by: Andrzej Głąbek <[email protected]>
Signed-off-by: Andrzej Puzdrowski <[email protected]>
Signed-off-by: Håkon Øye Amundsen <[email protected]>
Signed-off-by: Ioannis Glaropoulos <[email protected]>
Signed-off-by: Joakim Andersson <[email protected]>
Signed-off-by: Johann Fischer <[email protected]>
Signed-off-by: Martí Bolívar <[email protected]>
Signed-off-by: Ole Sæther <[email protected]>
Signed-off-by: Robert Lubos <[email protected]>
Signed-off-by: Sebastian Bøe <[email protected]>
Signed-off-by: Sigvart Hovland <[email protected]>
Signed-off-by: Thomas Stenersen <[email protected]>
Signed-off-by: Torsten Rasmussen <[email protected]>
Signed-off-by: Øyvind Rønningstad <[email protected]>
Signed-off-by: Trond Einar Snekvik <[email protected]>
Signed-off-by: Gerard Marull-Paretas <[email protected]>
Signed-off-by: Tomasz Moń <[email protected]>
Signed-off-by: Dominik Ermel <[email protected]>
Signed-off-by: Jamie McCrae <[email protected]>
(cherry picked from commit 7ad01c2)
@rlubos
[nrf noup] kernel: Disable boot banner if NCS_BOOT_BANNER is enabled
4b7a867 
Zephyr's boot banner should not be used if NCS boot banner is enabled.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit 11254fb)
@HaavardRei @rlubos
[nrf noup] Bluetooth: Mesh: remove legacy adv support
10b1361 
Removes explicit support for the legacy advertiser due
to incompatibility with SDC.

The legacy advertiser can be used (experimentally) with the
Zephyr Link Layer enabled, but is not recommended.

Signed-off-by: Håvard Reierstad <[email protected]>
(cherry picked from commit 4680ad7)
@nordicjm @rlubos
[nrf noup] board: nordic: thingy53: Default to update only MCUboot type
a06f958 
Changes the default MCUboot mode to update only for the thingy53,
to align with previous bootloader builds

Changes the thingy53 default configuration for sysbuild to enable
using all RAM in the MCUboot image

Signed-off-by: Jamie McCrae <[email protected]>
(cherry picked from commit af016f5)
@gmarull @rlubos
[nrf noup] samples: sysbuild: hello_world: support PM on nRF53
470fcc9 
PM support is still required for nRF53/nRF54l15 in the context of NCS.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
(cherry picked from commit 528a233)
@Damian-Nordic @rlubos
[nrf noup] settings: nvs: use dedicated lookup cache hash function
1deb982 
Introduce NVS_LOOKUP_CACHE_FOR_SETTINGS Kconfig option that
enables a dedicated hash function for the NVS lookup cache
that takes advantage of the NVS ID allocation scheme used
by the NVS settings backend. As such, this option should
only be used if an application uses NVS via the settings
layer.

Signed-off-by: Damian Krolik <[email protected]>
(cherry picked from commit aaa00db)
@SebastianBoe @rlubos
[nrf noup] soc: nordic: nrf54l: Skip clock configuration from ns
4a163d0 
Do not permit configuring clocks from non-secure as NRF_OSCILLATORS is
a not supported as a non-secure peripheral

Signed-off-by: Sebastian Bøe <[email protected]>
(cherry picked from commit 0099c2c)
@rugeGerritsen @rlubos
[nrf noup] ci: Enable action-manifest-pr
7cf6f4f 
This action will automatically manage a PR to sdk-nrf
once a PR to sdk-zephyr is created.

This includes:
 - Creating an initial PR
 - Updating and (optionally) rebase it once the PR
   to sdk-nrf is merged.

The action can be disabled by adding the string
"manifest-pr-skip" to the title or body of the PR.

This will simplify cherry-picking changes from upstream
zephyr.

Signed-off-by: Rubin Gerritsen <[email protected]>
(cherry picked from commit 40f8ba3)
@michalek-no @rlubos
[nrf noup] drivers: flash: kconfig: nrf_rram region resolution
73823d3 
adjusting region resolution to match erase-block-size

Signed-off-by: Mateusz Michalek <[email protected]>
(cherry picked from commit 743b21b)
@rugeGerritsen @rlubos
[nrf noup] dts: Add Bluetooth Controller to nRF54H20
6f17dc3 
The nRF54H20 supports a Bluetooth controller.
The HCI driver interface has changed upstream in
zephyrproject-rtos/zephyr#72323
so now we need to add it to device tree.

Signed-off-by: Rubin Gerritsen <[email protected]>
(cherry picked from commit e9a13c3)
@MarekPieta @rlubos
[nrf noup] bluetooth: att: Allow ATT sent callback after data TX is done
5b9ea8c 
By default, the BLE stack calls sent callback for ATT data when the data
is passed to BLE controller for transmission. Enabling this Kconfig
option delays calling the sent callback until data transmission is
finished by BLE controller (the callback is delayed until receiving the
num complete packets event).

Jira: NCSDK-27422

Signed-off-by: Marek Pieta <[email protected]>
(cherry picked from commit 5cb2c90)
@hellesvik-nordic @rlubos
[nrf noup] board: nordic_ thingy53: Enable QSPI by default
20671bc 
The Thingy:53 enabled MCUboot and external flash by default.
Therefore, it should also enable drivers for its external
flash by default.

Signed-off-by: Sigurd Hellesvik <[email protected]>
(cherry picked from commit d8ae490)
@nordicjm @rlubos
[nrf noup] mgmt: mcumgr: grp: img_mgmt: Add handling for QSPI XIP
a5b67c1 
Adds handling code to allow selecting the correct image slot when
using QSPI XIP in DirectXIP mode

In case of Direct XIP and multiple images, the radio active slot changes
accordingly (i.e. application slot 1 boots radio slot 1).

Signed-off-by: Jamie McCrae <[email protected]>
Signed-off-by: Tomasz Chyrowicz <[email protected]>
(cherry picked from commit eff4005)
@nvlsianpu @rlubos
[nrf noup] boards/nordic/nrf54l15dk: Set ROM offset by PM
cea66ec 
While Partition Manager is enabled, it manages the offset
by itself.

Signed-off-by: Andrzej Puzdrowski <[email protected]>
(cherry picked from commit 8e130db)
@jaz1-nordic @rlubos
[nrf noup] samples: basic: blinky: add eGPIO tests configuration
40b222d 
Add overlay for nrf54l15dk to enable eGPIO tests.

Signed-off-by: Jakub Zymelka <[email protected]>
Signed-off-by: Marcin Szymczyk <[email protected]>
(cherry picked from commit cdbdb26)
@frkv @rlubos
[nrf noup] modules: mbedtls: Disable configurations in Kconfig.tls-ge…
c62b6eb 
…neric

-This commit prevents legacy mbed TLS configurations from being in
 conflict with PSA Configurations while using nrf_security.
-This [nrf noup] is reworked from an earlier cherry-pick of
 commit d8c96cf, but has the following
 changes:
 - Endif's relevant that is using our pattern for masking
   configs duplicated or in conflict in nrf_security
  (by using if !(NRF_SECURITY || NORDIC_SECURITY_BACKEND)) is
  made clearer by adding a comment at their corresponding endif
  (# !(NRF_SECURITY || NORDIC_SECURITY_BACKEND))
 - Changes to zephyr_init.c for entropy_dev checking for
   CONFIG_NRF_CC3XX_PLATFORM is removed as the symbol entropy_dev
   doesn't exist in this file anymore

ref: NCSDK-13503

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
(cherry picked from commit 9e06da7)
@frkv @rlubos
[nrf noup] mbedtls: Enable PSA_WANT_GENERATE_RANDOM for PSA RNG
3c07722 
-This is a [nrf noup] because PSA_WANT_GENERATE_RANDOM is a Nordic
 configuration that is not found upstream. This was previously in
 commit 5cfe575 but has been split out

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
(cherry picked from commit 2a84cc8)
@frkv @rlubos
[nrf noup] mbedtls: Remove unsupported algorithms in PSA crypto
32fa048 
-This commit is a [nrf noup] because it removes configuration options
 for cryptographic algortihms available in Mbed TLS but which is not
 actively supported in nRF Connect SDK.
 The list of algorithms removed:
 - AES CFB - Cipher Feedback block cipher
 - AES OFB - Output Feedback block cipher
 - FFDH
 - RIPEMD160
 - Aria
 - Camellia
 - DES

 The removal of these algorithms is based both on a wish to remove
 weaker cryptography and unsupported features in the products we have
 today.

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
(cherry picked from commit f0c6253)
@frkv @rlubos
[nrf noup] mbedtls: Add dependency logic for PSA crypto configurations
2abfddf 
-This is a [nrf noup] as this the upstream version of PSA crypto configs
 is generated by tooling, and there is no mechanisms to qualify
 that dependent configurations are enabled (by depends or select).
-This adds dependency-mapping between configurations in the Kconfigs
 added for PSA crypto in upstream.
-Selecting CHACHA20 key type if PSA_WANT_ALG_STREAM_CIPHER is
 enabled

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
(cherry picked from commit abe1041)
@frkv @rlubos
[nrf noup] mbedtls: Adding helptext warnings for weak crypto
76dee8c 
-This commit is a [nrf noup] because PSA crypto configs in upstream
 Zephyr doesn't have help-text in their configurations and we don't
 want to duplicate configurations to control the value
-This adds warning for SHA-1 and MD5 usage

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
(cherry picked from commit e4e16eb)
@frkv @rlubos
[nrf noup] mbedtls: Adding missing configuration for RSA key type derive
414a6c3 
-This is a [nrf noup] because the upstream Zephyr is generated using
 a script and is not committed as-is as source-code. The relevant
 responsible person for this feature has received information about
 the missing configuration and this will be resolved upstream in
 Mbed TLS and will propagate down to zephyr. Once this has happened,
 this [nrf noup] can be dropped.
-Add missing PSA_WANT_KEY_TYPE_RSA_KEY_DERIVE

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
(cherry picked from commit 6ce5d2f)
@frkv @rlubos
[nrf noup] mbedtls: Don't select PSA_WANT_KEY_TYPE_RSA_KEY_PAIR_BASIC
34341c4 
-PSA_WANT_KEY_TYPE_RSA_KEY_PAIR_BASIC is internally resolved by
 Mbed TLS. It has been made promptless in a previous commit.
 Keeping this change separated since the Kconfig.psa is auto-generated
 and it would likely be a bit more complex to handle this in a
 single commit. Upstream maintainers have been notified about this
 mismatch in configurations.

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
(cherry picked from commit 1a18abd)
@nordicjm @rlubos
[nrf noup] kernel: banner: Make function weak
717e659 
Makes the boot banner function weak, this resolves an issue when
building with llext enabled which uses different build options
than a normal zephyr build

Signed-off-by: Jamie McCrae <[email protected]>
(cherry picked from commit 461df8c)
@bjarki-andreasen @rlubos
[nrf noup] lib: os: zvfs: Remove EXPERIMENTAL from ZVFS
f52f654 
Although ZVFS is experimental, the warning is annoying the matter team.
Therefore, remove the experimental selection.

This may be reverted once upstream unselects experimental.

Signed-off-by: Bjarki Arge Andreasen <[email protected]>
(cherry picked from commit 835d545)
@rlubos
[nrf noup] samples: net: Enable Wi-Fi driver in sysbuild builds
ccf6152 
Make sure Wi-Fi driver is enabled in networking samples supporting Wi-Fi
when sysbuild is used.
For shields we cannot automate this, as sysbuild doesn't recognize
shields, so, Wi-Fi has to be explicitly enabled, this is done for
twister in Wi-Fi sample.

Signed-off-by: Robert Lubos <[email protected]>
Signed-off-by: Chaitanya Tata <[email protected]>
(cherry picked from commit 48f4e3b)
@frkv @rlubos
[nrf noup] mbedtls: Don't enable auto-generation of Mbed TLS files
d268008 
- We don't use this mechanism in nRF Connect SDK

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
(cherry picked from commit 4bdc108)
Vge0rge and others added 20 commits September 8, 2025 18:03
@Vge0rge @rlubos
[nrf fromtree] samples: tfm: tfm_psa_test: Add temporary isolation wo…
82dae59 
…rkaround

Add a workaround which sets the isolation level to 2 in CMake.
The TF-M tests require the large TF-M profile because it supports
the full list of crypto algorithms needed, not because of the
solation level.

For the TF-M tests the isolation level is irrelevant so we
set it to 2 here so that we don't exclude the platforms which
don't support the isolation level 3.

I communicated with a TF-M maintainer and he informed me that this
workaround will be included in the TF-M 2.2 release branch later
so this is a short lived workaround that can be reverted when the
relevant commit from the TF-M branch is cherry-picked.

Signed-off-by: Georgios Vasilakis <[email protected]>
(cherry picked from commit 0081207)
@ceolin @rlubos
[nrf fromtree] tf-m: build: Treat warnings as errors
5b5eafa 
Starting in TF-M 2.2.0 there is a build option to treat warnings as
errors. Lets use it.

Signed-off-by: Flavio Ceolin <[email protected]>
(cherry picked from commit ca7d99a)
@msmttchr @rlubos
[nrf fromtree] arch: arm: cortex_m: Modifed FPU save and restore helpers
d9746f4 
z_arm_save_fp_context and z_arm_restore_fp_context
save and restore fpu context regardless of the
CONFIG_FPU_SHARING setting.

This modification is required to support suspend to ram use cases
where save and restore of FPU state is needed to ensure proper bahaviour
after wakeup.

Signed-off-by: Michele Sardo <[email protected]>
(cherry picked from commit 5981dc8)
@msmttchr @rlubos
[nrf fromtree] modules: trusted-firmware-m: Update in interface imple…
143520e 
…mentation

Following changes to arch/arm/core/cortex_m/fpu.c,
the dependency on CONFIG_FPU_SHARING is moved into this file.

Signed-off-by: Michele Sardo <[email protected]>
(cherry picked from commit 7bc1756)
@tomi-font @rlubos
[nrf fromlist] manifest: tf-m-tests: bring in fix for TF-M 2.2.0
b48a1f2 
Fixup to the TF-M 2.2.0 update.

Signed-off-by: Tomi Fontanilles <[email protected]>

Upstream PR #: 94729

Signed-off-by: Tomi Fontanilles <[email protected]>
@nika-nordic @nordicjm
[nrf fromtree] manifest: update hal_nordic revision to integrate nrfx…
70886df 
… 3.14.0

New nrfx release contains MDK 8.72.2
and various fixes for nRF devices.

Signed-off-by: Nikodem Kastelik <[email protected]>
(cherry picked from commit c1d28a4)
@mstasiaknordic @nordicjm
[nrf fromtree] soc: nordic: nrf54l: remove redundant inclusion of GLI…
e53ffc9 
…TCHDET

GLITCHDET is not used in this file
and causes issues for devices without GLITCHDET.

Signed-off-by: Michał Stasiak <[email protected]>
(cherry picked from commit a9eadb4)
@mstasiaknordic @nordicjm
[nrf fromtree] drivers: adc: adc_nrfx_saadc: add nRF54LS05B analog pins
886cc3d 
Added array of analong pins for nRF54LS05B.

Signed-off-by: Michał Stasiak <[email protected]>
(cherry picked from commit 2939ccd)
@carlescufi
[nrf fromtree] flash: spi_nor: Fix VLA error when building with clang
620baa9 
The following error is issued by clang when building with
SPI_NOR_SFDP_RUNTIME enabled:

error: fields must have a constant size:
'variable length array in structure' extension will never be supported
1379 | uint32_t dw[MIN(php->len_dw, 20)];

Instead, hardcode the array length to 20 32-bit words (it's instantiated
in the stack anyway).

Signed-off-by: Carles Cufi <[email protected]>
(cherry picked from commit 267469b)
@carlescufi
[nrf fromtree] Revert "boards: nordic: nRF54L15DK: set HFXO clock lat…
0f36da3 
…ency to 854"

This reverts commit d0a0769.

The reason for reverting this is that the only safe startup time for
54L series is the one defined in the SoC .dtsi files (1650us), and other
startup times are not reliable even when tested on a particular board.

Signed-off-by: Carles Cufi <[email protected]>
(cherry picked from commit 9e52df0)
@tomchy @rlubos
[nrf fromtree] samples: Enable Direct XIP through sysbuild
08ea5d1 
Enable Direct XIP with revert MCUboot mode through sysbuild
configuration.

Signed-off-by: Tomasz Chyrowicz <[email protected]>
(cherry picked from commit 7a54526)
@bjda @rlubos
[nrf fromtree] boards: nrf9280pdk: disable IEEE 802.15.4 support
bfbe1e0 
nRF92 series is not supported in the IEEE 802.15.4 stack yet,
so remove it from the board for now.

Signed-off-by: Bernt Johan Damslora <[email protected]>
(cherry picked from commit cb1b569)
@bjda @rlubos
[nrf fromtree] boards: nrf9280pdk: Add revision 0.2.0
b2b59e0 
Add revision support and overlays for rev. 0.2.0.

Signed-off-by: Bernt Johan Damslora <[email protected]>
(cherry picked from commit 46e9b4a)
@nordicjm
Revert "[nrf noup] mgmt/mcumgr: Add support for SHA512 in images"
6cf3d6e 
This reverts commit a56cf02.

Signed-off-by: Jamie McCrae <[email protected]>
@nordicjm
Revert "[nrf noup] samples/smp_svr: Increase stack size required"
1aa4426 
This reverts commit 018554d.

Signed-off-by: Jamie McCrae <[email protected]>
@nordicjm
[nrf fromlist] mgmt: mcumgr: grp: img_mgmt: Add support for SHA512 in…
1bf6d60 
… images

Adds support for images signed with SHA512.

Upstream PR #: 95332

Signed-off-by: Dominik Ermel <[email protected]>
Signed-off-by: Jamie McCrae <[email protected]>
@nordicjm
[nrf fromlist] samples/smp_svr: Increase stack size required
3143c02 
Increase stack size as support for SHA512 requires more data.

Upstream PR #: 95332

Signed-off-by: Dominik Ermel <[email protected]>
Signed-off-by: Jamie McCrae <[email protected]>
@nordicjm
Revert "[nrf noup] samples: smp_svr: configuration for nrf54l15"
dd89948 
This reverts commit 640dbac.

Signed-off-by: Jamie McCrae <[email protected]>
@nordicjm
Revert "[nrf noup] sample/**/smp_svr: nrf54l15pdk external memory dts…
f7a34c9 
… overlay"

This reverts commit 73df5dd.

Signed-off-by: Jamie McCrae <[email protected]>
@nordicjm
Revert "[nrf noup] samples: mgmt: mcumgr: smp_svr: Add nrf5340dk-opti…
cf180fe 
…mised sample"

This reverts commit d2ca243.

Signed-off-by: Jamie McCrae <[email protected]>
@NordicBuilder NordicBuilder mentioned this pull request Sep 10, 2025
Closed
tejlmand and others added 4 commits September 11, 2025 09:51
@tejlmand
Revert "[nrf noup] samples: sysbuild: hello_world: support PM on nRF53"
b13fb55 
This reverts commit 470fcc9.

Signed-off-by: Torsten Rasmussen <[email protected]>
@tejlmand
[nrf fromlist] samples: cleanup sysbuild/hello_world sample
38b0f12 
Remove the prompt for REMOTE_BOARD.
REMOTE_BOARD should not be having a prompt or be configurable on command
line.
Instead REMOTE_BOARD should be defined based on the SoC as this will
allow creation of new boards using the same SoC, and thereby be able
to build the sample out-of-the-box for any supported SoC.

This allows us to remove several single line config files.

For SoCs with more than two CPU clusters, such as the nRF54h20, then a
choice is provided to select specific core.

Remove the REMOTE_BOARD restriction, as this sample will build and run
even for single core SoCs, and may be useful for testing other sysbuild
multi-image features even for single cores SoCs.

Upstream PR #: 95745

Signed-off-by: Torsten Rasmussen <[email protected]>
@tejlmand
[nrf fromlist] samples: pass on board revision in sysbuild/hello_world
5b30793 
Pass on the board revision to remote image build if specified for the
application image board target.

Upstream PR #: 95745

Signed-off-by: Torsten Rasmussen <[email protected]>
@gmarull @tejlmand
[nrf noup] samples: sysbuild: hello_world: support PM on nRF53
ba5b69f 
PM support is still required for nRF53/nRF54l15 in the context of NCS.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
Signed-off-by: Torsten Rasmussen <[email protected]>
@tejlmand tejlmand force-pushed the sysbuild_hello_world_cleanup branch from 329d8cd to ba5b69f Compare September 11, 2025 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nordic-piks nordic-piks nordic-piks approved these changes

@nordicjm nordicjm Awaiting requested review from nordicjm

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.