See https://docs.zephyrproject.org/latest/contribute/guidelines.html#clang-format for more details. You may want to run clang-format on this change: -#define DROP_CHECK_PERIOD \ +#define DROP_CHECK_PERIOD \ File:drivers/debug/debug_nrf_etr.c Line:32 You may want to run clang-format on this change: -#define DBG_DATA(_data, _nlen, _marked) \ - do { \ - char *fmt; \ - switch (_nlen) { \ - case 2: \ - fmt = "D%s\t%02x "; \ - break; \ - case 4: \ - fmt = "D%s\t%04x "; \ - break; \ - case 8: \ - fmt = "D%s\t%08x "; \ - break; \ - default: \ - fmt = "D%s\t%016x "; \ - break; \ - } \ - DBG(fmt, _marked ? "M" : "", _data); \ - for (int i = 0; i < _nlen / 2; i++) { \ - DBG("%c ", ((char *)&_data)[i]); \ - } \ - DBG("\n"); \ +#define DBG_DATA(_data, _nlen, _marked) \ + do { \ + char *fmt; \ + switch (_nlen) { \ + case 2: \ + fmt = "D%s\t%02x "; \ + break; \ + case 4: \ + fmt = "D%s\t%04x "; \ + break; \ + case 8: \ + fmt = "D%s\t%08x "; \ + break; \ + default: \ + fmt = "D%s\t%016x "; \ + break; \ + } \ + DBG(fmt, _marked ? "M" : "", _data); \ + for (int i = 0; i < _nlen / 2; i++) { \ + DBG("%c ", ((char *)&_data)[i]); \ + } \ + DBG("\n"); \ File:drivers/debug/debug_nrf_etr.c Line:69 You may want to run clang-format on this change: -static uint32_t *frame_buf = IS_ENABLED(CONFIG_DEBUG_NRF_ETR_DECODE) ? - frame_buf_decode : frame_buf0; +static uint32_t *frame_buf = + IS_ENABLED(CONFIG_DEBUG_NRF_ETR_DECODE) ? frame_buf_decode : frame_buf0; File:drivers/debug/debug_nrf_etr.c Line:91 You may want to run clang-format on this change: - "CTI211_0", /* 0 CTI211 triger out 1 */ - "CTI211_1", /* 1 CTI211 triger out 1 inverted */ - "CTI211_2", /* 2 CTI211 triger out 2 */ - "CTI211_3", /* 3 CTI211 triger out 2 inverted*/ - "Sec up", /* 4 Secure Domain up */ - "Sec down", /* 5 Secure Domain down */ - "App up", /* 6 Application Domain up */ - "App down", /* 7 Application Domain down */ - "Rad up", /* 8 Radio Domain up */

