soc: riscv: telink_w91: create icmsg workqueue instead of system

Resolved IPC system workqueue conflict with drivers/samples

Signed-off-by: Alex Tsitsiura <s07641069@gmail.com>

Author: s07641069

samples/boards/tlsr9x/key_matrix/boards/tlsr9118bdk40d.conf

@@ -0,0 +1 @@
+CONFIG_GPIO=y

samples/boards/tlsr9x/key_matrix/boards/tlsr9118bdk40d.overlay

@@ -0,0 +1,21 @@
+/*
+ * Copyright (c) 2024 Telink Semiconductor
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+/ {
+	key_matrix {
+		compatible = "gpio-keys";
+
+		col {
+			gpios = <&gpio0 18 GPIO_ACTIVE_HIGH>,
+					<&gpio0 17 GPIO_ACTIVE_HIGH>;
+		};
+
+		row {
+			gpios = <&gpio0 16 (GPIO_PULL_DOWN | GPIO_ACTIVE_HIGH)>,
+					<&gpio0 15 (GPIO_PULL_DOWN | GPIO_ACTIVE_HIGH)>;
+		};
+	};
+};

samples/boards/tlsr9x/key_pool/boards/tlsr9118bdk40d.conf

@@ -0,0 +1 @@
+CONFIG_GPIO=y

samples/boards/tlsr9x/key_pool/boards/tlsr9118bdk40d.overlay

@@ -0,0 +1,17 @@
+/*
+ * Copyright (c) 2024 Telink Semiconductor
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+/ {
+	/* Short TL_Key1 (J20 pin 11) to ground (J20 pin 25-35) */
+	key_pool{
+		compatible = "gpio-keys";
+
+		inp {
+			gpios = <&gpio0 16 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>,
+					<&gpio0 15 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
+		};
+	};
+};

samples/boards/tlsr9x/led_pool/boards/tlsr9118bdk40d.conf

@@ -0,0 +1 @@
+CONFIG_GPIO=y

samples/boards/tlsr9x/led_pool/boards/tlsr9118bdk40d.overlay

@@ -0,0 +1,16 @@
+/*
+ * Copyright (c) 2024 Telink Semiconductor
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+/ {
+	led_pool{
+		compatible = "gpio-leds";
+
+		out {
+			gpios = <&gpio0 20 GPIO_ACTIVE_HIGH>,
+					<&gpio0 19 GPIO_ACTIVE_HIGH>;
+		};
+	};
+};

soc/riscv/riscv-privilege/telink_w91/ipc/Kconfig.ipc

@@ -45,6 +45,10 @@ config TELINK_W91_IPC_DISPATCHER_BOUND_TIMEOUT_MS
 config IPC_SERVICE_ICMSG_CB_BUF_SIZE
 	default 1024
 
+config IPC_SERVICE_ICMSG_WORK_QUEUE_STACK_SIZE
+	int "ICMSG work queue stack size"
+	default 1024
+
 if SPSC_PBUF_CACHE_FLAG || SPSC_PBUF_CACHE_ALWAYS
 
 config SPSC_PBUF_REMOTE_DCACHE_LINE

subsys/ipc/ipc_service/lib/icmsg.c

@@ -20,6 +20,9 @@
 #define SEND_BUFFER_UNUSED	0
 #define SEND_BUFFER_RESERVED	1
 
+static struct k_work_q icmsg_work_queue;
+K_THREAD_STACK_DEFINE(icmsg_work_queue_stack, CONFIG_IPC_SERVICE_ICMSG_WORK_QUEUE_STACK_SIZE);
+
 static const uint8_t magic[] = {0x45, 0x6d, 0x31, 0x6c, 0x31, 0x4b,
 				0x30, 0x72, 0x6e, 0x33, 0x6c, 0x69, 0x34};
 BUILD_ASSERT(sizeof(magic) <= RX_BUF_SIZE);
@@ -59,7 +62,7 @@ static void notify_process(struct k_work *item)
 	if (state != ICMSG_STATE_READY) {
 		int ret;
 
-		ret = k_work_reschedule(dwork, BOND_NOTIFY_REPEAT_TO);
+		ret = k_work_reschedule_for_queue(&icmsg_work_queue, dwork, BOND_NOTIFY_REPEAT_TO);
 		__ASSERT_NO_MSG(ret >= 0);
 		(void)ret;
 	}
@@ -127,7 +130,7 @@ static bool is_rx_data_available(struct icmsg_data_t *dev_data)
 
 static void submit_mbox_work(struct icmsg_data_t *dev_data)
 {
-	if (k_work_submit(&dev_data->mbox_work) < 0) {
+	if (k_work_submit_to_queue(&icmsg_work_queue, &dev_data->mbox_work) < 0) {
 		/* The mbox processing work is never canceled.
 		 * The negative error code should never be seen.
 		 */
@@ -211,6 +214,10 @@ static int mbox_init(const struct icmsg_config_t *conf,
 {
 	int err;
 
+	k_work_queue_start(&icmsg_work_queue, icmsg_work_queue_stack,
+					   K_THREAD_STACK_SIZEOF(icmsg_work_queue_stack),
+					   CONFIG_SYSTEM_WORKQUEUE_PRIORITY, NULL);
+
 	k_work_init(&dev_data->mbox_work, mbox_callback_process);
 	k_work_init_delayable(&dev_data->notify_work, notify_process);
 
@@ -263,7 +270,7 @@ int icmsg_open(const struct icmsg_config_t *conf,
 		return ret;
 	}
 
-	ret = k_work_schedule(&dev_data->notify_work, K_NO_WAIT);
+	ret = k_work_schedule_for_queue(&icmsg_work_queue, &dev_data->notify_work, K_NO_WAIT);
 	if (ret < 0) {
 		return ret;
 	}