log: use log backends to route data to usartx and/or cdc-acm

trying to find a replacement for shell log routing

Author: pennam

loader/main.c

@@ -14,6 +14,7 @@ LOG_MODULE_REGISTER(sketch);
 #include <zephyr/llext/buf_loader.h>
 #include <zephyr/shell/shell.h>
 #include <zephyr/shell/shell_uart.h>
+#include <zephyr/logging/log_ctrl.h>
 
 #include <stdlib.h>
 #include <zephyr/drivers/gpio.h>
@@ -35,11 +36,11 @@ struct sketch_header_v1 {
 #define SKETCH_FLAG_LINKED    0x02
 #define SKETCH_FLAG_IMMEDIATE 0x04
 
-#define TARGET_HAS_USB_CDC_SHELL                                                                   \
-	DT_NODE_HAS_PROP(DT_PATH(zephyr_user), cdc_acm) && CONFIG_SHELL &&                             \
+#define TARGET_HAS_USB_CDC                                                                         \
+	DT_NODE_HAS_PROP(DT_PATH(zephyr_user), cdc_acm) &&                                             \
 		(CONFIG_USB_DEVICE_STACK || CONFIG_USB_DEVICE_STACK_NEXT)
 
-#if TARGET_HAS_USB_CDC_SHELL
+#if TARGET_HAS_USB_CDC
 const struct device *const usb_dev =
 	DEVICE_DT_GET(DT_PHANDLE_BY_IDX(DT_PATH(zephyr_user), cdc_acm, 0));
 
@@ -63,6 +64,7 @@ int usb_enable(usb_dc_status_callback status_cb) {
 }
 #endif
 
+#if CONFIG_SHELL
 static int enable_shell_usb(void) {
 	bool log_backend = CONFIG_SHELL_BACKEND_SERIAL_LOG_LEVEL > 0;
 	uint32_t level = (CONFIG_SHELL_BACKEND_SERIAL_LOG_LEVEL > LOG_LEVEL_DBG) ?
@@ -75,6 +77,7 @@ static int enable_shell_usb(void) {
 	return 0;
 }
 #endif
+#endif
 
 #ifdef CONFIG_USERSPACE
 K_THREAD_STACK_DEFINE(llext_stack, CONFIG_MAIN_STACK_SIZE);
@@ -198,8 +201,9 @@ static int loader(const struct shell *sh) {
 
 	size_t sketch_buf_len = sketch_hdr->len;
 
-#if TARGET_HAS_USB_CDC_SHELL
+#if TARGET_HAS_USB_CDC
 	int debug = (!sketch_valid) || (sketch_hdr->flags & SKETCH_FLAG_DEBUG);
+#if CONFIG_SHELL
 	if (debug && strcmp(k_thread_name_get(k_current_get()), "main") == 0) {
 		// disables default shell on UART
 		shell_uninit(shell_backend_uart_get_ptr(), NULL);
@@ -214,6 +218,18 @@ static int loader(const struct shell *sh) {
 		enable_shell_usb();
 		return 0;
 	}
+#elif CONFIG_LOG
+	printk("Sketch debug mode: %s\n", debug ? "enabled" : "disabled");
+	if (!debug && log_backend_count_get() > 0) {
+		const struct log_backend *backend;
+		// Keep only the first log backend enabled
+		for (int i = 1; i < log_backend_count_get(); i++) {
+			backend = log_backend_get(i);
+			printk("Disabling log backend %p\n", backend);
+			log_backend_disable(backend);
+		}
+	}
+#endif
 #endif
 
 	if (sketch_hdr->flags & SKETCH_FLAG_LINKED) {

loader/prj.conf

@@ -6,18 +6,15 @@ CONFIG_USERSPACE=n
 CONFIG_ARM_MPU=n
 
 CONFIG_LOG=y
+CONFIG_LOG_BACKEND_UART=y
 CONFIG_LOG_MODE_IMMEDIATE=y
+CONFIG_LOG_BACKEND_UART_AUTOSTART=y
 
 CONFIG_HEAP_MEM_POOL_SIZE=32768
 CONFIG_MAIN_STACK_SIZE=32768
 
 CONFIG_ARDUINO_API=y
 
-CONFIG_SHELL=y
-CONFIG_SHELL_STACK_SIZE=2048
-CONFIG_SHELL_CMD_BUFF_SIZE=2048
-CONFIG_SHELL_LOG_LEVEL_DBG=n
-
 CONFIG_LLEXT=y
 CONFIG_LLEXT_LOG_LEVEL_ERR=y
 CONFIG_LLEXT_HEAP_SIZE=32

variants/arduino_giga_r1_stm32h747xx_m7/arduino_giga_r1_stm32h747xx_m7.overlay

@@ -320,9 +320,15 @@
 /{
 	chosen {
 		zephyr,camera = &dcmi;
+		zephyr,log-uart = &log_uarts;
 		/* zephyr,console = &board_cdc_acm_uart; */
 	};
 
+	log_uarts: log_uarts {
+		compatible = "zephyr,log-uart";
+		uarts = <&usart2 &board_cdc_acm_uart>;
+	};
+
 	/* used to overcome problems with _C analog pins */
 	gpioz: gpio@deadbeef {
 		compatible = "vnd,gpio";

variants/arduino_portenta_h7_stm32h747xx_m7/arduino_portenta_h7_stm32h747xx_m7.overlay

@@ -248,6 +248,12 @@
 		zephyr,console = &usart6;
 		zephyr,shell-uart = &usart6;
 		zephyr,cdc-acm-uart0 = &usart6;
+		zephyr,log-uart = &log_uarts;
+	};
+
+	log_uarts: log_uarts {
+		compatible = "zephyr,log-uart";
+		uarts = <&usart6 &board_cdc_acm_uart>;
 	};
 
 	/* used to overcome problems with _C analog pins */