drivers/debug/debug_nrf_etr.c:303 -static void decoder_cb_debug(enum mipi_stp_decoder_ctrl_type type, - union mipi_stp_decoder_data data, +static void decoder_cb_debug(enum mipi_stp_decoder_ctrl_type type, union mipi_stp_decoder_data data,

drivers/debug/debug_nrf_etr.c:257 - log_output_process(&log_output, packet->timestamp, dname, sname, NULL, - 1, (const uint8_t *)tp_p, NULL, 0, flags); + log_output_process(&log_output, packet->timestamp, dname, sname, NULL, 1, + (const uint8_t *)tp_p, NULL, 0, flags);

drivers/debug/debug_nrf_etr.c:242 - log_output_process(&log_output, packet->timestamp, dname, sname, NULL, - 1, (const uint8_t *)tp_p, NULL, 0, flags); + log_output_process(&log_output, packet->timestamp, dname, sname, NULL, 1, + (const uint8_t *)tp_p, NULL, 0, flags);

drivers/debug/debug_nrf_etr.c:212 - lptr = log_frontend_stmesp_demux_str_get(packet->major, - packet->id - CONFIG_LOG_FRONTEND_STMESP_TURBO_LOG_BASE); + lptr = log_frontend_stmesp_demux_str_get( + packet->major, packet->id - CONFIG_LOG_FRONTEND_STMESP_TURBO_LOG_BASE);

drivers/debug/debug_nrf_etr.c:168 - tx_buf = (tx_buf == (uint8_t *)frame_buf0) ? - (uint8_t *)frame_buf1 : (uint8_t *)frame_buf0; + tx_buf = (tx_buf == (uint8_t *)frame_buf0) ? (uint8_t *)frame_buf1 + : (uint8_t *)frame_buf0;

drivers/debug/debug_nrf_etr.c:141 -#define RX_BUF_SIZE \ - (CONFIG_DEBUG_NRF_ETR_SHELL_ASYNC_RX_BUFFER_SIZE * \ - CONFIG_DEBUG_NRF_ETR_SHELL_ASYNC_RX_BUFFER_COUNT) +#define RX_BUF_SIZE \ + (CONFIG_DEBUG_NRF_ETR_SHELL_ASYNC_RX_BUFFER_SIZE * \ + CONFIG_DEBUG_NRF_ETR_SHELL_ASYNC_RX_BUFFER_COUNT)

drivers/debug/debug_nrf_etr.c:129 - "CTI211_0", /* 0 CTI211 triger out 1 */ - "CTI211_1", /* 1 CTI211 triger out 1 inverted */ - "CTI211_2", /* 2 CTI211 triger out 2 */ - "CTI211_3", /* 3 CTI211 triger out 2 inverted*/ - "Sec up", /* 4 Secure Domain up */ - "Sec down", /* 5 Secure Domain down */ - "App up", /* 6 Application Domain up */ - "App down", /* 7 Application Domain down */ - "Rad up", /* 8 Radio Domain up */ - "Rad down", /* 9 Radio Domain down */ - "Radf up", /* 10 Radio fast up */ - "Radf down", /* 11 Radio fast down */ - NULL, /* Reserved */ - NULL, /* Reserved */ - NULL, /* Reserved */ - NULL, /* Reserved */ - NULL, /* Reserved */ - NULL, /* Reserved */ - NULL, /* Reserved */ - NULL, /* Reserved */ - NULL, /* Reserved */ - NULL, /* Reserved */ - NULL, /* Reserved */ - NULL, /* Reserved */ - NULL, /* Reserved */ - NULL, /* Reserved */ + "CTI211_0", /* 0 CTI211 triger out 1 */ + "CTI211_1", /* 1 CTI211 triger out 1 inverted */ + "CTI211_2", /* 2 CTI211 triger out 2 */ + "CTI211_3", /* 3 CTI211 triger out 2 inverted*/ + "Sec up", /* 4 Secure Domain up */ + "Sec down", /* 5 Secure Domain down */ + "App up", /* 6 Application Domain up */ + "App down", /* 7 Application Domain down */ + "Rad up", /* 8 Radio Domain up */ + "Rad down", /* 9 Radio Domain down */ + "Radf up", /* 10 Radio fast up */ + "Radf down", /* 11 Radio fast down */ + NULL, /* Reserved */ + NULL, /* Reserved */ + NULL, /* Reserved */ + NULL, /* Reserved */ + NULL, /* Reserved */ + NULL, /* Reserved */ + NULL, /* Reserved */ + NULL, /* Reserved */ + NULL, /* Reserved */ + NULL, /* Reserved */ + NULL, /* Reserved */ + NULL, /* Reserved */ + NULL, /* Reserved */ + NULL, /* Reserved */

drivers/debug/debug_nrf_etr.c:91 -static uint32_t *frame_buf = IS_ENABLED(CONFIG_DEBUG_NRF_ETR_DECODE) ? - frame_buf_decode : frame_buf0; +static uint32_t *frame_buf = + IS_ENABLED(CONFIG_DEBUG_NRF_ETR_DECODE) ? frame_buf_decode : frame_buf0;

drivers/debug/debug_nrf_etr.c:69 -#define DBG_DATA(_data, _nlen, _marked) \ - do { \ - char *fmt; \ - switch (_nlen) { \ - case 2: \ - fmt = "D%s\t%02x "; \ - break; \ - case 4: \ - fmt = "D%s\t%04x "; \ - break; \ - case 8: \ - fmt = "D%s\t%08x "; \ - break; \ - default: \ - fmt = "D%s\t%016x "; \ - break; \ - } \ - DBG(fmt, _marked ? "M" : "", _data); \ - for (int i = 0; i < _nlen / 2; i++) { \ - DBG("%c ", ((char *)&_data)[i]); \ - } \ - DBG("\n"); \ +#define DBG_DATA(_data, _nlen, _marked) \ + do { \ + char *fmt; \ + switch (_nlen) { \ + case 2: \ + fmt = "D%s\t%02x "; \ + break; \ + case 4: \ + fmt = "D%s\t%04x "; \ + break; \ + case 8: \ + fmt = "D%s\t%08x "; \ + break; \ + default: \ + fmt = "D%s\t%016x "; \ + break; \ + } \ + DBG(fmt, _marked ? "M" : "", _data); \ + for (int i = 0; i < _nlen / 2; i++) { \ + DBG("%c ", ((char *)&_data)[i]); \ + } \ + DBG("\n"); \