diff options
| author | Anton Vorontsov <avorontsov@ru.mvista.com> | 2009-10-12 08:00:33 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-10-13 08:54:02 +0200 |
| commit | 14231176b0dc358f8693f25b62017d222dd995e6 (patch) | |
| tree | c4340a05293f7a57b6c1a3e045b489349c6c2ffb /drivers/net/gianfar.c | |
| parent | gianfar: Simplify skb resources freeing code (diff) | |
| download | linux-14231176b0dc358f8693f25b62017d222dd995e6.tar.xz linux-14231176b0dc358f8693f25b62017d222dd995e6.zip | |
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/net/gianfar.c')
| -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) rxbdp++; } - /* Set the last descriptor in the ring to wrap */ - rxbdp--; - rxbdp->status |= RXBD_WRAP; - /* If the device has multiple interrupts, register for * them. Otherwise, only register for the one */ if (priv->device_flags & FSL_GIANFAR_DEV_HAS_MULTI_INTR) { |