Kevin, Todd, On Thu, Jan 12, 2012 at 1:55 AM, Kevin Hilman <[email protected]> wrote: > Todd Poynor <[email protected]> writes: > >>> +static irqreturn_t _interrupt(int irq, void *data) >>> +{ >>> + struct smartreflex *sr = (struct smartreflex *)data; >>> + u32 status = 0; >>> + >>> + /* Read the status bits */ >>> + sr_read_reg(sr, IRQSTATUS); >>> + >>> + /* Clear them by writing back */ >>> + sr_write_reg(sr, IRQSTATUS, status); >> >> Felipe Balbi sent a patch to the list fixing the write of zero, >> failing to clear the interrupts, to the list recently. > > Right, this patch is in mainline now[1], so rebasing this series should > pick up this change. I did include all the latest fixes from Felipe and Nishant in the latest version. More details to come in the chang e logs.
Jean > > Kevin > > [1] > commit 5a4f1844c2ba21f804d7729306d9b16eaeb724a8 > Author: Felipe Balbi <[email protected]> > Date: Wed Nov 23 14:43:37 2011 -0800 > > ARM: OMAP: smartreflex: fix IRQ handling bug > > Fix a bug which has been on this driver since > it was added by the original commit 984aa6db > which would never clear IRQSTATUS bits. > > Signed-off-by: Felipe Balbi <[email protected]> > Signed-off-by: Kevin Hilman <[email protected]> > Cc: [email protected] > Signed-off-by: Tony Lindgren <[email protected]> -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
