[dpdk-test-report] |WARNING| pw80135 [PATCH 04/13] net/bnxt: fixes for PMD PF support in SR-IOV mode

checkpatch at dpdk.org checkpatch at dpdk.org
Fri Oct 9 13:23:17 CEST 2020


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

_coding style issues_


WARNING:TYPO_SPELLING: 'forwared' may be misspelled - perhaps 'forwarded'?
#74: 
8. Update the HWRM commands list that can be forwared to the

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'cmd' - possible side-effects?
#98: FILE: drivers/net/bnxt/bnxt.h:170:
+#define BNXT_HWRM_CMD_TO_FORWARD(cmd)	\
+		(bp->pf->vf_req_fwd[(cmd) / 32] |= (1 << ((cmd) % 32)))

CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#389: FILE: drivers/net/bnxt/bnxt_hwrm.c:3354:
+	req->min_rsscos_ctx = req->max_rsscos_ctx =

CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#392: FILE: drivers/net/bnxt/bnxt_hwrm.c:3357:
+	req->min_stat_ctx = req->max_stat_ctx =

CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#394: FILE: drivers/net/bnxt/bnxt_hwrm.c:3359:
+	req->min_cmpl_rings = req->max_cmpl_rings =

CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#396: FILE: drivers/net/bnxt/bnxt_hwrm.c:3361:
+	req->min_tx_rings = req->max_tx_rings =

CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#398: FILE: drivers/net/bnxt/bnxt_hwrm.c:3363:
+	req->min_rx_rings = req->max_rx_rings =

CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#400: FILE: drivers/net/bnxt/bnxt_hwrm.c:3365:
+	req->min_l2_ctxs = req->max_l2_ctxs =

CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#403: FILE: drivers/net/bnxt/bnxt_hwrm.c:3368:
+	req->min_vnics = req->max_vnics = rte_cpu_to_le_16(1);

CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#404: FILE: drivers/net/bnxt/bnxt_hwrm.c:3369:
+	req->min_hw_ring_grps = req->max_hw_ring_grps =

total: 0 errors, 1 warnings, 9 checks, 757 lines checked


More information about the test-report mailing list