Re: [PATCH v2] pinctrl: nsp: Set irq handler based on trig type

From: Linus Walleij
Date: Sat Jul 11 2020 - 17:10:04 EST


On Fri, Jul 3, 2020 at 3:18 AM Mark Tomlinson
<mark.tomlinson@xxxxxxxxxxxxxxxxxxx> wrote:

> Rather than always using handle_simple_irq() as the gpio_irq_chip
> handler, set a more appropriate handler based on the IRQ trigger type
> requested. This is important for level triggered interrupts which need
> to be masked during handling. Also, fix the interrupt acknowledge so
> that it clears only one interrupt instead of all interrupts which are
> currently active. Finally there is no need to clear the interrupt during
> the interrupt handler, since the edge-triggered handler will do that for
> us.
>
> Signed-off-by: Mark Tomlinson <mark.tomlinson@xxxxxxxxxxxxxxxxxxx>
> ---
> Changes in v2:
> - Don't perform unnecessary acks.

Patch applied.

Yours,
Linus Walleij