[dpdk-test-report] |WARNING| pw24647 [PATCH 1/2] drivers/bonding: add other agg selection modes for mode4

checkpatch at dpdk.org checkpatch at dpdk.org
Fri May 26 16:42:48 CEST 2017


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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#16: 
This patch add support for setting additional aggregator modes for IEEE802.3AD

CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the previous line
#252: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1370:
+	if (agg_selection == AGG_COUNT || agg_selection == AGG_BANDWIDTH
+			|| agg_selection == AGG_STABLE)

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#379: FILE: drivers/net/bonding/rte_eth_bond_8023ad.h:332:
+ * ^I0 on success, negative value otherwise$

CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#478: FILE: drivers/net/bonding/rte_eth_bond_pmd.c:2597:
+				&agg_mode) != 0) {
+

CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#479: FILE: drivers/net/bonding/rte_eth_bond_pmd.c:2598:
+
+		}

total: 0 errors, 2 warnings, 3 checks, 450 lines checked


More information about the test-report mailing list