Skip to content

Commit bf2977a

Browse files
Zheng Yongjundavem330
authored andcommitted
net: micrel: Remove set but not used variable
Fixes gcc '-Wunused-but-set-variable' warning: drivers/net/ethernet/micrel/ksz884x.c: In function rx_proc: drivers/net/ethernet/micrel/ksz884x.c:4981:6: warning: variable ‘rx_status’ set but not used [-Wunused-but-set-variable] drivers/net/ethernet/micrel/ksz884x.c: In function netdev_get_ethtool_stats: drivers/net/ethernet/micrel/ksz884x.c:6512:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable] these variable is never used, so remove it. Signed-off-by: Zheng Yongjun <zhengyongjun3@huawei.com> Signed-off-by: David S. Miller <davem@davemloft.net>
1 parent f9d80a0 commit bf2977a

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

drivers/net/ethernet/micrel/ksz884x.c

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4978,7 +4978,6 @@ static inline int rx_proc(struct net_device *dev, struct ksz_hw* hw,
49784978
struct dev_info *hw_priv = priv->adapter;
49794979
struct ksz_dma_buf *dma_buf;
49804980
struct sk_buff *skb;
4981-
int rx_status;
49824981

49834982
/* Received length includes 4-byte CRC. */
49844983
packet_len = status.rx.frame_len - 4;
@@ -5014,7 +5013,7 @@ static inline int rx_proc(struct net_device *dev, struct ksz_hw* hw,
50145013
dev->stats.rx_bytes += packet_len;
50155014

50165015
/* Notify upper layer for received packet. */
5017-
rx_status = netif_rx(skb);
5016+
netif_rx(skb);
50185017

50195018
return 0;
50205019
}
@@ -6507,7 +6506,6 @@ static void netdev_get_ethtool_stats(struct net_device *dev,
65076506
int i;
65086507
int n;
65096508
int p;
6510-
int rc;
65116509
u64 counter[TOTAL_PORT_COUNTER_NUM];
65126510

65136511
mutex_lock(&hw_priv->lock);
@@ -6528,19 +6526,19 @@ static void netdev_get_ethtool_stats(struct net_device *dev,
65286526

65296527
if (1 == port->mib_port_cnt && n < SWITCH_PORT_NUM) {
65306528
p = n;
6531-
rc = wait_event_interruptible_timeout(
6529+
wait_event_interruptible_timeout(
65326530
hw_priv->counter[p].counter,
65336531
2 == hw_priv->counter[p].read,
65346532
HZ * 1);
65356533
} else
65366534
for (i = 0, p = n; i < port->mib_port_cnt - n; i++, p++) {
65376535
if (0 == i) {
6538-
rc = wait_event_interruptible_timeout(
6536+
wait_event_interruptible_timeout(
65396537
hw_priv->counter[p].counter,
65406538
2 == hw_priv->counter[p].read,
65416539
HZ * 2);
65426540
} else if (hw->port_mib[p].cnt_ptr) {
6543-
rc = wait_event_interruptible_timeout(
6541+
wait_event_interruptible_timeout(
65446542
hw_priv->counter[p].counter,
65456543
2 == hw_priv->counter[p].read,
65466544
HZ * 1);

0 commit comments

Comments
 (0)