linux-mtd.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: tegra: add "jedec,spi-nor" flash compatible binding
@ 2015-08-16  6:13 Rafał Miłecki
  2016-02-22  6:20 ` Rafał Miłecki
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Rafał Miłecki @ 2015-08-16  6:13 UTC (permalink / raw)
  To: linux-arm-kernel, Stephen Warren, Thierry Reding,
	Alexandre Courbot, linux-tegra
  Cc: linux-mtd, Rafał Miłecki

Starting with commit 8947e396a829 ("Documentation: dt: mtd: replace
"nor-jedec" binding with "jedec, spi-nor"") we have "jedec,spi-nor"
binding indicating support for JEDEC identification.

Use it for all flashes that are supposed to support READ ID op according
to the datasheets.

Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
---
 arch/arm/boot/dts/tegra114-dalmore.dts    | 2 +-
 arch/arm/boot/dts/tegra124-jetson-tk1.dts | 2 +-
 arch/arm/boot/dts/tegra124-nyan.dtsi      | 2 +-
 arch/arm/boot/dts/tegra124-venice2.dts    | 2 +-
 arch/arm/boot/dts/tegra20-trimslice.dts   | 2 +-
 arch/arm/boot/dts/tegra30-beaver.dts      | 2 +-
 arch/arm/boot/dts/tegra30-cardhu.dtsi     | 2 +-
 7 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/arch/arm/boot/dts/tegra114-dalmore.dts b/arch/arm/boot/dts/tegra114-dalmore.dts
index 8b7aa0d..5de23de 100644
--- a/arch/arm/boot/dts/tegra114-dalmore.dts
+++ b/arch/arm/boot/dts/tegra114-dalmore.dts
@@ -1082,7 +1082,7 @@
 		status = "okay";
 		spi-max-frequency = <25000000>;
 		spi-flash@0 {
-			compatible = "winbond,w25q32dw";
+			compatible = "winbond,w25q32dw", "jedec,spi-nor";
 			reg = <0>;
 			spi-max-frequency = <20000000>;
 		};
diff --git a/arch/arm/boot/dts/tegra124-jetson-tk1.dts b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
index bd43ed6..44097c6 100644
--- a/arch/arm/boot/dts/tegra124-jetson-tk1.dts
+++ b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
@@ -1611,7 +1611,7 @@
 		status = "okay";
 		spi-max-frequency = <25000000>;
 		spi-flash@0 {
-			compatible = "winbond,w25q32dw";
+			compatible = "winbond,w25q32dw", "jedec,spi-nor";
 			reg = <0>;
 			spi-max-frequency = <20000000>;
 		};
diff --git a/arch/arm/boot/dts/tegra124-nyan.dtsi b/arch/arm/boot/dts/tegra124-nyan.dtsi
index a9aec23..7d26f49 100644
--- a/arch/arm/boot/dts/tegra124-nyan.dtsi
+++ b/arch/arm/boot/dts/tegra124-nyan.dtsi
@@ -343,7 +343,7 @@
 		spi-max-frequency = <25000000>;
 
 		flash@0 {
-			compatible = "winbond,w25q32dw";
+			compatible = "winbond,w25q32dw", "jedec,spi-nor";
 			spi-max-frequency = <25000000>;
 			reg = <0>;
 		};
diff --git a/arch/arm/boot/dts/tegra124-venice2.dts b/arch/arm/boot/dts/tegra124-venice2.dts
index 79e724b..4decb3c 100644
--- a/arch/arm/boot/dts/tegra124-venice2.dts
+++ b/arch/arm/boot/dts/tegra124-venice2.dts
@@ -866,7 +866,7 @@
 		status = "okay";
 		spi-max-frequency = <25000000>;
 		spi-flash@0 {
-			compatible = "winbond,w25q32dw";
+			compatible = "winbond,w25q32dw", "jedec,spi-nor";
 			reg = <0>;
 			spi-max-frequency = <20000000>;
 		};
diff --git a/arch/arm/boot/dts/tegra20-trimslice.dts b/arch/arm/boot/dts/tegra20-trimslice.dts
index d99af4e..c4cff8b 100644
--- a/arch/arm/boot/dts/tegra20-trimslice.dts
+++ b/arch/arm/boot/dts/tegra20-trimslice.dts
@@ -282,7 +282,7 @@
 		status = "okay";
 		spi-max-frequency = <48000000>;
 		spi-flash@0 {
-			compatible = "winbond,w25q80bl";
+			compatible = "winbond,w25q80bl", "jedec,spi-nor";
 			reg = <0>;
 			spi-max-frequency = <48000000>;
 		};
diff --git a/arch/arm/boot/dts/tegra30-beaver.dts b/arch/arm/boot/dts/tegra30-beaver.dts
index 3dede39..c9d230c 100644
--- a/arch/arm/boot/dts/tegra30-beaver.dts
+++ b/arch/arm/boot/dts/tegra30-beaver.dts
@@ -1884,7 +1884,7 @@
 		status = "okay";
 		spi-max-frequency = <25000000>;
 		spi-flash@1 {
-			compatible = "winbond,w25q32";
+			compatible = "winbond,w25q32", "jedec,spi-nor";
 			reg = <1>;
 			spi-max-frequency = <20000000>;
 		};
diff --git a/arch/arm/boot/dts/tegra30-cardhu.dtsi b/arch/arm/boot/dts/tegra30-cardhu.dtsi
index bb1ca15..df7f845 100644
--- a/arch/arm/boot/dts/tegra30-cardhu.dtsi
+++ b/arch/arm/boot/dts/tegra30-cardhu.dtsi
@@ -354,7 +354,7 @@
 		status = "okay";
 		spi-max-frequency = <25000000>;
 		spi-flash@1 {
-			compatible = "winbond,w25q32";
+			compatible = "winbond,w25q32", "jedec,spi-nor";
 			reg = <1>;
 			spi-max-frequency = <20000000>;
 		};
-- 
1.8.4.5

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] ARM: tegra: add "jedec,spi-nor" flash compatible binding
  2015-08-16  6:13 [PATCH] ARM: tegra: add "jedec,spi-nor" flash compatible binding Rafał Miłecki
@ 2016-02-22  6:20 ` Rafał Miłecki
  2016-02-22 18:04 ` Brian Norris
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Rafał Miłecki @ 2016-02-22  6:20 UTC (permalink / raw)
  To: linux-arm-kernel, Stephen Warren, Thierry Reding,
	Alexandre Courbot, linux-tegra
  Cc: linux-mtd, Rafał Miłecki

On 16 August 2015 at 08:13, Rafał Miłecki <zajec5@gmail.com> wrote:
> Starting with commit 8947e396a829 ("Documentation: dt: mtd: replace
> "nor-jedec" binding with "jedec, spi-nor"") we have "jedec,spi-nor"
> binding indicating support for JEDEC identification.
>
> Use it for all flashes that are supposed to support READ ID op according
> to the datasheets.

Hi, for some reason it seems this patch wasn't applied. I didn't get
any comment and it was sent to all e-mails listed in MAINTAINERS in
TEGRA ARCHITECTURE SUPPORT
which I hope it enough.

Could someone pick it, please? FWIW you can find it in the patchwork:
http://patchwork.ozlabs.org/patch/507676/

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] ARM: tegra: add "jedec,spi-nor" flash compatible binding
  2015-08-16  6:13 [PATCH] ARM: tegra: add "jedec,spi-nor" flash compatible binding Rafał Miłecki
  2016-02-22  6:20 ` Rafał Miłecki
