[dpdk-test-report] |WARNING| pw31426 [PATCH 33/53] net/sfc/base: improve names for TXQ descriptor counts

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Nov 16 09:14:10 CET 2017


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

_coding style issues_


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

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#105: FILE: drivers/net/sfc/base/ef10_tx.c:215:
+	if ((rc = efx_mcdi_init_txq(enp, ndescs, eep->ee_index, label, index,

WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#247: FILE: drivers/net/sfc/base/efx_tx.c:98:
+	__checkReturn		efx_rc_t

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#314: FILE: drivers/net/sfc/base/efx_tx.c:376:
+	if ((rc = etxop->etxo_qpost(etp, eb, ndescs, completed, addedp)) != 0)

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ndescs < EFX_TXQ_MINNDESCS'
#392: FILE: drivers/net/sfc/base/efx_tx.c:913:
+	if (!ISP2(ndescs) ||
+	    (ndescs < EFX_TXQ_MINNDESCS) || (ndescs > EFX_EVQ_MAXNEVS)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ndescs > EFX_EVQ_MAXNEVS'
#392: FILE: drivers/net/sfc/base/efx_tx.c:913:
+	if (!ISP2(ndescs) ||
+	    (ndescs < EFX_TXQ_MINNDESCS) || (ndescs > EFX_EVQ_MAXNEVS)) {

total: 2 errors, 1 warnings, 3 checks, 380 lines checked


More information about the test-report mailing list