* [PATCH v9 0/3] dt-bindings: mtd: marvell-nand: Add YAML scheme
@ 2023-06-15 4:04 ` Chris Packham
0 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 4:04 UTC (permalink / raw)
To: miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Chris Packham
Add YAML scheme for the Marvell's NAND controller
to validate it's DT bindings. Old txt file is deleted.
Also fix node name in cp11x DTSI according to nand-controller.yaml
I've picked up this series to hopefully get it over the line. I think I've
addressed all the feedback from the earlier rounds of review.
As of v7 I've re-ordered the patches so the fixups for the device trees come
before the schema so if the series is applied in order the various checkers
should pass cleanly. This is reliant on a series from Miquel tightening up some
of the requirements on mtd/nand chips so the checkers may complain unless those
changes are picked up too.
Chris Packham (1):
ARM: dts: mvebu: align MTD partition nodes to dtschema
Vadym Kochan (2):
arm64: dts: marvell: cp11x: Fix nand_controller node name according to
YAML
dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
.../bindings/mtd/marvell,nand-controller.yaml | 218 ++++++++++++++++++
.../devicetree/bindings/mtd/marvell-nand.txt | 126 ----------
MAINTAINERS | 1 -
arch/arm/boot/dts/armada-385-atl-x530.dts | 14 +-
arch/arm64/boot/dts/marvell/armada-cp11x.dtsi | 2 +-
5 files changed, 226 insertions(+), 135 deletions(-)
create mode 100644 Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
delete mode 100644 Documentation/devicetree/bindings/mtd/marvell-nand.txt
--
2.40.1
^ permalink raw reply [flat|nested] 39+ messages in thread
* [PATCH v9 0/3] dt-bindings: mtd: marvell-nand: Add YAML scheme
@ 2023-06-15 4:04 ` Chris Packham
0 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 4:04 UTC (permalink / raw)
To: miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Chris Packham
Add YAML scheme for the Marvell's NAND controller
to validate it's DT bindings. Old txt file is deleted.
Also fix node name in cp11x DTSI according to nand-controller.yaml
I've picked up this series to hopefully get it over the line. I think I've
addressed all the feedback from the earlier rounds of review.
As of v7 I've re-ordered the patches so the fixups for the device trees come
before the schema so if the series is applied in order the various checkers
should pass cleanly. This is reliant on a series from Miquel tightening up some
of the requirements on mtd/nand chips so the checkers may complain unless those
changes are picked up too.
Chris Packham (1):
ARM: dts: mvebu: align MTD partition nodes to dtschema
Vadym Kochan (2):
arm64: dts: marvell: cp11x: Fix nand_controller node name according to
YAML
dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
.../bindings/mtd/marvell,nand-controller.yaml | 218 ++++++++++++++++++
.../devicetree/bindings/mtd/marvell-nand.txt | 126 ----------
MAINTAINERS | 1 -
arch/arm/boot/dts/armada-385-atl-x530.dts | 14 +-
arch/arm64/boot/dts/marvell/armada-cp11x.dtsi | 2 +-
5 files changed, 226 insertions(+), 135 deletions(-)
create mode 100644 Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
delete mode 100644 Documentation/devicetree/bindings/mtd/marvell-nand.txt
--
2.40.1
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 39+ messages in thread
* [PATCH v9 0/3] dt-bindings: mtd: marvell-nand: Add YAML scheme
@ 2023-06-15 4:04 ` Chris Packham
0 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 4:04 UTC (permalink / raw)
To: miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Chris Packham
Add YAML scheme for the Marvell's NAND controller
to validate it's DT bindings. Old txt file is deleted.
Also fix node name in cp11x DTSI according to nand-controller.yaml
I've picked up this series to hopefully get it over the line. I think I've
addressed all the feedback from the earlier rounds of review.
As of v7 I've re-ordered the patches so the fixups for the device trees come
before the schema so if the series is applied in order the various checkers
should pass cleanly. This is reliant on a series from Miquel tightening up some
of the requirements on mtd/nand chips so the checkers may complain unless those
changes are picked up too.
Chris Packham (1):
ARM: dts: mvebu: align MTD partition nodes to dtschema
Vadym Kochan (2):
arm64: dts: marvell: cp11x: Fix nand_controller node name according to
YAML
dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
.../bindings/mtd/marvell,nand-controller.yaml | 218 ++++++++++++++++++
.../devicetree/bindings/mtd/marvell-nand.txt | 126 ----------
MAINTAINERS | 1 -
arch/arm/boot/dts/armada-385-atl-x530.dts | 14 +-
arch/arm64/boot/dts/marvell/armada-cp11x.dtsi | 2 +-
5 files changed, 226 insertions(+), 135 deletions(-)
create mode 100644 Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
delete mode 100644 Documentation/devicetree/bindings/mtd/marvell-nand.txt
--
2.40.1
_______________________________________________
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] 39+ messages in thread
* [PATCH v9 1/3] arm64: dts: marvell: cp11x: Fix nand_controller node name according to YAML
2023-06-15 4:04 ` Chris Packham
(?)
@ 2023-06-15 4:04 ` Chris Packham
-1 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 4:04 UTC (permalink / raw)
To: miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel,
Vadym Kochan, Chris Packham
From: Vadym Kochan <vadym.kochan@plvision.eu>
Marvell NAND controller has now YAML to validate it's DT bindings, so
change the node name of cp11x DTSI as it is required by nand-controller.yaml
Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
---
Notes:
Changes in v9:
- None
Changes in v8:
- Add r-by from Miquel
arch/arm64/boot/dts/marvell/armada-cp11x.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
index 7d0043824f2a..982b180b33e6 100644
--- a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
@@ -468,7 +468,7 @@ CP11X_LABEL(uart3): serial@702300 {
status = "disabled";
};
- CP11X_LABEL(nand_controller): nand@720000 {
+ CP11X_LABEL(nand_controller): nand-controller@720000 {
/*
* Due to the limitation of the pins available
* this controller is only usable on the CPM
--
2.40.1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [PATCH v9 1/3] arm64: dts: marvell: cp11x: Fix nand_controller node name according to YAML
@ 2023-06-15 4:04 ` Chris Packham
0 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 4:04 UTC (permalink / raw)
To: miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel,
Vadym Kochan, Chris Packham
From: Vadym Kochan <vadym.kochan@plvision.eu>
Marvell NAND controller has now YAML to validate it's DT bindings, so
change the node name of cp11x DTSI as it is required by nand-controller.yaml
Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
---
Notes:
Changes in v9:
- None
Changes in v8:
- Add r-by from Miquel
arch/arm64/boot/dts/marvell/armada-cp11x.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
index 7d0043824f2a..982b180b33e6 100644
--- a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
@@ -468,7 +468,7 @@ CP11X_LABEL(uart3): serial@702300 {
status = "disabled";
};
- CP11X_LABEL(nand_controller): nand@720000 {
+ CP11X_LABEL(nand_controller): nand-controller@720000 {
/*
* Due to the limitation of the pins available
* this controller is only usable on the CPM
--
2.40.1
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [PATCH v9 1/3] arm64: dts: marvell: cp11x: Fix nand_controller node name according to YAML
@ 2023-06-15 4:04 ` Chris Packham
0 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 4:04 UTC (permalink / raw)
To: miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel,
Vadym Kochan, Chris Packham
From: Vadym Kochan <vadym.kochan@plvision.eu>
Marvell NAND controller has now YAML to validate it's DT bindings, so
change the node name of cp11x DTSI as it is required by nand-controller.yaml
Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
---
Notes:
Changes in v9:
- None
Changes in v8:
- Add r-by from Miquel
arch/arm64/boot/dts/marvell/armada-cp11x.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
index 7d0043824f2a..982b180b33e6 100644
--- a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
@@ -468,7 +468,7 @@ CP11X_LABEL(uart3): serial@702300 {
status = "disabled";
};
- CP11X_LABEL(nand_controller): nand@720000 {
+ CP11X_LABEL(nand_controller): nand-controller@720000 {
/*
* Due to the limitation of the pins available
* this controller is only usable on the CPM
--
2.40.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] 39+ messages in thread
* [PATCH v9 2/3] ARM: dts: mvebu: align MTD partition nodes to dtschema
2023-06-15 4:04 ` Chris Packham
(?)
@ 2023-06-15 4:04 ` Chris Packham
-1 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 4:04 UTC (permalink / raw)
To: miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Chris Packham
Update the node names for the SPI NOR and NAND partitions to conform to
the partition properties in the relevant dtschema.
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
---
Notes:
Changes in v9:
- None
Changes in v8:
- Add r-by from Miquel
arch/arm/boot/dts/armada-385-atl-x530.dts | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/arch/arm/boot/dts/armada-385-atl-x530.dts b/arch/arm/boot/dts/armada-385-atl-x530.dts
index 241f5d7c80e9..5a9ab8410b7b 100644
--- a/arch/arm/boot/dts/armada-385-atl-x530.dts
+++ b/arch/arm/boot/dts/armada-385-atl-x530.dts
@@ -179,19 +179,19 @@ partitions {
compatible = "fixed-partitions";
#address-cells = <1>;
#size-cells = <1>;
- partition@u-boot {
+ partition@0 {
reg = <0x00000000 0x00100000>;
label = "u-boot";
};
- partition@u-boot-env {
+ partition@100000 {
reg = <0x00100000 0x00040000>;
label = "u-boot-env";
};
- partition@unused {
+ partition@140000 {
reg = <0x00140000 0x00e80000>;
label = "unused";
};
- partition@idprom {
+ partition@fc0000 {
reg = <0x00fc0000 0x00040000>;
label = "idprom";
};
@@ -216,16 +216,16 @@ partitions {
compatible = "fixed-partitions";
#address-cells = <1>;
#size-cells = <1>;
- partition@user {
+ partition@0 {
reg = <0x00000000 0x0f000000>;
label = "user";
};
- partition@errlog {
+ partition@f000000 {
/* Maximum mtdoops size is 8MB, so set to that. */
reg = <0x0f000000 0x00800000>;
label = "errlog";
};
- partition@nand-bbt {
+ partition@f800000 {
reg = <0x0f800000 0x00800000>;
label = "nand-bbt";
};
--
2.40.1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [PATCH v9 2/3] ARM: dts: mvebu: align MTD partition nodes to dtschema
@ 2023-06-15 4:04 ` Chris Packham
0 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 4:04 UTC (permalink / raw)
To: miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Chris Packham
Update the node names for the SPI NOR and NAND partitions to conform to
the partition properties in the relevant dtschema.
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
---
Notes:
Changes in v9:
- None
Changes in v8:
- Add r-by from Miquel
arch/arm/boot/dts/armada-385-atl-x530.dts | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/arch/arm/boot/dts/armada-385-atl-x530.dts b/arch/arm/boot/dts/armada-385-atl-x530.dts
index 241f5d7c80e9..5a9ab8410b7b 100644
--- a/arch/arm/boot/dts/armada-385-atl-x530.dts
+++ b/arch/arm/boot/dts/armada-385-atl-x530.dts
@@ -179,19 +179,19 @@ partitions {
compatible = "fixed-partitions";
#address-cells = <1>;
#size-cells = <1>;
- partition@u-boot {
+ partition@0 {
reg = <0x00000000 0x00100000>;
label = "u-boot";
};
- partition@u-boot-env {
+ partition@100000 {
reg = <0x00100000 0x00040000>;
label = "u-boot-env";
};
- partition@unused {
+ partition@140000 {
reg = <0x00140000 0x00e80000>;
label = "unused";
};
- partition@idprom {
+ partition@fc0000 {
reg = <0x00fc0000 0x00040000>;
label = "idprom";
};
@@ -216,16 +216,16 @@ partitions {
compatible = "fixed-partitions";
#address-cells = <1>;
#size-cells = <1>;
- partition@user {
+ partition@0 {
reg = <0x00000000 0x0f000000>;
label = "user";
};
- partition@errlog {
+ partition@f000000 {
/* Maximum mtdoops size is 8MB, so set to that. */
reg = <0x0f000000 0x00800000>;
label = "errlog";
};
- partition@nand-bbt {
+ partition@f800000 {
reg = <0x0f800000 0x00800000>;
label = "nand-bbt";
};
--
2.40.1
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [PATCH v9 2/3] ARM: dts: mvebu: align MTD partition nodes to dtschema
@ 2023-06-15 4:04 ` Chris Packham
0 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 4:04 UTC (permalink / raw)
To: miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Chris Packham
Update the node names for the SPI NOR and NAND partitions to conform to
the partition properties in the relevant dtschema.
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
---
Notes:
Changes in v9:
- None
Changes in v8:
- Add r-by from Miquel
arch/arm/boot/dts/armada-385-atl-x530.dts | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/arch/arm/boot/dts/armada-385-atl-x530.dts b/arch/arm/boot/dts/armada-385-atl-x530.dts
index 241f5d7c80e9..5a9ab8410b7b 100644
--- a/arch/arm/boot/dts/armada-385-atl-x530.dts
+++ b/arch/arm/boot/dts/armada-385-atl-x530.dts
@@ -179,19 +179,19 @@ partitions {
compatible = "fixed-partitions";
#address-cells = <1>;
#size-cells = <1>;
- partition@u-boot {
+ partition@0 {
reg = <0x00000000 0x00100000>;
label = "u-boot";
};
- partition@u-boot-env {
+ partition@100000 {
reg = <0x00100000 0x00040000>;
label = "u-boot-env";
};
- partition@unused {
+ partition@140000 {
reg = <0x00140000 0x00e80000>;
label = "unused";
};
- partition@idprom {
+ partition@fc0000 {
reg = <0x00fc0000 0x00040000>;
label = "idprom";
};
@@ -216,16 +216,16 @@ partitions {
compatible = "fixed-partitions";
#address-cells = <1>;
#size-cells = <1>;
- partition@user {
+ partition@0 {
reg = <0x00000000 0x0f000000>;
label = "user";
};
- partition@errlog {
+ partition@f000000 {
/* Maximum mtdoops size is 8MB, so set to that. */
reg = <0x0f000000 0x00800000>;
label = "errlog";
};
- partition@nand-bbt {
+ partition@f800000 {
reg = <0x0f800000 0x00800000>;
label = "nand-bbt";
};
--
2.40.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] 39+ messages in thread
* [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
2023-06-15 4:04 ` Chris Packham
(?)
@ 2023-06-15 4:04 ` Chris Packham
-1 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 4:04 UTC (permalink / raw)
To: miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel,
Vadym Kochan, Chris Packham
From: Vadym Kochan <vadym.kochan@plvision.eu>
Switch the DT binding to a YAML schema to enable the DT validation.
The text binding didn't mention it as a requirement but existing usage
has
compatible = "marvell,armada-8k-nand-controller",
"marvell,armada370-nand-controller";
so the YAML allows this in addition to the individual compatible values.
There was also an incorrect reference to dma-names being "rxtx" where
the driver and existing device trees actually use dma-names = "data" so
this is corrected in the conversion.
Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
---
Notes:
Changes in v9:
- depend on series from Miquel
https://lore.kernel.org/linux-mtd/20230606175246.190465-1-miquel.raynal@bootlin.com/
- enforce minimum/maximum for nand-rb
- move required: block for controller
- move unevaluatedProperties: for nand chip
- remove label, partitions and nand-on-flash-bbt which are covered by
generic schema
Changes in v8:
- Mark deprecated compatible values as such
- Allow "marvell,armada-8k-nand-controller" without
"marvell,armada370-nand-controller"
- Make dma-names usage reflect reality
- Update commit message
Changes in v7:
- Restore "label" and "partitions" properties (should be picked up via
nand-controller.yaml but aren't)
- Add/restore nand-on-flash-bbt and nand-ecc-mode which aren't covered
by nand-controller.yaml.
- Use "unevalautedProperties: false"
- Corrections for clock-names, dma-names, nand-rb and nand-ecc-strength
- Add pxa3xx-nand-controller example
Changes in v6:
- remove properties covered by nand-controller.yaml
- add example using armada-8k compatible
earlier changes:
v5:
1) Get back "label" and "partitions" properties but without
ref to the "partition.yaml" which was wrongly used.
2) Add "additionalProperties: false" for nand@ because all possible
properties are described.
v4:
1) Remove "label" and "partitions" properties
2) Use 2 clocks for A7K/8K platform which is a requirement
v3:
1) Remove txt version from the MAINTAINERS list
2) Use enum for some of compatible strings
3) Drop:
#address-cells
#size-cells:
as they are inherited from the nand-controller.yaml
4) Add restriction to use 2 clocks for A8K SoC
5) Dropped description for clock-names and extend it with
minItems: 1
6) Drop description for "dmas"
7) Use "unevalautedProperties: false"
8) Drop quites from yaml refs.
9) Use 4-space indentation for the example section
v2:
1) Fixed warning by yamllint with incorrect indentation for compatible list
.../bindings/mtd/marvell,nand-controller.yaml | 218 ++++++++++++++++++
.../devicetree/bindings/mtd/marvell-nand.txt | 126 ----------
MAINTAINERS | 1 -
3 files changed, 218 insertions(+), 127 deletions(-)
create mode 100644 Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
delete mode 100644 Documentation/devicetree/bindings/mtd/marvell-nand.txt
diff --git a/Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml b/Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
new file mode 100644
index 000000000000..252bd0f3e222
--- /dev/null
+++ b/Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
@@ -0,0 +1,218 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Marvell NAND Flash Controller (NFC)
+
+maintainers:
+ - Miquel Raynal <miquel.raynal@bootlin.com>
+
+properties:
+ compatible:
+ oneOf:
+ - items:
+ - const: marvell,armada-8k-nand-controller
+ - const: marvell,armada370-nand-controller
+ - enum:
+ - marvell,armada-8k-nand-controller
+ - marvell,armada370-nand-controller
+ - marvell,pxa3xx-nand-controller
+ - description: legacy bindings
+ deprecated: true
+ enum:
+ - marvell,armada-8k-nand
+ - marvell,armada370-nand
+ - marvell,pxa3xx-nand
+
+ reg:
+ maxItems: 1
+
+ interrupts:
+ maxItems: 1
+
+ clocks:
+ description:
+ Shall reference the NAND controller clocks, the second one is
+ is only needed for the Armada 7K/8K SoCs
+ minItems: 1
+ maxItems: 2
+
+ clock-names:
+ minItems: 1
+ items:
+ - const: core
+ - const: reg
+
+ dmas:
+ maxItems: 1
+
+ dma-names:
+ items:
+ - const: data
+
+ marvell,system-controller:
+ $ref: /schemas/types.yaml#/definitions/phandle
+ description: Syscon node that handles NAND controller related registers
+
+patternProperties:
+ "^nand@[a-f0-9]$":
+ type: object
+ $ref: raw-nand-chip.yaml
+
+ properties:
+ reg:
+ minimum: 0
+ maximum: 3
+
+ nand-rb:
+ items:
+ - minimum: 0
+ maximum: 1
+
+ nand-ecc-step-size:
+ const: 512
+
+ nand-ecc-strength:
+ enum: [1, 4, 8, 12, 16]
+
+ nand-ecc-mode:
+ const: hw
+
+ marvell,nand-keep-config:
+ $ref: /schemas/types.yaml#/definitions/flag
+ description: |
+ Orders the driver not to take the timings from the core and
+ leaving them completely untouched. Bootloader timings will then
+ be used.
+
+ marvell,nand-enable-arbiter:
+ $ref: /schemas/types.yaml#/definitions/flag
+ description: |
+ To enable the arbiter, all boards blindly used it,
+ this bit was set by the bootloader for many boards and even if
+ it is marked reserved in several datasheets, it might be needed to set
+ it (otherwise it is harmless).
+ deprecated: true
+
+ required:
+ - reg
+ - nand-rb
+
+ unevaluatedProperties: false
+
+required:
+ - compatible
+ - reg
+ - interrupts
+ - clocks
+
+allOf:
+ - $ref: nand-controller.yaml#
+
+ - if:
+ properties:
+ compatible:
+ contains:
+ const: marvell,pxa3xx-nand-controller
+ then:
+ required:
+ - dmas
+ - dma-names
+
+ - if:
+ properties:
+ compatible:
+ contains:
+ const: marvell,armada-8k-nand-controller
+ then:
+ properties:
+ clocks:
+ minItems: 2
+
+ clock-names:
+ minItems: 2
+
+ required:
+ - marvell,system-controller
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/interrupt-controller/arm-gic.h>
+ nand_controller: nand-controller@d0000 {
+ compatible = "marvell,armada370-nand-controller";
+ reg = <0xd0000 0x54>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ interrupts = <GIC_SPI 84 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&coredivclk 0>;
+
+ nand@0 {
+ reg = <0>;
+ label = "main-storage";
+ nand-rb = <0>;
+ nand-ecc-mode = "hw";
+ marvell,nand-keep-config;
+ nand-on-flash-bbt;
+ nand-ecc-strength = <4>;
+ nand-ecc-step-size = <512>;
+
+ partitions {
+ compatible = "fixed-partitions";
+ #address-cells = <1>;
+ #size-cells = <1>;
+
+ partition@0 {
+ label = "Rootfs";
+ reg = <0x00000000 0x40000000>;
+ };
+ };
+ };
+ };
+
+ - |
+ cp0_nand_controller: nand-controller@720000 {
+ compatible = "marvell,armada-8k-nand-controller",
+ "marvell,armada370-nand-controller";
+ reg = <0x720000 0x54>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ interrupts = <115 IRQ_TYPE_LEVEL_HIGH>;
+ clock-names = "core", "reg";
+ clocks = <&cp0_clk 1 2>,
+ <&cp0_clk 1 17>;
+ marvell,system-controller = <&cp0_syscon0>;
+
+ nand@0 {
+ reg = <0>;
+ label = "main-storage";
+ nand-rb = <0>;
+ nand-ecc-mode = "hw";
+ nand-ecc-strength = <8>;
+ nand-ecc-step-size = <512>;
+ };
+ };
+
+ - |
+ nand-controller@43100000 {
+ compatible = "marvell,pxa3xx-nand-controller";
+ reg = <0x43100000 90>;
+ interrupts = <45>;
+ clocks = <&clks 1>;
+ clock-names = "core";
+ dmas = <&pdma 97 3>;
+ dma-names = "data";
+ #address-cells = <1>;
+ #size-cells = <0>;
+ nand@0 {
+ reg = <0>;
+ nand-rb = <0>;
+ nand-ecc-mode = "hw";
+ marvell,nand-keep-config;
+ };
+ };
+
+...
diff --git a/Documentation/devicetree/bindings/mtd/marvell-nand.txt b/Documentation/devicetree/bindings/mtd/marvell-nand.txt
deleted file mode 100644
index a2d9a0f2b683..000000000000
--- a/Documentation/devicetree/bindings/mtd/marvell-nand.txt
+++ /dev/null
@@ -1,126 +0,0 @@
-Marvell NAND Flash Controller (NFC)
-
-Required properties:
-- compatible: can be one of the following:
- * "marvell,armada-8k-nand-controller"
- * "marvell,armada370-nand-controller"
- * "marvell,pxa3xx-nand-controller"
- * "marvell,armada-8k-nand" (deprecated)
- * "marvell,armada370-nand" (deprecated)
- * "marvell,pxa3xx-nand" (deprecated)
- Compatibles marked deprecated support only the old bindings described
- at the bottom.
-- reg: NAND flash controller memory area.
-- #address-cells: shall be set to 1. Encode the NAND CS.
-- #size-cells: shall be set to 0.
-- interrupts: shall define the NAND controller interrupt.
-- clocks: shall reference the NAND controller clocks, the second one is
- is only needed for the Armada 7K/8K SoCs
-- clock-names: mandatory if there is a second clock, in this case there
- should be one clock named "core" and another one named "reg"
-- marvell,system-controller: Set to retrieve the syscon node that handles
- NAND controller related registers (only required with the
- "marvell,armada-8k-nand[-controller]" compatibles).
-
-Optional properties:
-- label: see partition.txt. New platforms shall omit this property.
-- dmas: shall reference DMA channel associated to the NAND controller.
- This property is only used with "marvell,pxa3xx-nand[-controller]"
- compatible strings.
-- dma-names: shall be "rxtx".
- This property is only used with "marvell,pxa3xx-nand[-controller]"
- compatible strings.
-
-Optional children nodes:
-Children nodes represent the available NAND chips.
-
-Required properties:
-- reg: shall contain the native Chip Select ids (0-3).
-- nand-rb: see nand-controller.yaml (0-1).
-
-Optional properties:
-- marvell,nand-keep-config: orders the driver not to take the timings
- from the core and leaving them completely untouched. Bootloader
- timings will then be used.
-- label: MTD name.
-- nand-on-flash-bbt: see nand-controller.yaml.
-- nand-ecc-mode: see nand-controller.yaml. Will use hardware ECC if not specified.
-- nand-ecc-algo: see nand-controller.yaml. This property is essentially useful when
- not using hardware ECC. Howerver, it may be added when using hardware
- ECC for clarification but will be ignored by the driver because ECC
- mode is chosen depending on the page size and the strength required by
- the NAND chip. This value may be overwritten with nand-ecc-strength
- property.
-- nand-ecc-strength: see nand-controller.yaml.
-- nand-ecc-step-size: see nand-controller.yaml. Marvell's NAND flash controller does
- use fixed strength (1-bit for Hamming, 16-bit for BCH), so the actual
- step size will shrink or grow in order to fit the required strength.
- Step sizes are not completely random for all and follow certain
- patterns described in AN-379, "Marvell SoC NFC ECC".
-
-See Documentation/devicetree/bindings/mtd/nand-controller.yaml for more details on
-generic bindings.
-
-
-Example:
-nand_controller: nand-controller@d0000 {
- compatible = "marvell,armada370-nand-controller";
- reg = <0xd0000 0x54>;
- #address-cells = <1>;
- #size-cells = <0>;
- interrupts = <GIC_SPI 84 IRQ_TYPE_LEVEL_HIGH>;
- clocks = <&coredivclk 0>;
-
- nand@0 {
- reg = <0>;
- label = "main-storage";
- nand-rb = <0>;
- nand-ecc-mode = "hw";
- marvell,nand-keep-config;
- nand-on-flash-bbt;
- nand-ecc-strength = <4>;
- nand-ecc-step-size = <512>;
-
- partitions {
- compatible = "fixed-partitions";
- #address-cells = <1>;
- #size-cells = <1>;
-
- partition@0 {
- label = "Rootfs";
- reg = <0x00000000 0x40000000>;
- };
- };
- };
-};
-
-
-Note on legacy bindings: One can find, in not-updated device trees,
-bindings slightly different than described above with other properties
-described below as well as the partitions node at the root of a so
-called "nand" node (without clear controller/chip separation).
-
-Legacy properties:
-- marvell,nand-enable-arbiter: To enable the arbiter, all boards blindly
- used it, this bit was set by the bootloader for many boards and even if
- it is marked reserved in several datasheets, it might be needed to set
- it (otherwise it is harmless) so whether or not this property is set,
- the bit is selected by the driver.
-- num-cs: Number of chip-select lines to use, all boards blindly set 1
- to this and for a reason, other values would have failed. The value of
- this property is ignored.
-
-Example:
-
- nand0: nand@43100000 {
- compatible = "marvell,pxa3xx-nand";
- reg = <0x43100000 90>;
- interrupts = <45>;
- dmas = <&pdma 97 0>;
- dma-names = "rxtx";
- #address-cells = <1>;
- marvell,nand-keep-config;
- marvell,nand-enable-arbiter;
- num-cs = <1>;
- /* Partitions (optional) */
- };
diff --git a/MAINTAINERS b/MAINTAINERS
index 5d463f171c90..5342798d4e4f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -12448,7 +12448,6 @@ MARVELL NAND CONTROLLER DRIVER
M: Miquel Raynal <miquel.raynal@bootlin.com>
L: linux-mtd@lists.infradead.org
S: Maintained
-F: Documentation/devicetree/bindings/mtd/marvell-nand.txt
F: drivers/mtd/nand/raw/marvell_nand.c
MARVELL OCTEONTX2 PHYSICAL FUNCTION DRIVER
--
2.40.1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-15 4:04 ` Chris Packham
0 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 4:04 UTC (permalink / raw)
To: miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel,
Vadym Kochan, Chris Packham
From: Vadym Kochan <vadym.kochan@plvision.eu>
Switch the DT binding to a YAML schema to enable the DT validation.
The text binding didn't mention it as a requirement but existing usage
has
compatible = "marvell,armada-8k-nand-controller",
"marvell,armada370-nand-controller";
so the YAML allows this in addition to the individual compatible values.
There was also an incorrect reference to dma-names being "rxtx" where
the driver and existing device trees actually use dma-names = "data" so
this is corrected in the conversion.
Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
---
Notes:
Changes in v9:
- depend on series from Miquel
https://lore.kernel.org/linux-mtd/20230606175246.190465-1-miquel.raynal@bootlin.com/
- enforce minimum/maximum for nand-rb
- move required: block for controller
- move unevaluatedProperties: for nand chip
- remove label, partitions and nand-on-flash-bbt which are covered by
generic schema
Changes in v8:
- Mark deprecated compatible values as such
- Allow "marvell,armada-8k-nand-controller" without
"marvell,armada370-nand-controller"
- Make dma-names usage reflect reality
- Update commit message
Changes in v7:
- Restore "label" and "partitions" properties (should be picked up via
nand-controller.yaml but aren't)
- Add/restore nand-on-flash-bbt and nand-ecc-mode which aren't covered
by nand-controller.yaml.
- Use "unevalautedProperties: false"
- Corrections for clock-names, dma-names, nand-rb and nand-ecc-strength
- Add pxa3xx-nand-controller example
Changes in v6:
- remove properties covered by nand-controller.yaml
- add example using armada-8k compatible
earlier changes:
v5:
1) Get back "label" and "partitions" properties but without
ref to the "partition.yaml" which was wrongly used.
2) Add "additionalProperties: false" for nand@ because all possible
properties are described.
v4:
1) Remove "label" and "partitions" properties
2) Use 2 clocks for A7K/8K platform which is a requirement
v3:
1) Remove txt version from the MAINTAINERS list
2) Use enum for some of compatible strings
3) Drop:
#address-cells
#size-cells:
as they are inherited from the nand-controller.yaml
4) Add restriction to use 2 clocks for A8K SoC
5) Dropped description for clock-names and extend it with
minItems: 1
6) Drop description for "dmas"
7) Use "unevalautedProperties: false"
8) Drop quites from yaml refs.
9) Use 4-space indentation for the example section
v2:
1) Fixed warning by yamllint with incorrect indentation for compatible list
.../bindings/mtd/marvell,nand-controller.yaml | 218 ++++++++++++++++++
.../devicetree/bindings/mtd/marvell-nand.txt | 126 ----------
MAINTAINERS | 1 -
3 files changed, 218 insertions(+), 127 deletions(-)
create mode 100644 Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
delete mode 100644 Documentation/devicetree/bindings/mtd/marvell-nand.txt
diff --git a/Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml b/Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
new file mode 100644
index 000000000000..252bd0f3e222
--- /dev/null
+++ b/Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
@@ -0,0 +1,218 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Marvell NAND Flash Controller (NFC)
+
+maintainers:
+ - Miquel Raynal <miquel.raynal@bootlin.com>
+
+properties:
+ compatible:
+ oneOf:
+ - items:
+ - const: marvell,armada-8k-nand-controller
+ - const: marvell,armada370-nand-controller
+ - enum:
+ - marvell,armada-8k-nand-controller
+ - marvell,armada370-nand-controller
+ - marvell,pxa3xx-nand-controller
+ - description: legacy bindings
+ deprecated: true
+ enum:
+ - marvell,armada-8k-nand
+ - marvell,armada370-nand
+ - marvell,pxa3xx-nand
+
+ reg:
+ maxItems: 1
+
+ interrupts:
+ maxItems: 1
+
+ clocks:
+ description:
+ Shall reference the NAND controller clocks, the second one is
+ is only needed for the Armada 7K/8K SoCs
+ minItems: 1
+ maxItems: 2
+
+ clock-names:
+ minItems: 1
+ items:
+ - const: core
+ - const: reg
+
+ dmas:
+ maxItems: 1
+
+ dma-names:
+ items:
+ - const: data
+
+ marvell,system-controller:
+ $ref: /schemas/types.yaml#/definitions/phandle
+ description: Syscon node that handles NAND controller related registers
+
+patternProperties:
+ "^nand@[a-f0-9]$":
+ type: object
+ $ref: raw-nand-chip.yaml
+
+ properties:
+ reg:
+ minimum: 0
+ maximum: 3
+
+ nand-rb:
+ items:
+ - minimum: 0
+ maximum: 1
+
+ nand-ecc-step-size:
+ const: 512
+
+ nand-ecc-strength:
+ enum: [1, 4, 8, 12, 16]
+
+ nand-ecc-mode:
+ const: hw
+
+ marvell,nand-keep-config:
+ $ref: /schemas/types.yaml#/definitions/flag
+ description: |
+ Orders the driver not to take the timings from the core and
+ leaving them completely untouched. Bootloader timings will then
+ be used.
+
+ marvell,nand-enable-arbiter:
+ $ref: /schemas/types.yaml#/definitions/flag
+ description: |
+ To enable the arbiter, all boards blindly used it,
+ this bit was set by the bootloader for many boards and even if
+ it is marked reserved in several datasheets, it might be needed to set
+ it (otherwise it is harmless).
+ deprecated: true
+
+ required:
+ - reg
+ - nand-rb
+
+ unevaluatedProperties: false
+
+required:
+ - compatible
+ - reg
+ - interrupts
+ - clocks
+
+allOf:
+ - $ref: nand-controller.yaml#
+
+ - if:
+ properties:
+ compatible:
+ contains:
+ const: marvell,pxa3xx-nand-controller
+ then:
+ required:
+ - dmas
+ - dma-names
+
+ - if:
+ properties:
+ compatible:
+ contains:
+ const: marvell,armada-8k-nand-controller
+ then:
+ properties:
+ clocks:
+ minItems: 2
+
+ clock-names:
+ minItems: 2
+
+ required:
+ - marvell,system-controller
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/interrupt-controller/arm-gic.h>
+ nand_controller: nand-controller@d0000 {
+ compatible = "marvell,armada370-nand-controller";
+ reg = <0xd0000 0x54>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ interrupts = <GIC_SPI 84 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&coredivclk 0>;
+
+ nand@0 {
+ reg = <0>;
+ label = "main-storage";
+ nand-rb = <0>;
+ nand-ecc-mode = "hw";
+ marvell,nand-keep-config;
+ nand-on-flash-bbt;
+ nand-ecc-strength = <4>;
+ nand-ecc-step-size = <512>;
+
+ partitions {
+ compatible = "fixed-partitions";
+ #address-cells = <1>;
+ #size-cells = <1>;
+
+ partition@0 {
+ label = "Rootfs";
+ reg = <0x00000000 0x40000000>;
+ };
+ };
+ };
+ };
+
+ - |
+ cp0_nand_controller: nand-controller@720000 {
+ compatible = "marvell,armada-8k-nand-controller",
+ "marvell,armada370-nand-controller";
+ reg = <0x720000 0x54>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ interrupts = <115 IRQ_TYPE_LEVEL_HIGH>;
+ clock-names = "core", "reg";
+ clocks = <&cp0_clk 1 2>,
+ <&cp0_clk 1 17>;
+ marvell,system-controller = <&cp0_syscon0>;
+
+ nand@0 {
+ reg = <0>;
+ label = "main-storage";
+ nand-rb = <0>;
+ nand-ecc-mode = "hw";
+ nand-ecc-strength = <8>;
+ nand-ecc-step-size = <512>;
+ };
+ };
+
+ - |
+ nand-controller@43100000 {
+ compatible = "marvell,pxa3xx-nand-controller";
+ reg = <0x43100000 90>;
+ interrupts = <45>;
+ clocks = <&clks 1>;
+ clock-names = "core";
+ dmas = <&pdma 97 3>;
+ dma-names = "data";
+ #address-cells = <1>;
+ #size-cells = <0>;
+ nand@0 {
+ reg = <0>;
+ nand-rb = <0>;
+ nand-ecc-mode = "hw";
+ marvell,nand-keep-config;
+ };
+ };
+
+...
diff --git a/Documentation/devicetree/bindings/mtd/marvell-nand.txt b/Documentation/devicetree/bindings/mtd/marvell-nand.txt
deleted file mode 100644
index a2d9a0f2b683..000000000000
--- a/Documentation/devicetree/bindings/mtd/marvell-nand.txt
+++ /dev/null
@@ -1,126 +0,0 @@
-Marvell NAND Flash Controller (NFC)
-
-Required properties:
-- compatible: can be one of the following:
- * "marvell,armada-8k-nand-controller"
- * "marvell,armada370-nand-controller"
- * "marvell,pxa3xx-nand-controller"
- * "marvell,armada-8k-nand" (deprecated)
- * "marvell,armada370-nand" (deprecated)
- * "marvell,pxa3xx-nand" (deprecated)
- Compatibles marked deprecated support only the old bindings described
- at the bottom.
-- reg: NAND flash controller memory area.
-- #address-cells: shall be set to 1. Encode the NAND CS.
-- #size-cells: shall be set to 0.
-- interrupts: shall define the NAND controller interrupt.
-- clocks: shall reference the NAND controller clocks, the second one is
- is only needed for the Armada 7K/8K SoCs
-- clock-names: mandatory if there is a second clock, in this case there
- should be one clock named "core" and another one named "reg"
-- marvell,system-controller: Set to retrieve the syscon node that handles
- NAND controller related registers (only required with the
- "marvell,armada-8k-nand[-controller]" compatibles).
-
-Optional properties:
-- label: see partition.txt. New platforms shall omit this property.
-- dmas: shall reference DMA channel associated to the NAND controller.
- This property is only used with "marvell,pxa3xx-nand[-controller]"
- compatible strings.
-- dma-names: shall be "rxtx".
- This property is only used with "marvell,pxa3xx-nand[-controller]"
- compatible strings.
-
-Optional children nodes:
-Children nodes represent the available NAND chips.
-
-Required properties:
-- reg: shall contain the native Chip Select ids (0-3).
-- nand-rb: see nand-controller.yaml (0-1).
-
-Optional properties:
-- marvell,nand-keep-config: orders the driver not to take the timings
- from the core and leaving them completely untouched. Bootloader
- timings will then be used.
-- label: MTD name.
-- nand-on-flash-bbt: see nand-controller.yaml.
-- nand-ecc-mode: see nand-controller.yaml. Will use hardware ECC if not specified.
-- nand-ecc-algo: see nand-controller.yaml. This property is essentially useful when
- not using hardware ECC. Howerver, it may be added when using hardware
- ECC for clarification but will be ignored by the driver because ECC
- mode is chosen depending on the page size and the strength required by
- the NAND chip. This value may be overwritten with nand-ecc-strength
- property.
-- nand-ecc-strength: see nand-controller.yaml.
-- nand-ecc-step-size: see nand-controller.yaml. Marvell's NAND flash controller does
- use fixed strength (1-bit for Hamming, 16-bit for BCH), so the actual
- step size will shrink or grow in order to fit the required strength.
- Step sizes are not completely random for all and follow certain
- patterns described in AN-379, "Marvell SoC NFC ECC".
-
-See Documentation/devicetree/bindings/mtd/nand-controller.yaml for more details on
-generic bindings.
-
-
-Example:
-nand_controller: nand-controller@d0000 {
- compatible = "marvell,armada370-nand-controller";
- reg = <0xd0000 0x54>;
- #address-cells = <1>;
- #size-cells = <0>;
- interrupts = <GIC_SPI 84 IRQ_TYPE_LEVEL_HIGH>;
- clocks = <&coredivclk 0>;
-
- nand@0 {
- reg = <0>;
- label = "main-storage";
- nand-rb = <0>;
- nand-ecc-mode = "hw";
- marvell,nand-keep-config;
- nand-on-flash-bbt;
- nand-ecc-strength = <4>;
- nand-ecc-step-size = <512>;
-
- partitions {
- compatible = "fixed-partitions";
- #address-cells = <1>;
- #size-cells = <1>;
-
- partition@0 {
- label = "Rootfs";
- reg = <0x00000000 0x40000000>;
- };
- };
- };
-};
-
-
-Note on legacy bindings: One can find, in not-updated device trees,
-bindings slightly different than described above with other properties
-described below as well as the partitions node at the root of a so
-called "nand" node (without clear controller/chip separation).
-
-Legacy properties:
-- marvell,nand-enable-arbiter: To enable the arbiter, all boards blindly
- used it, this bit was set by the bootloader for many boards and even if
- it is marked reserved in several datasheets, it might be needed to set
- it (otherwise it is harmless) so whether or not this property is set,
- the bit is selected by the driver.
-- num-cs: Number of chip-select lines to use, all boards blindly set 1
- to this and for a reason, other values would have failed. The value of
- this property is ignored.
-
-Example:
-
- nand0: nand@43100000 {
- compatible = "marvell,pxa3xx-nand";
- reg = <0x43100000 90>;
- interrupts = <45>;
- dmas = <&pdma 97 0>;
- dma-names = "rxtx";
- #address-cells = <1>;
- marvell,nand-keep-config;
- marvell,nand-enable-arbiter;
- num-cs = <1>;
- /* Partitions (optional) */
- };
diff --git a/MAINTAINERS b/MAINTAINERS
index 5d463f171c90..5342798d4e4f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -12448,7 +12448,6 @@ MARVELL NAND CONTROLLER DRIVER
M: Miquel Raynal <miquel.raynal@bootlin.com>
L: linux-mtd@lists.infradead.org
S: Maintained
-F: Documentation/devicetree/bindings/mtd/marvell-nand.txt
F: drivers/mtd/nand/raw/marvell_nand.c
MARVELL OCTEONTX2 PHYSICAL FUNCTION DRIVER
--
2.40.1
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-15 4:04 ` Chris Packham
0 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 4:04 UTC (permalink / raw)
To: miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel,
Vadym Kochan, Chris Packham
From: Vadym Kochan <vadym.kochan@plvision.eu>
Switch the DT binding to a YAML schema to enable the DT validation.
The text binding didn't mention it as a requirement but existing usage
has
compatible = "marvell,armada-8k-nand-controller",
"marvell,armada370-nand-controller";
so the YAML allows this in addition to the individual compatible values.
There was also an incorrect reference to dma-names being "rxtx" where
the driver and existing device trees actually use dma-names = "data" so
this is corrected in the conversion.
Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
---
Notes:
Changes in v9:
- depend on series from Miquel
https://lore.kernel.org/linux-mtd/20230606175246.190465-1-miquel.raynal@bootlin.com/
- enforce minimum/maximum for nand-rb
- move required: block for controller
- move unevaluatedProperties: for nand chip
- remove label, partitions and nand-on-flash-bbt which are covered by
generic schema
Changes in v8:
- Mark deprecated compatible values as such
- Allow "marvell,armada-8k-nand-controller" without
"marvell,armada370-nand-controller"
- Make dma-names usage reflect reality
- Update commit message
Changes in v7:
- Restore "label" and "partitions" properties (should be picked up via
nand-controller.yaml but aren't)
- Add/restore nand-on-flash-bbt and nand-ecc-mode which aren't covered
by nand-controller.yaml.
- Use "unevalautedProperties: false"
- Corrections for clock-names, dma-names, nand-rb and nand-ecc-strength
- Add pxa3xx-nand-controller example
Changes in v6:
- remove properties covered by nand-controller.yaml
- add example using armada-8k compatible
earlier changes:
v5:
1) Get back "label" and "partitions" properties but without
ref to the "partition.yaml" which was wrongly used.
2) Add "additionalProperties: false" for nand@ because all possible
properties are described.
v4:
1) Remove "label" and "partitions" properties
2) Use 2 clocks for A7K/8K platform which is a requirement
v3:
1) Remove txt version from the MAINTAINERS list
2) Use enum for some of compatible strings
3) Drop:
#address-cells
#size-cells:
as they are inherited from the nand-controller.yaml
4) Add restriction to use 2 clocks for A8K SoC
5) Dropped description for clock-names and extend it with
minItems: 1
6) Drop description for "dmas"
7) Use "unevalautedProperties: false"
8) Drop quites from yaml refs.
9) Use 4-space indentation for the example section
v2:
1) Fixed warning by yamllint with incorrect indentation for compatible list
.../bindings/mtd/marvell,nand-controller.yaml | 218 ++++++++++++++++++
.../devicetree/bindings/mtd/marvell-nand.txt | 126 ----------
MAINTAINERS | 1 -
3 files changed, 218 insertions(+), 127 deletions(-)
create mode 100644 Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
delete mode 100644 Documentation/devicetree/bindings/mtd/marvell-nand.txt
diff --git a/Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml b/Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
new file mode 100644
index 000000000000..252bd0f3e222
--- /dev/null
+++ b/Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
@@ -0,0 +1,218 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Marvell NAND Flash Controller (NFC)
+
+maintainers:
+ - Miquel Raynal <miquel.raynal@bootlin.com>
+
+properties:
+ compatible:
+ oneOf:
+ - items:
+ - const: marvell,armada-8k-nand-controller
+ - const: marvell,armada370-nand-controller
+ - enum:
+ - marvell,armada-8k-nand-controller
+ - marvell,armada370-nand-controller
+ - marvell,pxa3xx-nand-controller
+ - description: legacy bindings
+ deprecated: true
+ enum:
+ - marvell,armada-8k-nand
+ - marvell,armada370-nand
+ - marvell,pxa3xx-nand
+
+ reg:
+ maxItems: 1
+
+ interrupts:
+ maxItems: 1
+
+ clocks:
+ description:
+ Shall reference the NAND controller clocks, the second one is
+ is only needed for the Armada 7K/8K SoCs
+ minItems: 1
+ maxItems: 2
+
+ clock-names:
+ minItems: 1
+ items:
+ - const: core
+ - const: reg
+
+ dmas:
+ maxItems: 1
+
+ dma-names:
+ items:
+ - const: data
+
+ marvell,system-controller:
+ $ref: /schemas/types.yaml#/definitions/phandle
+ description: Syscon node that handles NAND controller related registers
+
+patternProperties:
+ "^nand@[a-f0-9]$":
+ type: object
+ $ref: raw-nand-chip.yaml
+
+ properties:
+ reg:
+ minimum: 0
+ maximum: 3
+
+ nand-rb:
+ items:
+ - minimum: 0
+ maximum: 1
+
+ nand-ecc-step-size:
+ const: 512
+
+ nand-ecc-strength:
+ enum: [1, 4, 8, 12, 16]
+
+ nand-ecc-mode:
+ const: hw
+
+ marvell,nand-keep-config:
+ $ref: /schemas/types.yaml#/definitions/flag
+ description: |
+ Orders the driver not to take the timings from the core and
+ leaving them completely untouched. Bootloader timings will then
+ be used.
+
+ marvell,nand-enable-arbiter:
+ $ref: /schemas/types.yaml#/definitions/flag
+ description: |
+ To enable the arbiter, all boards blindly used it,
+ this bit was set by the bootloader for many boards and even if
+ it is marked reserved in several datasheets, it might be needed to set
+ it (otherwise it is harmless).
+ deprecated: true
+
+ required:
+ - reg
+ - nand-rb
+
+ unevaluatedProperties: false
+
+required:
+ - compatible
+ - reg
+ - interrupts
+ - clocks
+
+allOf:
+ - $ref: nand-controller.yaml#
+
+ - if:
+ properties:
+ compatible:
+ contains:
+ const: marvell,pxa3xx-nand-controller
+ then:
+ required:
+ - dmas
+ - dma-names
+
+ - if:
+ properties:
+ compatible:
+ contains:
+ const: marvell,armada-8k-nand-controller
+ then:
+ properties:
+ clocks:
+ minItems: 2
+
+ clock-names:
+ minItems: 2
+
+ required:
+ - marvell,system-controller
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/interrupt-controller/arm-gic.h>
+ nand_controller: nand-controller@d0000 {
+ compatible = "marvell,armada370-nand-controller";
+ reg = <0xd0000 0x54>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ interrupts = <GIC_SPI 84 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&coredivclk 0>;
+
+ nand@0 {
+ reg = <0>;
+ label = "main-storage";
+ nand-rb = <0>;
+ nand-ecc-mode = "hw";
+ marvell,nand-keep-config;
+ nand-on-flash-bbt;
+ nand-ecc-strength = <4>;
+ nand-ecc-step-size = <512>;
+
+ partitions {
+ compatible = "fixed-partitions";
+ #address-cells = <1>;
+ #size-cells = <1>;
+
+ partition@0 {
+ label = "Rootfs";
+ reg = <0x00000000 0x40000000>;
+ };
+ };
+ };
+ };
+
+ - |
+ cp0_nand_controller: nand-controller@720000 {
+ compatible = "marvell,armada-8k-nand-controller",
+ "marvell,armada370-nand-controller";
+ reg = <0x720000 0x54>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ interrupts = <115 IRQ_TYPE_LEVEL_HIGH>;
+ clock-names = "core", "reg";
+ clocks = <&cp0_clk 1 2>,
+ <&cp0_clk 1 17>;
+ marvell,system-controller = <&cp0_syscon0>;
+
+ nand@0 {
+ reg = <0>;
+ label = "main-storage";
+ nand-rb = <0>;
+ nand-ecc-mode = "hw";
+ nand-ecc-strength = <8>;
+ nand-ecc-step-size = <512>;
+ };
+ };
+
+ - |
+ nand-controller@43100000 {
+ compatible = "marvell,pxa3xx-nand-controller";
+ reg = <0x43100000 90>;
+ interrupts = <45>;
+ clocks = <&clks 1>;
+ clock-names = "core";
+ dmas = <&pdma 97 3>;
+ dma-names = "data";
+ #address-cells = <1>;
+ #size-cells = <0>;
+ nand@0 {
+ reg = <0>;
+ nand-rb = <0>;
+ nand-ecc-mode = "hw";
+ marvell,nand-keep-config;
+ };
+ };
+
+...
diff --git a/Documentation/devicetree/bindings/mtd/marvell-nand.txt b/Documentation/devicetree/bindings/mtd/marvell-nand.txt
deleted file mode 100644
index a2d9a0f2b683..000000000000
--- a/Documentation/devicetree/bindings/mtd/marvell-nand.txt
+++ /dev/null
@@ -1,126 +0,0 @@
-Marvell NAND Flash Controller (NFC)
-
-Required properties:
-- compatible: can be one of the following:
- * "marvell,armada-8k-nand-controller"
- * "marvell,armada370-nand-controller"
- * "marvell,pxa3xx-nand-controller"
- * "marvell,armada-8k-nand" (deprecated)
- * "marvell,armada370-nand" (deprecated)
- * "marvell,pxa3xx-nand" (deprecated)
- Compatibles marked deprecated support only the old bindings described
- at the bottom.
-- reg: NAND flash controller memory area.
-- #address-cells: shall be set to 1. Encode the NAND CS.
-- #size-cells: shall be set to 0.
-- interrupts: shall define the NAND controller interrupt.
-- clocks: shall reference the NAND controller clocks, the second one is
- is only needed for the Armada 7K/8K SoCs
-- clock-names: mandatory if there is a second clock, in this case there
- should be one clock named "core" and another one named "reg"
-- marvell,system-controller: Set to retrieve the syscon node that handles
- NAND controller related registers (only required with the
- "marvell,armada-8k-nand[-controller]" compatibles).
-
-Optional properties:
-- label: see partition.txt. New platforms shall omit this property.
-- dmas: shall reference DMA channel associated to the NAND controller.
- This property is only used with "marvell,pxa3xx-nand[-controller]"
- compatible strings.
-- dma-names: shall be "rxtx".
- This property is only used with "marvell,pxa3xx-nand[-controller]"
- compatible strings.
-
-Optional children nodes:
-Children nodes represent the available NAND chips.
-
-Required properties:
-- reg: shall contain the native Chip Select ids (0-3).
-- nand-rb: see nand-controller.yaml (0-1).
-
-Optional properties:
-- marvell,nand-keep-config: orders the driver not to take the timings
- from the core and leaving them completely untouched. Bootloader
- timings will then be used.
-- label: MTD name.
-- nand-on-flash-bbt: see nand-controller.yaml.
-- nand-ecc-mode: see nand-controller.yaml. Will use hardware ECC if not specified.
-- nand-ecc-algo: see nand-controller.yaml. This property is essentially useful when
- not using hardware ECC. Howerver, it may be added when using hardware
- ECC for clarification but will be ignored by the driver because ECC
- mode is chosen depending on the page size and the strength required by
- the NAND chip. This value may be overwritten with nand-ecc-strength
- property.
-- nand-ecc-strength: see nand-controller.yaml.
-- nand-ecc-step-size: see nand-controller.yaml. Marvell's NAND flash controller does
- use fixed strength (1-bit for Hamming, 16-bit for BCH), so the actual
- step size will shrink or grow in order to fit the required strength.
- Step sizes are not completely random for all and follow certain
- patterns described in AN-379, "Marvell SoC NFC ECC".
-
-See Documentation/devicetree/bindings/mtd/nand-controller.yaml for more details on
-generic bindings.
-
-
-Example:
-nand_controller: nand-controller@d0000 {
- compatible = "marvell,armada370-nand-controller";
- reg = <0xd0000 0x54>;
- #address-cells = <1>;
- #size-cells = <0>;
- interrupts = <GIC_SPI 84 IRQ_TYPE_LEVEL_HIGH>;
- clocks = <&coredivclk 0>;
-
- nand@0 {
- reg = <0>;
- label = "main-storage";
- nand-rb = <0>;
- nand-ecc-mode = "hw";
- marvell,nand-keep-config;
- nand-on-flash-bbt;
- nand-ecc-strength = <4>;
- nand-ecc-step-size = <512>;
-
- partitions {
- compatible = "fixed-partitions";
- #address-cells = <1>;
- #size-cells = <1>;
-
- partition@0 {
- label = "Rootfs";
- reg = <0x00000000 0x40000000>;
- };
- };
- };
-};
-
-
-Note on legacy bindings: One can find, in not-updated device trees,
-bindings slightly different than described above with other properties
-described below as well as the partitions node at the root of a so
-called "nand" node (without clear controller/chip separation).
-
-Legacy properties:
-- marvell,nand-enable-arbiter: To enable the arbiter, all boards blindly
- used it, this bit was set by the bootloader for many boards and even if
- it is marked reserved in several datasheets, it might be needed to set
- it (otherwise it is harmless) so whether or not this property is set,
- the bit is selected by the driver.
-- num-cs: Number of chip-select lines to use, all boards blindly set 1
- to this and for a reason, other values would have failed. The value of
- this property is ignored.
-
-Example:
-
- nand0: nand@43100000 {
- compatible = "marvell,pxa3xx-nand";
- reg = <0x43100000 90>;
- interrupts = <45>;
- dmas = <&pdma 97 0>;
- dma-names = "rxtx";
- #address-cells = <1>;
- marvell,nand-keep-config;
- marvell,nand-enable-arbiter;
- num-cs = <1>;
- /* Partitions (optional) */
- };
diff --git a/MAINTAINERS b/MAINTAINERS
index 5d463f171c90..5342798d4e4f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -12448,7 +12448,6 @@ MARVELL NAND CONTROLLER DRIVER
M: Miquel Raynal <miquel.raynal@bootlin.com>
L: linux-mtd@lists.infradead.org
S: Maintained
-F: Documentation/devicetree/bindings/mtd/marvell-nand.txt
F: drivers/mtd/nand/raw/marvell_nand.c
MARVELL OCTEONTX2 PHYSICAL FUNCTION DRIVER
--
2.40.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] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
2023-06-15 4:04 ` Chris Packham
(?)
@ 2023-06-15 5:25 ` Rob Herring
-1 siblings, 0 replies; 39+ messages in thread
From: Rob Herring @ 2023-06-15 5:25 UTC (permalink / raw)
To: Chris Packham
Cc: linux-mtd, linux-kernel, robh+dt, vigneshr, miquel.raynal,
krzysztof.kozlowski+dt, richard, Vadym Kochan, linux-arm-kernel,
gregory.clement, andrew, sebastian.hesselbarth, devicetree
On Thu, 15 Jun 2023 16:04:47 +1200, Chris Packham wrote:
> From: Vadym Kochan <vadym.kochan@plvision.eu>
>
> Switch the DT binding to a YAML schema to enable the DT validation.
>
> The text binding didn't mention it as a requirement but existing usage
> has
>
> compatible = "marvell,armada-8k-nand-controller",
> "marvell,armada370-nand-controller";
>
> so the YAML allows this in addition to the individual compatible values.
>
> There was also an incorrect reference to dma-names being "rxtx" where
> the driver and existing device trees actually use dma-names = "data" so
> this is corrected in the conversion.
>
> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> ---
>
> Notes:
> Changes in v9:
> - depend on series from Miquel
> https://lore.kernel.org/linux-mtd/20230606175246.190465-1-miquel.raynal@bootlin.com/
> - enforce minimum/maximum for nand-rb
> - move required: block for controller
> - move unevaluatedProperties: for nand chip
> - remove label, partitions and nand-on-flash-bbt which are covered by
> generic schema
>
> Changes in v8:
> - Mark deprecated compatible values as such
> - Allow "marvell,armada-8k-nand-controller" without
> "marvell,armada370-nand-controller"
> - Make dma-names usage reflect reality
> - Update commit message
>
> Changes in v7:
> - Restore "label" and "partitions" properties (should be picked up via
> nand-controller.yaml but aren't)
> - Add/restore nand-on-flash-bbt and nand-ecc-mode which aren't covered
> by nand-controller.yaml.
> - Use "unevalautedProperties: false"
> - Corrections for clock-names, dma-names, nand-rb and nand-ecc-strength
> - Add pxa3xx-nand-controller example
>
> Changes in v6:
> - remove properties covered by nand-controller.yaml
> - add example using armada-8k compatible
>
> earlier changes:
>
> v5:
> 1) Get back "label" and "partitions" properties but without
> ref to the "partition.yaml" which was wrongly used.
>
> 2) Add "additionalProperties: false" for nand@ because all possible
> properties are described.
>
> v4:
> 1) Remove "label" and "partitions" properties
>
> 2) Use 2 clocks for A7K/8K platform which is a requirement
>
> v3:
> 1) Remove txt version from the MAINTAINERS list
>
> 2) Use enum for some of compatible strings
>
> 3) Drop:
> #address-cells
> #size-cells:
>
> as they are inherited from the nand-controller.yaml
>
> 4) Add restriction to use 2 clocks for A8K SoC
>
> 5) Dropped description for clock-names and extend it with
> minItems: 1
>
> 6) Drop description for "dmas"
>
> 7) Use "unevalautedProperties: false"
>
> 8) Drop quites from yaml refs.
>
> 9) Use 4-space indentation for the example section
>
> v2:
> 1) Fixed warning by yamllint with incorrect indentation for compatible list
>
> .../bindings/mtd/marvell,nand-controller.yaml | 218 ++++++++++++++++++
> .../devicetree/bindings/mtd/marvell-nand.txt | 126 ----------
> MAINTAINERS | 1 -
> 3 files changed, 218 insertions(+), 127 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
> delete mode 100644 Documentation/devicetree/bindings/mtd/marvell-nand.txt
>
My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
on your patch (DT_CHECKER_FLAGS is new in v5.13):
yamllint warnings/errors:
dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml: Error in referenced schema matching $id: http://devicetree.org/schemas/mtd/raw-nand-chip.yaml
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@d0000: nand@0: False schema does not allow {'reg': [[0]], 'label': ['main-storage'], 'nand-rb': [[0]], 'nand-ecc-mode': ['hw'], 'marvell,nand-keep-config': True, 'nand-on-flash-bbt': True, 'nand-ecc-strength': [[4]], 'nand-ecc-step-size': [[512]], 'partitions': {'compatible': ['fixed-partitions'], '#address-cells': [[1]], '#size-cells': [[1]], 'partition@0': {'label': ['Rootfs'], 'reg': [[0, 1073741824]]}}}
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@d0000: nand@0: Unevaluated properties are not allowed ('label', 'nand-on-flash-bbt', 'partitions' were unexpected)
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@720000: nand@0: False schema does not allow {'reg': [[0]], 'label': ['main-storage'], 'nand-rb': [[0]], 'nand-ecc-mode': ['hw'], 'nand-ecc-strength': [[8]], 'nand-ecc-step-size': [[512]]}
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@720000: nand@0: Unevaluated properties are not allowed ('label' was unexpected)
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@43100000: nand@0: False schema does not allow {'reg': [[0]], 'nand-rb': [[0]], 'nand-ecc-mode': ['hw'], 'marvell,nand-keep-config': True}
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20230615040447.3484564-4-chris.packham@alliedtelesis.co.nz
The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:
pip3 install dtschema --upgrade
Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-15 5:25 ` Rob Herring
0 siblings, 0 replies; 39+ messages in thread
From: Rob Herring @ 2023-06-15 5:25 UTC (permalink / raw)
To: Chris Packham
Cc: linux-mtd, linux-kernel, robh+dt, vigneshr, miquel.raynal,
krzysztof.kozlowski+dt, richard, Vadym Kochan, linux-arm-kernel,
gregory.clement, andrew, sebastian.hesselbarth, devicetree
On Thu, 15 Jun 2023 16:04:47 +1200, Chris Packham wrote:
> From: Vadym Kochan <vadym.kochan@plvision.eu>
>
> Switch the DT binding to a YAML schema to enable the DT validation.
>
> The text binding didn't mention it as a requirement but existing usage
> has
>
> compatible = "marvell,armada-8k-nand-controller",
> "marvell,armada370-nand-controller";
>
> so the YAML allows this in addition to the individual compatible values.
>
> There was also an incorrect reference to dma-names being "rxtx" where
> the driver and existing device trees actually use dma-names = "data" so
> this is corrected in the conversion.
>
> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> ---
>
> Notes:
> Changes in v9:
> - depend on series from Miquel
> https://lore.kernel.org/linux-mtd/20230606175246.190465-1-miquel.raynal@bootlin.com/
> - enforce minimum/maximum for nand-rb
> - move required: block for controller
> - move unevaluatedProperties: for nand chip
> - remove label, partitions and nand-on-flash-bbt which are covered by
> generic schema
>
> Changes in v8:
> - Mark deprecated compatible values as such
> - Allow "marvell,armada-8k-nand-controller" without
> "marvell,armada370-nand-controller"
> - Make dma-names usage reflect reality
> - Update commit message
>
> Changes in v7:
> - Restore "label" and "partitions" properties (should be picked up via
> nand-controller.yaml but aren't)
> - Add/restore nand-on-flash-bbt and nand-ecc-mode which aren't covered
> by nand-controller.yaml.
> - Use "unevalautedProperties: false"
> - Corrections for clock-names, dma-names, nand-rb and nand-ecc-strength
> - Add pxa3xx-nand-controller example
>
> Changes in v6:
> - remove properties covered by nand-controller.yaml
> - add example using armada-8k compatible
>
> earlier changes:
>
> v5:
> 1) Get back "label" and "partitions" properties but without
> ref to the "partition.yaml" which was wrongly used.
>
> 2) Add "additionalProperties: false" for nand@ because all possible
> properties are described.
>
> v4:
> 1) Remove "label" and "partitions" properties
>
> 2) Use 2 clocks for A7K/8K platform which is a requirement
>
> v3:
> 1) Remove txt version from the MAINTAINERS list
>
> 2) Use enum for some of compatible strings
>
> 3) Drop:
> #address-cells
> #size-cells:
>
> as they are inherited from the nand-controller.yaml
>
> 4) Add restriction to use 2 clocks for A8K SoC
>
> 5) Dropped description for clock-names and extend it with
> minItems: 1
>
> 6) Drop description for "dmas"
>
> 7) Use "unevalautedProperties: false"
>
> 8) Drop quites from yaml refs.
>
> 9) Use 4-space indentation for the example section
>
> v2:
> 1) Fixed warning by yamllint with incorrect indentation for compatible list
>
> .../bindings/mtd/marvell,nand-controller.yaml | 218 ++++++++++++++++++
> .../devicetree/bindings/mtd/marvell-nand.txt | 126 ----------
> MAINTAINERS | 1 -
> 3 files changed, 218 insertions(+), 127 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
> delete mode 100644 Documentation/devicetree/bindings/mtd/marvell-nand.txt
>
My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
on your patch (DT_CHECKER_FLAGS is new in v5.13):
yamllint warnings/errors:
dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml: Error in referenced schema matching $id: http://devicetree.org/schemas/mtd/raw-nand-chip.yaml
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@d0000: nand@0: False schema does not allow {'reg': [[0]], 'label': ['main-storage'], 'nand-rb': [[0]], 'nand-ecc-mode': ['hw'], 'marvell,nand-keep-config': True, 'nand-on-flash-bbt': True, 'nand-ecc-strength': [[4]], 'nand-ecc-step-size': [[512]], 'partitions': {'compatible': ['fixed-partitions'], '#address-cells': [[1]], '#size-cells': [[1]], 'partition@0': {'label': ['Rootfs'], 'reg': [[0, 1073741824]]}}}
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@d0000: nand@0: Unevaluated properties are not allowed ('label', 'nand-on-flash-bbt', 'partitions' were unexpected)
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@720000: nand@0: False schema does not allow {'reg': [[0]], 'label': ['main-storage'], 'nand-rb': [[0]], 'nand-ecc-mode': ['hw'], 'nand-ecc-strength': [[8]], 'nand-ecc-step-size': [[512]]}
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@720000: nand@0: Unevaluated properties are not allowed ('label' was unexpected)
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@43100000: nand@0: False schema does not allow {'reg': [[0]], 'nand-rb': [[0]], 'nand-ecc-mode': ['hw'], 'marvell,nand-keep-config': True}
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20230615040447.3484564-4-chris.packham@alliedtelesis.co.nz
The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:
pip3 install dtschema --upgrade
Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-15 5:25 ` Rob Herring
0 siblings, 0 replies; 39+ messages in thread
From: Rob Herring @ 2023-06-15 5:25 UTC (permalink / raw)
To: Chris Packham
Cc: linux-mtd, linux-kernel, robh+dt, vigneshr, miquel.raynal,
krzysztof.kozlowski+dt, richard, Vadym Kochan, linux-arm-kernel,
gregory.clement, andrew, sebastian.hesselbarth, devicetree
On Thu, 15 Jun 2023 16:04:47 +1200, Chris Packham wrote:
> From: Vadym Kochan <vadym.kochan@plvision.eu>
>
> Switch the DT binding to a YAML schema to enable the DT validation.
>
> The text binding didn't mention it as a requirement but existing usage
> has
>
> compatible = "marvell,armada-8k-nand-controller",
> "marvell,armada370-nand-controller";
>
> so the YAML allows this in addition to the individual compatible values.
>
> There was also an incorrect reference to dma-names being "rxtx" where
> the driver and existing device trees actually use dma-names = "data" so
> this is corrected in the conversion.
>
> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> ---
>
> Notes:
> Changes in v9:
> - depend on series from Miquel
> https://lore.kernel.org/linux-mtd/20230606175246.190465-1-miquel.raynal@bootlin.com/
> - enforce minimum/maximum for nand-rb
> - move required: block for controller
> - move unevaluatedProperties: for nand chip
> - remove label, partitions and nand-on-flash-bbt which are covered by
> generic schema
>
> Changes in v8:
> - Mark deprecated compatible values as such
> - Allow "marvell,armada-8k-nand-controller" without
> "marvell,armada370-nand-controller"
> - Make dma-names usage reflect reality
> - Update commit message
>
> Changes in v7:
> - Restore "label" and "partitions" properties (should be picked up via
> nand-controller.yaml but aren't)
> - Add/restore nand-on-flash-bbt and nand-ecc-mode which aren't covered
> by nand-controller.yaml.
> - Use "unevalautedProperties: false"
> - Corrections for clock-names, dma-names, nand-rb and nand-ecc-strength
> - Add pxa3xx-nand-controller example
>
> Changes in v6:
> - remove properties covered by nand-controller.yaml
> - add example using armada-8k compatible
>
> earlier changes:
>
> v5:
> 1) Get back "label" and "partitions" properties but without
> ref to the "partition.yaml" which was wrongly used.
>
> 2) Add "additionalProperties: false" for nand@ because all possible
> properties are described.
>
> v4:
> 1) Remove "label" and "partitions" properties
>
> 2) Use 2 clocks for A7K/8K platform which is a requirement
>
> v3:
> 1) Remove txt version from the MAINTAINERS list
>
> 2) Use enum for some of compatible strings
>
> 3) Drop:
> #address-cells
> #size-cells:
>
> as they are inherited from the nand-controller.yaml
>
> 4) Add restriction to use 2 clocks for A8K SoC
>
> 5) Dropped description for clock-names and extend it with
> minItems: 1
>
> 6) Drop description for "dmas"
>
> 7) Use "unevalautedProperties: false"
>
> 8) Drop quites from yaml refs.
>
> 9) Use 4-space indentation for the example section
>
> v2:
> 1) Fixed warning by yamllint with incorrect indentation for compatible list
>
> .../bindings/mtd/marvell,nand-controller.yaml | 218 ++++++++++++++++++
> .../devicetree/bindings/mtd/marvell-nand.txt | 126 ----------
> MAINTAINERS | 1 -
> 3 files changed, 218 insertions(+), 127 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
> delete mode 100644 Documentation/devicetree/bindings/mtd/marvell-nand.txt
>
My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
on your patch (DT_CHECKER_FLAGS is new in v5.13):
yamllint warnings/errors:
dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml: Error in referenced schema matching $id: http://devicetree.org/schemas/mtd/raw-nand-chip.yaml
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@d0000: nand@0: False schema does not allow {'reg': [[0]], 'label': ['main-storage'], 'nand-rb': [[0]], 'nand-ecc-mode': ['hw'], 'marvell,nand-keep-config': True, 'nand-on-flash-bbt': True, 'nand-ecc-strength': [[4]], 'nand-ecc-step-size': [[512]], 'partitions': {'compatible': ['fixed-partitions'], '#address-cells': [[1]], '#size-cells': [[1]], 'partition@0': {'label': ['Rootfs'], 'reg': [[0, 1073741824]]}}}
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@d0000: nand@0: Unevaluated properties are not allowed ('label', 'nand-on-flash-bbt', 'partitions' were unexpected)
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@720000: nand@0: False schema does not allow {'reg': [[0]], 'label': ['main-storage'], 'nand-rb': [[0]], 'nand-ecc-mode': ['hw'], 'nand-ecc-strength': [[8]], 'nand-ecc-step-size': [[512]]}
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@720000: nand@0: Unevaluated properties are not allowed ('label' was unexpected)
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mtd/marvell,nand-controller.example.dtb: nand-controller@43100000: nand@0: False schema does not allow {'reg': [[0]], 'nand-rb': [[0]], 'nand-ecc-mode': ['hw'], 'marvell,nand-keep-config': True}
from schema $id: http://devicetree.org/schemas/mtd/marvell,nand-controller.yaml#
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20230615040447.3484564-4-chris.packham@alliedtelesis.co.nz
The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:
pip3 install dtschema --upgrade
Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.
_______________________________________________
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] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
2023-06-15 4:04 ` Chris Packham
(?)
@ 2023-06-15 7:45 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 39+ messages in thread
From: Krzysztof Kozlowski @ 2023-06-15 7:45 UTC (permalink / raw)
To: Chris Packham, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Vadym Kochan
On 15/06/2023 06:04, Chris Packham wrote:
> From: Vadym Kochan <vadym.kochan@plvision.eu>
>
> Switch the DT binding to a YAML schema to enable the DT validation.
>
> The text binding didn't mention it as a requirement but existing usage
> has
>
> compatible = "marvell,armada-8k-nand-controller",
> "marvell,armada370-nand-controller";
>
> so the YAML allows this in addition to the individual compatible values.
>
> There was also an incorrect reference to dma-names being "rxtx" where
> the driver and existing device trees actually use dma-names = "data" so
> this is corrected in the conversion.
>
> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> ---
>
> Notes:
> Changes in v9:
> - depend on series from Miquel
> https://lore.kernel.org/linux-mtd/20230606175246.190465-1-miquel.raynal@bootlin.com/
> - enforce minimum/maximum for nand-rb
> - move required: block for controller
> - move unevaluatedProperties: for nand chip
> - remove label, partitions and nand-on-flash-bbt which are covered by
> generic schema
>
> Changes in v8:
> - Mark deprecated compatible values as such
> - Allow "marvell,armada-8k-nand-controller" without
> "marvell,armada370-nand-controller"
???
> +
> +properties:
> + compatible:
> + oneOf:
> + - items:
> + - const: marvell,armada-8k-nand-controller
> + - const: marvell,armada370-nand-controller
> + - enum:
> + - marvell,armada-8k-nand-controller
This is wrong. 8k cannot be both: compatible and not compatible with
370. It's not someone's cat to be in both states at the same time...
> + - marvell,armada370-nand-controller
> + - marvell,pxa3xx-nand-controller
> + - description: legacy bindings
> + deprecated: true
> + enum:
> + - marvell,armada-8k-nand
> + - marvell,armada370-nand
> + - marvell,pxa3xx-nand
> +
> + reg:
> + maxItems: 1
> +
> + interrupts:
> + maxItems: 1
> +
> + clocks:
> + description:
> + Shall reference the NAND controller clocks, the second one is
> + is only needed for the Armada 7K/8K SoCs
> + minItems: 1
> + maxItems: 2
> +
> + clock-names:
> + minItems: 1
> + items:
> + - const: core
> + - const: reg
> +
> + dmas:
> + maxItems: 1
> +
> + dma-names:
> + items:
> + - const: data
> +
> + marvell,system-controller:
> + $ref: /schemas/types.yaml#/definitions/phandle
> + description: Syscon node that handles NAND controller related registers
> +
> +patternProperties:
> + "^nand@[a-f0-9]$":
> + type: object
> + $ref: raw-nand-chip.yaml
> +
> + properties:
> + reg:
> + minimum: 0
> + maximum: 3
> +
> + nand-rb:
> + items:
> + - minimum: 0
> + maximum: 1
> +
> + nand-ecc-step-size:
> + const: 512
> +
> + nand-ecc-strength:
> + enum: [1, 4, 8, 12, 16]
> +
> + nand-ecc-mode:
> + const: hw
> +
> + marvell,nand-keep-config:
> + $ref: /schemas/types.yaml#/definitions/flag
> + description: |
Do not need '|' unless you need to preserve formatting.
> + Orders the driver not to take the timings from the core and
> + leaving them completely untouched. Bootloader timings will then
> + be used.
> +
> + marvell,nand-enable-arbiter:
> + $ref: /schemas/types.yaml#/definitions/flag
> + description: |
Do not need '|' unless you need to preserve formatting.
> + To enable the arbiter, all boards blindly used it,
> + this bit was set by the bootloader for many boards and even if
> + it is marked reserved in several datasheets, it might be needed to set
> + it (otherwise it is harmless).
> + deprecated: true
> +
> + required:
> + - reg
> + - nand-rb
> +
> + unevaluatedProperties: false
> +
> +required:
> + - compatible
> + - reg
> + - interrupts
> + - clocks
> +
> +allOf:
> + - $ref: nand-controller.yaml#
> +
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: marvell,pxa3xx-nand-controller
> + then:
> + required:
> + - dmas
> + - dma-names
> +
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: marvell,armada-8k-nand-controller
> + then:
> + properties:
> + clocks:
> + minItems: 2
> +
> + clock-names:
> + minItems: 2
> +
> + required:
> + - marvell,system-controller
else:
narrow clocks to 1? Or is the second clod also valid for other variants?
Does not look like from your example.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-15 7:45 ` Krzysztof Kozlowski
0 siblings, 0 replies; 39+ messages in thread
From: Krzysztof Kozlowski @ 2023-06-15 7:45 UTC (permalink / raw)
To: Chris Packham, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Vadym Kochan
On 15/06/2023 06:04, Chris Packham wrote:
> From: Vadym Kochan <vadym.kochan@plvision.eu>
>
> Switch the DT binding to a YAML schema to enable the DT validation.
>
> The text binding didn't mention it as a requirement but existing usage
> has
>
> compatible = "marvell,armada-8k-nand-controller",
> "marvell,armada370-nand-controller";
>
> so the YAML allows this in addition to the individual compatible values.
>
> There was also an incorrect reference to dma-names being "rxtx" where
> the driver and existing device trees actually use dma-names = "data" so
> this is corrected in the conversion.
>
> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> ---
>
> Notes:
> Changes in v9:
> - depend on series from Miquel
> https://lore.kernel.org/linux-mtd/20230606175246.190465-1-miquel.raynal@bootlin.com/
> - enforce minimum/maximum for nand-rb
> - move required: block for controller
> - move unevaluatedProperties: for nand chip
> - remove label, partitions and nand-on-flash-bbt which are covered by
> generic schema
>
> Changes in v8:
> - Mark deprecated compatible values as such
> - Allow "marvell,armada-8k-nand-controller" without
> "marvell,armada370-nand-controller"
???
> +
> +properties:
> + compatible:
> + oneOf:
> + - items:
> + - const: marvell,armada-8k-nand-controller
> + - const: marvell,armada370-nand-controller
> + - enum:
> + - marvell,armada-8k-nand-controller
This is wrong. 8k cannot be both: compatible and not compatible with
370. It's not someone's cat to be in both states at the same time...
> + - marvell,armada370-nand-controller
> + - marvell,pxa3xx-nand-controller
> + - description: legacy bindings
> + deprecated: true
> + enum:
> + - marvell,armada-8k-nand
> + - marvell,armada370-nand
> + - marvell,pxa3xx-nand
> +
> + reg:
> + maxItems: 1
> +
> + interrupts:
> + maxItems: 1
> +
> + clocks:
> + description:
> + Shall reference the NAND controller clocks, the second one is
> + is only needed for the Armada 7K/8K SoCs
> + minItems: 1
> + maxItems: 2
> +
> + clock-names:
> + minItems: 1
> + items:
> + - const: core
> + - const: reg
> +
> + dmas:
> + maxItems: 1
> +
> + dma-names:
> + items:
> + - const: data
> +
> + marvell,system-controller:
> + $ref: /schemas/types.yaml#/definitions/phandle
> + description: Syscon node that handles NAND controller related registers
> +
> +patternProperties:
> + "^nand@[a-f0-9]$":
> + type: object
> + $ref: raw-nand-chip.yaml
> +
> + properties:
> + reg:
> + minimum: 0
> + maximum: 3
> +
> + nand-rb:
> + items:
> + - minimum: 0
> + maximum: 1
> +
> + nand-ecc-step-size:
> + const: 512
> +
> + nand-ecc-strength:
> + enum: [1, 4, 8, 12, 16]
> +
> + nand-ecc-mode:
> + const: hw
> +
> + marvell,nand-keep-config:
> + $ref: /schemas/types.yaml#/definitions/flag
> + description: |
Do not need '|' unless you need to preserve formatting.
> + Orders the driver not to take the timings from the core and
> + leaving them completely untouched. Bootloader timings will then
> + be used.
> +
> + marvell,nand-enable-arbiter:
> + $ref: /schemas/types.yaml#/definitions/flag
> + description: |
Do not need '|' unless you need to preserve formatting.
> + To enable the arbiter, all boards blindly used it,
> + this bit was set by the bootloader for many boards and even if
> + it is marked reserved in several datasheets, it might be needed to set
> + it (otherwise it is harmless).
> + deprecated: true
> +
> + required:
> + - reg
> + - nand-rb
> +
> + unevaluatedProperties: false
> +
> +required:
> + - compatible
> + - reg
> + - interrupts
> + - clocks
> +
> +allOf:
> + - $ref: nand-controller.yaml#
> +
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: marvell,pxa3xx-nand-controller
> + then:
> + required:
> + - dmas
> + - dma-names
> +
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: marvell,armada-8k-nand-controller
> + then:
> + properties:
> + clocks:
> + minItems: 2
> +
> + clock-names:
> + minItems: 2
> +
> + required:
> + - marvell,system-controller
else:
narrow clocks to 1? Or is the second clod also valid for other variants?
Does not look like from your example.
Best regards,
Krzysztof
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-15 7:45 ` Krzysztof Kozlowski
0 siblings, 0 replies; 39+ messages in thread
From: Krzysztof Kozlowski @ 2023-06-15 7:45 UTC (permalink / raw)
To: Chris Packham, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Vadym Kochan
On 15/06/2023 06:04, Chris Packham wrote:
> From: Vadym Kochan <vadym.kochan@plvision.eu>
>
> Switch the DT binding to a YAML schema to enable the DT validation.
>
> The text binding didn't mention it as a requirement but existing usage
> has
>
> compatible = "marvell,armada-8k-nand-controller",
> "marvell,armada370-nand-controller";
>
> so the YAML allows this in addition to the individual compatible values.
>
> There was also an incorrect reference to dma-names being "rxtx" where
> the driver and existing device trees actually use dma-names = "data" so
> this is corrected in the conversion.
>
> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> ---
>
> Notes:
> Changes in v9:
> - depend on series from Miquel
> https://lore.kernel.org/linux-mtd/20230606175246.190465-1-miquel.raynal@bootlin.com/
> - enforce minimum/maximum for nand-rb
> - move required: block for controller
> - move unevaluatedProperties: for nand chip
> - remove label, partitions and nand-on-flash-bbt which are covered by
> generic schema
>
> Changes in v8:
> - Mark deprecated compatible values as such
> - Allow "marvell,armada-8k-nand-controller" without
> "marvell,armada370-nand-controller"
???
> +
> +properties:
> + compatible:
> + oneOf:
> + - items:
> + - const: marvell,armada-8k-nand-controller
> + - const: marvell,armada370-nand-controller
> + - enum:
> + - marvell,armada-8k-nand-controller
This is wrong. 8k cannot be both: compatible and not compatible with
370. It's not someone's cat to be in both states at the same time...
> + - marvell,armada370-nand-controller
> + - marvell,pxa3xx-nand-controller
> + - description: legacy bindings
> + deprecated: true
> + enum:
> + - marvell,armada-8k-nand
> + - marvell,armada370-nand
> + - marvell,pxa3xx-nand
> +
> + reg:
> + maxItems: 1
> +
> + interrupts:
> + maxItems: 1
> +
> + clocks:
> + description:
> + Shall reference the NAND controller clocks, the second one is
> + is only needed for the Armada 7K/8K SoCs
> + minItems: 1
> + maxItems: 2
> +
> + clock-names:
> + minItems: 1
> + items:
> + - const: core
> + - const: reg
> +
> + dmas:
> + maxItems: 1
> +
> + dma-names:
> + items:
> + - const: data
> +
> + marvell,system-controller:
> + $ref: /schemas/types.yaml#/definitions/phandle
> + description: Syscon node that handles NAND controller related registers
> +
> +patternProperties:
> + "^nand@[a-f0-9]$":
> + type: object
> + $ref: raw-nand-chip.yaml
> +
> + properties:
> + reg:
> + minimum: 0
> + maximum: 3
> +
> + nand-rb:
> + items:
> + - minimum: 0
> + maximum: 1
> +
> + nand-ecc-step-size:
> + const: 512
> +
> + nand-ecc-strength:
> + enum: [1, 4, 8, 12, 16]
> +
> + nand-ecc-mode:
> + const: hw
> +
> + marvell,nand-keep-config:
> + $ref: /schemas/types.yaml#/definitions/flag
> + description: |
Do not need '|' unless you need to preserve formatting.
> + Orders the driver not to take the timings from the core and
> + leaving them completely untouched. Bootloader timings will then
> + be used.
> +
> + marvell,nand-enable-arbiter:
> + $ref: /schemas/types.yaml#/definitions/flag
> + description: |
Do not need '|' unless you need to preserve formatting.
> + To enable the arbiter, all boards blindly used it,
> + this bit was set by the bootloader for many boards and even if
> + it is marked reserved in several datasheets, it might be needed to set
> + it (otherwise it is harmless).
> + deprecated: true
> +
> + required:
> + - reg
> + - nand-rb
> +
> + unevaluatedProperties: false
> +
> +required:
> + - compatible
> + - reg
> + - interrupts
> + - clocks
> +
> +allOf:
> + - $ref: nand-controller.yaml#
> +
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: marvell,pxa3xx-nand-controller
> + then:
> + required:
> + - dmas
> + - dma-names
> +
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: marvell,armada-8k-nand-controller
> + then:
> + properties:
> + clocks:
> + minItems: 2
> +
> + clock-names:
> + minItems: 2
> +
> + required:
> + - marvell,system-controller
else:
narrow clocks to 1? Or is the second clod also valid for other variants?
Does not look like from your example.
Best regards,
Krzysztof
_______________________________________________
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] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
2023-06-15 7:45 ` Krzysztof Kozlowski
(?)
@ 2023-06-15 21:06 ` Chris Packham
-1 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 21:06 UTC (permalink / raw)
To: Krzysztof Kozlowski, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Vadym Kochan
Hi Krzysztof,
On 15/06/23 19:45, Krzysztof Kozlowski wrote:
> On 15/06/2023 06:04, Chris Packham wrote:
>> From: Vadym Kochan <vadym.kochan@plvision.eu>
>>
>> Switch the DT binding to a YAML schema to enable the DT validation.
>>
>> The text binding didn't mention it as a requirement but existing usage
>> has
>>
>> compatible = "marvell,armada-8k-nand-controller",
>> "marvell,armada370-nand-controller";
>>
>> so the YAML allows this in addition to the individual compatible values.
>>
>> There was also an incorrect reference to dma-names being "rxtx" where
>> the driver and existing device trees actually use dma-names = "data" so
>> this is corrected in the conversion.
>>
>> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
>> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
>> ---
>>
>> Notes:
>> Changes in v9:
>> - depend on series from Miquel https://lore.kernel.org/linux-mtd/20230606175246.190465-1-miquel.raynal@bootlin.com/
>> - enforce minimum/maximum for nand-rb
>> - move required: block for controller
>> - move unevaluatedProperties: for nand chip
>> - remove label, partitions and nand-on-flash-bbt which are covered by
>> generic schema
>>
>> Changes in v8:
>> - Mark deprecated compatible values as such
>> - Allow "marvell,armada-8k-nand-controller" without
>> "marvell,armada370-nand-controller"
> ???
>
>
>> +
>> +properties:
>> + compatible:
>> + oneOf:
>> + - items:
>> + - const: marvell,armada-8k-nand-controller
>> + - const: marvell,armada370-nand-controller
>> + - enum:
>> + - marvell,armada-8k-nand-controller
> This is wrong. 8k cannot be both: compatible and not compatible with
> 370. It's not someone's cat to be in both states at the same time...
The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
= "marvell,armada-8k-nand-controller";` as there are some 8K specific
requirements that aren't present on the 370 (specifically the
system-controller and the 2nd clock). The only reason `compatible =
"marvell,armada-8k-nand-controller",
"marvell,armada370-nand-controller";` is supported is because that is
what is in use in the existing dtses.
>> + - marvell,armada370-nand-controller
>> + - marvell,pxa3xx-nand-controller
>> + - description: legacy bindings
>> + deprecated: true
>> + enum:
>> + - marvell,armada-8k-nand
>> + - marvell,armada370-nand
>> + - marvell,pxa3xx-nand
>> +
>> + reg:
>> + maxItems: 1
>> +
>> + interrupts:
>> + maxItems: 1
>> +
>> + clocks:
>> + description:
>> + Shall reference the NAND controller clocks, the second one is
>> + is only needed for the Armada 7K/8K SoCs
>> + minItems: 1
>> + maxItems: 2
>> +
>> + clock-names:
>> + minItems: 1
>> + items:
>> + - const: core
>> + - const: reg
>> +
>> + dmas:
>> + maxItems: 1
>> +
>> + dma-names:
>> + items:
>> + - const: data
>> +
>> + marvell,system-controller:
>> + $ref: /schemas/types.yaml#/definitions/phandle
>> + description: Syscon node that handles NAND controller related registers
>> +
>> +patternProperties:
>> + "^nand@[a-f0-9]$":
>> + type: object
>> + $ref: raw-nand-chip.yaml
>> +
>> + properties:
>> + reg:
>> + minimum: 0
>> + maximum: 3
>> +
>> + nand-rb:
>> + items:
>> + - minimum: 0
>> + maximum: 1
>> +
>> + nand-ecc-step-size:
>> + const: 512
>> +
>> + nand-ecc-strength:
>> + enum: [1, 4, 8, 12, 16]
>> +
>> + nand-ecc-mode:
>> + const: hw
>> +
>> + marvell,nand-keep-config:
>> + $ref: /schemas/types.yaml#/definitions/flag
>> + description: |
> Do not need '|' unless you need to preserve formatting.
Ack
>> + Orders the driver not to take the timings from the core and
>> + leaving them completely untouched. Bootloader timings will then
>> + be used.
>> +
>> + marvell,nand-enable-arbiter:
>> + $ref: /schemas/types.yaml#/definitions/flag
>> + description: |
> Do not need '|' unless you need to preserve formatting.
>
>> + To enable the arbiter, all boards blindly used it,
>> + this bit was set by the bootloader for many boards and even if
>> + it is marked reserved in several datasheets, it might be needed to set
>> + it (otherwise it is harmless).
>> + deprecated: true
>> +
>> + required:
>> + - reg
>> + - nand-rb
>> +
>> + unevaluatedProperties: false
>> +
>> +required:
>> + - compatible
>> + - reg
>> + - interrupts
>> + - clocks
>> +
>> +allOf:
>> + - $ref: nand-controller.yaml#
>> +
>> + - if:
>> + properties:
>> + compatible:
>> + contains:
>> + const: marvell,pxa3xx-nand-controller
>> + then:
>> + required:
>> + - dmas
>> + - dma-names
>> +
>> + - if:
>> + properties:
>> + compatible:
>> + contains:
>> + const: marvell,armada-8k-nand-controller
>> + then:
>> + properties:
>> + clocks:
>> + minItems: 2
>> +
>> + clock-names:
>> + minItems: 2
>> +
>> + required:
>> + - marvell,system-controller
> else:
> narrow clocks to 1? Or is the second clod also valid for other variants?
> Does not look like from your example.
Correct. Will update.
>
>
> Best regards,
> Krzysztof
>
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-15 21:06 ` Chris Packham
0 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 21:06 UTC (permalink / raw)
To: Krzysztof Kozlowski, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Vadym Kochan
Hi Krzysztof,
On 15/06/23 19:45, Krzysztof Kozlowski wrote:
> On 15/06/2023 06:04, Chris Packham wrote:
>> From: Vadym Kochan <vadym.kochan@plvision.eu>
>>
>> Switch the DT binding to a YAML schema to enable the DT validation.
>>
>> The text binding didn't mention it as a requirement but existing usage
>> has
>>
>> compatible = "marvell,armada-8k-nand-controller",
>> "marvell,armada370-nand-controller";
>>
>> so the YAML allows this in addition to the individual compatible values.
>>
>> There was also an incorrect reference to dma-names being "rxtx" where
>> the driver and existing device trees actually use dma-names = "data" so
>> this is corrected in the conversion.
>>
>> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
>> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
>> ---
>>
>> Notes:
>> Changes in v9:
>> - depend on series from Miquel https://lore.kernel.org/linux-mtd/20230606175246.190465-1-miquel.raynal@bootlin.com/
>> - enforce minimum/maximum for nand-rb
>> - move required: block for controller
>> - move unevaluatedProperties: for nand chip
>> - remove label, partitions and nand-on-flash-bbt which are covered by
>> generic schema
>>
>> Changes in v8:
>> - Mark deprecated compatible values as such
>> - Allow "marvell,armada-8k-nand-controller" without
>> "marvell,armada370-nand-controller"
> ???
>
>
>> +
>> +properties:
>> + compatible:
>> + oneOf:
>> + - items:
>> + - const: marvell,armada-8k-nand-controller
>> + - const: marvell,armada370-nand-controller
>> + - enum:
>> + - marvell,armada-8k-nand-controller
> This is wrong. 8k cannot be both: compatible and not compatible with
> 370. It's not someone's cat to be in both states at the same time...
The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
= "marvell,armada-8k-nand-controller";` as there are some 8K specific
requirements that aren't present on the 370 (specifically the
system-controller and the 2nd clock). The only reason `compatible =
"marvell,armada-8k-nand-controller",
"marvell,armada370-nand-controller";` is supported is because that is
what is in use in the existing dtses.
>> + - marvell,armada370-nand-controller
>> + - marvell,pxa3xx-nand-controller
>> + - description: legacy bindings
>> + deprecated: true
>> + enum:
>> + - marvell,armada-8k-nand
>> + - marvell,armada370-nand
>> + - marvell,pxa3xx-nand
>> +
>> + reg:
>> + maxItems: 1
>> +
>> + interrupts:
>> + maxItems: 1
>> +
>> + clocks:
>> + description:
>> + Shall reference the NAND controller clocks, the second one is
>> + is only needed for the Armada 7K/8K SoCs
>> + minItems: 1
>> + maxItems: 2
>> +
>> + clock-names:
>> + minItems: 1
>> + items:
>> + - const: core
>> + - const: reg
>> +
>> + dmas:
>> + maxItems: 1
>> +
>> + dma-names:
>> + items:
>> + - const: data
>> +
>> + marvell,system-controller:
>> + $ref: /schemas/types.yaml#/definitions/phandle
>> + description: Syscon node that handles NAND controller related registers
>> +
>> +patternProperties:
>> + "^nand@[a-f0-9]$":
>> + type: object
>> + $ref: raw-nand-chip.yaml
>> +
>> + properties:
>> + reg:
>> + minimum: 0
>> + maximum: 3
>> +
>> + nand-rb:
>> + items:
>> + - minimum: 0
>> + maximum: 1
>> +
>> + nand-ecc-step-size:
>> + const: 512
>> +
>> + nand-ecc-strength:
>> + enum: [1, 4, 8, 12, 16]
>> +
>> + nand-ecc-mode:
>> + const: hw
>> +
>> + marvell,nand-keep-config:
>> + $ref: /schemas/types.yaml#/definitions/flag
>> + description: |
> Do not need '|' unless you need to preserve formatting.
Ack
>> + Orders the driver not to take the timings from the core and
>> + leaving them completely untouched. Bootloader timings will then
>> + be used.
>> +
>> + marvell,nand-enable-arbiter:
>> + $ref: /schemas/types.yaml#/definitions/flag
>> + description: |
> Do not need '|' unless you need to preserve formatting.
>
>> + To enable the arbiter, all boards blindly used it,
>> + this bit was set by the bootloader for many boards and even if
>> + it is marked reserved in several datasheets, it might be needed to set
>> + it (otherwise it is harmless).
>> + deprecated: true
>> +
>> + required:
>> + - reg
>> + - nand-rb
>> +
>> + unevaluatedProperties: false
>> +
>> +required:
>> + - compatible
>> + - reg
>> + - interrupts
>> + - clocks
>> +
>> +allOf:
>> + - $ref: nand-controller.yaml#
>> +
>> + - if:
>> + properties:
>> + compatible:
>> + contains:
>> + const: marvell,pxa3xx-nand-controller
>> + then:
>> + required:
>> + - dmas
>> + - dma-names
>> +
>> + - if:
>> + properties:
>> + compatible:
>> + contains:
>> + const: marvell,armada-8k-nand-controller
>> + then:
>> + properties:
>> + clocks:
>> + minItems: 2
>> +
>> + clock-names:
>> + minItems: 2
>> +
>> + required:
>> + - marvell,system-controller
> else:
> narrow clocks to 1? Or is the second clod also valid for other variants?
> Does not look like from your example.
Correct. Will update.
>
>
> Best regards,
> Krzysztof
>
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-15 21:06 ` Chris Packham
0 siblings, 0 replies; 39+ messages in thread
From: Chris Packham @ 2023-06-15 21:06 UTC (permalink / raw)
To: Krzysztof Kozlowski, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Vadym Kochan
Hi Krzysztof,
On 15/06/23 19:45, Krzysztof Kozlowski wrote:
> On 15/06/2023 06:04, Chris Packham wrote:
>> From: Vadym Kochan <vadym.kochan@plvision.eu>
>>
>> Switch the DT binding to a YAML schema to enable the DT validation.
>>
>> The text binding didn't mention it as a requirement but existing usage
>> has
>>
>> compatible = "marvell,armada-8k-nand-controller",
>> "marvell,armada370-nand-controller";
>>
>> so the YAML allows this in addition to the individual compatible values.
>>
>> There was also an incorrect reference to dma-names being "rxtx" where
>> the driver and existing device trees actually use dma-names = "data" so
>> this is corrected in the conversion.
>>
>> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
>> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
>> ---
>>
>> Notes:
>> Changes in v9:
>> - depend on series from Miquel https://lore.kernel.org/linux-mtd/20230606175246.190465-1-miquel.raynal@bootlin.com/
>> - enforce minimum/maximum for nand-rb
>> - move required: block for controller
>> - move unevaluatedProperties: for nand chip
>> - remove label, partitions and nand-on-flash-bbt which are covered by
>> generic schema
>>
>> Changes in v8:
>> - Mark deprecated compatible values as such
>> - Allow "marvell,armada-8k-nand-controller" without
>> "marvell,armada370-nand-controller"
> ???
>
>
>> +
>> +properties:
>> + compatible:
>> + oneOf:
>> + - items:
>> + - const: marvell,armada-8k-nand-controller
>> + - const: marvell,armada370-nand-controller
>> + - enum:
>> + - marvell,armada-8k-nand-controller
> This is wrong. 8k cannot be both: compatible and not compatible with
> 370. It's not someone's cat to be in both states at the same time...
The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
= "marvell,armada-8k-nand-controller";` as there are some 8K specific
requirements that aren't present on the 370 (specifically the
system-controller and the 2nd clock). The only reason `compatible =
"marvell,armada-8k-nand-controller",
"marvell,armada370-nand-controller";` is supported is because that is
what is in use in the existing dtses.
>> + - marvell,armada370-nand-controller
>> + - marvell,pxa3xx-nand-controller
>> + - description: legacy bindings
>> + deprecated: true
>> + enum:
>> + - marvell,armada-8k-nand
>> + - marvell,armada370-nand
>> + - marvell,pxa3xx-nand
>> +
>> + reg:
>> + maxItems: 1
>> +
>> + interrupts:
>> + maxItems: 1
>> +
>> + clocks:
>> + description:
>> + Shall reference the NAND controller clocks, the second one is
>> + is only needed for the Armada 7K/8K SoCs
>> + minItems: 1
>> + maxItems: 2
>> +
>> + clock-names:
>> + minItems: 1
>> + items:
>> + - const: core
>> + - const: reg
>> +
>> + dmas:
>> + maxItems: 1
>> +
>> + dma-names:
>> + items:
>> + - const: data
>> +
>> + marvell,system-controller:
>> + $ref: /schemas/types.yaml#/definitions/phandle
>> + description: Syscon node that handles NAND controller related registers
>> +
>> +patternProperties:
>> + "^nand@[a-f0-9]$":
>> + type: object
>> + $ref: raw-nand-chip.yaml
>> +
>> + properties:
>> + reg:
>> + minimum: 0
>> + maximum: 3
>> +
>> + nand-rb:
>> + items:
>> + - minimum: 0
>> + maximum: 1
>> +
>> + nand-ecc-step-size:
>> + const: 512
>> +
>> + nand-ecc-strength:
>> + enum: [1, 4, 8, 12, 16]
>> +
>> + nand-ecc-mode:
>> + const: hw
>> +
>> + marvell,nand-keep-config:
>> + $ref: /schemas/types.yaml#/definitions/flag
>> + description: |
> Do not need '|' unless you need to preserve formatting.
Ack
>> + Orders the driver not to take the timings from the core and
>> + leaving them completely untouched. Bootloader timings will then
>> + be used.
>> +
>> + marvell,nand-enable-arbiter:
>> + $ref: /schemas/types.yaml#/definitions/flag
>> + description: |
> Do not need '|' unless you need to preserve formatting.
>
>> + To enable the arbiter, all boards blindly used it,
>> + this bit was set by the bootloader for many boards and even if
>> + it is marked reserved in several datasheets, it might be needed to set
>> + it (otherwise it is harmless).
>> + deprecated: true
>> +
>> + required:
>> + - reg
>> + - nand-rb
>> +
>> + unevaluatedProperties: false
>> +
>> +required:
>> + - compatible
>> + - reg
>> + - interrupts
>> + - clocks
>> +
>> +allOf:
>> + - $ref: nand-controller.yaml#
>> +
>> + - if:
>> + properties:
>> + compatible:
>> + contains:
>> + const: marvell,pxa3xx-nand-controller
>> + then:
>> + required:
>> + - dmas
>> + - dma-names
>> +
>> + - if:
>> + properties:
>> + compatible:
>> + contains:
>> + const: marvell,armada-8k-nand-controller
>> + then:
>> + properties:
>> + clocks:
>> + minItems: 2
>> +
>> + clock-names:
>> + minItems: 2
>> +
>> + required:
>> + - marvell,system-controller
> else:
> narrow clocks to 1? Or is the second clod also valid for other variants?
> Does not look like from your example.
Correct. Will update.
>
>
> Best regards,
> Krzysztof
>
_______________________________________________
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] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
2023-06-15 21:06 ` Chris Packham
(?)
@ 2023-06-16 8:15 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 39+ messages in thread
From: Krzysztof Kozlowski @ 2023-06-16 8:15 UTC (permalink / raw)
To: Chris Packham, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Vadym Kochan
On 15/06/2023 23:06, Chris Packham wrote:
>>
>>> +
>>> +properties:
>>> + compatible:
>>> + oneOf:
>>> + - items:
>>> + - const: marvell,armada-8k-nand-controller
>>> + - const: marvell,armada370-nand-controller
>>> + - enum:
>>> + - marvell,armada-8k-nand-controller
>> This is wrong. 8k cannot be both: compatible and not compatible with
>> 370. It's not someone's cat to be in both states at the same time...
>
> The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
> = "marvell,armada-8k-nand-controller";` as there are some 8K specific
> requirements that aren't present on the 370 (specifically the
> system-controller and the 2nd clock).
Presence of specific requirements does not invalidate compatibility. Two
devices are compatible if the 8k can bind and work with 370 compatible
string, even if this means some lower performance or less features (e.g.
subset of features).
Now whether they are really compatible or not - I don't know. I expect
the answer from you (as plural you).
> The only reason `compatible =
> "marvell,armada-8k-nand-controller",
> "marvell,armada370-nand-controller";` is supported is because that is
> what is in use in the existing dtses.
Fix the DTS, use here the correct combination and, if it is a deviation
from old binding, mention this in commit msg.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-16 8:15 ` Krzysztof Kozlowski
0 siblings, 0 replies; 39+ messages in thread
From: Krzysztof Kozlowski @ 2023-06-16 8:15 UTC (permalink / raw)
To: Chris Packham, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Vadym Kochan
On 15/06/2023 23:06, Chris Packham wrote:
>>
>>> +
>>> +properties:
>>> + compatible:
>>> + oneOf:
>>> + - items:
>>> + - const: marvell,armada-8k-nand-controller
>>> + - const: marvell,armada370-nand-controller
>>> + - enum:
>>> + - marvell,armada-8k-nand-controller
>> This is wrong. 8k cannot be both: compatible and not compatible with
>> 370. It's not someone's cat to be in both states at the same time...
>
> The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
> = "marvell,armada-8k-nand-controller";` as there are some 8K specific
> requirements that aren't present on the 370 (specifically the
> system-controller and the 2nd clock).
Presence of specific requirements does not invalidate compatibility. Two
devices are compatible if the 8k can bind and work with 370 compatible
string, even if this means some lower performance or less features (e.g.
subset of features).
Now whether they are really compatible or not - I don't know. I expect
the answer from you (as plural you).
> The only reason `compatible =
> "marvell,armada-8k-nand-controller",
> "marvell,armada370-nand-controller";` is supported is because that is
> what is in use in the existing dtses.
Fix the DTS, use here the correct combination and, if it is a deviation
from old binding, mention this in commit msg.
Best regards,
Krzysztof
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-16 8:15 ` Krzysztof Kozlowski
0 siblings, 0 replies; 39+ messages in thread
From: Krzysztof Kozlowski @ 2023-06-16 8:15 UTC (permalink / raw)
To: Chris Packham, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Vadym Kochan
On 15/06/2023 23:06, Chris Packham wrote:
>>
>>> +
>>> +properties:
>>> + compatible:
>>> + oneOf:
>>> + - items:
>>> + - const: marvell,armada-8k-nand-controller
>>> + - const: marvell,armada370-nand-controller
>>> + - enum:
>>> + - marvell,armada-8k-nand-controller
>> This is wrong. 8k cannot be both: compatible and not compatible with
>> 370. It's not someone's cat to be in both states at the same time...
>
> The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
> = "marvell,armada-8k-nand-controller";` as there are some 8K specific
> requirements that aren't present on the 370 (specifically the
> system-controller and the 2nd clock).
Presence of specific requirements does not invalidate compatibility. Two
devices are compatible if the 8k can bind and work with 370 compatible
string, even if this means some lower performance or less features (e.g.
subset of features).
Now whether they are really compatible or not - I don't know. I expect
the answer from you (as plural you).
> The only reason `compatible =
> "marvell,armada-8k-nand-controller",
> "marvell,armada370-nand-controller";` is supported is because that is
> what is in use in the existing dtses.
Fix the DTS, use here the correct combination and, if it is a deviation
from old binding, mention this in commit msg.
Best regards,
Krzysztof
_______________________________________________
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] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
2023-06-16 8:15 ` Krzysztof Kozlowski
(?)
@ 2023-06-16 8:24 ` Miquel Raynal
-1 siblings, 0 replies; 39+ messages in thread
From: Miquel Raynal @ 2023-06-16 8:24 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chris Packham, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth, linux-mtd, devicetree, linux-kernel,
linux-arm-kernel, Vadym Kochan
Hi Krzysztof,
krzysztof.kozlowski@linaro.org wrote on Fri, 16 Jun 2023 10:15:31 +0200:
> On 15/06/2023 23:06, Chris Packham wrote:
> >>
> >>> +
> >>> +properties:
> >>> + compatible:
> >>> + oneOf:
> >>> + - items:
> >>> + - const: marvell,armada-8k-nand-controller
> >>> + - const: marvell,armada370-nand-controller
> >>> + - enum:
> >>> + - marvell,armada-8k-nand-controller
> >> This is wrong. 8k cannot be both: compatible and not compatible with
> >> 370. It's not someone's cat to be in both states at the same time...
> >
> > The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
> > = "marvell,armada-8k-nand-controller";` as there are some 8K specific
> > requirements that aren't present on the 370 (specifically the
> > system-controller and the 2nd clock).
>
> Presence of specific requirements does not invalidate compatibility. Two
> devices are compatible if the 8k can bind and work with 370 compatible
> string, even if this means some lower performance or less features (e.g.
> subset of features).
Quoting myself from 2019 (comment in the driver):
"Some SoCs like A7k/A8k need to enable manually the NAND controller,
gated clocks and reset bits to avoid being bootloader dependent."
So can the 8k controller work using a 370 compatible : yes, if the
booloader enabled the NAND controller already, no otherwise. But in
practice it is the same controller.
Given this information I don't know whether it makes sense to qualify
the 8k controller compatible with the 370 compatible or not.
> Now whether they are really compatible or not - I don't know. I expect
> the answer from you (as plural you).
:-)
> > The only reason `compatible =
> > "marvell,armada-8k-nand-controller",
> > "marvell,armada370-nand-controller";` is supported is because that is
> > what is in use in the existing dtses.
>
> Fix the DTS, use here the correct combination and, if it is a deviation
> from old binding, mention this in commit msg.
Fine by me.
> Best regards,
> Krzysztof
>
Thanks,
Miquèl
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-16 8:24 ` Miquel Raynal
0 siblings, 0 replies; 39+ messages in thread
From: Miquel Raynal @ 2023-06-16 8:24 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chris Packham, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth, linux-mtd, devicetree, linux-kernel,
linux-arm-kernel, Vadym Kochan
Hi Krzysztof,
krzysztof.kozlowski@linaro.org wrote on Fri, 16 Jun 2023 10:15:31 +0200:
> On 15/06/2023 23:06, Chris Packham wrote:
> >>
> >>> +
> >>> +properties:
> >>> + compatible:
> >>> + oneOf:
> >>> + - items:
> >>> + - const: marvell,armada-8k-nand-controller
> >>> + - const: marvell,armada370-nand-controller
> >>> + - enum:
> >>> + - marvell,armada-8k-nand-controller
> >> This is wrong. 8k cannot be both: compatible and not compatible with
> >> 370. It's not someone's cat to be in both states at the same time...
> >
> > The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
> > = "marvell,armada-8k-nand-controller";` as there are some 8K specific
> > requirements that aren't present on the 370 (specifically the
> > system-controller and the 2nd clock).
>
> Presence of specific requirements does not invalidate compatibility. Two
> devices are compatible if the 8k can bind and work with 370 compatible
> string, even if this means some lower performance or less features (e.g.
> subset of features).
Quoting myself from 2019 (comment in the driver):
"Some SoCs like A7k/A8k need to enable manually the NAND controller,
gated clocks and reset bits to avoid being bootloader dependent."
So can the 8k controller work using a 370 compatible : yes, if the
booloader enabled the NAND controller already, no otherwise. But in
practice it is the same controller.
Given this information I don't know whether it makes sense to qualify
the 8k controller compatible with the 370 compatible or not.
> Now whether they are really compatible or not - I don't know. I expect
> the answer from you (as plural you).
:-)
> > The only reason `compatible =
> > "marvell,armada-8k-nand-controller",
> > "marvell,armada370-nand-controller";` is supported is because that is
> > what is in use in the existing dtses.
>
> Fix the DTS, use here the correct combination and, if it is a deviation
> from old binding, mention this in commit msg.
Fine by me.
> Best regards,
> Krzysztof
>
Thanks,
Miquèl
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-16 8:24 ` Miquel Raynal
0 siblings, 0 replies; 39+ messages in thread
From: Miquel Raynal @ 2023-06-16 8:24 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chris Packham, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth, linux-mtd, devicetree, linux-kernel,
linux-arm-kernel, Vadym Kochan
Hi Krzysztof,
krzysztof.kozlowski@linaro.org wrote on Fri, 16 Jun 2023 10:15:31 +0200:
> On 15/06/2023 23:06, Chris Packham wrote:
> >>
> >>> +
> >>> +properties:
> >>> + compatible:
> >>> + oneOf:
> >>> + - items:
> >>> + - const: marvell,armada-8k-nand-controller
> >>> + - const: marvell,armada370-nand-controller
> >>> + - enum:
> >>> + - marvell,armada-8k-nand-controller
> >> This is wrong. 8k cannot be both: compatible and not compatible with
> >> 370. It's not someone's cat to be in both states at the same time...
> >
> > The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
> > = "marvell,armada-8k-nand-controller";` as there are some 8K specific
> > requirements that aren't present on the 370 (specifically the
> > system-controller and the 2nd clock).
>
> Presence of specific requirements does not invalidate compatibility. Two
> devices are compatible if the 8k can bind and work with 370 compatible
> string, even if this means some lower performance or less features (e.g.
> subset of features).
Quoting myself from 2019 (comment in the driver):
"Some SoCs like A7k/A8k need to enable manually the NAND controller,
gated clocks and reset bits to avoid being bootloader dependent."
So can the 8k controller work using a 370 compatible : yes, if the
booloader enabled the NAND controller already, no otherwise. But in
practice it is the same controller.
Given this information I don't know whether it makes sense to qualify
the 8k controller compatible with the 370 compatible or not.
> Now whether they are really compatible or not - I don't know. I expect
> the answer from you (as plural you).
:-)
> > The only reason `compatible =
> > "marvell,armada-8k-nand-controller",
> > "marvell,armada370-nand-controller";` is supported is because that is
> > what is in use in the existing dtses.
>
> Fix the DTS, use here the correct combination and, if it is a deviation
> from old binding, mention this in commit msg.
Fine by me.
> Best regards,
> Krzysztof
>
Thanks,
Miquèl
_______________________________________________
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] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
2023-06-16 8:24 ` Miquel Raynal
(?)
@ 2023-06-16 8:38 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 39+ messages in thread
From: Krzysztof Kozlowski @ 2023-06-16 8:38 UTC (permalink / raw)
To: Miquel Raynal
Cc: Chris Packham, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth, linux-mtd, devicetree, linux-kernel,
linux-arm-kernel, Vadym Kochan
On 16/06/2023 10:24, Miquel Raynal wrote:
> Hi Krzysztof,
>
> krzysztof.kozlowski@linaro.org wrote on Fri, 16 Jun 2023 10:15:31 +0200:
>
>> On 15/06/2023 23:06, Chris Packham wrote:
>>>>
>>>>> +
>>>>> +properties:
>>>>> + compatible:
>>>>> + oneOf:
>>>>> + - items:
>>>>> + - const: marvell,armada-8k-nand-controller
>>>>> + - const: marvell,armada370-nand-controller
>>>>> + - enum:
>>>>> + - marvell,armada-8k-nand-controller
>>>> This is wrong. 8k cannot be both: compatible and not compatible with
>>>> 370. It's not someone's cat to be in both states at the same time...
>>>
>>> The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
>>> = "marvell,armada-8k-nand-controller";` as there are some 8K specific
>>> requirements that aren't present on the 370 (specifically the
>>> system-controller and the 2nd clock).
>>
>> Presence of specific requirements does not invalidate compatibility. Two
>> devices are compatible if the 8k can bind and work with 370 compatible
>> string, even if this means some lower performance or less features (e.g.
>> subset of features).
>
> Quoting myself from 2019 (comment in the driver):
> "Some SoCs like A7k/A8k need to enable manually the NAND controller,
> gated clocks and reset bits to avoid being bootloader dependent."
>
> So can the 8k controller work using a 370 compatible : yes, if the
> booloader enabled the NAND controller already, no otherwise. But in
> practice it is the same controller.
>
> Given this information I don't know whether it makes sense to qualify
> the 8k controller compatible with the 370 compatible or not.
Indeed, a bit tricky decision. For me they could stay compatible,
because apparently programming model is the same or similar.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-16 8:38 ` Krzysztof Kozlowski
0 siblings, 0 replies; 39+ messages in thread
From: Krzysztof Kozlowski @ 2023-06-16 8:38 UTC (permalink / raw)
To: Miquel Raynal
Cc: Chris Packham, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth, linux-mtd, devicetree, linux-kernel,
linux-arm-kernel, Vadym Kochan
On 16/06/2023 10:24, Miquel Raynal wrote:
> Hi Krzysztof,
>
> krzysztof.kozlowski@linaro.org wrote on Fri, 16 Jun 2023 10:15:31 +0200:
>
>> On 15/06/2023 23:06, Chris Packham wrote:
>>>>
>>>>> +
>>>>> +properties:
>>>>> + compatible:
>>>>> + oneOf:
>>>>> + - items:
>>>>> + - const: marvell,armada-8k-nand-controller
>>>>> + - const: marvell,armada370-nand-controller
>>>>> + - enum:
>>>>> + - marvell,armada-8k-nand-controller
>>>> This is wrong. 8k cannot be both: compatible and not compatible with
>>>> 370. It's not someone's cat to be in both states at the same time...
>>>
>>> The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
>>> = "marvell,armada-8k-nand-controller";` as there are some 8K specific
>>> requirements that aren't present on the 370 (specifically the
>>> system-controller and the 2nd clock).
>>
>> Presence of specific requirements does not invalidate compatibility. Two
>> devices are compatible if the 8k can bind and work with 370 compatible
>> string, even if this means some lower performance or less features (e.g.
>> subset of features).
>
> Quoting myself from 2019 (comment in the driver):
> "Some SoCs like A7k/A8k need to enable manually the NAND controller,
> gated clocks and reset bits to avoid being bootloader dependent."
>
> So can the 8k controller work using a 370 compatible : yes, if the
> booloader enabled the NAND controller already, no otherwise. But in
> practice it is the same controller.
>
> Given this information I don't know whether it makes sense to qualify
> the 8k controller compatible with the 370 compatible or not.
Indeed, a bit tricky decision. For me they could stay compatible,
because apparently programming model is the same or similar.
Best regards,
Krzysztof
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-16 8:38 ` Krzysztof Kozlowski
0 siblings, 0 replies; 39+ messages in thread
From: Krzysztof Kozlowski @ 2023-06-16 8:38 UTC (permalink / raw)
To: Miquel Raynal
Cc: Chris Packham, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth, linux-mtd, devicetree, linux-kernel,
linux-arm-kernel, Vadym Kochan
On 16/06/2023 10:24, Miquel Raynal wrote:
> Hi Krzysztof,
>
> krzysztof.kozlowski@linaro.org wrote on Fri, 16 Jun 2023 10:15:31 +0200:
>
>> On 15/06/2023 23:06, Chris Packham wrote:
>>>>
>>>>> +
>>>>> +properties:
>>>>> + compatible:
>>>>> + oneOf:
>>>>> + - items:
>>>>> + - const: marvell,armada-8k-nand-controller
>>>>> + - const: marvell,armada370-nand-controller
>>>>> + - enum:
>>>>> + - marvell,armada-8k-nand-controller
>>>> This is wrong. 8k cannot be both: compatible and not compatible with
>>>> 370. It's not someone's cat to be in both states at the same time...
>>>
>>> The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
>>> = "marvell,armada-8k-nand-controller";` as there are some 8K specific
>>> requirements that aren't present on the 370 (specifically the
>>> system-controller and the 2nd clock).
>>
>> Presence of specific requirements does not invalidate compatibility. Two
>> devices are compatible if the 8k can bind and work with 370 compatible
>> string, even if this means some lower performance or less features (e.g.
>> subset of features).
>
> Quoting myself from 2019 (comment in the driver):
> "Some SoCs like A7k/A8k need to enable manually the NAND controller,
> gated clocks and reset bits to avoid being bootloader dependent."
>
> So can the 8k controller work using a 370 compatible : yes, if the
> booloader enabled the NAND controller already, no otherwise. But in
> practice it is the same controller.
>
> Given this information I don't know whether it makes sense to qualify
> the 8k controller compatible with the 370 compatible or not.
Indeed, a bit tricky decision. For me they could stay compatible,
because apparently programming model is the same or similar.
Best regards,
Krzysztof
_______________________________________________
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] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
2023-06-16 8:38 ` Krzysztof Kozlowski
(?)
@ 2023-06-16 8:44 ` Miquel Raynal
-1 siblings, 0 replies; 39+ messages in thread
From: Miquel Raynal @ 2023-06-16 8:44 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chris Packham, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth, linux-mtd, devicetree, linux-kernel,
linux-arm-kernel, Vadym Kochan
Hi Krzysztof,
krzysztof.kozlowski@linaro.org wrote on Fri, 16 Jun 2023 10:38:38 +0200:
> On 16/06/2023 10:24, Miquel Raynal wrote:
> > Hi Krzysztof,
> >
> > krzysztof.kozlowski@linaro.org wrote on Fri, 16 Jun 2023 10:15:31 +0200:
> >
> >> On 15/06/2023 23:06, Chris Packham wrote:
> >>>>
> >>>>> +
> >>>>> +properties:
> >>>>> + compatible:
> >>>>> + oneOf:
> >>>>> + - items:
> >>>>> + - const: marvell,armada-8k-nand-controller
> >>>>> + - const: marvell,armada370-nand-controller
> >>>>> + - enum:
> >>>>> + - marvell,armada-8k-nand-controller
> >>>> This is wrong. 8k cannot be both: compatible and not compatible with
> >>>> 370. It's not someone's cat to be in both states at the same time...
> >>>
> >>> The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
> >>> = "marvell,armada-8k-nand-controller";` as there are some 8K specific
> >>> requirements that aren't present on the 370 (specifically the
> >>> system-controller and the 2nd clock).
> >>
> >> Presence of specific requirements does not invalidate compatibility. Two
> >> devices are compatible if the 8k can bind and work with 370 compatible
> >> string, even if this means some lower performance or less features (e.g.
> >> subset of features).
> >
> > Quoting myself from 2019 (comment in the driver):
> > "Some SoCs like A7k/A8k need to enable manually the NAND controller,
> > gated clocks and reset bits to avoid being bootloader dependent."
> >
> > So can the 8k controller work using a 370 compatible : yes, if the
> > booloader enabled the NAND controller already, no otherwise. But in
> > practice it is the same controller.
> >
> > Given this information I don't know whether it makes sense to qualify
> > the 8k controller compatible with the 370 compatible or not.
>
> Indeed, a bit tricky decision. For me they could stay compatible,
> because apparently programming model is the same or similar.
Agreed, the programming model wrt the controller itself is the same. So
we will expect 8k controllers to be described with both 8k and 370
compatible strings then.
Thanks for your feedback.
Cheers,
Miquèl
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-16 8:44 ` Miquel Raynal
0 siblings, 0 replies; 39+ messages in thread
From: Miquel Raynal @ 2023-06-16 8:44 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chris Packham, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth, linux-mtd, devicetree, linux-kernel,
linux-arm-kernel, Vadym Kochan
Hi Krzysztof,
krzysztof.kozlowski@linaro.org wrote on Fri, 16 Jun 2023 10:38:38 +0200:
> On 16/06/2023 10:24, Miquel Raynal wrote:
> > Hi Krzysztof,
> >
> > krzysztof.kozlowski@linaro.org wrote on Fri, 16 Jun 2023 10:15:31 +0200:
> >
> >> On 15/06/2023 23:06, Chris Packham wrote:
> >>>>
> >>>>> +
> >>>>> +properties:
> >>>>> + compatible:
> >>>>> + oneOf:
> >>>>> + - items:
> >>>>> + - const: marvell,armada-8k-nand-controller
> >>>>> + - const: marvell,armada370-nand-controller
> >>>>> + - enum:
> >>>>> + - marvell,armada-8k-nand-controller
> >>>> This is wrong. 8k cannot be both: compatible and not compatible with
> >>>> 370. It's not someone's cat to be in both states at the same time...
> >>>
> >>> The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
> >>> = "marvell,armada-8k-nand-controller";` as there are some 8K specific
> >>> requirements that aren't present on the 370 (specifically the
> >>> system-controller and the 2nd clock).
> >>
> >> Presence of specific requirements does not invalidate compatibility. Two
> >> devices are compatible if the 8k can bind and work with 370 compatible
> >> string, even if this means some lower performance or less features (e.g.
> >> subset of features).
> >
> > Quoting myself from 2019 (comment in the driver):
> > "Some SoCs like A7k/A8k need to enable manually the NAND controller,
> > gated clocks and reset bits to avoid being bootloader dependent."
> >
> > So can the 8k controller work using a 370 compatible : yes, if the
> > booloader enabled the NAND controller already, no otherwise. But in
> > practice it is the same controller.
> >
> > Given this information I don't know whether it makes sense to qualify
> > the 8k controller compatible with the 370 compatible or not.
>
> Indeed, a bit tricky decision. For me they could stay compatible,
> because apparently programming model is the same or similar.
Agreed, the programming model wrt the controller itself is the same. So
we will expect 8k controllers to be described with both 8k and 370
compatible strings then.
Thanks for your feedback.
Cheers,
Miquèl
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
@ 2023-06-16 8:44 ` Miquel Raynal
0 siblings, 0 replies; 39+ messages in thread
From: Miquel Raynal @ 2023-06-16 8:44 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chris Packham, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, gregory.clement,
sebastian.hesselbarth, linux-mtd, devicetree, linux-kernel,
linux-arm-kernel, Vadym Kochan
Hi Krzysztof,
krzysztof.kozlowski@linaro.org wrote on Fri, 16 Jun 2023 10:38:38 +0200:
> On 16/06/2023 10:24, Miquel Raynal wrote:
> > Hi Krzysztof,
> >
> > krzysztof.kozlowski@linaro.org wrote on Fri, 16 Jun 2023 10:15:31 +0200:
> >
> >> On 15/06/2023 23:06, Chris Packham wrote:
> >>>>
> >>>>> +
> >>>>> +properties:
> >>>>> + compatible:
> >>>>> + oneOf:
> >>>>> + - items:
> >>>>> + - const: marvell,armada-8k-nand-controller
> >>>>> + - const: marvell,armada370-nand-controller
> >>>>> + - enum:
> >>>>> + - marvell,armada-8k-nand-controller
> >>>> This is wrong. 8k cannot be both: compatible and not compatible with
> >>>> 370. It's not someone's cat to be in both states at the same time...
> >>>
> >>> The correct state (IMHO, Miquel correct me if I'm wrong) is `compatible
> >>> = "marvell,armada-8k-nand-controller";` as there are some 8K specific
> >>> requirements that aren't present on the 370 (specifically the
> >>> system-controller and the 2nd clock).
> >>
> >> Presence of specific requirements does not invalidate compatibility. Two
> >> devices are compatible if the 8k can bind and work with 370 compatible
> >> string, even if this means some lower performance or less features (e.g.
> >> subset of features).
> >
> > Quoting myself from 2019 (comment in the driver):
> > "Some SoCs like A7k/A8k need to enable manually the NAND controller,
> > gated clocks and reset bits to avoid being bootloader dependent."
> >
> > So can the 8k controller work using a 370 compatible : yes, if the
> > booloader enabled the NAND controller already, no otherwise. But in
> > practice it is the same controller.
> >
> > Given this information I don't know whether it makes sense to qualify
> > the 8k controller compatible with the 370 compatible or not.
>
> Indeed, a bit tricky decision. For me they could stay compatible,
> because apparently programming model is the same or similar.
Agreed, the programming model wrt the controller itself is the same. So
we will expect 8k controllers to be described with both 8k and 370
compatible strings then.
Thanks for your feedback.
Cheers,
Miquèl
_______________________________________________
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] 39+ messages in thread
* Re: [PATCH v9 1/3] arm64: dts: marvell: cp11x: Fix nand_controller node name according to YAML
2023-06-15 4:04 ` Chris Packham
(?)
@ 2023-06-16 11:55 ` Gregory CLEMENT
-1 siblings, 0 replies; 39+ messages in thread
From: Gregory CLEMENT @ 2023-06-16 11:55 UTC (permalink / raw)
To: Chris Packham, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel,
Vadym Kochan, Chris Packham
Chris Packham <chris.packham@alliedtelesis.co.nz> writes:
> From: Vadym Kochan <vadym.kochan@plvision.eu>
>
> Marvell NAND controller has now YAML to validate it's DT bindings, so
> change the node name of cp11x DTSI as it is required by nand-controller.yaml
>
> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
Applied on mvebu/dt64
Thanks,
Gregory
> ---
>
> Notes:
> Changes in v9:
> - None
> Changes in v8:
> - Add r-by from Miquel
>
> arch/arm64/boot/dts/marvell/armada-cp11x.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> index 7d0043824f2a..982b180b33e6 100644
> --- a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> @@ -468,7 +468,7 @@ CP11X_LABEL(uart3): serial@702300 {
> status = "disabled";
> };
>
> - CP11X_LABEL(nand_controller): nand@720000 {
> + CP11X_LABEL(nand_controller): nand-controller@720000 {
> /*
> * Due to the limitation of the pins available
> * this controller is only usable on the CPM
> --
> 2.40.1
>
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 1/3] arm64: dts: marvell: cp11x: Fix nand_controller node name according to YAML
@ 2023-06-16 11:55 ` Gregory CLEMENT
0 siblings, 0 replies; 39+ messages in thread
From: Gregory CLEMENT @ 2023-06-16 11:55 UTC (permalink / raw)
To: Chris Packham, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel,
Vadym Kochan, Chris Packham
Chris Packham <chris.packham@alliedtelesis.co.nz> writes:
> From: Vadym Kochan <vadym.kochan@plvision.eu>
>
> Marvell NAND controller has now YAML to validate it's DT bindings, so
> change the node name of cp11x DTSI as it is required by nand-controller.yaml
>
> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
Applied on mvebu/dt64
Thanks,
Gregory
> ---
>
> Notes:
> Changes in v9:
> - None
> Changes in v8:
> - Add r-by from Miquel
>
> arch/arm64/boot/dts/marvell/armada-cp11x.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> index 7d0043824f2a..982b180b33e6 100644
> --- a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> @@ -468,7 +468,7 @@ CP11X_LABEL(uart3): serial@702300 {
> status = "disabled";
> };
>
> - CP11X_LABEL(nand_controller): nand@720000 {
> + CP11X_LABEL(nand_controller): nand-controller@720000 {
> /*
> * Due to the limitation of the pins available
> * this controller is only usable on the CPM
> --
> 2.40.1
>
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 1/3] arm64: dts: marvell: cp11x: Fix nand_controller node name according to YAML
@ 2023-06-16 11:55 ` Gregory CLEMENT
0 siblings, 0 replies; 39+ messages in thread
From: Gregory CLEMENT @ 2023-06-16 11:55 UTC (permalink / raw)
To: Chris Packham, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel,
Vadym Kochan, Chris Packham
Chris Packham <chris.packham@alliedtelesis.co.nz> writes:
> From: Vadym Kochan <vadym.kochan@plvision.eu>
>
> Marvell NAND controller has now YAML to validate it's DT bindings, so
> change the node name of cp11x DTSI as it is required by nand-controller.yaml
>
> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
Applied on mvebu/dt64
Thanks,
Gregory
> ---
>
> Notes:
> Changes in v9:
> - None
> Changes in v8:
> - Add r-by from Miquel
>
> arch/arm64/boot/dts/marvell/armada-cp11x.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> index 7d0043824f2a..982b180b33e6 100644
> --- a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> @@ -468,7 +468,7 @@ CP11X_LABEL(uart3): serial@702300 {
> status = "disabled";
> };
>
> - CP11X_LABEL(nand_controller): nand@720000 {
> + CP11X_LABEL(nand_controller): nand-controller@720000 {
> /*
> * Due to the limitation of the pins available
> * this controller is only usable on the CPM
> --
> 2.40.1
>
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
_______________________________________________
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] 39+ messages in thread
* Re: [PATCH v9 2/3] ARM: dts: mvebu: align MTD partition nodes to dtschema
2023-06-15 4:04 ` Chris Packham
(?)
@ 2023-06-16 11:56 ` Gregory CLEMENT
-1 siblings, 0 replies; 39+ messages in thread
From: Gregory CLEMENT @ 2023-06-16 11:56 UTC (permalink / raw)
To: Chris Packham, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Chris Packham
Chris Packham <chris.packham@alliedtelesis.co.nz> writes:
> Update the node names for the SPI NOR and NAND partitions to conform to
> the partition properties in the relevant dtschema.
>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
Applied on mvebu/dt
Thanks,
Gregory
> ---
>
> Notes:
> Changes in v9:
> - None
> Changes in v8:
> - Add r-by from Miquel
>
> arch/arm/boot/dts/armada-385-atl-x530.dts | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm/boot/dts/armada-385-atl-x530.dts b/arch/arm/boot/dts/armada-385-atl-x530.dts
> index 241f5d7c80e9..5a9ab8410b7b 100644
> --- a/arch/arm/boot/dts/armada-385-atl-x530.dts
> +++ b/arch/arm/boot/dts/armada-385-atl-x530.dts
> @@ -179,19 +179,19 @@ partitions {
> compatible = "fixed-partitions";
> #address-cells = <1>;
> #size-cells = <1>;
> - partition@u-boot {
> + partition@0 {
> reg = <0x00000000 0x00100000>;
> label = "u-boot";
> };
> - partition@u-boot-env {
> + partition@100000 {
> reg = <0x00100000 0x00040000>;
> label = "u-boot-env";
> };
> - partition@unused {
> + partition@140000 {
> reg = <0x00140000 0x00e80000>;
> label = "unused";
> };
> - partition@idprom {
> + partition@fc0000 {
> reg = <0x00fc0000 0x00040000>;
> label = "idprom";
> };
> @@ -216,16 +216,16 @@ partitions {
> compatible = "fixed-partitions";
> #address-cells = <1>;
> #size-cells = <1>;
> - partition@user {
> + partition@0 {
> reg = <0x00000000 0x0f000000>;
> label = "user";
> };
> - partition@errlog {
> + partition@f000000 {
> /* Maximum mtdoops size is 8MB, so set to that. */
> reg = <0x0f000000 0x00800000>;
> label = "errlog";
> };
> - partition@nand-bbt {
> + partition@f800000 {
> reg = <0x0f800000 0x00800000>;
> label = "nand-bbt";
> };
> --
> 2.40.1
>
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 2/3] ARM: dts: mvebu: align MTD partition nodes to dtschema
@ 2023-06-16 11:56 ` Gregory CLEMENT
0 siblings, 0 replies; 39+ messages in thread
From: Gregory CLEMENT @ 2023-06-16 11:56 UTC (permalink / raw)
To: Chris Packham, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Chris Packham
Chris Packham <chris.packham@alliedtelesis.co.nz> writes:
> Update the node names for the SPI NOR and NAND partitions to conform to
> the partition properties in the relevant dtschema.
>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
Applied on mvebu/dt
Thanks,
Gregory
> ---
>
> Notes:
> Changes in v9:
> - None
> Changes in v8:
> - Add r-by from Miquel
>
> arch/arm/boot/dts/armada-385-atl-x530.dts | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm/boot/dts/armada-385-atl-x530.dts b/arch/arm/boot/dts/armada-385-atl-x530.dts
> index 241f5d7c80e9..5a9ab8410b7b 100644
> --- a/arch/arm/boot/dts/armada-385-atl-x530.dts
> +++ b/arch/arm/boot/dts/armada-385-atl-x530.dts
> @@ -179,19 +179,19 @@ partitions {
> compatible = "fixed-partitions";
> #address-cells = <1>;
> #size-cells = <1>;
> - partition@u-boot {
> + partition@0 {
> reg = <0x00000000 0x00100000>;
> label = "u-boot";
> };
> - partition@u-boot-env {
> + partition@100000 {
> reg = <0x00100000 0x00040000>;
> label = "u-boot-env";
> };
> - partition@unused {
> + partition@140000 {
> reg = <0x00140000 0x00e80000>;
> label = "unused";
> };
> - partition@idprom {
> + partition@fc0000 {
> reg = <0x00fc0000 0x00040000>;
> label = "idprom";
> };
> @@ -216,16 +216,16 @@ partitions {
> compatible = "fixed-partitions";
> #address-cells = <1>;
> #size-cells = <1>;
> - partition@user {
> + partition@0 {
> reg = <0x00000000 0x0f000000>;
> label = "user";
> };
> - partition@errlog {
> + partition@f000000 {
> /* Maximum mtdoops size is 8MB, so set to that. */
> reg = <0x0f000000 0x00800000>;
> label = "errlog";
> };
> - partition@nand-bbt {
> + partition@f800000 {
> reg = <0x0f800000 0x00800000>;
> label = "nand-bbt";
> };
> --
> 2.40.1
>
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v9 2/3] ARM: dts: mvebu: align MTD partition nodes to dtschema
@ 2023-06-16 11:56 ` Gregory CLEMENT
0 siblings, 0 replies; 39+ messages in thread
From: Gregory CLEMENT @ 2023-06-16 11:56 UTC (permalink / raw)
To: Chris Packham, miquel.raynal, richard, vigneshr, robh+dt,
krzysztof.kozlowski+dt, andrew, sebastian.hesselbarth
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-kernel, Chris Packham
Chris Packham <chris.packham@alliedtelesis.co.nz> writes:
> Update the node names for the SPI NOR and NAND partitions to conform to
> the partition properties in the relevant dtschema.
>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
Applied on mvebu/dt
Thanks,
Gregory
> ---
>
> Notes:
> Changes in v9:
> - None
> Changes in v8:
> - Add r-by from Miquel
>
> arch/arm/boot/dts/armada-385-atl-x530.dts | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm/boot/dts/armada-385-atl-x530.dts b/arch/arm/boot/dts/armada-385-atl-x530.dts
> index 241f5d7c80e9..5a9ab8410b7b 100644
> --- a/arch/arm/boot/dts/armada-385-atl-x530.dts
> +++ b/arch/arm/boot/dts/armada-385-atl-x530.dts
> @@ -179,19 +179,19 @@ partitions {
> compatible = "fixed-partitions";
> #address-cells = <1>;
> #size-cells = <1>;
> - partition@u-boot {
> + partition@0 {
> reg = <0x00000000 0x00100000>;
> label = "u-boot";
> };
> - partition@u-boot-env {
> + partition@100000 {
> reg = <0x00100000 0x00040000>;
> label = "u-boot-env";
> };
> - partition@unused {
> + partition@140000 {
> reg = <0x00140000 0x00e80000>;
> label = "unused";
> };
> - partition@idprom {
> + partition@fc0000 {
> reg = <0x00fc0000 0x00040000>;
> label = "idprom";
> };
> @@ -216,16 +216,16 @@ partitions {
> compatible = "fixed-partitions";
> #address-cells = <1>;
> #size-cells = <1>;
> - partition@user {
> + partition@0 {
> reg = <0x00000000 0x0f000000>;
> label = "user";
> };
> - partition@errlog {
> + partition@f000000 {
> /* Maximum mtdoops size is 8MB, so set to that. */
> reg = <0x0f000000 0x00800000>;
> label = "errlog";
> };
> - partition@nand-bbt {
> + partition@f800000 {
> reg = <0x0f800000 0x00800000>;
> label = "nand-bbt";
> };
> --
> 2.40.1
>
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
_______________________________________________
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] 39+ messages in thread
end of thread, other threads:[~2023-06-16 11:57 UTC | newest]
Thread overview: 39+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-15 4:04 [PATCH v9 0/3] dt-bindings: mtd: marvell-nand: Add YAML scheme Chris Packham
2023-06-15 4:04 ` Chris Packham
2023-06-15 4:04 ` Chris Packham
2023-06-15 4:04 ` [PATCH v9 1/3] arm64: dts: marvell: cp11x: Fix nand_controller node name according to YAML Chris Packham
2023-06-15 4:04 ` Chris Packham
2023-06-15 4:04 ` Chris Packham
2023-06-16 11:55 ` Gregory CLEMENT
2023-06-16 11:55 ` Gregory CLEMENT
2023-06-16 11:55 ` Gregory CLEMENT
2023-06-15 4:04 ` [PATCH v9 2/3] ARM: dts: mvebu: align MTD partition nodes to dtschema Chris Packham
2023-06-15 4:04 ` Chris Packham
2023-06-15 4:04 ` Chris Packham
2023-06-16 11:56 ` Gregory CLEMENT
2023-06-16 11:56 ` Gregory CLEMENT
2023-06-16 11:56 ` Gregory CLEMENT
2023-06-15 4:04 ` [PATCH v9 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme Chris Packham
2023-06-15 4:04 ` Chris Packham
2023-06-15 4:04 ` Chris Packham
2023-06-15 5:25 ` Rob Herring
2023-06-15 5:25 ` Rob Herring
2023-06-15 5:25 ` Rob Herring
2023-06-15 7:45 ` Krzysztof Kozlowski
2023-06-15 7:45 ` Krzysztof Kozlowski
2023-06-15 7:45 ` Krzysztof Kozlowski
2023-06-15 21:06 ` Chris Packham
2023-06-15 21:06 ` Chris Packham
2023-06-15 21:06 ` Chris Packham
2023-06-16 8:15 ` Krzysztof Kozlowski
2023-06-16 8:15 ` Krzysztof Kozlowski
2023-06-16 8:15 ` Krzysztof Kozlowski
2023-06-16 8:24 ` Miquel Raynal
2023-06-16 8:24 ` Miquel Raynal
2023-06-16 8:24 ` Miquel Raynal
2023-06-16 8:38 ` Krzysztof Kozlowski
2023-06-16 8:38 ` Krzysztof Kozlowski
2023-06-16 8:38 ` Krzysztof Kozlowski
2023-06-16 8:44 ` Miquel Raynal
2023-06-16 8:44 ` Miquel Raynal
2023-06-16 8:44 ` Miquel Raynal
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.