applications: protocols_serialization: refactor snippets

...to ease logging.

Reorganize the protocols serialization client and server
snippets to be able to test logging over RPC without verbose
nRF RPC core & transport logging.

In ps_client:
1. Enable logging by default.
2. Make "debug" snippet only enable nRF RPC logs and other
   debug features.

In protocols_serialization:
1. Make "log_rpc" snippet enable logging over RPC and
   OpenThread logs.
2. Make "debug" snippet enable (addtionally) logging over
   UART console.

Make other Kconfig adjustments to remove build warnings
and reduce config duplication.

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

Author: Damian-Nordic

applications/protocols_serialization/Kconfig

@@ -4,10 +4,30 @@
 # SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
 #
 
+menu "Protocols serialization server"
+
+choice LOG_MODE
+	default LOG_MODE_DEFERRED
+endchoice
+
+config LOG_BUFFER_SIZE
+	default 8192
+
+config LOG_FUNC_NAME_PREFIX_DBG
+	default n
+
+config LOG_BACKEND_RTT
+	default n
+
+config LOG_BACKEND_UART
+	default n
+
+module = NRF_PS_SERVER
+module-str = nrf_ps_server
+source "${ZEPHYR_BASE}/subsys/logging/Kconfig.template.log_config"
+
+endmenu # "Protocols serialization server"
+
 menu "Zephyr Kernel"
 source "Kconfig.zephyr"
 endmenu
-
-module = NRF_RPC_REMOTE
-module-str = nrf_rpc_remote
-source "${ZEPHYR_BASE}/subsys/logging/Kconfig.template.log_config"

applications/protocols_serialization/prj.conf

@@ -5,8 +5,6 @@
 #
 CONFIG_HEAP_MEM_POOL_SIZE=4096
 
-CONFIG_THREAD_CUSTOM_DATA=y
-
 CONFIG_NRF_RPC=y
 CONFIG_NRF_RPC_CBOR=y
 CONFIG_NRF_RPC_ZCBOR_BACKUPS=1

applications/protocols_serialization/snippets/debug/debug.conf

@@ -4,27 +4,18 @@
 # SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
 #
 
-# Enable multithread support in gdb
+# Reduce optimization level for easier debugging
 CONFIG_DEBUG=y
 
 # Enable asserts
 CONFIG_ASSERT=y
-#CONFIG_ASSERT_NO_COND_INFO=y
 
+# Enable UART console
 CONFIG_LOG=y
-
-# Use separate thread for logging
-CONFIG_LOG_MODE_DEFERRED=y
-CONFIG_LOG_BUFFER_SIZE=4096
-CONFIG_LOG_FUNC_NAME_PREFIX_DBG=n
-CONFIG_LOG_BACKEND_RTT=n
 CONFIG_LOG_BACKEND_UART=y
 
-# Increase max log level to informative
-CONFIG_LOG_MAX_LEVEL=4
-
-CONFIG_NRF_RPC_REMOTE_LOG_LEVEL_DBG=y
+# Enable verbose logging
+CONFIG_NRF_PS_SERVER_LOG_LEVEL_DBG=y
 CONFIG_NRF_RPC_TR_LOG_LEVEL_DBG=y
 CONFIG_NRF_RPC_LOG_LEVEL_DBG=y
 CONFIG_NRF_RPC_OS_LOG_LEVEL_DBG=y
-CONFIG_NET_L2_IEEE802154_LOG_LEVEL_ERR=y

applications/protocols_serialization/snippets/log_rpc/log_rpc.conf

@@ -4,8 +4,13 @@
 # SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
 #
 
+CONFIG_LOG=y
 CONFIG_LOG_BACKEND_RPC=y
 CONFIG_LOG_BACKEND_RPC_CRASH_LOG=y
 
 # nRF RPC requires slightly bigger stack than default
 CONFIG_LOG_PROCESS_THREAD_STACK_SIZE=1024
+
+# Enable OpenThread logs along with logging over RPC
+CONFIG_OPENTHREAD_DEBUG=y
+CONFIG_OPENTHREAD_LOG_LEVEL_INFO=y

applications/protocols_serialization/snippets/openthread/openthread.conf

@@ -4,11 +4,6 @@
 # SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
 #
 
-CONFIG_OPENTHREAD_SHELL=y
-
-CONFIG_SHELL_ARGC_MAX=26
-CONFIG_SHELL_CMD_BUFF_SIZE=416
-
 # Enable OpenThread features set
 CONFIG_OPENTHREAD_NORDIC_LIBRARY_MASTER=y
 CONFIG_OPENTHREAD_SOURCES=y

