automatic DPDK test reports
 help / color / mirror / Atom feed
* [dpdk-test-report] |WARNING| pw35165 [PATCH v2] net/i40e: fix link_state update for i40e_ethdev_vf drv
       [not found] <d3535e69b45a4139bb322a0ba8383479@sandvine.com>
@ 2018-02-14 12:01 ` checkpatch
  0 siblings, 0 replies; only message in thread
From: checkpatch @ 2018-02-14 12:01 UTC (permalink / raw)
  To: test-report; +Cc: Tushar Mulkar

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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#15: 
The check for bool was accounting unwanted bits in the calulation of truth value. In dpdk unsingned int is typedefed to bool but all it cares about is Least Significant Bit. But in calculation of condition expression the bits other than LSB was used which doesn't make sense. Some time these bits has values which results in to incorrect expression results. To fix this we just need to account LSB form the bool value . This can be easily done by anding the value with true.

ERROR:TRAILING_WHITESPACE: trailing whitespace
#35: FILE: drivers/net/i40e/i40e_ethdev_vf.c:2098:
+^Inew_link.link_status = (vf->link_up & true) ? $

ERROR:CODE_INDENT: code indent should use tabs where possible
#36: FILE: drivers/net/i40e/i40e_ethdev_vf.c:2099:
+                            ETH_LINK_UP : ETH_LINK_DOWN;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#36: FILE: drivers/net/i40e/i40e_ethdev_vf.c:2099:
+                            ETH_LINK_UP : ETH_LINK_DOWN;$

total: 2 errors, 2 warnings, 0 checks, 10 lines checked

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

only message in thread, other threads:[~2018-02-14 12:01 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <d3535e69b45a4139bb322a0ba8383479@sandvine.com>
2018-02-14 12:01 ` [dpdk-test-report] |WARNING| pw35165 [PATCH v2] net/i40e: fix link_state update for i40e_ethdev_vf drv 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).