From 75406d01f9a8ea4e32d940e0b7ac8b8cdaf04491 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Krzysztof=20Chru=C5=9Bci=C5=84ski?= Date: Wed, 22 Jan 2025 15:05:50 +0100 Subject: [PATCH] [nrf fromtree] logging: frontends: stmesp: Fix logging single argument log twice MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Macro was missing a break when single argument with non 32 bit word argument was detect. Because of that, there were two logging messages created for a single log entry. Signed-off-by: Krzysztof Chruściński (cherry picked from commit 9b2532ab431b9d7dd5c67f72ab67b8ea271acec3) --- subsys/logging/frontends/stmesp/zephyr_custom_log.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/subsys/logging/frontends/stmesp/zephyr_custom_log.h b/subsys/logging/frontends/stmesp/zephyr_custom_log.h index 9457123886d..62b949cd51f 100644 --- a/subsys/logging/frontends/stmesp/zephyr_custom_log.h +++ b/subsys/logging/frontends/stmesp/zephyr_custom_log.h @@ -69,7 +69,8 @@ extern "C" { /* Do turbo logging only if argument fits in 32 bit word. */ \ if (!Z_LOG_STMESP_1_ARG_CHECK(__VA_ARGS__)) { \ COND_CODE_1(CONFIG_LOG_FRONTEND_STMESP_TURBO_DROP_OTHERS, (), \ - (Z_LOG(_level, __VA_ARGS__))); \ + (Z_LOG(_level, __VA_ARGS__))); \ + break; \ } \ if (!Z_LOG_LEVEL_ALL_CHECK(_level, __log_current_const_data, _source)) { \ break; \