Commit 14231176 authored Oct 12, 2009 by Anton Vorontsov Committed by David S. Miller Oct 12, 2009

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>

parent e69edd21

drivers/net/gianfar.c

+0 −4

Original line number	Diff line number	Diff line
		@@ -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) {

Admin message