* [PATCH 1/6] arm64: dts: freescale: tqma9352: Add partitions subnode to spi-nor
[not found] <20240423113151.2145120-1-alexander.stein@ew.tq-group.com>
@ 2024-04-23 11:31 ` Alexander Stein
2024-06-03 1:12 ` Shawn Guo
2024-04-23 11:31 ` [PATCH 2/6] arm64: dts: freescale: tqma8mqml: " Alexander Stein
` (4 subsequent siblings)
5 siblings, 1 reply; 7+ messages in thread
From: Alexander Stein @ 2024-04-23 11:31 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: Alexander Stein, linux, devicetree, imx, linux-arm-kernel,
linux-kernel
The bootloader adds MTD partitions in this subnode if present, or in the
spi-nor node itself otherwise. Direct children cause warnings like:
425e0000.spi: ofpart partition /soc@0/bus@42000000/spi@425e0000/flash@0/
partition@600000 (/soc@0/bus@42000000/spi@425e0000/flash@0) #size-cells
is wrongly set to <0>, assuming <1> for parsing partitions.
This is due flexspi node having #size-cells = <0>. Setting #size-cells in
MTD nodes itself is deprecated by mtd.yaml.
Fix all this by adding an empty partitions node which the bootloader will
fill with configured MTD partitions.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm64/boot/dts/freescale/imx93-tqma9352.dtsi | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx93-tqma9352.dtsi b/arch/arm64/boot/dts/freescale/imx93-tqma9352.dtsi
index 9d2328c185c90..edbd8cad35bca 100644
--- a/arch/arm64/boot/dts/freescale/imx93-tqma9352.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx93-tqma9352.dtsi
@@ -75,6 +75,12 @@ flash0: flash@0 {
spi-max-frequency = <62000000>;
spi-tx-bus-width = <4>;
spi-rx-bus-width = <4>;
+
+ partitions {
+ compatible = "fixed-partitions";
+ #address-cells = <1>;
+ #size-cells = <1>;
+ };
};
};
--
2.34.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/6] arm64: dts: freescale: tqma8mqml: Add partitions subnode to spi-nor
[not found] <20240423113151.2145120-1-alexander.stein@ew.tq-group.com>
2024-04-23 11:31 ` [PATCH 1/6] arm64: dts: freescale: tqma9352: Add partitions subnode to spi-nor Alexander Stein
@ 2024-04-23 11:31 ` Alexander Stein
2024-04-23 11:31 ` [PATCH 3/6] arm64: dts: freescale: tqma8mqnl: " Alexander Stein
` (3 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Alexander Stein @ 2024-04-23 11:31 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: Alexander Stein, linux, devicetree, imx, linux-arm-kernel,
linux-kernel
The bootloader adds MTD partitions in this subnode if present, or in the
spi-nor node itself otherwise.
Setting #size-cells in MTD nodes itself is deprecated by mtd.yaml.
Fix all this by adding an empty partitions node which the bootloader will
fill with configured MTD partitions. Remove the deprecated properties
as well.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm64/boot/dts/freescale/imx8mm-tqma8mqml.dtsi | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-tqma8mqml.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-tqma8mqml.dtsi
index 8c0c6e7159247..ca0205b9019e6 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-tqma8mqml.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm-tqma8mqml.dtsi
@@ -62,11 +62,15 @@ &flexspi {
flash0: flash@0 {
compatible = "jedec,spi-nor";
reg = <0>;
- #address-cells = <1>;
- #size-cells = <1>;
spi-max-frequency = <84000000>;
spi-tx-bus-width = <1>;
spi-rx-bus-width = <4>;
+
+ partitions {
+ compatible = "fixed-partitions";
+ #address-cells = <1>;
+ #size-cells = <1>;
+ };
};
};
--
2.34.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 3/6] arm64: dts: freescale: tqma8mqnl: Add partitions subnode to spi-nor
[not found] <20240423113151.2145120-1-alexander.stein@ew.tq-group.com>
2024-04-23 11:31 ` [PATCH 1/6] arm64: dts: freescale: tqma9352: Add partitions subnode to spi-nor Alexander Stein
2024-04-23 11:31 ` [PATCH 2/6] arm64: dts: freescale: tqma8mqml: " Alexander Stein
@ 2024-04-23 11:31 ` Alexander Stein
2024-04-23 11:31 ` [PATCH 4/6] arm64: dts: freescale: tqma8mpql: " Alexander Stein
` (2 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Alexander Stein @ 2024-04-23 11:31 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: Alexander Stein, linux, devicetree, imx, linux-arm-kernel,
linux-kernel
The bootloader adds MTD partitions in this subnode if present, or in the
spi-nor node itself otherwise.
Setting #size-cells in MTD nodes itself is deprecated by mtd.yaml.
Fix all this by adding an empty partitions node which the bootloader will
fill with configured MTD partitions. Remove the deprecated properties
as well.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm64/boot/dts/freescale/imx8mn-tqma8mqnl.dtsi | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mn-tqma8mqnl.dtsi b/arch/arm64/boot/dts/freescale/imx8mn-tqma8mqnl.dtsi
index fb24b9aa1b938..e68a3fd73e17d 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-tqma8mqnl.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mn-tqma8mqnl.dtsi
@@ -60,11 +60,15 @@ &flexspi {
flash0: flash@0 {
compatible = "jedec,spi-nor";
reg = <0>;
- #address-cells = <1>;
- #size-cells = <1>;
spi-max-frequency = <84000000>;
spi-tx-bus-width = <1>;
spi-rx-bus-width = <4>;
+
+ partitions {
+ compatible = "fixed-partitions";
+ #address-cells = <1>;
+ #size-cells = <1>;
+ };
};
};
--
2.34.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 4/6] arm64: dts: freescale: tqma8mpql: Add partitions subnode to spi-nor
[not found] <20240423113151.2145120-1-alexander.stein@ew.tq-group.com>
` (2 preceding siblings ...)
2024-04-23 11:31 ` [PATCH 3/6] arm64: dts: freescale: tqma8mqnl: " Alexander Stein
@ 2024-04-23 11:31 ` Alexander Stein
2024-04-23 11:31 ` [PATCH 5/6] arm64: dts: freescale: tqma8mq: " Alexander Stein
2024-04-23 11:31 ` [PATCH 6/6] arm64: dts: freescale: tqma8xx: " Alexander Stein
5 siblings, 0 replies; 7+ messages in thread
From: Alexander Stein @ 2024-04-23 11:31 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: Alexander Stein, linux, devicetree, imx, linux-arm-kernel,
linux-kernel
The bootloader adds MTD partitions in this subnode if present, or in the
spi-nor node itself otherwise.
Setting #size-cells in MTD nodes itself is deprecated by mtd.yaml.
Fix all this by adding an empty partitions node which the bootloader will
fill with configured MTD partitions. Remove the deprecated properties
as well.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi
index ebc29a950ba9a..336785a9fba89 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi
@@ -45,12 +45,16 @@ &flexspi {
flash0: flash@0 {
reg = <0>;
- #address-cells = <1>;
- #size-cells = <1>;
compatible = "jedec,spi-nor";
spi-max-frequency = <80000000>;
spi-tx-bus-width = <1>;
spi-rx-bus-width = <4>;
+
+ partitions {
+ compatible = "fixed-partitions";
+ #address-cells = <1>;
+ #size-cells = <1>;
+ };
};
};
--
2.34.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 5/6] arm64: dts: freescale: tqma8mq: Add partitions subnode to spi-nor
[not found] <20240423113151.2145120-1-alexander.stein@ew.tq-group.com>
` (3 preceding siblings ...)
2024-04-23 11:31 ` [PATCH 4/6] arm64: dts: freescale: tqma8mpql: " Alexander Stein
@ 2024-04-23 11:31 ` Alexander Stein
2024-04-23 11:31 ` [PATCH 6/6] arm64: dts: freescale: tqma8xx: " Alexander Stein
5 siblings, 0 replies; 7+ messages in thread
From: Alexander Stein @ 2024-04-23 11:31 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: Alexander Stein, linux, devicetree, imx, linux-arm-kernel,
linux-kernel
The bootloader adds MTD partitions in this subnode if present, or in the
spi-nor node itself otherwise.
Setting #size-cells in MTD nodes itself is deprecated by mtd.yaml.
Fix all this by adding an empty partitions node which the bootloader will
fill with configured MTD partitions. Remove the deprecated properties
as well.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm64/boot/dts/freescale/imx8mq-tqma8mq.dtsi | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mq-tqma8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-tqma8mq.dtsi
index 5ca6b22525469..01e5092e4c40a 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-tqma8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq-tqma8mq.dtsi
@@ -251,11 +251,15 @@ &qspi0 {
flash0: flash@0 {
compatible = "jedec,spi-nor";
reg = <0>;
- #address-cells = <1>;
- #size-cells = <1>;
spi-max-frequency = <84000000>;
spi-tx-bus-width = <1>;
spi-rx-bus-width = <4>;
+
+ partitions {
+ compatible = "fixed-partitions";
+ #address-cells = <1>;
+ #size-cells = <1>;
+ };
};
};
--
2.34.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 6/6] arm64: dts: freescale: tqma8xx: Add partitions subnode to spi-nor
[not found] <20240423113151.2145120-1-alexander.stein@ew.tq-group.com>
` (4 preceding siblings ...)
2024-04-23 11:31 ` [PATCH 5/6] arm64: dts: freescale: tqma8mq: " Alexander Stein
@ 2024-04-23 11:31 ` Alexander Stein
5 siblings, 0 replies; 7+ messages in thread
From: Alexander Stein @ 2024-04-23 11:31 UTC (permalink / raw)
To: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: Alexander Stein, imx, linux-arm-kernel, devicetree, linux-kernel
The bootloader adds MTD partitions in this subnode if present, or in the
spi-nor node itself otherwise.
Setting #size-cells in MTD nodes itself is deprecated by mtd.yaml.
Fix all this by adding an empty partitions node which the bootloader will
fill with configured MTD partitions. Remove the deprecated properties
as well.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm64/boot/dts/freescale/tqma8xx.dtsi | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/tqma8xx.dtsi b/arch/arm64/boot/dts/freescale/tqma8xx.dtsi
index d98469a7c47cc..366912bf3d5e5 100644
--- a/arch/arm64/boot/dts/freescale/tqma8xx.dtsi
+++ b/arch/arm64/boot/dts/freescale/tqma8xx.dtsi
@@ -61,12 +61,16 @@ &flexspi0 {
flash0: flash@0 {
reg = <0>;
- #address-cells = <1>;
- #size-cells = <1>;
compatible = "jedec,spi-nor";
spi-max-frequency = <66000000>;
spi-tx-bus-width = <1>;
spi-rx-bus-width = <4>;
+
+ partitions {
+ compatible = "fixed-partitions";
+ #address-cells = <1>;
+ #size-cells = <1>;
+ };
};
};
--
2.34.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/6] arm64: dts: freescale: tqma9352: Add partitions subnode to spi-nor
2024-04-23 11:31 ` [PATCH 1/6] arm64: dts: freescale: tqma9352: Add partitions subnode to spi-nor Alexander Stein
@ 2024-06-03 1:12 ` Shawn Guo
0 siblings, 0 replies; 7+ messages in thread
From: Shawn Guo @ 2024-06-03 1:12 UTC (permalink / raw)
To: Alexander Stein
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, linux,
devicetree, imx, linux-arm-kernel, linux-kernel
On Tue, Apr 23, 2024 at 01:31:46PM +0200, Alexander Stein wrote:
> The bootloader adds MTD partitions in this subnode if present, or in the
> spi-nor node itself otherwise. Direct children cause warnings like:
> 425e0000.spi: ofpart partition /soc@0/bus@42000000/spi@425e0000/flash@0/
> partition@600000 (/soc@0/bus@42000000/spi@425e0000/flash@0) #size-cells
> is wrongly set to <0>, assuming <1> for parsing partitions.
>
> This is due flexspi node having #size-cells = <0>. Setting #size-cells in
> MTD nodes itself is deprecated by mtd.yaml.
> Fix all this by adding an empty partitions node which the bootloader will
> fill with configured MTD partitions.
>
> Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Applied all, thanks!
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2024-06-03 1:12 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20240423113151.2145120-1-alexander.stein@ew.tq-group.com>
2024-04-23 11:31 ` [PATCH 1/6] arm64: dts: freescale: tqma9352: Add partitions subnode to spi-nor Alexander Stein
2024-06-03 1:12 ` Shawn Guo
2024-04-23 11:31 ` [PATCH 2/6] arm64: dts: freescale: tqma8mqml: " Alexander Stein
2024-04-23 11:31 ` [PATCH 3/6] arm64: dts: freescale: tqma8mqnl: " Alexander Stein
2024-04-23 11:31 ` [PATCH 4/6] arm64: dts: freescale: tqma8mpql: " Alexander Stein
2024-04-23 11:31 ` [PATCH 5/6] arm64: dts: freescale: tqma8mq: " Alexander Stein
2024-04-23 11:31 ` [PATCH 6/6] arm64: dts: freescale: tqma8xx: " Alexander Stein
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).