All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nicolas Ferre <nicolas.ferre@atmel.com>
To: Wenyou Yang <wenyou.yang@atmel.com>,
	<linux-arm-kernel@lists.infradead.org>
Cc: <davidm@egauge.net>, <romain.izard.pro@gmail.com>,
	<linux-kernel@vger.kernel.org>,
	Nicolas Ferre <nicolas.ferre@atmel.com>
Subject: [PATCH] ARM: dts: at91: sama5d2: use "atmel,sama5d3-nfc" compatible for nfc
Date: Mon, 9 May 2016 18:34:29 +0200	[thread overview]
Message-ID: <1462811669-10107-1-git-send-email-nicolas.ferre@atmel.com> (raw)
In-Reply-To: <1462776680-5284-3-git-send-email-wenyou.yang@atmel.com>

From: Wenyou Yang <wenyou.yang@atmel.com>

An error in documentation of the NAND Flash Controller (NFC) led to choose
another compatibility string for sama5d2 with an impact on the NAND flash
ready/busy information. It was producing the error message:

atmel_nand 80000000.nand: Time out to wait for interrupt: 0x08000000

and had an impact on performance.

So, switch back to the classical "atmel,sama5d3-nfc" compatibility string for
this SoC which gives the proper ready/busy bit information. The NAND flash
driver will be updated to remove the support for this different
implementation.

Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
[nicolas.ferre@atmel.com: change commit message]
Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com>
---
Wenyou, Romain,

I plan to send this patch for 4.6-rc as a fix tomorrow (my time). Can you
please tell me if it's okay on your side.
I revised the commit message just to justify the very late "fix" sending of
this patch...

Thanks for your help, bye.
  Nicolas.


 arch/arm/boot/dts/sama5d2.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi
index 78996bdbd3df..9817090c1b73 100644
--- a/arch/arm/boot/dts/sama5d2.dtsi
+++ b/arch/arm/boot/dts/sama5d2.dtsi
@@ -280,7 +280,7 @@
 			status = "disabled";
 
 			nfc@c0000000 {
-				compatible = "atmel,sama5d4-nfc";
+				compatible = "atmel,sama5d3-nfc";
 				#address-cells = <1>;
 				#size-cells = <1>;
 				reg = < /* NFC Command Registers */
-- 
2.1.3

WARNING: multiple messages have this Message-ID (diff)
From: nicolas.ferre@atmel.com (Nicolas Ferre)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: dts: at91: sama5d2: use "atmel, sama5d3-nfc" compatible for nfc
Date: Mon, 9 May 2016 18:34:29 +0200	[thread overview]
Message-ID: <1462811669-10107-1-git-send-email-nicolas.ferre@atmel.com> (raw)
In-Reply-To: <1462776680-5284-3-git-send-email-wenyou.yang@atmel.com>

From: Wenyou Yang <wenyou.yang@atmel.com>

An error in documentation of the NAND Flash Controller (NFC) led to choose
another compatibility string for sama5d2 with an impact on the NAND flash
ready/busy information. It was producing the error message:

atmel_nand 80000000.nand: Time out to wait for interrupt: 0x08000000

and had an impact on performance.

So, switch back to the classical "atmel,sama5d3-nfc" compatibility string for
this SoC which gives the proper ready/busy bit information. The NAND flash
driver will be updated to remove the support for this different
implementation.

Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
[nicolas.ferre at atmel.com: change commit message]
Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com>
---
Wenyou, Romain,

I plan to send this patch for 4.6-rc as a fix tomorrow (my time). Can you
please tell me if it's okay on your side.
I revised the commit message just to justify the very late "fix" sending of
this patch...

Thanks for your help, bye.
  Nicolas.


 arch/arm/boot/dts/sama5d2.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi
index 78996bdbd3df..9817090c1b73 100644
--- a/arch/arm/boot/dts/sama5d2.dtsi
+++ b/arch/arm/boot/dts/sama5d2.dtsi
@@ -280,7 +280,7 @@
 			status = "disabled";
 
 			nfc at c0000000 {
-				compatible = "atmel,sama5d4-nfc";
+				compatible = "atmel,sama5d3-nfc";
 				#address-cells = <1>;
 				#size-cells = <1>;
 				reg = < /* NFC Command Registers */
-- 
2.1.3

  reply	other threads:[~2016-05-09 16:33 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-09  6:51 [PATCH 0/2] mtd: atmel_nand: remove compatible "atmel,sama5d4-nfc" Wenyou Yang
2016-05-09  6:51 ` Wenyou Yang
2016-05-09  6:51 ` Wenyou Yang
2016-05-09  6:51 ` [PATCH 1/2] Revert "mtd: atmel_nand: Support variable RB_EDGE interrupts" Wenyou Yang
2016-05-09  6:51   ` Wenyou Yang
2016-05-09  6:51   ` Wenyou Yang
2016-05-10  8:55   ` Boris Brezillon
2016-05-10  8:55     ` Boris Brezillon
2016-05-10  8:55     ` Boris Brezillon
2016-05-10  8:59     ` Yang, Wenyou
2016-05-10  8:59       ` Yang, Wenyou
2016-05-10  8:59       ` Yang, Wenyou
2016-05-10  8:59       ` Yang, Wenyou
2016-05-10  9:59     ` Romain Izard
2016-05-10  9:59       ` Romain Izard
2016-05-23  7:55   ` Boris Brezillon
2016-05-23  7:55     ` Boris Brezillon
2016-05-26  3:10     ` Brian Norris
2016-05-26  3:10       ` Brian Norris
2016-05-26  3:10       ` Brian Norris
2016-05-26  3:16   ` Brian Norris
2016-05-26  3:16     ` Brian Norris
2016-05-26  3:16     ` Brian Norris
2016-05-26  6:36     ` Boris Brezillon
2016-05-26  6:36       ` Boris Brezillon
2016-05-09  6:51 ` [PATCH 2/2] ARM: at91/dt: use "atmel,sama5d3-nfc" compatible for nfc Wenyou Yang
2016-05-09  6:51   ` Wenyou Yang
2016-05-09  6:51   ` Wenyou Yang
2016-05-09 16:34   ` Nicolas Ferre [this message]
2016-05-09 16:34     ` [PATCH] ARM: dts: at91: sama5d2: use "atmel, sama5d3-nfc" " Nicolas Ferre
2016-05-09 17:17     ` [PATCH] ARM: dts: at91: sama5d2: use "atmel,sama5d3-nfc" " Romain Izard
2016-05-09 17:17       ` Romain Izard
2016-05-09 20:13 ` [PATCH 0/2] mtd: atmel_nand: remove compatible "atmel,sama5d4-nfc" Rob Herring
2016-05-09 20:13   ` Rob Herring
2016-05-10  4:50   ` Yang, Wenyou
2016-05-10  4:50     ` Yang, Wenyou
2016-05-10  4:50     ` Yang, Wenyou
2016-05-10  4:50     ` Yang, Wenyou
2016-05-10 11:52     ` Alexandre Belloni
2016-05-10 11:52       ` Alexandre Belloni
2016-05-10 11:52       ` Alexandre Belloni
2016-05-10 11:52       ` Alexandre Belloni

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=1462811669-10107-1-git-send-email-nicolas.ferre@atmel.com \
    --to=nicolas.ferre@atmel.com \
    --cc=davidm@egauge.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=romain.izard.pro@gmail.com \
    --cc=wenyou.yang@atmel.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 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.