applications/protocols_serialization/src/main.c

@@ -8,7 +8,7 @@
 
 #include <nrf_rpc.h>
 
-LOG_MODULE_REGISTER(nrf_rpc_remote, CONFIG_NRF_RPC_REMOTE_LOG_LEVEL);
+LOG_MODULE_REGISTER(nrf_ps_server, CONFIG_NRF_PS_SERVER_LOG_LEVEL);
 
 static void err_handler(const struct nrf_rpc_err_report *report)
 {

samples/nrf_rpc/ps_client/Kconfig

@@ -4,7 +4,14 @@
 # SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
 #
 
-mainmenu "Protocols serialization client"
+menu "Protocols serialization client"
+
+config BT_CTLR
+	default n
+
+choice BT_NUS_LOG_LEVEL_CHOICE
+	default BT_NUS_LOG_LEVEL_DBG if DEBUG
+endchoice
 
 config RPC_CRASH_LOG_READ_BUFFER_SIZE
 	int "Crash log read buffer size"
@@ -14,13 +21,12 @@ config RPC_CRASH_LOG_READ_BUFFER_SIZE
 	  Size of the stack buffer used for reading a single chunk of the crash
 	  log from the server device.
 
-config BT_CTLR
-	default n
+module = NRF_PS_CLIENT
+module-str = nrf_ps_client
+source "${ZEPHYR_BASE}/subsys/logging/Kconfig.template.log_config"
+
+endmenu # "Protocols serialization client"
 
 menu "Zephyr Kernel"
 source "Kconfig.zephyr"
 endmenu
-
-module = NRF_PS_CLIENT
-module-str = nrf_ps_client
-source "${ZEPHYR_BASE}/subsys/logging/Kconfig.template.log_config"

samples/nrf_rpc/ps_client/prj.conf

@@ -5,8 +5,6 @@
 #
 CONFIG_HEAP_MEM_POOL_SIZE=4096
 
-CONFIG_THREAD_CUSTOM_DATA=y
-
 CONFIG_NRF_RPC=y
 CONFIG_NRF_RPC_CBOR=y
 CONFIG_NRF_RPC_ZCBOR_BACKUPS=1
@@ -18,4 +16,9 @@ CONFIG_UART_LINE_CTRL=y
 CONFIG_UART_INTERRUPT_DRIVEN=y
 
 CONFIG_SHELL=y
-CONFIG_NET_SHELL=y
+CONFIG_LOG=y
+CONFIG_LOG_MODE_DEFERRED=y
+CONFIG_LOG_BUFFER_SIZE=4096
+CONFIG_LOG_FUNC_NAME_PREFIX_DBG=n
+CONFIG_LOG_BACKEND_RTT=n
+CONFIG_SHELL_LOG_BACKEND=y

samples/nrf_rpc/ps_client/snippets/debug/debug.conf

@@ -4,27 +4,14 @@
 # SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
 #
 
-# Enable multithread support in gdb
+# Reduce optimization level for easier debugging.
 CONFIG_DEBUG=y
 
 # Enable asserts
 CONFIG_ASSERT=y
-#CONFIG_ASSERT_NO_COND_INFO=y
-
-CONFIG_LOG=y
-
-# Use separate thread for logging
-CONFIG_LOG_MODE_DEFERRED=y
-CONFIG_LOG_BUFFER_SIZE=4096
-CONFIG_LOG_FUNC_NAME_PREFIX_DBG=n
-CONFIG_LOG_BACKEND_RTT=n
-CONFIG_SHELL_LOG_BACKEND=y
-
-# Increase max log level to informative
-CONFIG_LOG_MAX_LEVEL=4
 
+# Enable verbose logging
 CONFIG_NRF_PS_CLIENT_LOG_LEVEL_DBG=y
 CONFIG_NRF_RPC_TR_LOG_LEVEL_DBG=y
 CONFIG_NRF_RPC_LOG_LEVEL_DBG=y
 CONFIG_NRF_RPC_OS_LOG_LEVEL_DBG=y
-CONFIG_BT_NUS_LOG_LEVEL_DBG=y

samples/nrf_rpc/ps_client/snippets/log_rpc/log_rpc.conf

@@ -4,5 +4,4 @@
 # SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
 #
 
-CONFIG_LOG=y
 CONFIG_LOG_FORWARDER_RPC=y