All of lore.kernel.org
 help / color / mirror / Atom feed
From: voice.shen@atmel.com (Bo Shen)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/1] ARM: at91: fix missing #interrupt-cells on gpio-controller
Date: Fri, 14 Sep 2012 09:29:10 +0800	[thread overview]
Message-ID: <50528866.6060409@atmel.com> (raw)
In-Reply-To: <1347532826-32733-1-git-send-email-plagnioj@jcrosoft.com>

Hi Jean-Christophe PLAGNIOL-VILLARD,

On 9/13/2012 18:40, Jean-Christophe PLAGNIOL-VILLARD wrote:
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> Cc: Nicolas Ferre <nicolas.ferre@atmel.com>
> ---
>   arch/arm/boot/dts/at91sam9260.dtsi |    3 +++
>   arch/arm/boot/dts/at91sam9263.dtsi |    5 +++++
>   arch/arm/boot/dts/at91sam9g45.dtsi |    5 +++++
>   arch/arm/boot/dts/at91sam9n12.dtsi |    4 ++++
>   arch/arm/boot/dts/at91sam9x5.dtsi  |    4 ++++
>   5 files changed, 21 insertions(+)
>

Test this on sam9x5ek board with qt1070. It works. Thanks.

Tested-by: voice.shen at atmel.com

