diff options
author | Sascha Hauer <s.hauer@pengutronix.de> | 2009-01-28 18:03:09 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-02-01 03:58:24 -0500 |
commit | ccdc4f198193eb4956b8dbc00745270525c4cd6e (patch) | |
tree | 77efa737a2ee47692e6c09250f0cf0da3ae5a220 /drivers/net/fec.c | |
parent | 43268dcea7512cc10bc2542f20ce37971ecfeb48 (diff) |
fec: replace flush_dcache_range with dma_sync_single
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>
Diffstat (limited to 'drivers/net/fec.c')
-rw-r--r-- | drivers/net/fec.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/net/fec.c b/drivers/net/fec.c index cab07eced06e..a17dc6af30c3 100644 --- a/drivers/net/fec.c +++ b/drivers/net/fec.c | |||
@@ -356,8 +356,8 @@ fec_enet_start_xmit(struct sk_buff *skb, struct net_device *dev) | |||
356 | /* Push the data cache so the CPM does not get stale memory | 356 | /* Push the data cache so the CPM does not get stale memory |
357 | * data. | 357 | * data. |
358 | */ | 358 | */ |
359 | flush_dcache_range((unsigned long)skb->data, | 359 | dma_sync_single(NULL, bdp->cbd_bufaddr, |
360 | (unsigned long)skb->data + skb->len); | 360 | bdp->cbd_datlen, DMA_TO_DEVICE); |
361 | 361 | ||
362 | /* Send it on its way. Tell FEC it's ready, interrupt when done, | 362 | /* Send it on its way. Tell FEC it's ready, interrupt when done, |
363 | * it's the last BD of the frame, and to put the CRC on the end. | 363 | * 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)) { | |||
630 | dev->stats.rx_bytes += pkt_len; | 630 | dev->stats.rx_bytes += pkt_len; |
631 | data = (__u8*)__va(bdp->cbd_bufaddr); | 631 | data = (__u8*)__va(bdp->cbd_bufaddr); |
632 | 632 | ||
633 | dma_sync_single(NULL, (unsigned long)__pa(data), | ||
634 | pkt_len - 4, DMA_FROM_DEVICE); | ||
635 | |||
633 | /* This does 16 byte alignment, exactly what we need. | 636 | /* This does 16 byte alignment, exactly what we need. |
634 | * The packet length includes FCS, but we don't want to | 637 | * The packet length includes FCS, but we don't want to |
635 | * include that when passing upstream as it messes up | 638 | * include that when passing upstream as it messes up |