Skip to content

Conversation

@nordic-segl
Copy link
Contributor

@nordic-segl nordic-segl commented Mar 14, 2025

Add board overlays required to run mbox sample on
nRF54L09 and nRF54L20 targets.

Upstream PR #: 87124

manifest-pr-skip

@nordic-segl
Copy link
Contributor Author

Works on L20:

14:07:45  INFO    - 1/3 nrf54l20pdk/nrf54l20/cpuapp samples/drivers/mbox/sample.drivers.mbox.nrf54l_remote_no_multithreading PASSED (device: 175104789, 6.214s <zephyr>)
14:07:53  INFO    - 2/3 nrf54l20pdk/nrf54l20/cpuapp samples/drivers/mbox/sample.drivers.mbox.nrf54l    PASSED (device: 175104789, 6.184s <zephyr>)
14:08:00  INFO    - 3/3 nrf54l20pdk/nrf54l20/cpuapp samples/drivers/mbox/sample.drivers.mbox.nrf54l_no_multithreading PASSED (device: 175104789, 6.041s <zephyr>)

Fails on L09:

14:13:21  INFO    - 1/3 nrf54l09pdk/nrf54l09/cpuapp samples/drivers/mbox/sample.drivers.mbox.nrf54l_remote_no_multithreading FAILED Timeout (device: 851006632, 36.664s <zephyr>)
14:13:21  INFO    - /jenkins_workspace_b/workspace/ster-firelight-customizable_main/outcomes/results/sanitycheck_build/nrf54l09pdk_nrf54l09_cpuapp/zephyr/samples/drivers/mbox/sample.drivers.mbox.nrf54l_remote_no_multithreading/handler.log
14:13:21  ERROR   - *** Booting nRF Connect SDK v2.9.99-c8ef70c6dbe1 ***
14:13:21  *** Using Zephyr OS v4.0.99-b30e388b1df6 ***
14:13:21  Hello from HOST - nrf54l09pdk/nrf54l09/cpuapp
14:13:21  Maximum RX channels: 1
14:13:21  Maximum bytes of data in the TX message: 0
14:13:21  Maximum TX channels: 7
14:13:21  
14:13:21  INFO    - /jenkins_workspace_b/workspace/ster-firelight-customizable_main/outcomes/results/sanitycheck_build/nrf54l09pdk_nrf54l09_cpuapp/zephyr/samples/drivers/mbox/sample.drivers.mbox.nrf54l_remote_no_multithreading/handler.log
14:13:57  INFO    - 2/3 nrf54l09pdk/nrf54l09/cpuapp samples/drivers/mbox/sample.drivers.mbox.nrf54l    FAILED Timeout (device: 851006632, 36.652s <zephyr>)
14:13:57  INFO    - /jenkins_workspace_b/workspace/ster-firelight-customizable_main/outcomes/results/sanitycheck_build/nrf54l09pdk_nrf54l09_cpuapp/zephyr/samples/drivers/mbox/sample.drivers.mbox.nrf54l/handler.log
14:13:57  ERROR   - *** Booting nRF Connect SDK v2.9.99-c8ef70c6dbe1 ***
14:13:57  *** Using Zephyr OS v4.0.99-b30e388b1df6 ***
14:13:57  Hello from HOST - nrf54l09pdk/nrf54l09/cpuapp
14:13:57  Maximum RX channels: 1
14:13:57  Maximum bytes of data in the TX message: 0
14:13:57  Maximum TX channels: 7
14:13:57  
14:13:57  INFO    - /jenkins_workspace_b/workspace/ster-firelight-customizable_main/outcomes/results/sanitycheck_build/nrf54l09pdk_nrf54l09_cpuapp/zephyr/samples/drivers/mbox/sample.drivers.mbox.nrf54l/handler.log
14:14:39  INFO    - 3/3 nrf54l09pdk/nrf54l09/cpuapp samples/drivers/mbox/sample.drivers.mbox.nrf54l_no_multithreading FAILED Timeout (device: 851006632, 36.445s <zephyr>)
14:14:39  INFO    - /jenkins_workspace_b/workspace/ster-firelight-customizable_main/outcomes/results/sanitycheck_build/nrf54l09pdk_nrf54l09_cpuapp/zephyr/samples/drivers/mbox/sample.drivers.mbox.nrf54l_no_multithreading/handler.log
14:14:39  ERROR   - *** Booting nRF Connect SDK v2.9.99-c8ef70c6dbe1 ***
14:14:39  *** Using Zephyr OS v4.0.99-b30e388b1df6 ***
14:14:39  Hello from HOST - nrf54l09pdk/nrf54l09/cpuapp
14:14:39  Maximum RX channels: 1
14:14:39  Maximum bytes of data in the TX message: 0
14:14:39  Maximum TX channels: 7
14:14:39  
14:14:39  INFO    - /jenkins_workspace_b/workspace/ster-firelight-customizable_main/outcomes/results/sanitycheck_build/nrf54l09pdk_nrf54l09_cpuapp/zephyr/samples/drivers/mbox/sample.drivers.mbox.nrf54l_no_multithreading/handler.log
14:14:39  
14:14:39  INFO    - 8 test scenarios (3 configurations) selected, 0 configurations filtered (0 by static filter, 0 at runtime).
14:14:39  INFO    - 0 of 3 executed test configurations passed (0.00%), 0 built (not run), 3 failed, 0 errored, with no warnings in 327.98 seconds.

