diff options
| author | Horatiu Vultur <horatiu.vultur@microchip.com> | 2021-12-01 17:53:51 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2021-12-02 15:17:28 +0300 |
| commit | a290cf692779ccce6dbb05398fad32d035bfd313 (patch) | |
| tree | 11d6d853c2b70d7e7808fea9e6fea90d3ae125b8 | |
| parent | dfb40cba6d4500f8196b2df66c29c54f5c17af2b (diff) | |
| download | linux-a290cf692779ccce6dbb05398fad32d035bfd313.tar.xz | |
net: lan966x: Fix duplicate check in frame extraction
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>
| -rw-r--r-- | drivers/net/ethernet/microchip/lan966x/lan966x_main.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/net/ethernet/microchip/lan966x/lan966x_main.c b/drivers/net/ethernet/microchip/lan966x/lan966x_main.c index e9e4dca6542d..be5e2b3a7f43 100644 --- a/drivers/net/ethernet/microchip/lan966x/lan966x_main.c +++ b/drivers/net/ethernet/microchip/lan966x/lan966x_main.c @@ -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); |