From: Thierry Reding <thierry.reding@avionic-design.de>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: Linus Walleij <linus.walleij@stericsson.com>,
Grant Likely <grant.likely@secretlab.ca>,
Rob Herring <rob.herring@calxeda.com>,
devicetree-discuss@lists.ozlabs.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3] gpio: Add Avionic Design N-bit GPIO expander support
Date: Wed, 19 Sep 2012 09:18:35 +0200 [thread overview]
Message-ID: <20120919071835.GA7686@avionic-0098.mockup.avionic-design.de> (raw)
In-Reply-To: <20120919054050.GB611@avionic-0098.mockup.avionic-design.de>
[-- Attachment #1: Type: text/plain, Size: 987 bytes --]
On Wed, Sep 19, 2012 at 07:40:50AM +0200, Thierry Reding wrote:
> On Tue, Sep 18, 2012 at 11:29:46PM +0200, Linus Walleij wrote:
> > On Tue, Sep 18, 2012 at 10:57 AM, 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.
> >
> > And you followed up on absolutely everything I commented so how
> > could I resist this patch ... applied with Rob's ACK!
>
> There's this one issue that was discussed in the GPIO binding update
> that I posted earlier and I was going to send an updated patch and then
> make this patch reference the documentation but I can just go and do
> that in a follow-up patch as well.
On the other hand it has been suggested that this kind of information
doesn't need to be included in every binding because they belong to the
set of standard properties.
Thierry
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
prev parent reply other threads:[~2012-09-19 7:18 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-18 8:57 [PATCH v3] gpio: Add Avionic Design N-bit GPIO expander support Thierry Reding
2012-09-18 13:37 ` Rob Herring
2012-09-18 13:48 ` Thierry Reding
2012-09-18 21:29 ` Linus Walleij
2012-09-19 5:40 ` Thierry Reding
2012-09-19 7:18 ` Thierry Reding [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120919071835.GA7686@avionic-0098.mockup.avionic-design.de \
--to=thierry.reding@avionic-design.de \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=grant.likely@secretlab.ca \
--cc=linus.walleij@linaro.org \
--cc=linus.walleij@stericsson.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rob.herring@calxeda.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox