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@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html