* [PATCH v2 1/3] ARM: dts: ti: drop linux,mtd-name from NAND nodes
2024-10-09 13:16 [PATCH v2 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend Roger Quadros
@ 2024-10-09 13:16 ` Roger Quadros
2024-10-09 13:16 ` [PATCH v2 2/3] ARM: dts: ti: omap: am335x-baltos: drop "gpmc,device-nand" from NAND node Roger Quadros
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: Roger Quadros @ 2024-10-09 13:16 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 v2 2/3] ARM: dts: ti: omap: am335x-baltos: drop "gpmc,device-nand" from NAND node
2024-10-09 13:16 [PATCH v2 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend Roger Quadros
2024-10-09 13:16 ` [PATCH v2 1/3] ARM: dts: ti: drop linux,mtd-name from NAND nodes Roger Quadros
@ 2024-10-09 13:16 ` Roger Quadros
2024-10-09 13:16 ` [PATCH v2 3/3] ARM: dts: ti: omap3434-sdp: drop linux,mtd-name from onenand node Roger Quadros
2024-10-10 9:35 ` [PATCH v2 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend Andreas Kemnade
3 siblings, 0 replies; 6+ messages in thread
From: Roger Quadros @ 2024-10-09 13:16 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 v2 3/3] ARM: dts: ti: omap3434-sdp: drop linux,mtd-name from onenand node
2024-10-09 13:16 [PATCH v2 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend Roger Quadros
2024-10-09 13:16 ` [PATCH v2 1/3] ARM: dts: ti: drop linux,mtd-name from NAND nodes Roger Quadros
2024-10-09 13:16 ` [PATCH v2 2/3] ARM: dts: ti: omap: am335x-baltos: drop "gpmc,device-nand" from NAND node Roger Quadros
@ 2024-10-09 13:16 ` Roger Quadros
2024-10-10 9:35 ` [PATCH v2 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend Andreas Kemnade
3 siblings, 0 replies; 6+ messages in thread
From: Roger Quadros @ 2024-10-09 13:16 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 v2 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend
2024-10-09 13:16 [PATCH v2 0/3] ARM: dts: ti: omap: fix dtbs_check warnings for ti,gpmc-nand and ti,gpmc-onenend Roger Quadros
` (2 preceding siblings ...)
2024-10-09 13:16 ` [PATCH v2 3/3] ARM: dts: ti: omap3434-sdp: drop linux,mtd-name from onenand node Roger Quadros
@ 2024-10-10 9:35 ` Andreas Kemnade
2024-10-10 11:31 ` Roger Quadros
3 siblings, 1 reply; 6+ messages in thread
From: Andreas Kemnade @ 2024-10-10 9:35 UTC (permalink / raw)
To: Roger Quadros
Cc: Kevin Hilman, Tony Lindgren, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Enric Balletbo i Serra, Javier Martinez Canillas,
Nishanth Menon, srk, linux-omap, devicetree, linux-kernel,
linux-arm-kernel
Am Wed, 09 Oct 2024 16:16:53 +0300
schrieb Roger Quadros <rogerq@kernel.org>:
> 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)"
>
> Signed-off-by: Roger Quadros <rogerq@kernel.org>
> ---
> Changes in v2:
> - Rebased on v6.12-rc1
> - Link to v1:
> https://lore.kernel.org/r/20240903-gpmc-dtb-v1-0-380952952e34@kernel.org
>
hmm, v1 seems to be applied.
see
https://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap.git/commit/?h=for-next&id=ea453dc2d4d6b7bed89386fe76916252993676ab
Kevin seems to only have pushed the for-next branch and not the
individual topic-6.13 branches.
Regards,
Andreas
^ permalink raw reply [flat|nested] 6+ messages in thread