From: Cindy H Kao <[email protected]>

In order to avoid issues during high-load traffic, the interrupt
status register has to be cleared ONLY after the RX size is read.

Signed-off-by: Cindy H Kao <[email protected]>
Signed-off-by: Inaky Perez-Gonzalez <[email protected]>
---
 drivers/net/wimax/i2400m/sdio-rx.c |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/drivers/net/wimax/i2400m/sdio-rx.c 
b/drivers/net/wimax/i2400m/sdio-rx.c
index f6ca51a..1c90469 100644
--- a/drivers/net/wimax/i2400m/sdio-rx.c
+++ b/drivers/net/wimax/i2400m/sdio-rx.c
@@ -138,6 +138,11 @@ void i2400ms_rx(struct i2400ms *i2400ms)
                ret = rx_size;
                goto error_get_size;
        }
+       /*
+        * Hardware quirk: make sure to clear the INTR status register
+        * AFTER getting the data transfer size.
+        */
+       sdio_writeb(func, 1, I2400MS_INTR_CLEAR_ADDR, &ret);
 
        ret = -ENOMEM;
        skb = alloc_skb(rx_size, GFP_ATOMIC);
@@ -209,7 +214,6 @@ void i2400ms_irq(struct sdio_func *func)
                dev_err(dev, "RX: BUG? got IRQ but no interrupt ready?\n");
                goto error_no_irq;
        }
-       sdio_writeb(func, 1, I2400MS_INTR_CLEAR_ADDR, &ret);
        i2400ms_rx(i2400ms);
 error_no_irq:
        d_fnend(6, dev, "(i2400ms %p) = void\n", i2400ms);
-- 
1.6.2.5

_______________________________________________
wimax mailing list
[email protected]
http://lists.linuxwimax.org/listinfo/wimax

Reply via email to