|WARNING| pw135089 [PATCH 15/26] net/qede: replace RTE_LOG_DP with rte_log_dp
checkpatch at dpdk.org
checkpatch at dpdk.org
Wed Dec 13 02:48:43 CET 2023
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/135089
_coding style issues_
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'q' - possible side-effects?
#101: FILE: drivers/net/qede/qede_logs.h:61:
+#define PMD_TX_LOG(level, q, fmt, args...) \
+ rte_log(RTE_LOG_ ## level, qded_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
#101: FILE: drivers/net/qede/qede_logs.h:61:
+#define PMD_TX_LOG(level, q, fmt, args...) \
+ rte_log(RTE_LOG_ ## level, qded_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?
#112: FILE: drivers/net/qede/qede_logs.h:70:
+#define PMD_RX_LOG(level, q, fmt, args...) \
+ rte_log(RTE_LOG_ ## level, qded_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
#112: FILE: drivers/net/qede/qede_logs.h:70:
+#define PMD_RX_LOG(level, q, fmt, args...) \
+ rte_log(RTE_LOG_ ## level, qded_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
More information about the test-report
mailing list