net: fec: remove useless status check in tx reap path
authorRussell King <rmk+kernel@arm.linux.org.uk>
Tue, 8 Jul 2014 12:02:04 +0000 (13:02 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 9 Jul 2014 03:57:46 +0000 (20:57 -0700)
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>
drivers/net/ethernet/freescale/fec_main.c

index 09fcdc768931ea192637e0cee6771c9f463bd156..8679c919419c7115db13651d86f7f0383bf976e0 100644 (file)
@@ -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.
                 */