* |WARNING| pw135149 [PATCH v3 14/37] net/qede: replace RTE_LOG_DP with rte_log_dp
[not found] <20231213163820.894987-15-stephen@networkplumber.org>
@ 2023-12-13 16:41 ` checkpatch
0 siblings, 0 replies; only message in thread
From: checkpatch @ 2023-12-13 16:41 UTC (permalink / raw)
To: test-report; +Cc: Stephen Hemminger
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/135149
_coding style issues_
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'q' - possible side-effects?
#103: FILE: drivers/net/qede/qede_logs.h:61:
+#define PMD_TX_LOG(level, q, fmt, args...) \
+ rte_log(RTE_LOG_ ## level, qede_logtype_driver, \
+ "%s(): port=%u queue=%u " fmt "
", \
__func__, q->port_id, q->queue_id, ## args)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'q' may be better as '(q)' to avoid precedence issues
#103: FILE: drivers/net/qede/qede_logs.h:61:
+#define PMD_TX_LOG(level, q, fmt, args...) \
+ rte_log(RTE_LOG_ ## level, qede_logtype_driver, \
+ "%s(): port=%u queue=%u " fmt "
", \
__func__, q->port_id, q->queue_id, ## args)
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'q' - possible side-effects?
#114: FILE: drivers/net/qede/qede_logs.h:70:
+#define PMD_RX_LOG(level, q, fmt, args...) \
+ rte_log(RTE_LOG_ ## level, qede_logtype_driver, \
+ "%s(): port=%u queue=%u " fmt "
", \
__func__, q->port_id, q->queue_id, ## args)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'q' may be better as '(q)' to avoid precedence issues
#114: FILE: drivers/net/qede/qede_logs.h:70:
+#define PMD_RX_LOG(level, q, fmt, args...) \
+ rte_log(RTE_LOG_ ## level, qede_logtype_driver, \
+ "%s(): port=%u queue=%u " fmt "
", \
__func__, q->port_id, q->queue_id, ## args)
total: 0 errors, 0 warnings, 4 checks, 22 lines checked
^ permalink raw reply [flat|nested] only message in thread