Skip to content

Commit

Permalink
logging: frontends: stmesp: Fix logging single argument log twice
Browse files Browse the repository at this point in the history
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 <[email protected]>
  • Loading branch information
nordic-krch authored and kartben committed Jan 24, 2025
1 parent cb60881 commit 9b2532a
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion subsys/logging/frontends/stmesp/zephyr_custom_log.h
Original file line number Diff line number Diff line change
Expand Up @@ -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; \
Expand Down

0 comments on commit 9b2532a

Please sign in to comment.