Re: [RFC] Potential issue with GPIO/IRQ flags

2015-09-17 Thread Andrew F. Davis
On 09/17/2015 05:35 PM, Stephen Warren wrote: On 09/17/2015 11:21 AM, Andrew F. Davis wrote: On 09/17/2015 12:20 PM, Rob Herring wrote: On Thu, Sep 17, 2015 at 10:53 AM, Andrew F. Davis wrote: On 09/16/2015 08:26 PM, Rob Herring wrote: On Wed, Sep 16, 2015 at 4:07 PM, Andrew F. Davis wro

Re: [RFC] Potential issue with GPIO/IRQ flags

2015-09-17 Thread Stephen Warren
On 09/17/2015 11:21 AM, Andrew F. Davis wrote: > > > On 09/17/2015 12:20 PM, Rob Herring wrote: >> On Thu, Sep 17, 2015 at 10:53 AM, Andrew F. Davis wrote: >>> On 09/16/2015 08:26 PM, Rob Herring wrote: On Wed, Sep 16, 2015 at 4:07 PM, Andrew F. Davis wrote: > > Hello all, >>>

Re: [RFC] Potential issue with GPIO/IRQ flags

2015-09-17 Thread Andrew F. Davis
On 09/17/2015 12:20 PM, Rob Herring wrote: On Thu, Sep 17, 2015 at 10:53 AM, Andrew F. Davis wrote: On 09/16/2015 08:26 PM, Rob Herring wrote: On Wed, Sep 16, 2015 at 4:07 PM, Andrew F. Davis wrote: Hello all, I've noticed that in a few DT bindings GPIO_ACTIVE_* defines are incorrectly

Re: [RFC] Potential issue with GPIO/IRQ flags

2015-09-17 Thread Rob Herring
On Thu, Sep 17, 2015 at 10:53 AM, Andrew F. Davis wrote: > On 09/16/2015 08:26 PM, Rob Herring wrote: >> >> On Wed, Sep 16, 2015 at 4:07 PM, Andrew F. Davis wrote: >>> >>> Hello all, >>> >>> I've noticed that in a few DT bindings GPIO_ACTIVE_* defines are >>> incorrectly used as interrupt flags.

Re: [RFC] Potential issue with GPIO/IRQ flags

2015-09-17 Thread Andrew F. Davis
On 09/16/2015 08:26 PM, Rob Herring wrote: On Wed, Sep 16, 2015 at 4:07 PM, Andrew F. Davis wrote: Hello all, I've noticed that in a few DT bindings GPIO_ACTIVE_* defines are incorrectly used as interrupt flags. GPIO_ACTIVE_*'s are defined in: include/dt-bindings/gpio/gpio.h and are used to

Re: [RFC] Potential issue with GPIO/IRQ flags

2015-09-16 Thread Rob Herring
On Wed, Sep 16, 2015 at 4:07 PM, Andrew F. Davis wrote: > Hello all, > > I've noticed that in a few DT bindings GPIO_ACTIVE_* defines are > incorrectly used as interrupt flags. GPIO_ACTIVE_*'s are defined > in: > > include/dt-bindings/gpio/gpio.h > > and are used to describe GPIO pins. IRQ types a

[RFC] Potential issue with GPIO/IRQ flags

2015-09-16 Thread Andrew F. Davis
Hello all, I've noticed that in a few DT bindings GPIO_ACTIVE_* defines are incorrectly used as interrupt flags. GPIO_ACTIVE_*'s are defined in: include/dt-bindings/gpio/gpio.h and are used to describe GPIO pins. IRQ types are defined in: include/dt-bindings/interrupt-controller/irq.h and are