automatic DPDK test reports
 help / color / mirror / Atom feed
* [dpdk-test-report] |WARNING| pw24647 [PATCH 1/2] drivers/bonding: add other agg selection modes for mode4
       [not found] <20170526143528.16533-2-danielx.t.mrzyglod@intel.com>
@ 2017-05-26 14:42 ` checkpatch
  0 siblings, 0 replies; only message in thread
From: checkpatch @ 2017-05-26 14:42 UTC (permalink / raw)
  To: test-report; +Cc: Daniel Mrzyglod

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

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2017-05-26 14:42 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20170526143528.16533-2-danielx.t.mrzyglod@intel.com>
2017-05-26 14:42 ` [dpdk-test-report] |WARNING| pw24647 [PATCH 1/2] drivers/bonding: add other agg selection modes for mode4 checkpatch

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).