> diff --git a/arch/arm/boot/dts/at91sam9260.dtsi b/arch/arm/boot/dts/at91sam9260.dtsi
> index 66389c1..7c95f76 100644
> --- a/arch/arm/boot/dts/at91sam9260.dtsi
> +++ b/arch/arm/boot/dts/at91sam9260.dtsi
> @@ -104,6 +104,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioB: gpio at fffff600 {
> @@ -113,6 +114,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioC: gpio at fffff800 {
> @@ -122,6 +124,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			dbgu: serial at fffff200 {
> diff --git a/arch/arm/boot/dts/at91sam9263.dtsi b/arch/arm/boot/dts/at91sam9263.dtsi
> index b460d6c..195019b 100644
> --- a/arch/arm/boot/dts/at91sam9263.dtsi
> +++ b/arch/arm/boot/dts/at91sam9263.dtsi
> @@ -95,6 +95,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioB: gpio at fffff400 {
> @@ -104,6 +105,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioC: gpio at fffff600 {
> @@ -113,6 +115,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioD: gpio at fffff800 {
> @@ -122,6 +125,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioE: gpio at fffffa00 {
> @@ -131,6 +135,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			dbgu: serial at ffffee00 {
> diff --git a/arch/arm/boot/dts/at91sam9g45.dtsi b/arch/arm/boot/dts/at91sam9g45.dtsi
> index bafa880..63751b1 100644
> --- a/arch/arm/boot/dts/at91sam9g45.dtsi
> +++ b/arch/arm/boot/dts/at91sam9g45.dtsi
> @@ -113,6 +113,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioB: gpio at fffff400 {
> @@ -122,6 +123,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioC: gpio at fffff600 {
> @@ -131,6 +133,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioD: gpio at fffff800 {
> @@ -140,6 +143,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioE: gpio at fffffa00 {
> @@ -149,6 +153,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			dbgu: serial at ffffee00 {
> diff --git a/arch/arm/boot/dts/at91sam9n12.dtsi b/arch/arm/boot/dts/at91sam9n12.dtsi
> index bfac0df..ef9336a 100644
> --- a/arch/arm/boot/dts/at91sam9n12.dtsi
> +++ b/arch/arm/boot/dts/at91sam9n12.dtsi
> @@ -107,6 +107,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioB: gpio at fffff600 {
> @@ -116,6 +117,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioC: gpio at fffff800 {
> @@ -125,6 +127,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioD: gpio at fffffa00 {
> @@ -134,6 +137,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			dbgu: serial at fffff200 {
> diff --git a/arch/arm/boot/dts/at91sam9x5.dtsi b/arch/arm/boot/dts/at91sam9x5.dtsi
> index 4a18c39..8a387a8 100644
> --- a/arch/arm/boot/dts/at91sam9x5.dtsi
> +++ b/arch/arm/boot/dts/at91sam9x5.dtsi
> @@ -115,6 +115,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioB: gpio at fffff600 {
> @@ -124,6 +125,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioC: gpio at fffff800 {
> @@ -133,6 +135,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioD: gpio at fffffa00 {
> @@ -142,6 +145,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			dbgu: serial at fffff200 {
>

WARNING: multiple messages have this Message-ID (diff)
From: Bo Shen <voice.shen@atmel.com>
To: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Cc: devicetree-discuss@lists.ozlabs.org,
	Nicolas Ferre <nicolas.ferre@atmel.com>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/1] ARM: at91: fix missing #interrupt-cells on gpio-controller
Date: Fri, 14 Sep 2012 09:29:10 +0800	[thread overview]
Message-ID: <50528866.6060409@atmel.com> (raw)
In-Reply-To: <1347532826-32733-1-git-send-email-plagnioj@jcrosoft.com>

Hi Jean-Christophe PLAGNIOL-VILLARD,

On 9/13/2012 18:40, Jean-Christophe PLAGNIOL-VILLARD wrote:
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> Cc: Nicolas Ferre <nicolas.ferre@atmel.com>
> ---
>   arch/arm/boot/dts/at91sam9260.dtsi |    3 +++
>   arch/arm/boot/dts/at91sam9263.dtsi |    5 +++++
>   arch/arm/boot/dts/at91sam9g45.dtsi |    5 +++++
>   arch/arm/boot/dts/at91sam9n12.dtsi |    4 ++++
>   arch/arm/boot/dts/at91sam9x5.dtsi  |    4 ++++
>   5 files changed, 21 insertions(+)
>

Test this on sam9x5ek board with qt1070. It works. Thanks.

Tested-by: voice.shen@atmel.com

> diff --git a/arch/arm/boot/dts/at91sam9260.dtsi b/arch/arm/boot/dts/at91sam9260.dtsi
> index 66389c1..7c95f76 100644
> --- a/arch/arm/boot/dts/at91sam9260.dtsi
> +++ b/arch/arm/boot/dts/at91sam9260.dtsi
> @@ -104,6 +104,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioB: gpio@fffff600 {
> @@ -113,6 +114,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioC: gpio@fffff800 {
> @@ -122,6 +124,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			dbgu: serial@fffff200 {
> diff --git a/arch/arm/boot/dts/at91sam9263.dtsi b/arch/arm/boot/dts/at91sam9263.dtsi
> index b460d6c..195019b 100644
> --- a/arch/arm/boot/dts/at91sam9263.dtsi
> +++ b/arch/arm/boot/dts/at91sam9263.dtsi
> @@ -95,6 +95,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioB: gpio@fffff400 {
> @@ -104,6 +105,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioC: gpio@fffff600 {
> @@ -113,6 +115,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioD: gpio@fffff800 {
> @@ -122,6 +125,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioE: gpio@fffffa00 {
> @@ -131,6 +135,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			dbgu: serial@ffffee00 {
> diff --git a/arch/arm/boot/dts/at91sam9g45.dtsi b/arch/arm/boot/dts/at91sam9g45.dtsi
> index bafa880..63751b1 100644
> --- a/arch/arm/boot/dts/at91sam9g45.dtsi
> +++ b/arch/arm/boot/dts/at91sam9g45.dtsi
> @@ -113,6 +113,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioB: gpio@fffff400 {
> @@ -122,6 +123,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioC: gpio@fffff600 {
> @@ -131,6 +133,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioD: gpio@fffff800 {
> @@ -140,6 +143,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioE: gpio@fffffa00 {
> @@ -149,6 +153,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			dbgu: serial@ffffee00 {
> diff --git a/arch/arm/boot/dts/at91sam9n12.dtsi b/arch/arm/boot/dts/at91sam9n12.dtsi
> index bfac0df..ef9336a 100644
> --- a/arch/arm/boot/dts/at91sam9n12.dtsi
> +++ b/arch/arm/boot/dts/at91sam9n12.dtsi
> @@ -107,6 +107,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioB: gpio@fffff600 {
> @@ -116,6 +117,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioC: gpio@fffff800 {
> @@ -125,6 +127,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioD: gpio@fffffa00 {
> @@ -134,6 +137,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			dbgu: serial@fffff200 {
> diff --git a/arch/arm/boot/dts/at91sam9x5.dtsi b/arch/arm/boot/dts/at91sam9x5.dtsi
> index 4a18c39..8a387a8 100644
> --- a/arch/arm/boot/dts/at91sam9x5.dtsi
> +++ b/arch/arm/boot/dts/at91sam9x5.dtsi
> @@ -115,6 +115,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioB: gpio@fffff600 {
> @@ -124,6 +125,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioC: gpio@fffff800 {
> @@ -133,6 +135,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			pioD: gpio@fffffa00 {
> @@ -142,6 +145,7 @@
>   				#gpio-cells = <2>;
>   				gpio-controller;
>   				interrupt-controller;
> +				#interrupt-cells = <2>;
>   			};
>
>   			dbgu: serial@fffff200 {
>

  reply	other threads:[~2012-09-14  1:29 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-13 10:40 [PATCH 1/1] ARM: at91: fix missing #interrupt-cells on gpio-controller Jean-Christophe PLAGNIOL-VILLARD
2012-09-13 10:40 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-14  1:29 ` Bo Shen [this message]
2012-09-14  1:29   ` Bo Shen

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=50528866.6060409@atmel.com \
    --to=voice.shen@atmel.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.