Nrfx 7821 run samples subsys ipc ipc service icmsg on more targets #23019

nordic-segl · 2025-06-27T09:34:23Z

No description provided.

NordicBuilder · 2025-06-27T09:35:40Z

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

Name	Old Revision	New Revision	Diff
zephyr	nrfconnect/sdk-zephyr@`51e6f94`	nrfconnect/sdk-zephyr#2998	nrfconnect/sdk-zephyr#2998/files

⛔ DNM label due to: 1 project with PR revision

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

NordicBuilder · 2025-06-27T09:39:04Z

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 12

Inputs:

Sources:

sdk-nrf: PR head: ca2500d690ba6a5b2c46ac9a5d010116cb0bba05
zephyr: PR head: 573b30abaa7d01a053c2cc0005480626f91f8626

more details

sdk-nrf:

PR head: ca2500d690ba6a5b2c46ac9a5d010116cb0bba05
merge base: b8a88cb47afccd55f2ba78084715c1e66c7cb7de
target head (main): f6fed40c2cb15d6c0ffee81758fb3eab9d436e4f
Diff

zephyr:

PR head: 573b30abaa7d01a053c2cc0005480626f91f8626
merge base: 53824876e35648dc8c2d6fb99e3756e7cd4d779f
target head (main): 5408ed3f43481b3e6c031a16c7ea128e8fb1c0ae
Diff

Github labels

Enabled	Name	Description
	ci-disabled	Disable the ci execution
	ci-all-test	Run all of ci, no test spec filtering will be done
	ci-force-downstream	Force execution of downstream even if twister fails
	ci-run-twister	Force run twister
	ci-run-zephyr-twister	Force run zephyr twister

List of changed files detected by CI (12)

west.yml
zephyr
│  ├── samples
│  │  ├── subsys
│  │  │  ├── ipc
│  │  │  │  ├── ipc_service
│  │  │  │  │  ├── icmsg
│  │  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  │  ├── Kconfig.sysbuild
│  │  │  │  │  │  ├── boards
│  │  │  │  │  │  │  ├── nrf54h20dk_nrf54h20_cpuapp.overlay
│  │  │  │  │  │  │  ├── nrf54lm20dk_nrf54lm20a_cpuapp.overlay
│  │  │  │  │  │  │  │ nrf54lm20dk_nrf54lm20a_cpuapp_icbmsg.overlay
│  │  │  │  │  │  ├── remote
│  │  │  │  │  │  │  ├── boards
│  │  │  │  │  │  │  │  ├── nrf54h20dk_nrf54h20_cpuppr.conf
│  │  │  │  │  │  │  │  ├── nrf54h20dk_nrf54h20_cpuppr.overlay
│  │  │  │  │  │  │  │  ├── nrf54lm20dk_nrf54lm20a_cpuflpr.overlay
│  │  │  │  │  │  │  │  │ nrf54lm20dk_nrf54lm20a_cpuflpr_icbmsg.overlay
│  │  │  │  │  │  ├── sample.yaml
│  │  │  │  │  │  │ sysbuild.cmake

Outputs:

Toolchain

Version: 2b2cd9579a
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:2b2cd9579a_bba2ea5f2e

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

◻️ Toolchain - Skipped: existing toolchain is used
✅ Build twister
- sdk-nrf test count: 12
- sdk-zephyr test count: 9
❌ Integration tests
- ❌ test-low-level

Disabled integration tests

