Kevin, Todd,
On Thu, Jan 12, 2012 at 1:55 AM, Kevin Hilman <khilman@xxxxxx> wrote:
> 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.
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 <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
