ftgmac100: Don't clear tx desc fields unnecessarily
authorBenjamin Herrenschmidt <benh@kernel.crashing.org>
Mon, 10 Apr 2017 01:15:24 +0000 (11:15 +1000)
committerDavid S. Miller <davem@davemloft.net>
Mon, 10 Apr 2017 20:03:57 +0000 (16:03 -0400)
Those are non-cachable stores, let's avoid those we don't need. Remove
the helper, it's not particularly helpful and since it uses "priv"
I can't move it to the header file.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/faraday/ftgmac100.c

index 802f5e73006c72179aa1aaed044017659c953721..d303c597e9b8a8ceec04792321eea332a0be441d 100644 (file)
@@ -468,16 +468,6 @@ static bool ftgmac100_rx_packet(struct ftgmac100 *priv, int *processed)
        return true;
 }
 
-static void ftgmac100_txdes_reset(const struct ftgmac100 *priv,
-                                 struct ftgmac100_txdes *txdes)
-{
-       /* clear all except end of ring bit */
-       txdes->txdes0 &= cpu_to_le32(priv->txdes0_edotr_mask);
-       txdes->txdes1 = 0;
-       txdes->txdes2 = 0;
-       txdes->txdes3 = 0;
-}
-
 static bool ftgmac100_txdes_owned_by_dma(struct ftgmac100_txdes *txdes)
 {
        return txdes->txdes0 & cpu_to_le32(FTGMAC100_TXDES0_TXDMA_OWN);
@@ -577,7 +567,12 @@ static void ftgmac100_free_tx_packet(struct ftgmac100 *priv,
        dev_kfree_skb(skb);
        priv->tx_skbs[pointer] = NULL;
 
-       ftgmac100_txdes_reset(priv, txdes);
+       /* Clear txdes0 except end of ring bit, clear txdes1 as we
+        * only "OR" into it, leave 2 and 3 alone as 2 is unused
+        * and 3 will be overwritten entirely
+        */
+       txdes->txdes0 &= cpu_to_le32(priv->txdes0_edotr_mask);
+       txdes->txdes1 = 0;
 }
 
 static bool ftgmac100_tx_complete_packet(struct ftgmac100 *priv)