[dpdk-test-report] |WARNING| pw100242 [PATCH v4 2/5] net/enetfec: add UIO support

checkpatch at dpdk.org checkpatch at dpdk.org
Fri Oct 1 13:43:56 CEST 2021


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

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse 'X' - possible side-effects?
#404: FILE: drivers/net/enetfec/enet_regs.h:41:
+#define ENETFEC_RD_START(X)	(((X) == 1) ? ENETFEC_RD_START_1 : \
+				(((X) == 2) ? \
+				   ENETFEC_RD_START_2 : ENETFEC_RD_START_0))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'X' - possible side-effects?
#407: FILE: drivers/net/enetfec/enet_regs.h:44:
+#define ENETFEC_TD_START(X)	(((X) == 1) ? ENETFEC_TD_START_1 : \
+				(((X) == 2) ? \
+				   ENETFEC_TD_START_2 : ENETFEC_TD_START_0))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'X' - possible side-effects?
#410: FILE: drivers/net/enetfec/enet_regs.h:47:
+#define ENETFEC_MRB_SIZE(X)	(((X) == 1) ? ENETFEC_MRB_SIZE_1 : \
+				(((X) == 2) ? \
+				   ENETFEC_MRB_SIZE_2 : ENETFEC_MRB_SIZE_0))

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#719: FILE: drivers/net/enetfec/enet_uio.c:244:
+		if (file_name_match_extract (dir->d_name, "uio")) {

WARNING:NAKED_SSCANF: unchecked sscanf return value
#724: FILE: drivers/net/enetfec/enet_uio.c:249:
+			sscanf(dir->d_name + strlen("uio"), "%d", &uio_minor_number);

total: 0 errors, 2 warnings, 3 checks, 714 lines checked


More information about the test-report mailing list