@nordic-segl nordic-segl force-pushed the NRFX-7216_Enable-mbox-on-nrf54l20 branch 5 times, most recently from 61665c7 to 06ba3f1 Compare March 18, 2025 12:16
@nordic-segl nordic-segl marked this pull request as ready for review March 18, 2025 12:16
@nordic-segl nordic-segl requested a review from a team March 18, 2025 12:16
@nordic-segl nordic-segl force-pushed the NRFX-7216_Enable-mbox-on-nrf54l20 branch from 06ba3f1 to 437a6f6 Compare March 19, 2025 06:44
@nordic-segl nordic-segl force-pushed the NRFX-7216_Enable-mbox-on-nrf54l20 branch 2 times, most recently from 99eb1cc to 13c3b88 Compare March 19, 2025 07:49
@nordic-segl nordic-segl force-pushed the NRFX-7216_Enable-mbox-on-nrf54l20 branch 2 times, most recently from 02d287e to 606c864 Compare March 20, 2025 10:17
@nordic-segl nordic-segl force-pushed the NRFX-7216_Enable-mbox-on-nrf54l20 branch from 606c864 to b0acaf5 Compare March 20, 2025 13:03
@nordic-segl nordic-segl force-pushed the NRFX-7216_Enable-mbox-on-nrf54l20 branch from b0acaf5 to 7ca68f5 Compare March 21, 2025 12:31
extra_configs:
- CONFIG_DUAL_UART_TEST=y
- CONFIG_SETUP_MISMATCH_TEST=y
drivers.uart.uart_elementary_dual_nrf54l09:
Copy link
Contributor Author

@nordic-segl nordic-segl Mar 21, 2025

Choose a reason for hiding this comment

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

@nordic-piks : Dual uart tests on L09 require some tricks; shall I add here

    tags:
      - nrf54l09-switch-uart

same as in nrf.extended tests at sdk-nrf? Or, remove these tests from this yaml?

Copy link
Contributor

Choose a reason for hiding this comment

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

FLPR will already use 2nd uart.
so maybe uart removal from app core is only needed?

@nordic-segl nordic-segl force-pushed the NRFX-7216_Enable-mbox-on-nrf54l20 branch from 7ca68f5 to a7c12e2 Compare March 27, 2025 07:58
@nordic-segl nordic-segl force-pushed the NRFX-7216_Enable-mbox-on-nrf54l20 branch from a7c12e2 to 78ace71 Compare March 28, 2025 07:37
@nordic-segl nordic-segl force-pushed the NRFX-7216_Enable-mbox-on-nrf54l20 branch from b76fda6 to 5e06a39 Compare April 14, 2025 06:04
TomasGalbickaNXP and others added 21 commits April 14, 2025 12:16
This commit adds support for these multicore samples:
- samples/drivers/mbox
- samples/drivers/mbox_data
- samples/subsys/ipc/ipc_service/static_vrings
- samples/subsys/ipc/openamp

Signed-off-by: Tomas Galbicka <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 64585b7)
This commit adds support for these multicore samples:
- samples/drivers/mbox
- samples/drivers/mbox_data
- samples/subsys/ipc/ipc_service/static_vrings
- samples/subsys/ipc/openamp

Signed-off-by: Tomas Galbicka <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 2589717)
s/bellow/below/

Signed-off-by: Benjamin Cabé <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit ae4d217)
using filter to add potential platform in

