Kalle Jokiniemi <kalle.jokiniemi@xxxxxxxxxxxxxxx> writes: > The irqs are enabled one-by-one in pm core resume_noirq phase. > This leads to situation where the twl4030 primary interrupt > handler (PIH) is enabled before the chained secondary handlers > (SIH). As the PIH cannot clear the pending interrupt, and > SIHs have not been enabled yet, a flood of interrupts hangs > the device. > > Fixed the issue by setting the SIH irqs with IRQF_EARLY_RESUME > flags, so they get enabled before the PIH. > > Signed-off-by: Kalle Jokiniemi <kalle.jokiniemi@xxxxxxxxxxxxxxx> Acked-by: Kevin Hilman <khilman@xxxxxx> Thanks, I like this better than the I2C driver fix. Samuel, feel free to take this for v3.7 Kevin > --- > drivers/mfd/twl4030-irq.c | 3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/drivers/mfd/twl4030-irq.c b/drivers/mfd/twl4030-irq.c > index ad733d7..cdd1173 100644 > --- a/drivers/mfd/twl4030-irq.c > +++ b/drivers/mfd/twl4030-irq.c > @@ -672,7 +672,8 @@ int twl4030_sih_setup(struct device *dev, int module, int irq_base) > irq = sih_mod + twl4030_irq_base; > irq_set_handler_data(irq, agent); > agent->irq_name = kasprintf(GFP_KERNEL, "twl4030_%s", sih->name); > - status = request_threaded_irq(irq, NULL, handle_twl4030_sih, 0, > + status = request_threaded_irq(irq, NULL, handle_twl4030_sih, > + IRQF_EARLY_RESUME, > agent->irq_name ?: sih->name, NULL); > > dev_info(dev, "%s (irq %d) chaining IRQs %d..%d\n", sih->name, -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