@ 2016-02-22 18:04 ` Brian Norris
  2019-01-15 15:09 ` Jon Hunter
  2019-02-06 13:16 ` Thierry Reding
  3 siblings, 0 replies; 5+ messages in thread
From: Brian Norris @ 2016-02-22 18:04 UTC (permalink / raw)
  To: Rafał Miłecki
  Cc: linux-arm-kernel, Stephen Warren, Thierry Reding,
	Alexandre Courbot, linux-tegra, linux-mtd

On Sun, Aug 16, 2015 at 08:13:25AM +0200, Rafał Miłecki wrote:
> Starting with commit 8947e396a829 ("Documentation: dt: mtd: replace
> "nor-jedec" binding with "jedec, spi-nor"") we have "jedec,spi-nor"
> binding indicating support for JEDEC identification.
> 
> Use it for all flashes that are supposed to support READ ID op according
> to the datasheets.
> 
> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
> ---
>  arch/arm/boot/dts/tegra114-dalmore.dts    | 2 +-
>  arch/arm/boot/dts/tegra124-jetson-tk1.dts | 2 +-
>  arch/arm/boot/dts/tegra124-nyan.dtsi      | 2 +-
>  arch/arm/boot/dts/tegra124-venice2.dts    | 2 +-
>  arch/arm/boot/dts/tegra20-trimslice.dts   | 2 +-
>  arch/arm/boot/dts/tegra30-beaver.dts      | 2 +-
>  arch/arm/boot/dts/tegra30-cardhu.dtsi     | 2 +-
>  7 files changed, 7 insertions(+), 7 deletions(-)

FWIW:

Acked-by: Brian Norris <computersforpeace@gmail.com>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] ARM: tegra: add "jedec,spi-nor" flash compatible binding
  2015-08-16  6:13 [PATCH] ARM: tegra: add "jedec,spi-nor" flash compatible binding Rafał Miłecki
  2016-02-22  6:20 ` Rafał Miłecki
  2016-02-22 18:04 ` Brian Norris
@ 2019-01-15 15:09 ` Jon Hunter
  2019-02-06 13:16 ` Thierry Reding
  3 siblings, 0 replies; 5+ messages in thread
From: Jon Hunter @ 2019-01-15 15:09 UTC (permalink / raw)
  To: Rafał Miłecki, linux-arm-kernel, Stephen Warren,
	Thierry Reding, linux-tegra
  Cc: linux-mtd


On 16/08/2015 07:13, Rafał Miłecki wrote:
> Starting with commit 8947e396a829 ("Documentation: dt: mtd: replace
> "nor-jedec" binding with "jedec, spi-nor"") we have "jedec,spi-nor"
> binding indicating support for JEDEC identification.
> 
> Use it for all flashes that are supposed to support READ ID op according
> to the datasheets.
> 
> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
> ---
>  arch/arm/boot/dts/tegra114-dalmore.dts    | 2 +-
>  arch/arm/boot/dts/tegra124-jetson-tk1.dts | 2 +-
>  arch/arm/boot/dts/tegra124-nyan.dtsi      | 2 +-
>  arch/arm/boot/dts/tegra124-venice2.dts    | 2 +-
>  arch/arm/boot/dts/tegra20-trimslice.dts   | 2 +-
>  arch/arm/boot/dts/tegra30-beaver.dts      | 2 +-
>  arch/arm/boot/dts/tegra30-cardhu.dtsi     | 2 +-
>  7 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/tegra114-dalmore.dts b/arch/arm/boot/dts/tegra114-dalmore.dts
> index 8b7aa0d..5de23de 100644
> --- a/arch/arm/boot/dts/tegra114-dalmore.dts
> +++ b/arch/arm/boot/dts/tegra114-dalmore.dts
> @@ -1082,7 +1082,7 @@
>  		status = "okay";
>  		spi-max-frequency = <25000000>;
>  		spi-flash@0 {
> -			compatible = "winbond,w25q32dw";
> +			compatible = "winbond,w25q32dw", "jedec,spi-nor";
>  			reg = <0>;
>  			spi-max-frequency = <20000000>;
>  		};
> diff --git a/arch/arm/boot/dts/tegra124-jetson-tk1.dts b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
> index bd43ed6..44097c6 100644
> --- a/arch/arm/boot/dts/tegra124-jetson-tk1.dts
> +++ b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
> @@ -1611,7 +1611,7 @@
>  		status = "okay";
>  		spi-max-frequency = <25000000>;
>  		spi-flash@0 {
> -			compatible = "winbond,w25q32dw";
> +			compatible = "winbond,w25q32dw", "jedec,spi-nor";
>  			reg = <0>;
>  			spi-max-frequency = <20000000>;
>  		};
> diff --git a/arch/arm/boot/dts/tegra124-nyan.dtsi b/arch/arm/boot/dts/tegra124-nyan.dtsi
> index a9aec23..7d26f49 100644
> --- a/arch/arm/boot/dts/tegra124-nyan.dtsi
> +++ b/arch/arm/boot/dts/tegra124-nyan.dtsi
> @@ -343,7 +343,7 @@
>  		spi-max-frequency = <25000000>;
>  
>  		flash@0 {
> -			compatible = "winbond,w25q32dw";
> +			compatible = "winbond,w25q32dw", "jedec,spi-nor";
>  			spi-max-frequency = <25000000>;
>  			reg = <0>;
>  		};
> diff --git a/arch/arm/boot/dts/tegra124-venice2.dts b/arch/arm/boot/dts/tegra124-venice2.dts
> index 79e724b..4decb3c 100644
> --- a/arch/arm/boot/dts/tegra124-venice2.dts
> +++ b/arch/arm/boot/dts/tegra124-venice2.dts
> @@ -866,7 +866,7 @@
>  		status = "okay";
>  		spi-max-frequency = <25000000>;
>  		spi-flash@0 {
> -			compatible = "winbond,w25q32dw";
> +			compatible = "winbond,w25q32dw", "jedec,spi-nor";
>  			reg = <0>;
>  			spi-max-frequency = <20000000>;
>  		};
> diff --git a/arch/arm/boot/dts/tegra20-trimslice.dts b/arch/arm/boot/dts/tegra20-trimslice.dts
> index d99af4e..c4cff8b 100644
> --- a/arch/arm/boot/dts/tegra20-trimslice.dts
> +++ b/arch/arm/boot/dts/tegra20-trimslice.dts
> @@ -282,7 +282,7 @@
>  		status = "okay";
>  		spi-max-frequency = <48000000>;
>  		spi-flash@0 {
> -			compatible = "winbond,w25q80bl";
> +			compatible = "winbond,w25q80bl", "jedec,spi-nor";
>  			reg = <0>;
>  			spi-max-frequency = <48000000>;
>  		};
> diff --git a/arch/arm/boot/dts/tegra30-beaver.dts b/arch/arm/boot/dts/tegra30-beaver.dts
> index 3dede39..c9d230c 100644
> --- a/arch/arm/boot/dts/tegra30-beaver.dts
> +++ b/arch/arm/boot/dts/tegra30-beaver.dts
> @@ -1884,7 +1884,7 @@
>  		status = "okay";
>  		spi-max-frequency = <25000000>;
>  		spi-flash@1 {
> -			compatible = "winbond,w25q32";
> +			compatible = "winbond,w25q32", "jedec,spi-nor";
>  			reg = <1>;
>  			spi-max-frequency = <20000000>;
>  		};
> diff --git a/arch/arm/boot/dts/tegra30-cardhu.dtsi b/arch/arm/boot/dts/tegra30-cardhu.dtsi
> index bb1ca15..df7f845 100644
> --- a/arch/arm/boot/dts/tegra30-cardhu.dtsi
> +++ b/arch/arm/boot/dts/tegra30-cardhu.dtsi
> @@ -354,7 +354,7 @@
>  		status = "okay";
>  		spi-max-frequency = <25000000>;
>  		spi-flash@1 {
> -			compatible = "winbond,w25q32";
> +			compatible = "winbond,w25q32", "jedec,spi-nor";
>  			reg = <1>;
>  			spi-max-frequency = <20000000>;
>  		};
> 

Looks like this one got lost, but still applies cleanly so ...

Acked-by: Jon Hunter <jonathanh@nvidia.com>

Cheers
Jon

-- 
nvpublic

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] ARM: tegra: add "jedec,spi-nor" flash compatible binding
  2015-08-16  6:13 [PATCH] ARM: tegra: add "jedec,spi-nor" flash compatible binding Rafał Miłecki
                   ` (2 preceding siblings ...)
  2019-01-15 15:09 ` Jon Hunter
@ 2019-02-06 13:16 ` Thierry Reding
  3 siblings, 0 replies; 5+ messages in thread
