fec: replace flush_dcache_range with dma_sync_single
authorSascha Hauer <s.hauer@pengutronix.de>
Wed, 28 Jan 2009 23:03:09 +0000 (23:03 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sun, 1 Feb 2009 08:58:24 +0000 (00:58 -0800)
flush_dcache_range is not portable across architectures. Use
dma_sync_single instead. Also, the memory must be synchronised in the
receive path aswell.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Acked-by: Greg Ungerer <gerg@uclinux.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/fec.c

index cab07eced06eb894aac9de9a8b42143503ec876b..a17dc6af30c33211955a08cc10fc4c57eb875b30 100644 (file)
@@ -356,8 +356,8 @@ fec_enet_start_xmit(struct sk_buff *skb, struct net_device *dev)
        /* Push the data cache so the CPM does not get stale memory
         * data.
         */
-       flush_dcache_range((unsigned long)skb->data,
-                          (unsigned long)skb->data + skb->len);
+       dma_sync_single(NULL, bdp->cbd_bufaddr,
+                       bdp->cbd_datlen, DMA_TO_DEVICE);
 
        /* Send it on its way.  Tell FEC it's ready, interrupt when done,
         * it's the last BD of the frame, and to put the CRC on the end.
@@ -630,6 +630,9 @@ while (!((status = bdp->cbd_sc) & BD_ENET_RX_EMPTY)) {
        dev->stats.rx_bytes += pkt_len;
        data = (__u8*)__va(bdp->cbd_bufaddr);
 
+       dma_sync_single(NULL, (unsigned long)__pa(data),
+                       pkt_len - 4, DMA_FROM_DEVICE);
+
        /* This does 16 byte alignment, exactly what we need.
         * The packet length includes FCS, but we don't want to
         * include that when passing upstream as it messes up