2023-06-15 04:21:28

by Chris Packham

[permalink] [raw]
Subject: [PATCH v9 1/3] arm64: dts: marvell: cp11x: Fix nand_controller node name according to YAML

From: Vadym Kochan <[email protected]>

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 <[email protected]>
Signed-off-by: Chris Packham <[email protected]>
Reviewed-by: Miquel Raynal <[email protected]>
---

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



2023-06-16 12:24:20

by Gregory CLEMENT

[permalink] [raw]
Subject: Re: [PATCH v9 1/3] arm64: dts: marvell: cp11x: Fix nand_controller node name according to YAML

Chris Packham <[email protected]> writes:

> From: Vadym Kochan <[email protected]>
>
> 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 <[email protected]>
> Signed-off-by: Chris Packham <[email protected]>
> Reviewed-by: Miquel Raynal <[email protected]>

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