[dpdk-test-report] |WARNING| pw37132 [PATCH v2 4/5] net/sfc/base: support FW subvariant choice

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Apr 4 16:18:14 CEST 2018


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

_coding style issues_


CHECK:CAMELCASE: Avoid CamelCase: <__checkReturn>
#39: FILE: drivers/net/sfc/base/ef10_impl.h:1172:
+extern	__checkReturn	efx_rc_t

CHECK:SPACING: No space is necessary after a cast
#77: FILE: drivers/net/sfc/base/ef10_nic.c:2313:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#77: FILE: drivers/net/sfc/base/ef10_nic.c:2313:
+	(void) memset(payload, 0, sizeof (payload));

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#100: FILE: drivers/net/sfc/base/ef10_nic.c:2336:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#107: FILE: drivers/net/sfc/base/ef10_nic.c:2343:
+	return (rc);

CHECK:SPACING: No space is necessary after a cast
#120: FILE: drivers/net/sfc/base/ef10_nic.c:2356:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#120: FILE: drivers/net/sfc/base/ef10_nic.c:2356:
+	(void) memset(payload, 0, sizeof (payload));

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#142: FILE: drivers/net/sfc/base/ef10_nic.c:2378:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#238: FILE: drivers/net/sfc/base/efx_nic.c:986:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#263: FILE: drivers/net/sfc/base/efx_nic.c:1011:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#271: FILE: drivers/net/sfc/base/efx_nic.c:1019:
+	return (rc);

total: 6 errors, 2 warnings, 3 checks, 229 lines checked


More information about the test-report mailing list