On 6/14/2013 3:50 PM, Philip Avinash wrote: > From: KV Sujith <sujithkv@xxxxxx> > > - Add of_device_id for Davinci GPIO driver. > - Add function to populate data from DT. > - Modify the probe to read from DT if DT match is found. > - Add DT binding documentation for Davinci GPIO properties in a new file > gpio-davinci.txt located at Documentation/devicetree/bindings/gpio/. > > Signed-off-by: KV Sujith <sujithkv@xxxxxx> > Signed-off-by: Philip Avinash <avinashphilip@xxxxxx> > --- > Changes since v1: > - description for interrupts changed to reflecti > interrupt array usage. > > .../devicetree/bindings/gpio/gpio-davinci.txt | 32 +++++++++++ > drivers/gpio/gpio-davinci.c | 57 ++++++++++++++++++-- > 2 files changed, 86 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/gpio/gpio-davinci.txt b/Documentation/devicetree/bindings/gpio/gpio-davinci.txt > new file mode 100644 > index 0000000..1c31638 > --- /dev/null > +++ b/Documentation/devicetree/bindings/gpio/gpio-davinci.txt > @@ -0,0 +1,32 @@ > +Davinci GPIO controller bindings > + > +Required Properties: > +- compatible:"ti,da830-gpio" "ti,dm6441-gpio" instead since this came first on DM6441? > + > +- reg: Physical base address of the controller and length of memory mapped > + region. > + > +- interrupts: Array of GPIO interrupt number. > + > +- ngpio: The number of GPIO pins supported Is this a generic GPIO property? I could not find it in documentation. Looks like Marvell GPIO uses a similar property too, but even there it is not marked as Marvell specific. Should this be added as a generic GPIO property? > +- intc_irq_num: The number of IRQs supported by the Interrupt Controller You are not actually looking at the number of IRQs interrupt controller on the SoC supports (which is quite unrelated to GPIO module). What you are actually looking for is the base from where GPIO interrupt numbering can start. So "ti,davinci-gpio-irq-base" is more meaningful. That said, this property is probably not required if irqdomains are used (I myself have to read about that). Not sure if it is okay to add this property now waiting for irqdomain conversion. I will let Linus take a call. > + > +- gpio_unbanked: The number of GPIOs that have an individual interrupt > + line to processor. This is also a TI specific property so it should be "ti,davinci-gpio-unbanked". Thanks, Sekhar -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
