nrf_rpc: unify nRF RPC initialization

Currently, almost each component implementing RPC function
encoders and decoders, provides an option to automatically
start nRF RPC during the system startup. Reduce the code
duplication by introducing CONFIG_NRF_RPC_INIT that handles
starting nRF RPC regardless of the enabled RPC serialization
libraries.

Signed-off-by: Damian Krolik <[email protected]>

Author: Damian-Nordic

samples/nrf_rpc/protocols_serialization/client/snippets/ble/ble.conf

@@ -9,7 +9,7 @@ CONFIG_SYSTEM_WORKQUEUE_STACK_SIZE=2144
 # Enable and configure Bluetooth LE
 CONFIG_BT=y
 CONFIG_BT_RPC_STACK=y
-CONFIG_BT_RPC_INITIALIZE_NRF_RPC=n
+CONFIG_NRF_RPC_INIT=n
 CONFIG_BT_RPC_GATT_SRV_MAX=4
 CONFIG_BT_PERIPHERAL=y
 CONFIG_BT_MAX_CONN=1

samples/nrf_rpc/protocols_serialization/server/snippets/ble/ble.conf

@@ -8,7 +8,7 @@
 CONFIG_BT=y
 CONFIG_BT_RPC=y
 CONFIG_BT_RPC_HOST=y
-CONFIG_BT_RPC_INITIALIZE_NRF_RPC=n
+CONFIG_NRF_RPC_INIT=n
 
 CONFIG_BT_PERIPHERAL=y
 CONFIG_BT_MAX_CONN=1

subsys/bluetooth/rpc/Kconfig

@@ -9,6 +9,7 @@ config BT_RPC
 	default y if BT_RPC_STACK
 	select NRF_RPC
 	select NRF_RPC_CBOR
+	imply NRF_RPC_INIT
 	select EXPERIMENTAL
 	depends on BT
 	depends on !BT_ISO_UNICAST
@@ -48,13 +49,6 @@ config BT_RPC_HOST
 
 endchoice
 
-config BT_RPC_INITIALIZE_NRF_RPC
-	bool "Automatically initialize nRF RPC library"
-	default y
-	help
-	  Initialize nRF RPC library during the system startup. Disabling this
-	  option allow user to initialize it in a different way.
-
 config BT_RPC_GATT_SRV_MAX
 	int "Maximum number of GATT services"
 	default 4

subsys/bluetooth/rpc/common/bt_rpc_common.c

