Skip to content
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.

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

[nrf fromlist] samples: drivers: mbox: Enable sample on L09 and L20 #2622

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

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 added a commit to nordic-segl/sdk-nrf that referenced this pull request Mar 18, 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 <sebastian.glab@nordicsemi.no>
@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 added a commit to nordic-segl/sdk-nrf that referenced this pull request Mar 19, 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 <sebastian.glab@nordicsemi.no>
@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 added a commit to nordic-segl/sdk-nrf that referenced this pull request Mar 19, 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 <sebastian.glab@nordicsemi.no>
@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 added a commit to nordic-segl/sdk-nrf that referenced this pull request Mar 20, 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 <sebastian.glab@nordicsemi.no>
@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 added a commit to nordic-segl/sdk-nrf that referenced this pull request Mar 20, 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 <sebastian.glab@nordicsemi.no>
@nordic-segl nordic-segl force-pushed the NRFX-7216_Enable-mbox-on-nrf54l20 branch from b0acaf5 to 7ca68f5 Compare March 21, 2025 12:31
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Mar 21, 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 <sebastian.glab@nordicsemi.no>
extra_args: DTC_OVERLAY_FILE="boards/nrf54l15dk_nrf54l15_cpuflpr_dual_uart.overlay"
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 added a commit to nordic-segl/sdk-nrf that referenced this pull request Mar 27, 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 <sebastian.glab@nordicsemi.no>
@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 added a commit to nordic-segl/sdk-nrf that referenced this pull request Mar 28, 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 <sebastian.glab@nordicsemi.no>
TomasGalbickaNXP and others added 13 commits March 28, 2025 15:33
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 <tomas.galbicka@nxp.com>

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
(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 <tomas.galbicka@nxp.com>

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
(cherry picked from commit 2589717)
s/bellow/below/

Signed-off-by: Benjamin Cabé <benjamin@zephyrproject.org>

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
(cherry picked from commit ae4d217)
using filter to add potential platform in

Signed-off-by: Hake Huang <hake.huang@nxp.com>

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
(cherry picked from commit 9d7ee12)
… 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 <hake.huang@oss.nxp.com>

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
(cherry picked from commit 6eb176a2795a5217a542d4f30ec50d6531f5daeb)
Add board overlays required to run mbox sample on
nRF54L09 and nRF54L20 targets.

Upstream PR #: 87124

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
…RF54L* cpuflpr

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

Upstream PR #: 87124

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
Enable nrf_grtc_timer test on:
- nrf54l09pdk/nrf54l09/cpuflpr and
- nrf54l20pdk/nrf54l20/cpuflpr.

Upstream PR #: 87124

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
Add overlay required to run qdec tests
on nrf54l20pdk/nrf54l20/cpuflpr.

Upstream PR #: 87124

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
…and L20

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

Upstream PR #: 87124

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
…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).

Upstream PR #: 87124

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
…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 <krzysztof.chruscinski@nordicsemi.no>

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
(cherry picked from commit 72d0ed7)
…54L* FLPR

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

Upstream PR #: 87124

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
@nordic-segl nordic-segl force-pushed the NRFX-7216_Enable-mbox-on-nrf54l20 branch from 78ace71 to fb080f0 Compare March 28, 2025 14:33
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Mar 28, 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 <sebastian.glab@nordicsemi.no>
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.

6 participants