* [PATCH 1/3] ARM: dts: ti: drop linux,mtd-name from NAND nodes
2024-09-03 16:43 [PATCH 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend Roger Quadros
@ 2024-09-03 16:43 ` Roger Quadros
2024-09-03 16:43 ` [PATCH 2/3] ARM: dts: ti: omap: am335x-baltos: drop "gpmc,device-nand" from NAND node Roger Quadros
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Roger Quadros @ 2024-09-03 16:43 UTC (permalink / raw)
To: Kevin Hilman, Tony Lindgren, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Enric Balletbo i Serra, Javier Martinez Canillas
Cc: Nishanth Menon, srk, linux-omap, devicetree, linux-kernel,
linux-arm-kernel, 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
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 2/3] ARM: dts: ti: omap: am335x-baltos: drop "gpmc,device-nand" from NAND node
2024-09-03 16:43 [PATCH 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend Roger Quadros
2024-09-03 16:43 ` [PATCH 1/3] ARM: dts: ti: drop linux,mtd-name from NAND nodes Roger Quadros
@ 2024-09-03 16:43 ` Roger Quadros
2024-09-03 16:43 ` [PATCH 3/3] ARM: dts: ti: omap3434-sdp: drop linux,mtd-name from onenand node Roger Quadros
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Roger Quadros @ 2024-09-03 16:43 UTC (permalink / raw)
To: Kevin Hilman, Tony Lindgren, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Enric Balletbo i Serra, Javier Martinez Canillas
Cc: Nishanth Menon, srk, linux-omap, devicetree, linux-kernel,
linux-arm-kernel, 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
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 3/3] ARM: dts: ti: omap3434-sdp: drop linux,mtd-name from onenand node
2024-09-03 16:43 [PATCH 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend Roger Quadros
2024-09-03 16:43 ` [PATCH 1/3] ARM: dts: ti: drop linux,mtd-name from NAND nodes Roger Quadros
2024-09-03 16:43 ` [PATCH 2/3] ARM: dts: ti: omap: am335x-baltos: drop "gpmc,device-nand" from NAND node Roger Quadros
@ 2024-09-03 16:43 ` Roger Quadros
2024-10-04 21:01 ` [PATCH 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend Kevin Hilman
2024-10-04 21:33 ` Kevin Hilman
4 siblings, 0 replies; 6+ messages in thread
From: Roger Quadros @ 2024-09-03 16:43 UTC (permalink / raw)
To: Kevin Hilman, Tony Lindgren, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Enric Balletbo i Serra, Javier Martinez Canillas
Cc: Nishanth Menon, srk, linux-omap, devicetree, linux-kernel,
linux-arm-kernel, 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
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend
2024-09-03 16:43 [PATCH 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend Roger Quadros
` (2 preceding siblings ...)
2024-09-03 16:43 ` [PATCH 3/3] ARM: dts: ti: omap3434-sdp: drop linux,mtd-name from onenand node Roger Quadros
@ 2024-10-04 21:01 ` Kevin Hilman
2024-10-04 21:33 ` Kevin Hilman
4 siblings, 0 replies; 6+ messages in thread
From: Kevin Hilman @ 2024-10-04 21:01 UTC (permalink / raw)
To: Roger Quadros, Tony Lindgren, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Enric Balletbo i Serra, Javier Martinez Canillas
Cc: Nishanth Menon, srk, linux-omap, devicetree, linux-kernel,
linux-arm-kernel, Roger Quadros
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/
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend
2024-09-03 16:43 [PATCH 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend Roger Quadros
` (3 preceding siblings ...)
2024-10-04 21:01 ` [PATCH 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend Kevin Hilman
@ 2024-10-04 21:33 ` Kevin Hilman
4 siblings, 0 replies; 6+ messages in thread
From: Kevin Hilman @ 2024-10-04 21:33 UTC (permalink / raw)
To: Tony Lindgren, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Enric Balletbo i Serra, Javier Martinez Canillas, Roger Quadros
Cc: Nishanth Menon, srk, linux-omap, devicetree, linux-kernel,
linux-arm-kernel
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>
^ permalink raw reply [flat|nested] 6+ messages in thread