Skip to content

Commit

Permalink
net: ethernet: dnet: Remove set but unused variable 'len'
Browse files Browse the repository at this point in the history
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 4796417 ("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>
  • Loading branch information
Wang Hai authored and Jakub Kicinski committed Sep 7, 2020
1 parent f3b1144 commit 30ebaf8
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions drivers/net/ethernet/dnet.c
Original file line number Diff line number Diff line change
Expand Up @@ -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;

Expand All @@ -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);
Expand Down

0 comments on commit 30ebaf8

Please sign in to comment.