* [dpdk-test-report] |WARNING| pw28095 [PATCH 06/11] net/sfc/base: add API to allocate and free RSS contexts
[not found] <1504117061-17906-7-git-send-email-arybchenko@solarflare.com>
@ 2017-08-30 18:18 ` checkpatch
0 siblings, 0 replies; only message in thread
From: checkpatch @ 2017-08-30 18:18 UTC (permalink / raw)
To: test-report; +Cc: Andrew Rybchenko
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/28095
_coding style issues_
CHECK:CAMELCASE: Avoid CamelCase: <__checkReturn>
#43: FILE: drivers/net/sfc/base/ef10_impl.h:907:
+extern __checkReturn efx_rc_t
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#72: FILE: drivers/net/sfc/base/ef10_rx.c:506:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#76: FILE: drivers/net/sfc/base/ef10_rx.c:510:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#92: FILE: drivers/net/sfc/base/ef10_rx.c:526:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#96: FILE: drivers/net/sfc/base/ef10_rx.c:530:
+ return (rc);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#189: FILE: drivers/net/sfc/base/efx_rx.c:387:
+ if ((rc = erxop->erxo_scale_context_alloc(enp, type,
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#194: FILE: drivers/net/sfc/base/efx_rx.c:392:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#200: FILE: drivers/net/sfc/base/efx_rx.c:398:
+ return (rc);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#220: FILE: drivers/net/sfc/base/efx_rx.c:418:
+ if ((rc = erxop->erxo_scale_context_free(enp, rss_context)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#223: FILE: drivers/net/sfc/base/efx_rx.c:421:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#229: FILE: drivers/net/sfc/base/efx_rx.c:427:
+ return (rc);
total: 10 errors, 0 warnings, 1 checks, 181 lines checked
^ permalink raw reply [flat|nested] only message in thread