Todd Poynor <toddpoynor@xxxxxxxxxx> 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. Kevin [1] commit 5a4f1844c2ba21f804d7729306d9b16eaeb724a8 Author: Felipe Balbi <balbi@xxxxxx> 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 <balbi@xxxxxx> Signed-off-by: Kevin Hilman <khilman@xxxxxx> Cc: stable@xxxxxxxxxxxxxxx Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html