[PATCH 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend

[PATCH 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend

[Roger Quadros]

This series fixes dtbs_check warnings on OMAP platforms
for ti,gpmc-nand and ti,gpmc-onenand.

The following warnings are fixed
- "nand@0,0: Unevaluated properties are not allowed ('linux,mtd-name' was unexpected)"
- "nand@0,0: Unevaluated properties are not allowed ('gpmc,device-nand' was unexpected)"
- "omap3430-sdp.dtb: onenand@2,0: Unevaluated properties are not allowed ('linux,mtd-name' was unexpected)"

---
Roger Quadros (3):
      ARM: dts: ti: drop linux,mtd-name from NAND nodes
      ARM: dts: ti: omap: am335x-baltos: drop "gpmc,device-nand" from NAND node
      ARM: dts: ti: omap3434-sdp: drop linux,mtd-name from onenand node

 arch/arm/boot/dts/ti/omap/am335x-baltos.dtsi       | 1 -
 arch/arm/boot/dts/ti/omap/am3517-som.dtsi          | 1 -
 arch/arm/boot/dts/ti/omap/dm8148-evm.dts           | 1 -
 arch/arm/boot/dts/ti/omap/dm8168-evm.dts           | 1 -
 arch/arm/boot/dts/ti/omap/dra62x-j5eco-evm.dts     | 1 -
 arch/arm/boot/dts/ti/omap/logicpd-som-lv.dtsi      | 1 -
 arch/arm/boot/dts/ti/omap/logicpd-torpedo-som.dtsi | 1 -
 arch/arm/boot/dts/ti/omap/omap3-evm-37xx.dts       | 1 -
 arch/arm/boot/dts/ti/omap/omap3-evm.dts            | 1 -
 arch/arm/boot/dts/ti/omap/omap3-igep.dtsi          | 1 -
 arch/arm/boot/dts/ti/omap/omap3-ldp.dts            | 1 -
 arch/arm/boot/dts/ti/omap/omap3-overo-base.dtsi    | 1 -
 arch/arm/boot/dts/ti/omap/omap3430-sdp.dts         | 2 --
 13 files changed, 14 deletions(-)
---
base-commit: 5be63fc19fcaa4c236b307420483578a56986a37
change-id: 20240830-gpmc-dtb-de2ce28acfb6

Best regards,
-- 
Roger Quadros <rogerq@kernel.org>

[PATCH 1/3] ARM: dts: ti: drop linux,mtd-name from NAND nodes

[Roger Quadros]

linux,mtd-name is only supported by PHYSMAP driver and not applicable
to NAND controller/flash nodes.

Fixes the below dtbs_check warning on many OMAP boards.

"nand@0,0: Unevaluated properties are not allowed ('linux,mtd-name' was unexpected)"

Signed-off-by: Roger Quadros <rogerq@kernel.org>
---
 arch/arm/boot/dts/ti/omap/am3517-som.dtsi          | 1 -
 arch/arm/boot/dts/ti/omap/dm8148-evm.dts           | 1 -
 arch/arm/boot/dts/ti/omap/dm8168-evm.dts           | 1 -
 arch/arm/boot/dts/ti/omap/dra62x-j5eco-evm.dts     | 1 -
 arch/arm/boot/dts/ti/omap/logicpd-som-lv.dtsi      | 1 -
 arch/arm/boot/dts/ti/omap/logicpd-torpedo-som.dtsi | 1 -
 arch/arm/boot/dts/ti/omap/omap3-evm-37xx.dts       | 1 -
 arch/arm/boot/dts/ti/omap/omap3-evm.dts            | 1 -
 arch/arm/boot/dts/ti/omap/omap3-igep.dtsi          | 1 -
 arch/arm/boot/dts/ti/omap/omap3-ldp.dts            | 1 -
 arch/arm/boot/dts/ti/omap/omap3-overo-base.dtsi    | 1 -
 arch/arm/boot/dts/ti/omap/omap3430-sdp.dts         | 1 -
 12 files changed, 12 deletions(-)

diff --git a/arch/arm/boot/dts/ti/omap/am3517-som.dtsi b/arch/arm/boot/dts/ti/omap/am3517-som.dtsi
index bd0a6c95afa1..e36cd98f57fa 100644
--- a/arch/arm/boot/dts/ti/omap/am3517-som.dtsi
+++ b/arch/arm/boot/dts/ti/omap/am3517-som.dtsi
@@ -44,7 +44,6 @@ &gpmc {
 
 	nand@0,0 {
 		compatible = "ti,omap2-nand";
-		linux,mtd-name = "micron,mt29f4g16abchch";
 		reg = <0 0 4>; /* CS0, offset 0, IO size 4 */
 		nand-bus-width = <16>;
 		ti,nand-ecc-opt = "bch8";
diff --git a/arch/arm/boot/dts/ti/omap/dm8148-evm.dts b/arch/arm/boot/dts/ti/omap/dm8148-evm.dts
index ae8d9fa09d16..57a9eef09f6f 100644
--- a/arch/arm/boot/dts/ti/omap/dm8148-evm.dts
+++ b/arch/arm/boot/dts/ti/omap/dm8148-evm.dts
@@ -51,7 +51,6 @@ nand@0,0 {
 		interrupt-parent = <&gpmc>;
 		interrupts = <0 IRQ_TYPE_NONE>, /* fifoevent */
 			     <1 IRQ_TYPE_NONE>; /* termcount */
-		linux,mtd-name = "micron,mt29f2g16aadwp";
 		#address-cells = <1>;
 		#size-cells = <1>;
 		ti,nand-ecc-opt = "bch8";
diff --git a/arch/arm/boot/dts/ti/omap/dm8168-evm.dts b/arch/arm/boot/dts/ti/omap/dm8168-evm.dts
index 1d80288f6ba5..6130b9a5f660 100644
--- a/arch/arm/boot/dts/ti/omap/dm8168-evm.dts
+++ b/arch/arm/boot/dts/ti/omap/dm8168-evm.dts
@@ -119,7 +119,6 @@ &gpmc {
 
 	nand@0,0 {
 		compatible = "ti,omap2-nand";
-		linux,mtd-name = "micron,mt29f2g16aadwp";
 		reg = <0 0 4>; /* CS0, offset 0, IO size 4 */
 		interrupt-parent = <&gpmc>;
 		interrupts = <0 IRQ_TYPE_NONE>, /* fifoevent */
diff --git a/arch/arm/boot/dts/ti/omap/dra62x-j5eco-evm.dts b/arch/arm/boot/dts/ti/omap/dra62x-j5eco-evm.dts
index 2f6ac267fc15..df05a0682322 100644
--- a/arch/arm/boot/dts/ti/omap/dra62x-j5eco-evm.dts
+++ b/arch/arm/boot/dts/ti/omap/dra62x-j5eco-evm.dts
@@ -51,7 +51,6 @@ nand@0,0 {
 		interrupt-parent = <&gpmc>;
 		interrupts = <0 IRQ_TYPE_NONE>, /* fifoevent */
 			     <1 IRQ_TYPE_NONE>; /* termcount */
-		linux,mtd-name = "micron,mt29f2g16aadwp";
 		#address-cells = <1>;
 		#size-cells = <1>;
 		ti,nand-ecc-opt = "bch8";
diff --git a/arch/arm/boot/dts/ti/omap/logicpd-som-lv.dtsi b/arch/arm/boot/dts/ti/omap/logicpd-som-lv.dtsi
index c0e6b73fa472..d51a436d9774 100644
--- a/arch/arm/boot/dts/ti/omap/logicpd-som-lv.dtsi
+++ b/arch/arm/boot/dts/ti/omap/logicpd-som-lv.dtsi
@@ -51,7 +51,6 @@ nand@0,0 {
 		interrupt-parent = <&gpmc>;
 		interrupts = <0 IRQ_TYPE_NONE>, /* fifoevent */
 			     <1 IRQ_TYPE_NONE>;	/* termcount */
-		linux,mtd-name = "micron,mt29f4g16abbda3w";
 		nand-bus-width = <16>;
 		ti,nand-ecc-opt = "bch8";
 		rb-gpios = <&gpmc 0 GPIO_ACTIVE_HIGH>; /* gpmc_wait0 */
diff --git a/arch/arm/boot/dts/ti/omap/logicpd-torpedo-som.dtsi b/arch/arm/boot/dts/ti/omap/logicpd-torpedo-som.dtsi
index 227699890890..28feb235482d 100644
--- a/arch/arm/boot/dts/ti/omap/logicpd-torpedo-som.dtsi
+++ b/arch/arm/boot/dts/ti/omap/logicpd-torpedo-som.dtsi
@@ -49,7 +49,6 @@ nand@0,0 {
 		interrupt-parent = <&gpmc>;
 		interrupts = <0 IRQ_TYPE_NONE>, /* fifoevent */
 			     <1 IRQ_TYPE_NONE>;	/* termcount */
-		linux,mtd-name = "micron,mt29f4g16abbda3w";
 		nand-bus-width = <16>;
 		ti,nand-ecc-opt = "bch8";
 		rb-gpios = <&gpmc 0 GPIO_ACTIVE_HIGH>; /* gpmc_wait0 */
diff --git a/arch/arm/boot/dts/ti/omap/omap3-evm-37xx.dts b/arch/arm/boot/dts/ti/omap/omap3-evm-37xx.dts
index e0346bf842fc..9c60ac853a40 100644
--- a/arch/arm/boot/dts/ti/omap/omap3-evm-37xx.dts
+++ b/arch/arm/boot/dts/ti/omap/omap3-evm-37xx.dts
@@ -60,7 +60,6 @@ nand@0,0 {
 		interrupt-parent = <&gpmc>;
 		interrupts = <0 IRQ_TYPE_NONE>, /* fifoevent */
 			     <1 IRQ_TYPE_NONE>;	/* termcount */
-		linux,mtd-name = "hynix,h8kds0un0mer-4em";
 		nand-bus-width = <16>;
 		gpmc,device-width = <2>;
 		ti,nand-ecc-opt = "bch8";
diff --git a/arch/arm/boot/dts/ti/omap/omap3-evm.dts b/arch/arm/boot/dts/ti/omap/omap3-evm.dts
index a2a1613c45c3..28caa5d93b87 100644
--- a/arch/arm/boot/dts/ti/omap/omap3-evm.dts
+++ b/arch/arm/boot/dts/ti/omap/omap3-evm.dts
@@ -60,7 +60,6 @@ nand@0,0 {
 		interrupt-parent = <&gpmc>;
 		interrupts = <0 IRQ_TYPE_NONE>, /* fifoevent */
 			     <1 IRQ_TYPE_NONE>;	/* termcount */
-		linux,mtd-name = "micron,mt29f2g16abdhc";
 		nand-bus-width = <16>;
 		gpmc,device-width = <2>;
 		ti,nand-ecc-opt = "bch8";
diff --git a/arch/arm/boot/dts/ti/omap/omap3-igep.dtsi b/arch/arm/boot/dts/ti/omap/omap3-igep.dtsi
index e068ecf86b8f..7346cad84eda 100644
--- a/arch/arm/boot/dts/ti/omap/omap3-igep.dtsi
+++ b/arch/arm/boot/dts/ti/omap/omap3-igep.dtsi
@@ -111,7 +111,6 @@ nand@0,0 {
 		interrupt-parent = <&gpmc>;
 		interrupts = <0 IRQ_TYPE_NONE>, /* fifoevent */
 			     <1 IRQ_TYPE_NONE>;	/* termcount */
-		linux,mtd-name = "micron,mt29c4g96maz";
 		nand-bus-width = <16>;
 		gpmc,device-width = <2>;
 		ti,nand-ecc-opt = "bch8";
diff --git a/arch/arm/boot/dts/ti/omap/omap3-ldp.dts b/arch/arm/boot/dts/ti/omap/omap3-ldp.dts
index bb6fab9fa47d..cf325f56b464 100644
--- a/arch/arm/boot/dts/ti/omap/omap3-ldp.dts
+++ b/arch/arm/boot/dts/ti/omap/omap3-ldp.dts
@@ -103,7 +103,6 @@ nand@0,0 {
 		interrupt-parent = <&gpmc>;
 		interrupts = <0 IRQ_TYPE_NONE>, /* fifoevent */
 			     <1 IRQ_TYPE_NONE>;	/* termcount */
-		linux,mtd-name = "micron,nand";
 		nand-bus-width = <16>;
 		gpmc,device-width = <2>;
 		ti,nand-ecc-opt = "bch8";
diff --git a/arch/arm/boot/dts/ti/omap/omap3-overo-base.dtsi b/arch/arm/boot/dts/ti/omap/omap3-overo-base.dtsi
index cc57626ea607..2793821b2c33 100644
--- a/arch/arm/boot/dts/ti/omap/omap3-overo-base.dtsi
+++ b/arch/arm/boot/dts/ti/omap/omap3-overo-base.dtsi
@@ -222,7 +222,6 @@ &gpmc {
 
 	nand@0,0 {
 		compatible = "ti,omap2-nand";
-		linux,mtd-name = "micron,mt29c4g96maz";
 		reg = <0 0 4>;	/* CS0, offset 0, IO size 4 */
 		interrupt-parent = <&gpmc>;
 		interrupts = <0 IRQ_TYPE_NONE>, /* fifoevent */
diff --git a/arch/arm/boot/dts/ti/omap/omap3430-sdp.dts b/arch/arm/boot/dts/ti/omap/omap3430-sdp.dts
index 258ecd9e4519..1454a89aa980 100644
--- a/arch/arm/boot/dts/ti/omap/omap3430-sdp.dts
+++ b/arch/arm/boot/dts/ti/omap/omap3430-sdp.dts
@@ -105,7 +105,6 @@ nand@1,0 {
 		interrupt-parent = <&gpmc>;
 		interrupts = <0 IRQ_TYPE_NONE>, /* fifoevent */
 			     <1 IRQ_TYPE_NONE>;	/* termcount */
-		linux,mtd-name = "micron,mt29f1g08abb";
 		#address-cells = <1>;
 		#size-cells = <1>;
 		ti,nand-ecc-opt = "sw";

-- 
2.34.1

[PATCH 2/3] ARM: dts: ti: omap: am335x-baltos: drop "gpmc,device-nand" from NAND node

[Roger Quadros]

"gpmc,device-nand" is not used any more and leads to below dtbs_check
warning so drop it.

"nand@0,0: Unevaluated properties are not allowed ('gpmc,device-nand' was unexpected)"

Signed-off-by: Roger Quadros <rogerq@kernel.org>
---
 arch/arm/boot/dts/ti/omap/am335x-baltos.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/ti/omap/am335x-baltos.dtsi b/arch/arm/boot/dts/ti/omap/am335x-baltos.dtsi
index a4beb718559c..d0efbeadd7d2 100644
--- a/arch/arm/boot/dts/ti/omap/am335x-baltos.dtsi
+++ b/arch/arm/boot/dts/ti/omap/am335x-baltos.dtsi
@@ -199,7 +199,6 @@ nand@0,0 {
 		ti,nand-ecc-opt = "bch8";
 		ti,nand-xfer-type = "prefetch-dma";
 
-		gpmc,device-nand = "true";
 		gpmc,device-width = <1>;
 		gpmc,sync-clk-ps = <0>;
 		gpmc,cs-on-ns = <0>;

-- 
2.34.1

[PATCH 3/3] ARM: dts: ti: omap3434-sdp: drop linux,mtd-name from onenand node

[Roger Quadros]

linux,mtd-name is only applicable for PHYSMAP driver and not for
OneNAND.

Fixes the below dtbs_check warning

"omap3430-sdp.dtb: onenand@2,0: Unevaluated properties are not allowed ('linux,mtd-name' was unexpected)"

Signed-off-by: Roger Quadros <rogerq@kernel.org>
---
 arch/arm/boot/dts/ti/omap/omap3430-sdp.dts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/ti/omap/omap3430-sdp.dts b/arch/arm/boot/dts/ti/omap/omap3430-sdp.dts
index 1454a89aa980..cc5e9035ef73 100644
--- a/arch/arm/boot/dts/ti/omap/omap3430-sdp.dts
+++ b/arch/arm/boot/dts/ti/omap/omap3430-sdp.dts
@@ -147,7 +147,6 @@ partition@780000 {
 	};
 
 	onenand@2,0 {
-		linux,mtd-name = "samsung,kfm2g16q2m-deb8";
 		#address-cells = <1>;
 		#size-cells = <1>;
 		compatible = "ti,omap2-onenand";

-- 
2.34.1

Re: [PATCH 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend

[Kevin Hilman]

Hi Roger,

Roger Quadros <rogerq@kernel.org> writes:

> This series fixes dtbs_check warnings on OMAP platforms
> for ti,gpmc-nand and ti,gpmc-onenand.
>
> The following warnings are fixed
> - "nand@0,0: Unevaluated properties are not allowed ('linux,mtd-name' was unexpected)"
> - "nand@0,0: Unevaluated properties are not allowed ('gpmc,device-nand' was unexpected)"
> - "omap3430-sdp.dtb: onenand@2,0: Unevaluated properties are not allowed ('linux,mtd-name' was unexpected)"
>
> ---
> Roger Quadros (3):
>       ARM: dts: ti: drop linux,mtd-name from NAND nodes
>       ARM: dts: ti: omap: am335x-baltos: drop "gpmc,device-nand" from NAND node
>       ARM: dts: ti: omap3434-sdp: drop linux,mtd-name from onenand node
>
>  arch/arm/boot/dts/ti/omap/am335x-baltos.dtsi       | 1 -
>  arch/arm/boot/dts/ti/omap/am3517-som.dtsi          | 1 -
>  arch/arm/boot/dts/ti/omap/dm8148-evm.dts           | 1 -
>  arch/arm/boot/dts/ti/omap/dm8168-evm.dts           | 1 -
>  arch/arm/boot/dts/ti/omap/dra62x-j5eco-evm.dts     | 1 -
>  arch/arm/boot/dts/ti/omap/logicpd-som-lv.dtsi      | 1 -
>  arch/arm/boot/dts/ti/omap/logicpd-torpedo-som.dtsi | 1 -
>  arch/arm/boot/dts/ti/omap/omap3-evm-37xx.dts       | 1 -
>  arch/arm/boot/dts/ti/omap/omap3-evm.dts            | 1 -
>  arch/arm/boot/dts/ti/omap/omap3-igep.dtsi          | 1 -
>  arch/arm/boot/dts/ti/omap/omap3-ldp.dts            | 1 -
>  arch/arm/boot/dts/ti/omap/omap3-overo-base.dtsi    | 1 -
>  arch/arm/boot/dts/ti/omap/omap3430-sdp.dts         | 2 --
>  13 files changed, 14 deletions(-)
> ---
> base-commit: 5be63fc19fcaa4c236b307420483578a56986a37
> change-id: 20240830-gpmc-dtb-de2ce28acfb6

FYI: you seem to have used the same change-id in this series as in a
previous patch[1], which means when I point b4 at this series, notices
the same change-id in the other patch, thinks it's a v2 of this series,
and tries to apply the v2 patch instead.

I was able to "encourage" b4 to apply this series by using -v1, but it
took me a bit to figure out why I pointed it at a 3 patch series and it
was applying a single patch from a different thread.

Kevin

[1] https://lore.kernel.org/all/20240903-gpmc-dtb-v2-1-8046c1915b96@kernel.org/

Re: [PATCH 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend

[Kevin Hilman]

On Tue, 03 Sep 2024 19:43:43 +0300, Roger Quadros wrote:
> This series fixes dtbs_check warnings on OMAP platforms
> for ti,gpmc-nand and ti,gpmc-onenand.
> 
> The following warnings are fixed
> - "nand@0,0: Unevaluated properties are not allowed ('linux,mtd-name' was unexpected)"
> - "nand@0,0: Unevaluated properties are not allowed ('gpmc,device-nand' was unexpected)"
> - "omap3430-sdp.dtb: onenand@2,0: Unevaluated properties are not allowed ('linux,mtd-name' was unexpected)"
> 
> [...]

Applied, thanks!

[1/3] ARM: dts: ti: drop linux,mtd-name from NAND nodes
      commit: ea453dc2d4d6b7bed89386fe76916252993676ab
[2/3] ARM: dts: ti: omap: am335x-baltos: drop "gpmc,device-nand" from NAND node
      commit: a9c81b1d47baf1a187d240da6e4e6cac2dd668e5
[3/3] ARM: dts: ti: omap3434-sdp: drop linux,mtd-name from onenand node
      commit: 9fe9af0ba275f0109d118ccbd8a438989ca6708a

Best regards,
-- 
Kevin Hilman <khilman@baylibre.com>