Re: [PATCH v2] gpio/gpiolib: Forbid irqchip default trigger if probed over DT

2016-09-12 Thread Linus Walleij
On Thu, Sep 8, 2016 at 7:13 PM, Marc Zyngier wrote: > On 07/09/16 23:18, Linus Walleij wrote: >> On Wed, Sep 7, 2016 at 10:12 AM, Marc Zyngier wrote: >> >>> Using a default trigger is a bad idea if using DT to configure >>> interrupts, as the device's

Re: [PATCH v2] gpio/gpiolib: Forbid irqchip default trigger if probed over DT

2016-09-12 Thread Linus Walleij
On Thu, Sep 8, 2016 at 7:13 PM, Marc Zyngier wrote: > On 07/09/16 23:18, Linus Walleij wrote: >> On Wed, Sep 7, 2016 at 10:12 AM, Marc Zyngier wrote: >> >>> Using a default trigger is a bad idea if using DT to configure >>> interrupts, as the device's interrupt specifier will always contain >>>

Re: [PATCH v2] gpio/gpiolib: Forbid irqchip default trigger if probed over DT

2016-09-08 Thread Marc Zyngier
On 07/09/16 23:18, Linus Walleij wrote: > On Wed, Sep 7, 2016 at 10:12 AM, Marc Zyngier wrote: > >> Using a default trigger is a bad idea if using DT to configure >> interrupts, as the device's interrupt specifier will always contain >> the trigger configuration. >> >>

Re: [PATCH v2] gpio/gpiolib: Forbid irqchip default trigger if probed over DT

2016-09-08 Thread Marc Zyngier
On 07/09/16 23:18, Linus Walleij wrote: > On Wed, Sep 7, 2016 at 10:12 AM, Marc Zyngier wrote: > >> Using a default trigger is a bad idea if using DT to configure >> interrupts, as the device's interrupt specifier will always contain >> the trigger configuration. >> >> Let's warn about that

Re: [PATCH v2] gpio/gpiolib: Forbid irqchip default trigger if probed over DT

2016-09-07 Thread Linus Walleij
On Wed, Sep 7, 2016 at 10:12 AM, Marc Zyngier wrote: > Using a default trigger is a bad idea if using DT to configure > interrupts, as the device's interrupt specifier will always contain > the trigger configuration. > > Let's warn about that particular situation, and

Re: [PATCH v2] gpio/gpiolib: Forbid irqchip default trigger if probed over DT

2016-09-07 Thread Linus Walleij
On Wed, Sep 7, 2016 at 10:12 AM, Marc Zyngier wrote: > Using a default trigger is a bad idea if using DT to configure > interrupts, as the device's interrupt specifier will always contain > the trigger configuration. > > Let's warn about that particular situation, and revert to not > having a

[PATCH v2] gpio/gpiolib: Forbid irqchip default trigger if probed over DT

2016-09-07 Thread Marc Zyngier
Using a default trigger is a bad idea if using DT to configure interrupts, as the device's interrupt specifier will always contain the trigger configuration. Let's warn about that particular situation, and revert to not having a default. Hopefully, the couple of drivers still using this feature

[PATCH v2] gpio/gpiolib: Forbid irqchip default trigger if probed over DT

2016-09-07 Thread Marc Zyngier
Using a default trigger is a bad idea if using DT to configure interrupts, as the device's interrupt specifier will always contain the trigger configuration. Let's warn about that particular situation, and revert to not having a default. Hopefully, the couple of drivers still using this feature