memory: tegra: Do not handle spurious interrupts
authorDmitry Osipenko <digetx@gmail.com>
Mon, 9 Apr 2018 19:28:27 +0000 (22:28 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 6 Aug 2018 14:24:38 +0000 (16:24 +0200)
[ Upstream commit bf3fbdfbec947cdd04b2f2c4bce11534c8786eee ]

The ISR reads interrupts-enable mask, but doesn't utilize it. Apply the
mask to the interrupt status and don't handle interrupts that MC driver
haven't asked for. Kernel would disable spurious MC IRQ and report the
error. This would happen only in a case of a very severe bug.

Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/memory/tegra/mc.c

index a1ae0cc2b86d50dd24e84cf66871dd3148d86e01..13dcee4ed0cf757489bf95a9d923cf035d9f30e6 100644 (file)
@@ -252,8 +252,11 @@ static irqreturn_t tegra_mc_irq(int irq, void *data)
        unsigned int bit;
 
        /* mask all interrupts to avoid flooding */
-       status = mc_readl(mc, MC_INTSTATUS);
        mask = mc_readl(mc, MC_INTMASK);
+       status = mc_readl(mc, MC_INTSTATUS) & mask;
+
+       if (!status)
+               return IRQ_NONE;
 
        for_each_set_bit(bit, &status, 32) {
                const char *error = status_names[bit] ?: "unknown";