Re: [RFC 3/4] dt-bindings: iio: add support for GPIO triggers

[Fabrice Gasnier]

On 02/23/2017 04:03 PM, Linus Walleij wrote:
> On Fri, Feb 17, 2017 at 5:03 PM, Fabrice Gasnier <fabrice.gasnier@xxxxxx> wrote:
>
> > Document Industrial I/O GPIO trigger support.
> >
> > Signed-off-by: Fabrice Gasnier <fabrice.gasnier@xxxxxx>
> (...)
> > +- gpio-trigger-rising-edge: A rising edge on GPIO will trigger.
> > +- gpio-trigger-falling-edge: A falling edge on GPIO will trigger.
> > +  Note: Both rising and falling edge may be used. In case none of
> > +  rising or falling edge is selected, rising edge is selected by
> > +  default.
>
> We have GPIO_ACTIVE*_HIGH and GPIO_ACTIVE_LOW in the
> gpios second cell already.
>
> Can't we simply use that and specify that for this usecase, reading
> that cell as GPIO_ACTIVE_HIGH == trigger on rising edge and
> GPIO_ACTIVE_LOW == trigger on falling edge?

Hi Linus,

Thanks for reviewing,

I think I'll simply drop patch 3 & 4, as suggested by Lars and Jonathan,
on patch 4, to use interrupt binding directly. Then standard interrupt
binding can be used instead.
Only thing is to add OF support for iio_interrupt_trigger.

BR,
Fabrice
> Yours,
> Linus Walleij