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

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Apr 3 17:07:34 CEST 2018


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

_coding style issues_


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

CHECK:SPACING: No space is necessary after a cast
#76: 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 '('
#76: 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
#99: FILE: drivers/net/sfc/base/ef10_nic.c:2336:
+	return (0);

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

CHECK:SPACING: No space is necessary after a cast
#119: 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 '('
#119: 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
#141: FILE: drivers/net/sfc/base/ef10_nic.c:2378:
+	return (rc);

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

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

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#270: 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