diff options
author | Anton Vorontsov <avorontsov@ru.mvista.com> | 2009-10-12 02:00:33 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-10-13 02:54:02 -0400 |
commit | 14231176b0dc358f8693f25b62017d222dd995e6 (patch) | |
tree | c4340a05293f7a57b6c1a3e045b489349c6c2ffb /drivers | |
parent | e69edd21819823bbad06d1d02f9fa21713fad173 (diff) |
gianfar: Don't needlessly set the wrap bit for the last RX BD
startup_gfar() sets the wrap bit for the last rxbd just after
gfar_new_rxbdp() call, which is issued for all rxbds. And
gfar_new_rxbdp() has the following check already:
if (bdp == priv->rx_bd_base + priv->rx_ring_size - 1)
lstatus |= BD_LFLAG(RXBD_WRAP);
So we don't need to set the bit again.
Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/gianfar.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/net/gianfar.c b/drivers/net/gianfar.c index a8b50c9b2d94..f84974195507 100644 --- a/drivers/net/gianfar.c +++ b/drivers/net/gianfar.c | |||
@@ -1028,10 +1028,6 @@ int startup_gfar(struct net_device *ndev) | |||
1028 | rxbdp++; | 1028 | rxbdp++; |
1029 | } | 1029 | } |
1030 | 1030 | ||
1031 | /* Set the last descriptor in the ring to wrap */ | ||
1032 | rxbdp--; | ||
1033 | rxbdp->status |= RXBD_WRAP; | ||
1034 | |||
1035 | /* If the device has multiple interrupts, register for | 1031 | /* If the device has multiple interrupts, register for |
1036 | * them. Otherwise, only register for the one */ | 1032 | * them. Otherwise, only register for the one */ |
1037 | if (priv->device_flags & FSL_GIANFAR_DEV_HAS_MULTI_INTR) { | 1033 | if (priv->device_flags & FSL_GIANFAR_DEV_HAS_MULTI_INTR) { |