[1/2] net/netvsc: fix lgtm static checker complaint
diff mbox series

Message ID 20190327210843.13813-2-stephen@networkplumber.org
State Accepted, archived
Delegated to: Ferruh Yigit
Headers show
  • minor lgtm warning fixes
Related show


Context Check Description
ci/checkpatch warning coding style issues
ci/Intel-compilation success Compilation OK
ci/intel-Performance-Testing success Performance Testing PASS
ci/mellanox-Performance-Testing fail Performance Testing issues

Commit Message

Stephen Hemminger March 27, 2019, 9:08 p.m. UTC
Since previous test is for mtu < 1519 the next else if
is always true. This causes the lgtm static tool to complain.
Not a real issue, just cosmetic.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
 drivers/net/netvsc/hn_rxtx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


Rami Rosen March 28, 2019, 12:25 a.m. UTC | #1
Acked-by: Rami Rosen <ramirose@gmail.com>

diff mbox series

diff --git a/drivers/net/netvsc/hn_rxtx.c b/drivers/net/netvsc/hn_rxtx.c
index 6197118b01ee..8818d68cea5c 100644
--- a/drivers/net/netvsc/hn_rxtx.c
+++ b/drivers/net/netvsc/hn_rxtx.c
@@ -123,7 +123,7 @@  hn_update_packet_stats(struct hn_stats *stats, const struct rte_mbuf *m)
 		else if (s < 1519)
-		else if (s >= 1519)
+		else