Re: [PATCH 1/2] gpio: eic: Add edge trigger emulation for EIC

From: Linus Walleij
Date: Thu Apr 26 2018 - 04:10:37 EST


On Fri, Mar 30, 2018 at 9:02 AM, Baolin Wang <baolin.wang@xxxxxxxxxx> wrote:

> The Spreadtrum debounce EIC and latch EIC can not support edge trigger,
> but most GPIO users (like gpio-key driver) only use the edge trigger,
> thus the EIC driver need add some support to emulate the edge trigger
> to satisfy this requirement.
>
> Signed-off-by: Baolin Wang <baolin.wang@xxxxxxxxxx>

Patch applied.

The patch is fine and in line with other kernel drivers doing edge
trigger emulation and we do not hold back on useful features.

But I want to take this opportunity to ping Uwe Kleine-KÃnig about
his ideas for an edge trigger emulation library in the gpiolib core,
and whether this also follows the pattern.

I think we have a slew of these drivers doing similar stuff around.

Yours,
Linus Walleij