public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Rob Herring <robherring2@gmail.com>
To: Thierry Reding <thierry.reding@avionic-design.de>
Cc: devicetree-discuss@lists.ozlabs.org,
	Linus Walleij <linus.walleij@stericsson.com>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/2] gpio: adnp: dt: Reference generic interrupt binding
Date: Wed, 19 Sep 2012 10:45:16 -0500	[thread overview]
Message-ID: <5059E88C.7090801@gmail.com> (raw)
In-Reply-To: <1348045056-29769-2-git-send-email-thierry.reding@avionic-design.de>

On 09/19/2012 03:57 AM, Thierry Reding wrote:
> Instead of having to duplicate the description of the properties needed
> for interrupt support, reference the new standard document.
> 
> Cc: Linus Walleij <linus.walleij@stericsson.com>
> Cc: Grant Likely <grant.likely@secretlab.ca>
> Cc: Rob Herring <rob.herring@calxeda.com>
> Cc: devicetree-discuss@lists.ozlabs.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Thierry Reding <thierry.reding@avionic-design.de>
> ---
> This patch should go through LinusW's tree because it already carries
> the patch that created this documentation.
> 
>  Documentation/devicetree/bindings/gpio/gpio-adnp.txt | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/gpio/gpio-adnp.txt b/Documentation/devicetree/bindings/gpio/gpio-adnp.txt
> index 5a09a21..625eee7 100644
> --- a/Documentation/devicetree/bindings/gpio/gpio-adnp.txt
> +++ b/Documentation/devicetree/bindings/gpio/gpio-adnp.txt
> @@ -11,6 +11,10 @@ Required properties:
>  - gpio-controller: Marks the device as a GPIO controller
>  - nr-gpios: The number of pins supported by the controller.
>  
> +The GPIO exander can optionally be used as an interrupt controller, in

s/exander/expander/

Otherwise,

Acked-by: Rob Herring <rob.herring@calxeda.com>

> +which case it uses the default two cell specifier as described in
> +Documentation/devicetree/bindings/interrupt-controller/interrupts.txt.
> +
>  Example:
>  
>  	gpioext: gpio-controller@41 {
> 


  reply	other threads:[~2012-09-19 15:45 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-19  8:57 [PATCH 1/2] dt: Document general interrupt controller bindings Thierry Reding
2012-09-19  8:57 ` [PATCH 2/2] gpio: adnp: dt: Reference generic interrupt binding Thierry Reding
2012-09-19 15:45   ` Rob Herring [this message]
2012-09-20  6:55   ` Linus Walleij
2012-09-19 15:25 ` [PATCH 1/2] dt: Document general interrupt controller bindings Stephen Warren
2012-09-19 15:45 ` Rob Herring

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=5059E88C.7090801@gmail.com \
    --to=robherring2@gmail.com \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=linus.walleij@stericsson.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=thierry.reding@avionic-design.de \
    /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