- test-fw-nrfconnect-nrf_lrcs_mosh
- test-fw-nrfconnect-nrf_lrcs_positioning
- desktop52_verification
- doc-internal
- test_ble_nrf_config
- test-fw-nrfconnect-apps
- test-fw-nrfconnect-ble_mesh
- test-fw-nrfconnect-ble_samples
- test-fw-nrfconnect-chip
- test-fw-nrfconnect-fem
- test-fw-nrfconnect-nfc
- test-fw-nrfconnect-nrf-iot_libmodem-nrf
- test-fw-nrfconnect-nrf-iot_lwm2m
- test-fw-nrfconnect-nrf-iot_samples
- test-fw-nrfconnect-nrf-iot_serial_lte_modem
- test-fw-nrfconnect-nrf-iot_thingy91
- test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
- test-fw-nrfconnect-nrf_crypto
- test-fw-nrfconnect-proprietary_esb
- test-fw-nrfconnect-ps-main
- test-fw-nrfconnect-rpc
- test-fw-nrfconnect-rs
- test-fw-nrfconnect-tfm
- test-fw-nrfconnect-thread-main
- test-sdk-audio
- test-sdk-dfu
- test-sdk-find-my
- test-sdk-mcuboot
- test-sdk-pmic-samples
- test-sdk-wifi
- test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

github-actions · 2025-06-27T10:32:32Z

You can find the documentation preview for this PR here.

nordic-segl · 2025-06-27T12:59:40Z

There are some failures

- scenarios:
    - nrf.extended.sample.ipc.icbmsg.nrf54l_no_multithreading
  platforms:
    - [email protected]/nrf54lm20a/cpuapp
    - [email protected]/nrf54lm20a/cpuapp
    - [email protected]/nrf54lm20a/cpuapp
    - [email protected]/nrf54lv10a/cpuapp
    - [email protected]/nrf54lv10a/cpuapp
  comment: "https://nordicsemi.atlassian.net/browse/NRFX-6589"

- scenarios:
    - nrf.extended.sample.ipc.icmsg.nrf54l_remote_no_multithreading
  platforms:
    - [email protected]/nrf54lv10a/cpuapp
  comment: "FPGA issue. Test works on PCA10188"

But generally works in CI:

