automatic DPDK test reports
 help / color / mirror / Atom feed
* [dpdk-test-report] |WARNING| pw31754 [PATCH] net/bonding: fix link status check
       [not found] <20171129145349.43448-1-tomaszx.kulasek@intel.com>
@ 2017-11-29 14:57 ` checkpatch
  0 siblings, 0 replies; only message in thread
From: checkpatch @ 2017-11-29 14:57 UTC (permalink / raw)
  To: test-report; +Cc: Tomasz Kulasek

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

_coding style issues_


CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#93: FILE: drivers/net/bonding/rte_eth_bond_api.c:289:
 	if (internals->slave_count < 1) {
+

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#95: FILE: drivers/net/bonding/rte_eth_bond_api.c:291:
+		 * from first active slave) */

WARNING:LONG_LINE: line over 90 characters
#132: FILE: drivers/net/bonding/rte_eth_bond_api.c:368:
+						slave_port_id) == internals->active_slave_count) {

WARNING:LONG_LINE_COMMENT: line over 90 characters
#135: FILE: drivers/net/bonding/rte_eth_bond_api.c:371:
+					/* Inherit dev link properties from first active slave */

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around slave_eth_dev->data->dev_link
#137: FILE: drivers/net/bonding/rte_eth_bond_api.c:373:
+							&(slave_eth_dev->data->dev_link));

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (32, 48)
#138: FILE: drivers/net/bonding/rte_eth_bond_api.c:374:
+				} else if (link_properties_valid(bonded_eth_dev,
[...]
+						RTE_BOND_LOG(ERR, "Invalid link properties for slave %d"

WARNING:LONG_LINE: line over 90 characters
#139: FILE: drivers/net/bonding/rte_eth_bond_api.c:375:
+							&slave_eth_dev->data->dev_link) != 0) {

WARNING:LONG_LINE: line over 90 characters
#141: FILE: drivers/net/bonding/rte_eth_bond_api.c:377:
+								" in bonding mode %d", slave_port_id,

total: 0 errors, 6 warnings, 2 checks, 99 lines checked

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

only message in thread, other threads:[~2017-11-29 14:57 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20171129145349.43448-1-tomaszx.kulasek@intel.com>
2017-11-29 14:57 ` [dpdk-test-report] |WARNING| pw31754 [PATCH] net/bonding: fix link status check 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).