[dpdk-test-report] |WARNING| pw54163 [PATCH v3 01/11] net/hinic/base: add registers for Huawei Hi1822 NIC

checkpatch at dpdk.org checkpatch at dpdk.org
Mon Jun 3 13:26:39 CEST 2019


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

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse 'q_id' - possible side-effects?
#116: FILE: drivers/net/hinic/base/hinic_csr.h:83:
+#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?
#116: FILE: drivers/net/hinic/base/hinic_csr.h:83:
+#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?
#121: FILE: drivers/net/hinic/base/hinic_csr.h:88:
+#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?
#121: FILE: drivers/net/hinic/base/hinic_csr.h:88:
+#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, 164 lines checked


More information about the test-report mailing list