From: Thierry Reding @ 2019-02-06 13:16 UTC (permalink / raw)
  To: Rafał Miłecki
  Cc: linux-tegra, Alexandre Courbot, linux-mtd, linux-arm-kernel,
	Stephen Warren


[-- Attachment #1.1: Type: text/plain, Size: 885 bytes --]

On Sun, Aug 16, 2015 at 08:13:25AM +0200, Rafał Miłecki wrote:
> Starting with commit 8947e396a829 ("Documentation: dt: mtd: replace
> "nor-jedec" binding with "jedec, spi-nor"") we have "jedec,spi-nor"
> binding indicating support for JEDEC identification.
> 
> Use it for all flashes that are supposed to support READ ID op according
> to the datasheets.
> 
> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
> ---
>  arch/arm/boot/dts/tegra114-dalmore.dts    | 2 +-
>  arch/arm/boot/dts/tegra124-jetson-tk1.dts | 2 +-
>  arch/arm/boot/dts/tegra124-nyan.dtsi      | 2 +-
>  arch/arm/boot/dts/tegra124-venice2.dts    | 2 +-
>  arch/arm/boot/dts/tegra20-trimslice.dts   | 2 +-
>  arch/arm/boot/dts/tegra30-beaver.dts      | 2 +-
>  arch/arm/boot/dts/tegra30-cardhu.dtsi     | 2 +-
>  7 files changed, 7 insertions(+), 7 deletions(-)

Applied, thanks.

Thierry

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 144 bytes --]

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2019-02-06 13:16 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-08-16  6:13 [PATCH] ARM: tegra: add "jedec,spi-nor" flash compatible binding Rafał Miłecki
2016-02-22  6:20 ` Rafał Miłecki
2016-02-22 18:04 ` Brian Norris
2019-01-15 15:09 ` Jon Hunter
2019-02-06 13:16 ` Thierry Reding

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).