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.
Cc: Grant Likely
Cc: Rob Herring
Cc: devicetree-discuss@lists.ozlabs.org
Cc: Linus Walleij
Signed-off-by: Thierry Reding
---
.../devicetree/bindin
On Thu, 12 Apr 2012 13:24:18 +0200, Thierry Reding
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.
Hi Thierry, comments below...
> diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kc
* Grant Likely wrote:
> On Thu, 12 Apr 2012 13:24:18 +0200, Thierry Reding
> 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.
>
> Hi Thierry, comments below...
>
> > diff --git
On Mon, 21 May 2012 12:20:32 +0200, Thierry Reding
wrote:
> * Grant Likely wrote:
> > On Thu, 12 Apr 2012 13:24:18 +0200, Thierry Reding
> > wrote:
> > > This commit adds a driver for the Avionic Design N-bit GPIO expander.
> > > The expander provides a variable number of GPIO pins with interru
* Grant Likely wrote:
> On Mon, 21 May 2012 12:20:32 +0200, Thierry Reding
> wrote:
> > * Grant Likely wrote:
> > > On Thu, 12 Apr 2012 13:24:18 +0200, Thierry Reding
> > > wrote:
[...]
> > > > +struct adnp_platform_data {
> > > > + unsigned gpio_base;
> > > > + unsigned nr_gpios;
>
On Thursday 07 June 2012, Thierry Reding wrote:
> > Unless you have an actual user of platform_data, don't borrow
> > trouble by adding it.
>
> If I make the driver DT-only I should probably add a "select OF" to it's
> Kconfig entry, right? Or should it rather be "depends on OF"?
"depends on OF".