diff options
author | Russell King <rmk+kernel@arm.linux.org.uk> | 2014-07-08 13:02:04 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-07-08 20:57:46 -0700 |
commit | 96018f52c532b69424d354a848ac7812136650f6 (patch) | |
tree | 8a42529815222710c46e9690e4782f5fb8abfffb /drivers | |
parent | 344756f6e36b056ed361eedbd68244b108bdb1c6 (diff) | |
download | linux-96018f52c532b69424d354a848ac7812136650f6.tar.bz2 |
net: fec: remove useless status check in tx reap path
Remove a useless status check in the transmit reap path - we have
already checked that the BD_ENET_TX_READY bit is clear, and as the
hardware only ever clears this bit, there is no way this test can ever
be true.
Acked-by: Fugang Duan <B38611@freescale.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/ethernet/freescale/fec_main.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/net/ethernet/freescale/fec_main.c b/drivers/net/ethernet/freescale/fec_main.c index 09fcdc768931..8679c919419c 100644 --- a/drivers/net/ethernet/freescale/fec_main.c +++ b/drivers/net/ethernet/freescale/fec_main.c @@ -1133,9 +1133,6 @@ fec_enet_tx(struct net_device *ndev) skb_tstamp_tx(skb, &shhwtstamps); } - if (status & BD_ENET_TX_READY) - netdev_err(ndev, "HEY! Enet xmit interrupt and TX_READY\n"); - /* Deferred means some collisions occurred during transmit, * but we eventually sent the packet OK. */ |