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

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Jun 6 13:03:00 CEST 2019


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

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse 'q_id' - possible side-effects?
#117: 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?
#117: 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?
#122: 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?
#122: 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