This saves 1 comparison.

Signed-off-by: Troy Kisky <troy.ki...@boundarydevices.com>
---
 drivers/net/ethernet/freescale/fec_main.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/net/ethernet/freescale/fec_main.c 
b/drivers/net/ethernet/freescale/fec_main.c
index b5ed287..1902897 100644
--- a/drivers/net/ethernet/freescale/fec_main.c
+++ b/drivers/net/ethernet/freescale/fec_main.c
@@ -1157,8 +1157,13 @@ static void fec_txq(struct net_device *ndev, struct 
fec_enet_private *fep,
                /* Order the load of bd.cur and cbd_sc */
                rmb();
                status = fec16_to_cpu(READ_ONCE(bdp->cbd_sc));
-               if (status & BD_ENET_TX_READY)
+               if (status & BD_ENET_TX_READY) {
+                       if (!readl(txq->bd.reg_desc_active)) {
+                               /* ERR006358 has hit, restart tx */
+                               writel(0, txq->bd.reg_desc_active);
+                       }
                        break;
+               }
 
                index = fec_enet_get_bd_index(bdp, &txq->bd);
 
@@ -1230,11 +1235,6 @@ static void fec_txq(struct net_device *ndev, struct 
fec_enet_private *fep,
                                netif_tx_wake_queue(nq);
                }
        }
-
-       /* ERR006538: Keep the transmitter going */
-       if (bdp != txq->bd.cur &&
-           readl(txq->bd.reg_desc_active) == 0)
-               writel(0, txq->bd.reg_desc_active);
 }
 
 static int
-- 
2.5.0

Reply via email to