* [dpdk-test-report] |WARNING| pw38969 [PATCH v3 2/8] net/sfc/base: add a new means to control RSS hash
[not found] <1524678704-29354-3-git-send-email-arybchenko@solarflare.com>
@ 2018-04-25 17:53 ` checkpatch
0 siblings, 0 replies; only message in thread
From: checkpatch @ 2018-04-25 17:53 UTC (permalink / raw)
To: test-report; +Cc: Andrew Rybchenko
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/38969
_coding style issues_
CHECK:CAMELCASE: Avoid CamelCase: <__checkReturn>
#199: FILE: drivers/net/sfc/base/efx.h:2178:
+extern __checkReturn efx_rc_t
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_entryp' - possible side-effects?
#232: FILE: drivers/net/sfc/base/efx_rx.c:312:
+#define LIST_FLAGS(_entryp, _class, _l4_hashing) \
+ do { \
+ if (_l4_hashing) \
+ *(_entryp++) = EFX_RX_HASH(_class, 4TUPLE); \
+ \
+ *(_entryp++) = EFX_RX_HASH(_class, 2TUPLE); \
+ *(_entryp++) = EFX_RX_HASH(_class, DISABLE); \
+ \
+ _NOTE(CONSTANTCONDITION) \
+ } while (B_FALSE)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_class' - possible side-effects?
#232: FILE: drivers/net/sfc/base/efx_rx.c:312:
+#define LIST_FLAGS(_entryp, _class, _l4_hashing) \
+ do { \
+ if (_l4_hashing) \
+ *(_entryp++) = EFX_RX_HASH(_class, 4TUPLE); \
+ \
+ *(_entryp++) = EFX_RX_HASH(_class, 2TUPLE); \
+ *(_entryp++) = EFX_RX_HASH(_class, DISABLE); \
+ \
+ _NOTE(CONSTANTCONDITION) \
+ } while (B_FALSE)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#261: FILE: drivers/net/sfc/base/efx_rx.c:341:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#269: FILE: drivers/net/sfc/base/efx_rx.c:349:
+ return (rc);
total: 2 errors, 0 warnings, 3 checks, 331 lines checked
^ permalink raw reply [flat|nested] only message in thread