net: lan966x: Fix duplicate check in frame extraction
authorHoratiu Vultur <horatiu.vultur@microchip.com>
Wed, 1 Dec 2021 14:53:51 +0000 (15:53 +0100)
committerDavid S. Miller <davem@davemloft.net>
Thu, 2 Dec 2021 12:17:28 +0000 (12:17 +0000)
The blamed commit generates the following smatch static checker warning:

 drivers/net/ethernet/microchip/lan966x/lan966x_main.c:515 lan966x_xtr_irq_handler()
         warn: duplicate check 'sz < 0' (previous on line 502)

This patch fixes this issue removing the duplicate check 'sz < 0'

Fixes: d28d6d2e37d10d ("net: lan966x: add port module support")
Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/microchip/lan966x/lan966x_main.c

index e9e4dca..be5e2b3 100644 (file)
@@ -512,11 +512,6 @@ static irqreturn_t lan966x_xtr_irq_handler(int irq, void *args)
                        *buf = val;
                }
 
-               if (sz < 0) {
-                       err = sz;
-                       break;
-               }
-
                skb->protocol = eth_type_trans(skb, dev);
 
                netif_rx_ni(skb);