Signed-off-by: Hake Huang <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 9d7ee12)
In 9d7ee12
the regex used to verify the sample is doing what it should was changed
in a way in which it cannot possibly pass.
Let's fix it.

While at it, in the same commit, the simulator version of the test was
merged with the real_hw version, but the test name kept "real_hw" which
is now missleading => let's fix it.

And let's set as integration platform the simulated target, so it
can actually be runtime tested in CI.

Signed-off-by: Alberto Escolar Piedras <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 7250293)
The mbox sample uses bellboard events enabled by default
in the nRF54H20dk board definition.
Thus, it's not easily obvious that IRQ bitmask must be aligned
when other mbox/bellboard channel is used.

Add comment to the nRF54H20 overlays.
Inform that bellboard channel used for RX must be enabled in
the 'nordic,interrupt-mapping' property.

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit dd882dc)
… not supported

remove the FATAL_ERROR message, and leave it just as a message.
also enhance filter for this case with "1pc0" anmd "1pc1"

Signed-off-by: Hake Huang <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 6eb176a)
Where applicable, pick a simulator as the integration platform.

Signed-off-by: Anas Nashif <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit d1af028)
Enable flash tests for s32z board

Signed-off-by: Cong Nguyen Huu <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit c4d1c55)
…six overlay

native_posix is being removed as it has reached its end of life in 4.2,
after being deprecated since 4.0.

So this overlay will not serve any purpose anymore.

Note native_sim did not have an overlay of its own.

Signed-off-by: Alberto Escolar Piedras <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit db62135)
…tegration platforms

We want to be able to run tests and get results in CI, not just build
for platforms we do not have access to.
Where possible, use a simulator instead of hw platforms.

Signed-off-by: Anas Nashif <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit abb5f52)
add new board Ophelia-IV

Signed-off-by: Matthias Hauser <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 941a548)
…support

Add overlay for nrf54l09pdk/nrf54l09/cpuapp target.

Signed-off-by: Krzysztof Chruściński <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit eda1b3f)
…RF54L* cpuflpr

Add overlay required to run gpio_basic_api test on
- nrf54l09pdk/nrf54l09/cpuflpr and
- nrf54l20pdk/nrf54l20/cpuflpr.

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit fb1d730)
Add overlay required to run qdec tests
on nrf54l20pdk/nrf54l20/cpuflpr.

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 729e39a)
…and L20

Add overlays required to run watchdog sample on:
- nrf54l09pdk/nrf54l09/cpuapp,
- nrf54l09pdk/nrf54l09/cpuflpr,
- nrf54l20pdk/nrf54l20/cpuflpr.

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit a2f4154)
…n nRF54L09 and L20

Enable wdt_basic_api test on:
- nrf54l09pdk/nrf54l09/cpuapp (missing supported),
- nrf54l09pdk/nrf54l09/cpuflpr (missing overlay),
- nrf54l20pdk/nrf54l20/cpuflpr (missing supported and overlay).

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit a9e822e)
…target

Add new target to the following tests:
- uart_elementary
- uart_async_api
- uart_mix_fifo_poll
- uart_pm

Signed-off-by: Krzysztof Chruściński <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 72d0ed7)
…54L* FLPR

Add overlays required to run the test on:
- nrf54l09pdk/nrf54l09/cpuflpr,
- nrf54l20pdk/nrf54l20/cpuflpr.

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 3c34f88)
Add board overlays required to run mbox sample on
nRF54L09 and nRF54L20 targets.

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 9bb5e2f)
Enable nrf_grtc_timer test on:
- nrf54l09pdk/nrf54l09/cpuflpr and
- nrf54l20pdk/nrf54l20/cpuflpr.

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 6a782cf)
@nordic-segl nordic-segl force-pushed the NRFX-7216_Enable-mbox-on-nrf54l20 branch from 5e06a39 to de7227e Compare April 14, 2025 10:16
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Apr 14, 2025
Include:
samples: drivers: mbox: Enable sample on L09 and L20
nrfconnect/sdk-zephyr#2622
zephyrproject-rtos/zephyr#87124

Signed-off-by: Sebastian Głąb <[email protected]>
@sonarqubecloud
Copy link

@nordic-segl
Copy link
Contributor Author

Pulled in with the upmerge.

@nordic-segl nordic-segl deleted the NRFX-7216_Enable-mbox-on-nrf54l20 branch May 14, 2025 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.