[dpdk-test-report] |WARNING| pw55439 [PATCH v6 01/15] net/hinic/base: add HW registers definition

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Jun 27 10:00:28 CEST 2019


Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/55439

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse 'q_id' - possible side-effects?
#111: FILE: drivers/net/hinic/base/hinic_csr.h:79:
+#define HINIC_EQ_HI_PHYS_ADDR_REG(type, q_id, pg_num)	\
+		((u32)((type == HINIC_AEQ) ? \
+		HINIC_AEQ_HI_PHYS_ADDR_REG(q_id, pg_num) : \
+		HINIC_CEQ_HI_PHYS_ADDR_REG(q_id, pg_num)))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pg_num' - possible side-effects?
#111: FILE: drivers/net/hinic/base/hinic_csr.h:79:
+#define HINIC_EQ_HI_PHYS_ADDR_REG(type, q_id, pg_num)	\
+		((u32)((type == HINIC_AEQ) ? \
+		HINIC_AEQ_HI_PHYS_ADDR_REG(q_id, pg_num) : \
+		HINIC_CEQ_HI_PHYS_ADDR_REG(q_id, pg_num)))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'q_id' - possible side-effects?
#116: FILE: drivers/net/hinic/base/hinic_csr.h:84:
+#define HINIC_EQ_LO_PHYS_ADDR_REG(type, q_id, pg_num)	\
+		((u32)((type == HINIC_AEQ) ? \
+		HINIC_AEQ_LO_PHYS_ADDR_REG(q_id, pg_num) : \
+		HINIC_CEQ_LO_PHYS_ADDR_REG(q_id, pg_num)))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pg_num' - possible side-effects?
#116: FILE: drivers/net/hinic/base/hinic_csr.h:84:
+#define HINIC_EQ_LO_PHYS_ADDR_REG(type, q_id, pg_num)	\
+		((u32)((type == HINIC_AEQ) ? \
+		HINIC_AEQ_LO_PHYS_ADDR_REG(q_id, pg_num) : \
+		HINIC_CEQ_LO_PHYS_ADDR_REG(q_id, pg_num)))

total: 0 errors, 0 warnings, 4 checks, 160 lines checked


More information about the test-report mailing list