lkml.org 
[lkml]   [2013]   [Oct]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] regmap: irq: clear status when disable irq
On Mon, Oct 14, 2013 at 12:23:53PM +0800, Yi Zhang wrote:

> Change-Id: I371201f365c5a8470073a393068cfeb4e3d14a03

Don't include noise like this in upstream submissions.

> + /* Ack masked but set interrupts */
> + reg = d->chip->status_base +
> + (i * map->reg_stride * d->irq_reg_stride);
> + ret = regmap_read(d->map, reg, &d->status_buf[i]);
> + if (ret != 0)
> + dev_err(d->map->dev, "Failed to read IRQ status: %d\n",
> + ret);

No, this isn't good - it'll read the hardware interrupt status again.
This will break any devices that are clear on read since enabled
interrupts will also be read. I'd suggest unconditionally acknowledging
all masked interrupts as the simplest approach, obviously it'd be better
to only acknowledge newly masked interrupts but that is more complex to
implement.
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2013-10-14 13:21    [W:0.043 / U:1.440 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site