samples: basic: add multitarget_hello_world #69652
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gmarull
requested review from
kartben,
tejlmand,
carlescufi,
anangl,
nordicjm and
57300
nordicjm
requested changes
Mar 1, 2024
gmarull
added a commit
to teslabs/sdk-zephyr
that referenced
this pull request
Mar 1, 2024
Add a new "hello world" sample that boots on multiple cores. This sample can be used to see if Zephyr boots successfully with _minimal_ configuration on SoCs with multiple cores. The following configurations are provided for now: - nRF5340 DK: cpuapp+cpunet - nRF54H20 PDK: cpuapp+cpurad, cpuapp+cpuppr, cpuapp+cpuppr(ram) Note that because of a quoting bug somewhere in the build system, SB_CONFIG_REMOTE_BOARD is specified in .conf files (see the sysbuild/ directory). This should be fixed in the near future. Upstream PR: zephyrproject-rtos/zephyr#69652 Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com> (cherry picked from commit d1b3b3ad11878c99abf8501537c4704661573f84)
gmarull
force-pushed
the
multicore-hello-world
branch
2 times, most recently
from
0c793e3
to
57e04ba
Compare
gmarull
force-pushed
the
multicore-hello-world
branch
from
57e04ba
to
be00980
Compare
gmarull
changed the title
gmarull
force-pushed
the
multicore-hello-world
branch
from
be00980
to
46a92d9
Compare
gmarull
force-pushed
the
multicore-hello-world
branch
from
46a92d9
to
b14ae38
Compare
gmarull
changed the title
gmarull
force-pushed
the
multicore-hello-world
branch
from
b14ae38
to
2afb636
Compare
57300
reviewed
Mar 11, 2024
|
|
||
| Run a hello world sample on multiple board targets | ||
|
|
||
| Overview |
There was a problem hiding this comment.
I think the overview should mention somewhere that the main use case for this sample is when the remote core/target needs to be booted by the primary one.
Otherwise, someone may ask: why not just build hello_world twice?
There was a problem hiding this comment.
I think that's one of the usecases, but there may be e.g. a board with 2 independent socs.
samples/basic/multitarget_hello_world/remote/boards/nrf54h20pdk_nrf54h20_cpuppr_ram.conf
Outdated
Show resolved
Hide resolved
|
last push: created a new category |
gmarull
force-pushed
the
multicore-hello-world
branch
2 times, most recently
from
4299703
to
f975788
Compare
gmarull
force-pushed
the
multicore-hello-world
branch
from
f975788
to
ea9c7cb
Compare
carlescufi
previously approved these changes
May 10, 2024
|
@kartben please take another look |
MaureenHelm
previously approved these changes
May 10, 2024
fabiobaltieri
previously approved these changes
May 12, 2024
kartben
requested changes
May 13, 2024
This creates a new category of samples that demonstrate certain features of sysbuild. "application development" was a very generic name. Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
Add a new "hello world" sample that can be built for multiple board targets. This sample can be used to see if Zephyr boots successfully with _minimal_ configuration on SoCs with multiple cores, or boards with multiple SoCs. The following configurations are provided for now: - nRF5340 DK: cpuapp+cpunet - nRF54H20 DK: cpuapp+cpurad, cpuapp+cpuppr, cpuapp+cpuppr(xip) Note that because of a quoting bug somewhere in the build system, SB_CONFIG_REMOTE_BOARD is specified in .conf files (see the sysbuild/ directory). This should be fixed in the near future. Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
gmarull
dismissed stale reviews from fabiobaltieri, MaureenHelm, and carlescufi
via
dcd6fc6
gmarull
force-pushed
the
multicore-hello-world
branch
from
ea9c7cb
to
dcd6fc6
Compare
kartben
approved these changes
May 13, 2024
fabiobaltieri
approved these changes
May 13, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add a new "hello world" sample that can be built for multiple board
targets. This sample can be used to see if Zephyr boots successfully
with minimal configuration on SoCs with multiple cores, or boards with
multiple SoCs.
The following configurations are provided for now:
Note that because of a quoting bug somewhere in the build system, SB_CONFIG_REMOTE_BOARD is specified in .conf files (see the sysbuild/ directory). This should be fixed in the near future.