Re: [PATCH v2] mfd: arizona: Add DT binding for irq-gpios property

[Lee Jones <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>]

On Thu, 05 Nov 2015, Sebastien Jan wrote:

> On 03/11/2015 17:52, Charles Keepax wrote:
> >On Tue, Nov 03, 2015 at 03:04:50PM +0000, Mark Brown wrote:
> >>On Tue, Nov 03, 2015 at 03:10:06PM +0100, Sebastien Jan wrote:
> >>
> >>>+  - irq-gpios : GPIO specifier for the GPIO connected to /IRQ
> >>The platform data this is providing is essentially a workaround for the
> >>lack of an irq_to_gpio() function and the use of it is a workaround for
> >>the lack of level triggered IRQ emulation in genirq...  do we really
> >>want this in the ABI (especially as a generic thing)?
> >Personally I would rather not add this to the DT unless we really
> >had to.
> >
> >Also are you sure this is necessary on your system? Specifically
> >you only need this entry if the CODEC is connected to an IRQ line
> >that only supports edge triggered IRQs.
> 
> Mark, Charles, thanks for pointing me to this information.
> 
> I don't have this specific constraint on my system and could finally
> also have it working using the regular interrupts / interrup-parent
> properties.
> 
> So this new binding is not required for my case. I suppose we can
> forget about it.

My favourite kind of property. ;)

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html