From: Gabor Juhos <juhosg@openwrt.org>
Date: Sun, 7 Dec 2008 09:46:41 +0000 (+0000)
Subject: ag71xx driver: NAPI poll cleanup
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=c33682316ecb98071d196c61762162143f8248b3;p=openwrt%2Fstaging%2Frobimarko.git

ag71xx driver: NAPI poll cleanup

SVN-Revision: 13538
---

diff --git a/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx.h b/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx.h
index d5f7743d4c..80c8e18ab1 100644
--- a/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx.h
+++ b/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx.h
@@ -38,7 +38,7 @@
 #define ETH_FCS_LEN	4
 
 #define AG71XX_DRV_NAME		"ag71xx"
-#define AG71XX_DRV_VERSION	"0.5.11"
+#define AG71XX_DRV_VERSION	"0.5.12"
 
 #define AG71XX_NAPI_WEIGHT	64
 
diff --git a/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c b/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c
index 7818969355..aa4730ce20 100644
--- a/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c
+++ b/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c
@@ -687,9 +687,16 @@ static int ag71xx_poll(struct napi_struct *napi, int limit)
 		ag71xx_wr(ag, AG71XX_REG_RX_CTRL, RX_CTRL_RXE);
 	}
 
-	if ((done < limit) && ((status & RX_STATUS_PR) == 0)) {
-		DBG("%s: disable polling mode, done=%d, status=%x\n",
-			dev->name, done, status);
+	if (done < limit) {
+		if (status & RX_STATUS_PR)
+			goto more;
+
+		status = ag71xx_rr(ag, AG71XX_REG_TX_STATUS);
+		if (status & TX_STATUS_PS)
+			goto more;
+
+		DBG("%s: disable polling mode, done=%d, limit=%d\n",
+			dev->name, done, limit);
 
 		netif_rx_complete(dev, napi);
 
@@ -697,12 +704,13 @@ static int ag71xx_poll(struct napi_struct *napi, int limit)
 		spin_lock_irqsave(&ag->lock, flags);
 		ag71xx_int_enable(ag, AG71XX_INT_POLL);
 		spin_unlock_irqrestore(&ag->lock, flags);
-		return 0;
+		return done;
 	}
 
-	DBG("%s: stay in polling mode, done=%d, status=%x\n",
-			dev->name, done, status);
-	return 1;
+ more:
+	DBG("%s: stay in polling mode, done=%d, limit=%d\n",
+			dev->name, done, limit);
+	return done;
 }
 
 static irqreturn_t ag71xx_interrupt(int irq, void *dev_id)