[dpdk-test-report] |WARNING| pw95629 [PATCH 1/4] ethdev: fix max Rx packet length

[checkpatch at dpdk.org]

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

_coding style issues_


CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'max_rx_pktlen > avp->guest_mbuf_size'
#882: FILE: drivers/net/avp/avp_ethdev.c:1067:
+	if ((max_rx_pktlen > avp->guest_mbuf_size) ||
+	    (max_rx_pktlen > avp->host_mbuf_size)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'max_rx_pktlen > avp->host_mbuf_size'
#882: FILE: drivers/net/avp/avp_ethdev.c:1067:
+	if ((max_rx_pktlen > avp->guest_mbuf_size) ||
+	    (max_rx_pktlen > avp->host_mbuf_size)) {

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1058: FILE: drivers/net/bonding/rte_eth_bond_pmd.c:1731:
+	slave_eth_dev->data->dev_conf.rxmode.mtu =

WARNING:CONSTANT_COMPARISON: Comparisons should place the constant on the right side of the test
#1072: FILE: drivers/net/cnxk/cnxk_ethdev.c:56:
+	if (eth_dev->data->mtu + (uint32_t)CNXK_NIX_L2_OVERHEAD > buffsz) {

ERROR:SPACING: space required before the open brace '{'
#1482: FILE: drivers/net/e1000/igb_rxtx.c:2425:
+			if ((max_len + 2 * VLAN_TAG_SIZE) > buf_size){

ERROR:SPACING: space required before the open brace '{'
#1511: FILE: drivers/net/e1000/igb_rxtx.c:2714:
+			if ((max_len + 2 * VLAN_TAG_SIZE) > buf_size){

WARNING:CONSTANT_COMPARISON: Comparisons should place the constant on the right side of the test
#3060: FILE: drivers/net/octeontx2/otx2_ethdev.c:915:
+	if (eth_dev->data->mtu + (uint32_t)NIX_L2_OVERHEAD > buffsz) {

total: 2 errors, 3 warnings, 3272 lines checked