@@ -28,34 +28,6 @@ NRF_RPC_IPC_TRANSPORT(bt_rpc_tr, DEVICE_DT_GET(DT_NODELABEL(ipc0)), "bt_rpc_ept"
 #endif
 NRF_RPC_GROUP_DEFINE(bt_rpc_grp, "bt_rpc", &bt_rpc_tr, NULL, NULL, NULL);
 
-#if CONFIG_BT_RPC_INITIALIZE_NRF_RPC
-static void err_handler(const struct nrf_rpc_err_report *report)
-{
-	LOG_ERR("nRF RPC error %d ocurred. See nRF RPC logs for more details.",
-		report->code);
-	k_oops();
-}
-
-static int serialization_init(void)
-{
-
-	int err;
-
-	LOG_DBG("Init begin");
-
-	err = nrf_rpc_init(err_handler);
-	if (err) {
-		return -EINVAL;
-	}
-
-	LOG_DBG("Init done\n");
-
-	return 0;
-}
-
-SYS_INIT(serialization_init, APPLICATION, CONFIG_APPLICATION_INIT_PRIORITY);
-#endif /* CONFIG_BT_RPC_INITIALIZE_NRF_RPC */
-
 enum {
 	CHECK_ENTRY_FLAGS,
 	CHECK_ENTRY_UINT,

subsys/net/openthread/rpc/Kconfig

@@ -96,11 +96,4 @@ config OPENTHREAD_RPC_ERASE_SETTINGS
 
 endmenu # "OpenThread over RPC server configuration"
 
-config OPENTHREAD_RPC_INITIALIZE_NRF_RPC
-	bool "Automatically initialize nRF RPC library"
-	default n
-	help
-	  Initialize nRF RPC library during the system startup. Disabling this
-	  option allow user to initialize it in a different way.
-
 endif # OPENTHREAD_RPC

subsys/net/openthread/rpc/common/ot_rpc_group.c

@@ -24,25 +24,3 @@ NRF_RPC_IPC_TRANSPORT(ot_rpc_tr, DEVICE_DT_GET(DT_NODELABEL(ipc0)), "ot_rpc_ept"
 #endif
 NRF_RPC_GROUP_DEFINE(ot_group, "ot", &ot_rpc_tr, NULL, NULL, NULL);
 LOG_MODULE_REGISTER(ot_rpc, LOG_LEVEL_DBG);
-
-#ifdef CONFIG_OPENTHREAD_RPC_INITIALIZE_NRF_RPC
-static void err_handler(const struct nrf_rpc_err_report *report)
-{
-	LOG_ERR("nRF RPC error %d ocurred. See nRF RPC logs for more details", report->code);
-	k_oops();
-}
-
-static int serialization_init(void)
-{
-	int err;
-
-	err = nrf_rpc_init(err_handler);
-	if (err) {
-		return -EINVAL;
-	}
-
-	return 0;
-}
-
-SYS_INIT(serialization_init, POST_KERNEL, CONFIG_APPLICATION_INIT_PRIORITY);
-#endif /* CONFIG_OPENTHREAD_RPC_INITIALIZE_NRF_RPC */

subsys/nfc/rpc/common/nfc_rpc_group.c

@@ -25,25 +25,3 @@ NRF_RPC_IPC_TRANSPORT(nfc_rpc_tr, DEVICE_DT_GET(DT_NODELABEL(ipc0)), "nfc_rpc_ep
 #endif
 NRF_RPC_GROUP_DEFINE(nfc_group, "nfc", &nfc_rpc_tr, NULL, NULL, NULL);
 LOG_MODULE_REGISTER(nfc_rpc, LOG_LEVEL_DBG);
-
-#ifdef CONFIG_NFC_RPC_INITIALIZE_NRF_RPC
-static void err_handler(const struct nrf_rpc_err_report *report)
-{
-	LOG_ERR("nRF RPC error %d ocurred. See nRF RPC logs for more details", report->code);
-	k_oops();
-}
-
-static int serialization_init(void)
-{
-	int err;
-
-	err = nrf_rpc_init(err_handler);
-	if (err) {
-		return -EINVAL;
-	}
-
-	return 0;
-}
-
-SYS_INIT(serialization_init, POST_KERNEL, CONFIG_APPLICATION_INIT_PRIORITY);
-#endif /* CONFIG_NFC_RPC_INITIALIZE_NRF_RPC */

subsys/nrf_rpc/CMakeLists.txt

@@ -18,4 +18,6 @@ zephyr_library_sources_ifdef(CONFIG_NRF_RPC_CALLBACK_PROXY nrf_rpc_cbkproxy.c)
 
 zephyr_library_sources_ifdef(CONFIG_NRF_RPC_UART_TRANSPORT nrf_rpc_uart.c)
 
+zephyr_library_sources_ifdef(CONFIG_NRF_RPC_INIT nrf_rpc_init.c)
+
 add_subdirectory_ifdef(CONFIG_NRF_RPC_UTILS rpc_utils)

subsys/nrf_rpc/Kconfig

@@ -15,6 +15,12 @@ config _NRF_RPC_DUMMY_SELECT
 	default y
 	select THREAD_CUSTOM_DATA
 
+config NRF_RPC_CBOR
+	select ZCBOR
+	select ZCBOR_STOP_ON_ERROR
+
+# End of Zephyr port dependencies selection
+
 choice NRF_RPC_TRANSPORT
 	prompt "NRF RPC transport"
 	default NRF_RPC_IPC_SERVICE
@@ -109,13 +115,6 @@ endif # NRF_RPC_UART_RELIABLE
 
 endmenu # "nRF RPC over UART configuration"
 
-config NRF_RPC_CBOR
-	bool
-	select ZCBOR
-	select ZCBOR_STOP_ON_ERROR
-
-# End of Zephyr port dependencies selection
-
 config NRF_RPC_THREAD_STACK_SIZE
 	int "Stack size of thread from thread pool"
 	default 1024
@@ -174,6 +173,11 @@ config NRF_RPC_CBKPROXY_IN_SLOTS
 
 endif # NRF_RPC_CALLBACK_PROXY
 
+config NRF_RPC_INIT
+	bool "Automatically initialize nRF RPC library"
+	help
+	  Initialize nRF RPC library during the system startup.
+
 module = NRF_RPC
 module-str = NRF_RPC
 source "$(ZEPHYR_BASE)/subsys/logging/Kconfig.template.log_config"
@@ -194,6 +198,14 @@ source "$(ZEPHYR_BASE)/subsys/logging/Kconfig.template.log_config"
 
 endif # NRF_RPC_CBOR
 
+if NRF_RPC_INIT
+
+module = NRF_RPC_INIT
+module-str = NRF_RPC_INIT
+source "$(ZEPHYR_BASE)/subsys/logging/Kconfig.template.log_config"
+
+endif # NRF_RPC_INIT
+
 rsource "rpc_utils/Kconfig"
 
 endif # NRF_RPC

subsys/nrf_rpc/nrf_rpc_init.c

@@ -0,0 +1,37 @@
+/*
+ * Copyright (c) 2025 Nordic Semiconductor ASA
+ *
+ * SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
+ */
+
+#define NRF_RPC_LOG_MODULE NRF_RPC_INIT
+#include <nrf_rpc_log.h>
+
+#include <zephyr/init.h>
+
+#include <nrf_rpc.h>
+
+static void err_handler(const struct nrf_rpc_err_report *report)
+{
+	LOG_ERR("nRF RPC error %d ocurred. See nRF RPC logs for more details",
+		report->code);
+	k_oops();
+}
+
+static int nrf_rpc_sys_init(void)
+{
+	int err;
+
+	LOG_DBG("RPC init begin");
+
+	err = nrf_rpc_init(err_handler);
+	if (err) {
+		return -EINVAL;
+	}
+
+	LOG_DBG("RPC init done");
+
+	return 0;
+}
+
+SYS_INIT(nrf_rpc_sys_init, APPLICATION, CONFIG_APPLICATION_INIT_PRIORITY);