Messages in this thread | | | Date | Tue, 31 Jul 2012 17:22:40 -0600 | From | Stephen Warren <> | Subject | Re: [PATCH v2] gpio: Add Avionic Design N-bit GPIO expander support |
| |
On 07/31/2012 04:03 PM, Rob Herring wrote: > On 07/30/2012 02:47 AM, Thierry Reding wrote: >> On Sun, Jul 29, 2012 at 07:13:57PM +0200, Linus Walleij wrote: >>> On Mon, Jul 23, 2012 at 1:59 PM, Thierry Reding >>> <thierry.reding@avionic-design.de> wrote: >>> >>>> This commit adds a driver for the Avionic Design N-bit GPIO expander. >>>> The expander provides a variable number of GPIO pins with interrupt >>>> support.
>>>> diff --git a/Documentation/devicetree/bindings/gpio/gpio-adnp.txt b/Documentation/devicetree/bindings/gpio/gpio-adnp.txt
>>>> +Required properties:
>>>> +- nr-gpios: The number of pins supported by the controller.
> For nr-gpios, I think it is typically not needed. Generally, you will > know how many gpio lines the h/w has based on the compatible string. If > this part really is the same part but different packages with different > numbers of gpio, then this property makes sense. Otherwise, I would say > drop it. > > If your goal is how many gpios are you using, you really need a bitmap > instead if you want it to be generic. > > IIRC, Tegra also needed this in that they N instances of some number of > bits and the registers are interleaved so they can't have separate nodes.
In the end, I got away without having a separate property to represent this. Instead, the code keys off the number of interrupts listed in the interrupts property, there being one interrupt per GPIO bank, and hence dynamically instantiates the number of banks based on that.
| |