net: ethernet: dnet: Remove set but unused variable 'len'
authorWang Hai <wanghai38@huawei.com>
Mon, 7 Sep 2020 14:12:07 +0000 (22:12 +0800)
committerJakub Kicinski <kuba@kernel.org>
Mon, 7 Sep 2020 20:56:24 +0000 (13:56 -0700)
Fixes gcc '-Wunused-but-set-variable' warning:

drivers/net/ethernet/dnet.c: In function dnet_start_xmit
drivers/net/ethernet/dnet.c:511:15: warning: variable ‘len’ set but not used [-Wunused-but-set-variable]

commit 4796417417a6 ("dnet: Dave DNET ethernet controller driver (updated)")
involved this unused variable, remove it.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Wang Hai <wanghai38@huawei.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/dnet.c

index db98274..3143df9 100644 (file)
@@ -508,7 +508,7 @@ static netdev_tx_t dnet_start_xmit(struct sk_buff *skb, struct net_device *dev)
 
        struct dnet *bp = netdev_priv(dev);
        u32 tx_status, irq_enable;
-       unsigned int len, i, tx_cmd, wrsz;
+       unsigned int i, tx_cmd, wrsz;
        unsigned long flags;
        unsigned int *bufp;
 
@@ -518,9 +518,6 @@ static netdev_tx_t dnet_start_xmit(struct sk_buff *skb, struct net_device *dev)
               skb->len, skb->head, skb->data);
        dnet_print_skb(skb);
 
-       /* frame size (words) */
-       len = (skb->len + 3) >> 2;
-
        spin_lock_irqsave(&bp->lock, flags);
 
        tx_status = dnet_readl(bp, TX_STATUS);