michaelh/zephyr
1
0
Fork 0
Code Releases Activity

log: formatting: Add configuration for blue DBG log

Browse source 
It's usual to show debug logs as blue.
Showing dbg and info with different colors are good,
but it's preferable to keep info logs with default
color.

Signed-off-by: Félix Piédallu <felix@piedallu.me>
This commit is contained in:
Félix Piédallu 2024-06-28 17:34:45 +02:00 • committed by Anas Nashif
commit 61e5958eff
4 changed files with 8 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

3
subsys/logging/Kconfig.formatting
View file

@ -145,6 +145,9 @@ if LOG_BACKEND_SHOW_COLOR
config LOG_INFO_COLOR_GREEN config LOG_INFO_COLOR_GREEN
bool "Use green color for info level logs" bool "Use green color for info level logs"
config LOG_DBG_COLOR_BLUE
bool "Use blue color for debug level logs"
endif # LOG_BACKEND_SHOW_COLOR endif # LOG_BACKEND_SHOW_COLOR
config LOG_TAG_MAX_LEN config LOG_TAG_MAX_LEN

3
subsys/logging/log_output.c
View file

@ -19,6 +19,7 @@
#define LOG_COLOR_CODE_RED "\x1B[1;31m" #define LOG_COLOR_CODE_RED "\x1B[1;31m"
#define LOG_COLOR_CODE_GREEN "\x1B[1;32m" #define LOG_COLOR_CODE_GREEN "\x1B[1;32m"
#define LOG_COLOR_CODE_YELLOW "\x1B[1;33m" #define LOG_COLOR_CODE_YELLOW "\x1B[1;33m"
#define LOG_COLOR_CODE_BLUE "\x1B[1;34m"
#define HEXDUMP_BYTES_IN_LINE 16 #define HEXDUMP_BYTES_IN_LINE 16
@ -41,7 +42,7 @@ static const char *const colors[] = {
LOG_COLOR_CODE_RED, /* err */ LOG_COLOR_CODE_RED, /* err */
LOG_COLOR_CODE_YELLOW, /* warn */ LOG_COLOR_CODE_YELLOW, /* warn */
IS_ENABLED(CONFIG_LOG_INFO_COLOR_GREEN) ? LOG_COLOR_CODE_GREEN : NULL, /* info */ IS_ENABLED(CONFIG_LOG_INFO_COLOR_GREEN) ? LOG_COLOR_CODE_GREEN : NULL, /* info */
NULL /* dbg */ IS_ENABLED(CONFIG_LOG_DBG_COLOR_BLUE) ? LOG_COLOR_CODE_BLUE : NULL, /* dbg */
}; };
static uint32_t freq; static uint32_t freq;

1
tests/subsys/logging/log_output/prj.conf
View file

@ -4,3 +4,4 @@ CONFIG_LOG=y
CONFIG_LOG_OUTPUT=y CONFIG_LOG_OUTPUT=y
CONFIG_LOG_PRINTK=n CONFIG_LOG_PRINTK=n
CONFIG_ZTEST_STACK_SIZE=1152 CONFIG_ZTEST_STACK_SIZE=1152
CONFIG_LOG_DBG_COLOR_BLUE=y

3
tests/subsys/logging/log_output/src/log_output_test.c
View file

@ -192,13 +192,14 @@ ZTEST(test_log_output, test_colors)
#define LOG_COLOR_CODE_RED "\x1B[1;31m" #define LOG_COLOR_CODE_RED "\x1B[1;31m"
#define LOG_COLOR_CODE_GREEN "\x1B[1;32m" #define LOG_COLOR_CODE_GREEN "\x1B[1;32m"
#define LOG_COLOR_CODE_YELLOW "\x1B[1;33m" #define LOG_COLOR_CODE_YELLOW "\x1B[1;33m"
#define LOG_COLOR_CODE_BLUE "\x1B[1;34m"
char package[256]; char package[256];
static const char *const exp_strs[] = { static const char *const exp_strs[] = {
LOG_COLOR_CODE_RED "<err> " SNAME ": " TEST_STR LOG_COLOR_CODE_DEFAULT "\r\n", LOG_COLOR_CODE_RED "<err> " SNAME ": " TEST_STR LOG_COLOR_CODE_DEFAULT "\r\n",
LOG_COLOR_CODE_YELLOW "<wrn> " SNAME ": " TEST_STR LOG_COLOR_CODE_DEFAULT "\r\n", LOG_COLOR_CODE_YELLOW "<wrn> " SNAME ": " TEST_STR LOG_COLOR_CODE_DEFAULT "\r\n",
LOG_COLOR_CODE_DEFAULT "<inf> " SNAME ": " TEST_STR LOG_COLOR_CODE_DEFAULT "\r\n", LOG_COLOR_CODE_DEFAULT "<inf> " SNAME ": " TEST_STR LOG_COLOR_CODE_DEFAULT "\r\n",
LOG_COLOR_CODE_DEFAULT "<dbg> " SNAME ": " TEST_STR LOG_COLOR_CODE_DEFAULT "\r\n" LOG_COLOR_CODE_BLUE "<dbg> " SNAME ": " TEST_STR LOG_COLOR_CODE_DEFAULT "\r\n"
}; };
uint8_t levels[] = {LOG_LEVEL_ERR, LOG_LEVEL_WRN, LOG_LEVEL_INF, LOG_LEVEL_DBG}; uint8_t levels[] = {LOG_LEVEL_ERR, LOG_LEVEL_WRN, LOG_LEVEL_INF, LOG_LEVEL_DBG};
uint32_t flags = LOG_OUTPUT_FLAG_LEVEL | LOG_OUTPUT_FLAG_COLORS; uint32_t flags = LOG_OUTPUT_FLAG_LEVEL | LOG_OUTPUT_FLAG_COLORS;