gianfar: Only process completed frames

If the LAST bit is not set in the RxBD, it's possible we're processing
an incomplete frame, which is bad.  While we're at it, add a constant
for the error bitmask, so the whole if-clause fits on one line,
and is more legible.

Signed-off-by: Dai Haruki <dai.haruki@freescale.com>
Signed-off-by: Andy Fleming <afleming@freescale.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>

1 files changed, 1 insertions, 3 deletions

diff --git a/drivers/net/gianfar.c b/drivers/net/gianfar.c
index 0ab4b26c79a5..a59edf7eacdc 100644
--- a/drivers/net/gianfar.c
+++ b/drivers/net/gianfar.c
@@ -1526,9 +1526,7 @@ int gfar_clean_rx_ring(struct net_device *dev, int rx_work_limit)
                 rmb();
                 skb = priv->rx_skbuff[priv->skb_currx];
 
-                if (!(bdp->status &
-                      (RXBD_LARGE | RXBD_SHORT | RXBD_NONOCTET
-                       | RXBD_CRCERR | RXBD_OVERRUN | RXBD_TRUNCATED))) {
+                if ((bdp->status & RXBD_LAST) && !(bdp->status & RXBD_ERR)) {
                         /* Increment the number of packets */
                         dev->stats.rx_packets++;
                         howmany++;