12:09:19  INFO    - 1/2 [email protected]/nrf54h20/cpuapp samples/subsys/ipc/ipc_service/icmsg/sample.ipc.icmsg.nrf54h20_remote_no_multithreading PASSED (device: 1051164192, 6.594s <zephyr>)
12:09:19  INFO    - 2/2 [email protected]/nrf54h20/cpuapp samples/subsys/ipc/ipc_service/icmsg/sample.ipc.icmsg.nrf54h20 PASSED (device: 1051149552, 6.710s <zephyr>)
(no_multithreading on cpuapp doesn't compile due to NRFS/GDP)
(all three icbmsg configurations fail due to kernel panic, mcause 5 on cpuppr)

12:43:42  INFO    - 1/5 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l_no_multithreading PASSED (device: 821006974, 3.916s <zephyr>)
12:43:42  INFO    - 2/5 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l    PASSED (device: 821006974, 5.022s <zephyr>)
12:43:42  INFO    - 3/5 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icbmsg.nrf54l_remote_no_multithreading PASSED (device: 821006974, 5.275s <zephyr>)
12:43:47  INFO    - 4/5 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l_remote_no_multithreading PASSED (device: 821006974, 4.652s <zephyr>)
12:44:19  INFO    - 5/5 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icbmsg.nrf54l   PASSED (device: 821006974, 5.195s <zephyr>)

12:54:57  INFO    -  1/10 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l_remote_no_multithreading PASSED (device: 1051885387, 6.466s <zephyr>)
12:54:57  INFO    -  2/10 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l    PASSED (device: 1051757207, 6.969s <zephyr>)
12:54:57  INFO    -  3/10 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icbmsg.nrf54l_remote_no_multithreading PASSED (device: 1051811315, 7.070s <zephyr>)
12:54:57  INFO    -  4/10 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icbmsg.nrf54l   PASSED (device: 1051500724, 7.189s <zephyr>)
12:55:02  INFO    -  5/10 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l_no_multithreading PASSED (device: 1051885387, 5.678s <zephyr>)
12:55:02  INFO    -  6/10 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l_no_multithreading PASSED (device: 1051757207, 6.225s <zephyr>)
12:55:04  INFO    -  7/10 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icbmsg.nrf54l_remote_no_multithreading PASSED (device: 1051500724, 7.330s <zephyr>)
12:55:04  INFO    -  8/10 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l PASSED (device: 1051811315, 7.511s <zephyr>)
12:55:54  INFO    -  9/10 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l_remote_no_multithreading PASSED (device: 1051500724, 5.830s <zephyr>)
12:55:54  INFO    - 10/10 [email protected]/nrf54lm20a/cpuapp nrf.extended.sample.ipc.icbmsg.nrf54l PASSED (device: 1051885387, 6.520s <zephyr>)

14:13:45  INFO    - 1/4 [email protected]/nrf54lv10a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l_no_multithreading PASSED (device: 175104789, 3.520s <zephyr>)
14:13:48  INFO    - 2/4 [email protected]/nrf54lv10a/cpuapp nrf.extended.sample.ipc.icbmsg.nrf54l_remote_no_multithreading PASSED (device: 175104789, 4.990s <zephyr>)
14:13:53  INFO    - 3/4 [email protected]/nrf54lv10a/cpuapp nrf.extended.sample.ipc.icbmsg.nrf54l    PASSED (device: 175104789, 5.094s <zephyr>)
14:13:58  INFO    - 4/4 [email protected]/nrf54lv10a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l     PASSED (device: 175104789, 4.464s <zephyr>)

12:55:05  INFO    - 1/5 [email protected]/nrf54lv10a/cpuapp nrf.extended.sample.ipc.icbmsg.nrf54l_remote_no_multithreading PASSED (device: 1051628602, 13.591s <zephyr>)
12:55:05  INFO    - 2/5 [email protected]/nrf54lv10a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l_no_multithreading PASSED (device: 1051628602, 6.920s <zephyr>)
12:55:11  INFO    - 3/5 [email protected]/nrf54lv10a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l_remote_no_multithreading PASSED (device: 1051628602, 8.174s <zephyr>)
12:55:20  INFO    - 4/5 [email protected]/nrf54lv10a/cpuapp nrf.extended.sample.ipc.icmsg.nrf54l     PASSED (device: 1051628602, 8.051s <zephyr>)
12:55:55  INFO    - 5/5 [email protected]/nrf54lv10a/cpuapp nrf.extended.sample.ipc.icbmsg.nrf54l    PASSED (device: 1051628602, 8.491s <zephyr>)

samples/zephyr/subsys/ipc/ipc_service/icmsg/Kconfig.sysbuild

nordicjm · 2025-06-30T08:49:54Z

samples/zephyr/subsys/ipc/ipc_service/icmsg/CMakeLists.txt

error if not using sysbuild?

Isn't SysBuild enabled by default for these targets?
Is CONFIG_BOARD_STM32H747I_DISCO also using SysBuild?

someone can do west build --no-sysbuild and would get a broken image, we don't care about stm32 builds not working

As stated above, I don't think there is need to improve this sample because this is a temporary solution.

I added check at line 15.

However, I feel this breaks idea of copying samples/tests from upstream to sdk-nrf so it can be executed on target existing only in downstream (54LV10).
With this check, sample is no longer identical with the upstream version.

katgiadla

LGTM

Include samples: subsys: ipc: ipc_service: icmsg: Enable sample on nRF54H20 sdk-zephyr: pull/2998/head zephyrproject-rtos: pull/92287/head Signed-off-by: Sebastian Głąb <[email protected]>

nordic-segl requested review from a team, PerMac, katgiadla and nordic-piks as code owners June 27, 2025 09:34

github-actions bot added the manifest label Jun 27, 2025

nordic-piks approved these changes Jun 27, 2025

View reviewed changes

NordicBuilder added manifest-zephyr DNM labels Jun 27, 2025

nordic-segl force-pushed the NRFX-7821_Run-samples-subsys-ipc-ipc_service-icmsg-on-more-targets branch from 8b37fa2 to 0d51959 Compare June 27, 2025 09:42

miha-nordic approved these changes Jun 27, 2025

View reviewed changes

nordic-segl force-pushed the NRFX-7821_Run-samples-subsys-ipc-ipc_service-icmsg-on-more-targets branch 2 times, most recently from 4540abc to 1972095 Compare June 27, 2025 10:06

nordicjm reviewed Jun 30, 2025

View reviewed changes

nordic-segl force-pushed the NRFX-7821_Run-samples-subsys-ipc-ipc_service-icmsg-on-more-targets branch 2 times, most recently from f8382a7 to 7687978 Compare July 8, 2025 06:05

nordic-segl force-pushed the NRFX-7821_Run-samples-subsys-ipc-ipc_service-icmsg-on-more-targets branch from 7687978 to 271a774 Compare July 11, 2025 07:43

nordic-segl force-pushed the NRFX-7821_Run-samples-subsys-ipc-ipc_service-icmsg-on-more-targets branch 3 times, most recently from 541e761 to 0a578c8 Compare July 25, 2025 07:42

nordic-segl requested a review from nordicjm July 25, 2025 07:54

katgiadla approved these changes Jul 25, 2025

View reviewed changes

nordic-segl force-pushed the NRFX-7821_Run-samples-subsys-ipc-ipc_service-icmsg-on-more-targets branch from 0a578c8 to 6442615 Compare August 18, 2025 06:43

manifest: update sdk-zephyr

ca2500d

Include samples: subsys: ipc: ipc_service: icmsg: Enable sample on nRF54H20 sdk-zephyr: pull/2998/head zephyrproject-rtos: pull/92287/head Signed-off-by: Sebastian Głąb <[email protected]>

nordic-segl force-pushed the NRFX-7821_Run-samples-subsys-ipc-ipc_service-icmsg-on-more-targets branch from 6442615 to ca2500d Compare September 16, 2025 09:37

NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Sep 16, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Nrfx 7821 run samples subsys ipc ipc service icmsg on more targets #23019

Nrfx 7821 run samples subsys ipc ipc service icmsg on more targets #23019

nordic-segl commented Jun 27, 2025

Uh oh!

NordicBuilder commented Jun 27, 2025 •

edited

Loading

Uh oh!

NordicBuilder commented Jun 27, 2025 •

edited

Loading

Uh oh!

github-actions bot commented Jun 27, 2025

Uh oh!

nordic-segl commented Jun 27, 2025

Uh oh!

Uh oh!

nordicjm Jun 30, 2025

Uh oh!

nordic-segl Jun 30, 2025

Uh oh!

nordicjm Jun 30, 2025

Uh oh!

nordic-segl Jun 30, 2025

Uh oh!

nordic-segl Jul 25, 2025 •

edited

Loading

Uh oh!

katgiadla left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Nrfx 7821 run samples subsys ipc ipc service icmsg on more targets #23019

Are you sure you want to change the base?

Nrfx 7821 run samples subsys ipc ipc service icmsg on more targets #23019

Conversation

nordic-segl commented Jun 27, 2025

Uh oh!

NordicBuilder commented Jun 27, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

NordicBuilder commented Jun 27, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CI Information

Inputs:

Sources:

Github labels

Outputs:

Toolchain

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

Uh oh!

github-actions bot commented Jun 27, 2025

Uh oh!

nordic-segl commented Jun 27, 2025

Uh oh!

Uh oh!

nordicjm Jun 30, 2025

Choose a reason for hiding this comment

Uh oh!

nordic-segl Jun 30, 2025

Choose a reason for hiding this comment

Uh oh!

nordicjm Jun 30, 2025

Choose a reason for hiding this comment

Uh oh!

nordic-segl Jun 30, 2025

Choose a reason for hiding this comment

Uh oh!

nordic-segl Jul 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

katgiadla left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

NordicBuilder commented Jun 27, 2025 •

edited

Loading

NordicBuilder commented Jun 27, 2025 •

edited

Loading

nordic-segl Jul 25, 2025 •

edited

Loading