diff options
| author | Gabor Juhos <juhosg@openwrt.org> | 2008-12-07 09:46:41 +0000 |
|---|---|---|
| committer | Gabor Juhos <juhosg@openwrt.org> | 2008-12-07 09:46:41 +0000 |
| commit | c33682316ecb98071d196c61762162143f8248b3 (patch) | |
| tree | 7469ec3126cdec7367ed87d517718dbe8750e94c | |
| parent | 398ff859daa5943995a4f8d0cd95920069a42455 (diff) | |
| download | master-31e0f0ae-c33682316ecb98071d196c61762162143f8248b3.tar.gz master-31e0f0ae-c33682316ecb98071d196c61762162143f8248b3.tar.bz2 master-31e0f0ae-c33682316ecb98071d196c61762162143f8248b3.zip | |
ag71xx driver: NAPI poll cleanup
SVN-Revision: 13538
| -rw-r--r-- | target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx.h | 2 | ||||
| -rw-r--r-- | target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c | 22 |
2 files changed, 16 insertions, 8 deletions
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) |