samples: ipc: Modify icmsg_me sample allowing any multiendpoint backend

There are more ipc_service backends that supports multiple endpoint. This sample can be used for any of those backends, so this commits makes the sample more generic. The default backend it still icmsg_me, but now, the sample has files and instructions for icmsg_with_buf backend. Signed-off-by: Dominik Kilian <[email protected]>
zephyrproject-rtos · Nov 13, 2023 · 23254f2 · 23254f2
1 parent 28df449
commit 23254f2
Show file tree

Hide file tree

Showing 17 changed files with 208 additions and 12 deletions.
diff --git a/samples/subsys/ipc/ipc_service/icmsg_me/sample.yaml b/samples/subsys/ipc/ipc_service/icmsg_me/sample.yaml
diff --git a/...s/ipc/ipc_service/icmsg_me/CMakeLists.txt → ...ipc_service/multi_endpoint/CMakeLists.txt b/...s/ipc/ipc_service/icmsg_me/CMakeLists.txt → ...ipc_service/multi_endpoint/CMakeLists.txt
@@ -15,5 +15,3 @@ endif()
 project(ipc_service)
 
 target_sources(app PRIVATE src/main.c)
-
-include(ExternalProject)
diff --git a/...ipc/ipc_service/icmsg_me/Kconfig.sysbuild → ...c_service/multi_endpoint/Kconfig.sysbuild b/...ipc/ipc_service/icmsg_me/Kconfig.sysbuild → ...c_service/multi_endpoint/Kconfig.sysbuild
diff --git a/samples/subsys/ipc/ipc_service/multi_endpoint/README.rst b/samples/subsys/ipc/ipc_service/multi_endpoint/README.rst
@@ -0,0 +1,73 @@
+.. _ipc_multi_endpoint_sample:
+
+IPC Service - Multi-endpoint Sample Application
+###############################################
+
+This application demonstrates how to use IPC Service with multiple endpoints.
+By default, it uses the ``icmsg_me`` backend.
+You can also configure it to use the ``icbmsg`` backend.
+
+Building the application for nrf5340dk_nrf5340_cpuapp
+*****************************************************
+
+.. zephyr-app-commands::
+   :zephyr-app: samples/subsys/ipc/ipc_service/multi_endpoint
+   :board: nrf5340dk_nrf5340_cpuapp
+   :goals: debug
+
+Open a serial terminal (for example Minicom or PuTTY) and connect the board with the following settings:
+
+* Speed: 115200
+* Data: 8 bits
+* Parity: None
+* Stop bits: 1
+
+After resetting the board, the following message will appear on the corresponding
+serial port:
+
+.. code-block:: console
+
+   *** Booting Zephyr OS build v3.4.0-rc1-108-gccfbac8b0721 ***
+   IPC-service HOST [INST 0 - ENDP A] demo started
+   IPC-service HOST [INST 0 - ENDP B] demo started
+   IPC-service HOST [INST 1] demo started
+   HOST [0A]: 1
+   HOST [0A]: 3
+   HOST [0B]: 1
+   HOST [1]: 1
+   ...
+   HOST [0A]: 99
+   IPC-service HOST [INST 0 - ENDP A] demo ended.
+   HOST [0B]: 99
+   IPC-service HOST [INST 0 - ENDP B] demo ended.
+   HOST [1]: 99
+   IPC-service HOST [INST 1] demo ended.
+
+.. code-block:: console
+
+   *** Booting Zephyr OS build v3.4.0-rc1-108-gccfbac8b0721 ***
+   IPC-service REMOTE [INST 0 - ENDP A] demo started
+   IPC-service REMOTE [INST 0 - ENDP B] demo started
+   IPC-service REMOTE [INST 1] demo started
+   REMOTE [0A]: 0
+   REMOTE [0A]: 2
+   ...
+   REMOTE [0A]: 98
+   IPC-service REMOTE [INST 0 - ENDP A] demo ended.
+   REMOTE [0B]: 98
+   IPC-service REMOTE [INST 0 - ENDP B] demo ended.
+   REMOTE [1]: 98
+   IPC-service REMOTE [INST 1] demo ended.
+
+
+Changing the backend
+********************
+
+To change the backend to ``icbmsg``, switch the devicetree
+overlay files as follows:
+
+.. code-block:: console
+
+   west build -b nrf5340dk_nrf5340_cpuapp --sysbuild -- \
+   -DDTC_OVERLAY_FILE=boards/nrf5340dk_nrf5340_cpuapp_icbmsg.overlay \
+   -Dremote_DTC_OVERLAY_FILE=boards/nrf5340dk_nrf5340_cpunet_icbmsg.overlay
diff --git a/...g_me/boards/nrf5340dk_nrf5340_cpuapp.conf → ...oint/boards/nrf5340dk_nrf5340_cpuapp.conf b/...g_me/boards/nrf5340dk_nrf5340_cpuapp.conf → ...oint/boards/nrf5340dk_nrf5340_cpuapp.conf
diff --git a/...e/boards/nrf5340dk_nrf5340_cpuapp.overlay → ...t/boards/nrf5340dk_nrf5340_cpuapp.overlay b/...e/boards/nrf5340dk_nrf5340_cpuapp.overlay → ...t/boards/nrf5340dk_nrf5340_cpuapp.overlay
diff --git a/samples/subsys/ipc/ipc_service/multi_endpoint/boards/nrf5340dk_nrf5340_cpuapp_icbmsg.overlay b/samples/subsys/ipc/ipc_service/multi_endpoint/boards/nrf5340dk_nrf5340_cpuapp_icbmsg.overlay
@@ -0,0 +1,57 @@
+/*
+ * Copyright (c) 2023 Nordic Semiconductor ASA
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+/ {
+	chosen {
+		/delete-property/ zephyr,ipc_shm;
+	};
+
+	reserved-memory {
+		/delete-node/ memory@20070000;
+
+		sram_ipc0_tx: memory@20070000 {
+			reg = <0x20070000 0x4000>;
+		};
+
+		sram_ipc0_rx: memory@20074000 {
+			reg = <0x20074000 0x4000>;
+		};
+
+		sram_ipc1_tx: memory@20078000 {
+			reg = <0x20078000 0x4000>;
+		};
+
+		sram_ipc1_rx: memory@2007C000 {
+			reg = <0x2007C000 0x4000>;
+		};
+	};
+
+	ipc {
+		/delete-node/ ipc0;
+
+		ipc0: ipc0 {
+			compatible = "zephyr,ipc-icbmsg";
+			tx-region = <&sram_ipc0_tx>;
+			rx-region = <&sram_ipc0_rx>;
+			tx-blocks = <16>;
+			rx-blocks = <24>;
+			mboxes = <&mbox 0>, <&mbox 1>;
+			mbox-names = "tx", "rx";
+			status = "okay";
+		};
+
+		ipc1: ipc1 {
+			compatible = "zephyr,ipc-icbmsg";
+			tx-region = <&sram_ipc1_tx>;
+			rx-region = <&sram_ipc1_rx>;
+			tx-blocks = <32>;
+			rx-blocks = <48>;
+			mboxes = <&mbox 2>, <&mbox 3>;
+			mbox-names = "tx", "rx";
+			status = "okay";
+		};
+	};
+};
diff --git a/.../subsys/ipc/ipc_service/icmsg_me/prj.conf → ...s/ipc/ipc_service/multi_endpoint/prj.conf b/.../subsys/ipc/ipc_service/icmsg_me/prj.conf → ...s/ipc/ipc_service/multi_endpoint/prj.conf
@@ -1,4 +1,5 @@
 CONFIG_PRINTK=y
+CONFIG_LOG_PRINTK=n
 
 CONFIG_IPC_SERVICE=y
 CONFIG_MBOX=y

diff --git a/...pc_service/icmsg_me/remote/CMakeLists.txt → ...vice/multi_endpoint/remote/CMakeLists.txt b/...pc_service/icmsg_me/remote/CMakeLists.txt → ...vice/multi_endpoint/remote/CMakeLists.txt
diff --git a/...mote/boards/nrf5340dk_nrf5340_cpunet.conf → ...mote/boards/nrf5340dk_nrf5340_cpunet.conf b/...mote/boards/nrf5340dk_nrf5340_cpunet.conf → ...mote/boards/nrf5340dk_nrf5340_cpunet.conf
diff --git a/...e/boards/nrf5340dk_nrf5340_cpunet.overlay → ...e/boards/nrf5340dk_nrf5340_cpunet.overlay b/...e/boards/nrf5340dk_nrf5340_cpunet.overlay → ...e/boards/nrf5340dk_nrf5340_cpunet.overlay
diff --git a/...bsys/ipc/ipc_service/multi_endpoint/remote/boards/nrf5340dk_nrf5340_cpunet_icbmsg.overlay b/...bsys/ipc/ipc_service/multi_endpoint/remote/boards/nrf5340dk_nrf5340_cpunet_icbmsg.overlay
@@ -0,0 +1,57 @@
+/*
+ * Copyright (c) 2023 Nordic Semiconductor ASA
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+/ {
+	chosen {
+		/delete-property/ zephyr,ipc_shm;
+	};
+
+	reserved-memory {
+		/delete-node/ memory@20070000;
+
+		sram_ipc0_rx: memory@20070000 {
+			reg = <0x20070000 0x4000>;
+		};
+
+		sram_ipc0_tx: memory@20074000 {
+			reg = <0x20074000 0x4000>;
+		};
+
+		sram_ipc1_rx: memory@20078000 {
+			reg = <0x20078000 0x4000>;
+		};
+
+		sram_ipc1_tx: memory@2007C000 {
+			reg = <0x2007C000 0x4000>;
+		};
+	};
+
+	ipc {
+		/delete-node/ ipc0;
+
+		ipc0: ipc0 {
+			compatible = "zephyr,ipc-icbmsg";
+			tx-region = <&sram_ipc0_tx>;
+			rx-region = <&sram_ipc0_rx>;
+			tx-blocks = <24>;
+			rx-blocks = <16>;
+			mboxes = <&mbox 0>, <&mbox 1>;
+			mbox-names = "rx", "tx";
+			status = "okay";
+		};
+
+		ipc1: ipc1 {
+			compatible = "zephyr,ipc-icbmsg";
+			tx-region = <&sram_ipc1_tx>;
+			rx-region = <&sram_ipc1_rx>;
+			tx-blocks = <48>;
+			rx-blocks = <32>;
+			mboxes = <&mbox 2>, <&mbox 3>;
+			mbox-names = "rx", "tx";
+			status = "okay";
+		};
+	};
+};
diff --git a/.../ipc/ipc_service/icmsg_me/remote/prj.conf → ...pc_service/multi_endpoint/remote/prj.conf b/.../ipc/ipc_service/icmsg_me/remote/prj.conf → ...pc_service/multi_endpoint/remote/prj.conf
@@ -1,4 +1,5 @@
 CONFIG_PRINTK=y
+CONFIG_LOG_PRINTK=n
 
 CONFIG_IPC_SERVICE=y
 CONFIG_MBOX=y

diff --git a/...pc/ipc_service/icmsg_me/remote/src/main.c → ..._service/multi_endpoint/remote/src/main.c b/...pc/ipc_service/icmsg_me/remote/src/main.c → ..._service/multi_endpoint/remote/src/main.c
diff --git a/samples/subsys/ipc/ipc_service/multi_endpoint/sample.yaml b/samples/subsys/ipc/ipc_service/multi_endpoint/sample.yaml
@@ -0,0 +1,19 @@
+sample:
+  name: IPC Service example integration (icmsg multi endpoint backend)
+tests:
+  sample.ipc.multi_endpoint:
+    platform_allow: nrf5340dk_nrf5340_cpuapp
+    integration_platforms:
+      - nrf5340dk_nrf5340_cpuapp
+    tags: ipc
+    sysbuild: true
+    harness: remote
+  sample.ipc.multi_endpoint.icbmsg:
+    platform_allow: nrf5340dk_nrf5340_cpuapp
+    integration_platforms:
+      - nrf5340dk_nrf5340_cpuapp
+    tags: ipc
+    sysbuild: true
+    extra_args:
+      DTC_OVERLAY_FILE=boards/nrf5340dk_nrf5340_cpuapp_icbmsg.overlay
+      remote_DTC_OVERLAY_FILE=boards/nrf5340dk_nrf5340_cpunet_icbmsg.overlay
diff --git a/...ubsys/ipc/ipc_service/icmsg_me/src/main.c → ...ipc/ipc_service/multi_endpoint/src/main.c b/...ubsys/ipc/ipc_service/icmsg_me/src/main.c → ...ipc/ipc_service/multi_endpoint/src/main.c
diff --git a/...s/ipc/ipc_service/icmsg_me/sysbuild.cmake → ...ipc_service/multi_endpoint/sysbuild.cmake b/...s/ipc/ipc_service/icmsg_me/sysbuild.cmake → ...ipc_service/multi_endpoint/sysbuild.cmake
Original file line number	Diff line number	Diff line change
Expand Up		@@ -15,5 +15,3 @@ endif()
		project(ipc_service)

		target_sources(app PRIVATE src/main.c)

		include(ExternalProject)