net: macb: clear interrupts when disabling them
authorNathan Sullivan <nathan.sullivan@ni.com>
Thu, 14 Jan 2016 19:27:27 +0000 (13:27 -0600)
committerDavid S. Miller <davem@davemloft.net>
Fri, 15 Jan 2016 19:47:09 +0000 (14:47 -0500)
Disabling interrupts with the IDR register does not stop the macb hardware
from asserting its interrupt line if there are interrupts pending.  Always
clear the interrupts using ISR, and be sure to write it on hardware that
is not read-to-clear, like Zynq.  Not doing so will cause interrupts when
the driver doesn't expect them.

Signed-off-by: Nathan Sullivan <nathan.sullivan@ni.com>
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/cadence/macb.c

index c56347536f6b9b32d846ff587a8e8e6af9782d83..9d9984a87d4227f0b79c01e5ff1a3877da41b7e3 100644 (file)
@@ -1040,6 +1040,8 @@ static irqreturn_t macb_interrupt(int irq, void *dev_id)
                /* close possible race with dev_close */
                if (unlikely(!netif_running(dev))) {
                        queue_writel(queue, IDR, -1);
+                       if (bp->caps & MACB_CAPS_ISR_CLEAR_ON_WRITE)
+                               queue_writel(queue, ISR, -1);
                        break;
                }
 
@@ -1561,6 +1563,8 @@ static void macb_reset_hw(struct macb *bp)
        for (q = 0, queue = bp->queues; q < bp->num_queues; ++q, ++queue) {
                queue_writel(queue, IDR, -1);
                queue_readl(queue, ISR);
+               if (bp->caps & MACB_CAPS_ISR_CLEAR_ON_WRITE)
+                       queue_writel(queue, ISR, -1);
        }
 }