Messages in this thread | | | Date | Fri, 31 May 2013 13:17:47 +0100 | From | James Hogan <> | Subject | Re: [PATCH v2 9/9] gpio-tz1090-pdc: add TZ1090 PDC gpio driver |
| |
On 29/05/13 17:12, James Hogan wrote: > On 29/05/13 16:38, Linus Walleij wrote: >> On Fri, May 24, 2013 at 6:21 PM, James Hogan <james.hogan@imgtec.com> wrote: >>> +Optional properties: >>> +- gpio-ranges: Mapping to pin controller pins >>> + >>> +- interrupts: Individual syswake interrupts (other GPIOs cannot interrupt) >>> + >>> + >>> +Example: >>> + >>> + pdc_gpios: gpio-controller@02006500 { >>> + gpio-controller; >>> + #gpio-cells = <2>; >>> + >>> + compatible = "img,tz1090-pdc-gpio"; >>> + reg = <0x02006500 0x100>; >>> + >>> + interrupt-parent = <&pdc>; >>> + interrupts = <1 0 0>, /* Syswake 0 */ >>> + <1 1 0>, /* Syswake 1 */ >>> + <1 2 0>; /* Syswake 2 */ >>> + }; >> >> Interrupt flags seem to be undocumented here. > > Hmmm yes, good spot.
Looking again at this, the driver doesn't expose any new interrupts. These 3 interrupts are simply passed on by the gpio chip's to_irq() callback. The flags in the example depend entirely on the interrupt parent, in this case the PDC irq driver in patch 2, so they don't need documenting here.
Cheers James
| |