[PATCH 1/3] dt-bindings: marvell: add D-Link DNS-320L

[PATCH 1/3] dt-bindings: marvell: add D-Link DNS-320L

[Zoltan HERPAI]

Add DT-compatible for the D-Link DNS-320L NAS.

Signed-off-by: Zoltan HERPAI <wigyori@uid0.hu>
---
 .../devicetree/bindings/arm/marvell/marvell,kirkwood.txt         | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/arm/marvell/marvell,kirkwood.txt b/Documentation/devicetree/bindings/arm/marvell/marvell,kirkwood.txt
index 7d28fe4bf654..4a70dd1b6b65 100644
--- a/Documentation/devicetree/bindings/arm/marvell/marvell,kirkwood.txt
+++ b/Documentation/devicetree/bindings/arm/marvell/marvell,kirkwood.txt
@@ -30,6 +30,7 @@ board. Currently known boards are:
 "cloudengines,pogo02"
 "cloudengines,pogoplugv4"
 "dlink,dns-320"
+"dlink,dns-320l"
 "dlink,dns-320-a1"
 "dlink,dns-325"
 "dlink,dns-325-a1"
-- 
2.30.2

[PATCH 2/3] dt-bindings: marvell: typo fix in SoC name

[Zoltan HERPAI]

Fix typo in the SoC name of Marvell Kirkwood. This change is in the
description which shouldn't affect functionality.

Signed-off-by: Zoltan HERPAI <wigyori@uid0.hu>
---
 .../devicetree/bindings/arm/marvell/marvell,kirkwood.txt        | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/arm/marvell/marvell,kirkwood.txt b/Documentation/devicetree/bindings/arm/marvell/marvell,kirkwood.txt
index 4a70dd1b6b65..7edfaa0b0b73 100644
--- a/Documentation/devicetree/bindings/arm/marvell/marvell,kirkwood.txt
+++ b/Documentation/devicetree/bindings/arm/marvell/marvell,kirkwood.txt
@@ -1,7 +1,7 @@
 Marvell Kirkwood SoC Family Device Tree Bindings
 ------------------------------------------------
 
-Boards with a SoC of the Marvell Kirkwook family, eg 88f6281
+Boards with a SoC of the Marvell Kirkwood family, eg 88f6281
 
 * Required root node properties:
 compatible: must contain "marvell,kirkwood"
-- 
2.30.2

[PATCH 3/3] ARM: dts: marvell: add support for D-Link DNS-320L

[Zoltan HERPAI]

Dual-slot NAS based on Marvell Kirkwood.

Specifications:
 - Marvell 88F6702 @1GHz
 - 256Mb RAM
 - 128Mb NAND
 - 1x GbE LAN (Marvell 88E1318R)
 - 1x USB 2.0
 - 2x SATA
 - Weltrend WT69P3 ("supervisor" MCU chip)
 - Serial on J2 (115200,8n1)
 - Newer bootROM so kwboot-ing via serial is possible

Signed-off-by: Zoltan HERPAI <wigyori@uid0.hu>
---
 arch/arm/boot/dts/marvell/Makefile            |   1 +
 .../arm/boot/dts/marvell/kirkwood-dns320l.dts | 197 ++++++++++++++++++
 2 files changed, 198 insertions(+)
 create mode 100644 arch/arm/boot/dts/marvell/kirkwood-dns320l.dts

diff --git a/arch/arm/boot/dts/marvell/Makefile b/arch/arm/boot/dts/marvell/Makefile
index 1e0f5ff492f7..cadd4039b783 100644
--- a/arch/arm/boot/dts/marvell/Makefile
+++ b/arch/arm/boot/dts/marvell/Makefile
@@ -92,6 +92,7 @@ dtb-$(CONFIG_MACH_KIRKWOOD) += \
 	kirkwood-db-88f6282.dtb \
 	kirkwood-dir665.dtb \
 	kirkwood-dns320.dtb \
+	kirkwood-dns320l.dtb \
 	kirkwood-dns325.dtb \
 	kirkwood-dockstar.dtb \
 	kirkwood-dreamplug.dtb \
diff --git a/arch/arm/boot/dts/marvell/kirkwood-dns320l.dts b/arch/arm/boot/dts/marvell/kirkwood-dns320l.dts
new file mode 100644
index 000000000000..82a2d60376f7
--- /dev/null
+++ b/arch/arm/boot/dts/marvell/kirkwood-dns320l.dts
@@ -0,0 +1,197 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Device Tree file for D-Link DNS-320L
+ *
+ * Copyright (C) 2024, Zoltan HERPAI <wigyori@uid0.hu>
+ * Copyright (C) 2015, Sunke Schluters <sunke-dev@schlueters.de>
+ *
+ * This file is based on the works of:
+ * - Sunke Schluters <sunke-dev@schlueters.de>
+ *   - https://github.com/scus1/dns320l/blob/master/kernel/dts/kirkwood-dns320l.dts
+ * - Andreas Bohler <dev@aboehler.at>:
+ *   - http://www.aboehler.at/doku/doku.php/projects:dns320l
+ *   - http://www.aboehler.at/hg/linux-dns320l/file/ba7a60ad7687/linux-3.12/kirkwood-dns320l.dts
+ */
+
+/dts-v1/;
+
+#include "kirkwood.dtsi"
+#include "kirkwood-6281.dtsi"
+
+/ {
+	model = "D-Link DNS-320L";
+	compatible = "dlink,dns320l", "marvell,kirkwood-88f6702", "marvell,kirkwood";
+
+	memory {
+		device_type = "memory";
+		reg = <0x00000000 0x10000000>;
+	};
+
+	chosen {
+		bootargs = "console=ttyS0,115200n8 earlyprintk";
+		stdout-path = &uart0;
+	};
+
+	gpio-keys {
+		compatible = "gpio-keys";
+		#address-cells = <1>;
+		#size-cells = <0>;
+		pinctrl-0 = <&pmx_buttons>;
+		pinctrl-names = "default";
+
+		button@1 {
+			label = "Reset push button";
+			linux,code = <KEY_RESTART>;
+			gpios = <&gpio0 28 1>;
+		};
+
+		button@2 {
+			label = "USB unmount button";
+			linux,code = <KEY_EJECTCD>;
+			gpios = <&gpio0 27 1>;
+		};
+	};
+
+	gpio-leds {
+		compatible = "gpio-leds";
+		pinctrl-0 = <&pmx_leds>;
+		pinctrl-names = "default";
+
+		blue-usb {
+			label = "dns320l:usb:blue";
+			gpios = <&gpio0 25 GPIO_ACTIVE_HIGH>;
+			linux,default-trigger = "usbport";
+		};
+
+		orange-usb {
+			label = "dns320l:usb:orange";
+			gpios = <&gpio0 26 GPIO_ACTIVE_HIGH>;
+		};
+
+		orange-l-hdd {
+			label = "dns320l:orange:l_hdd";
+			gpios = <&gpio0 23 GPIO_ACTIVE_HIGH>;
+		};
+
+		orange-r-hdd {
+			label = "dns320l:orange:r_hdd";
+			gpios = <&gpio0 22 GPIO_ACTIVE_HIGH>;
+		};
+	};
+
+	ocp@f1000000 {
+		sata@80000 {
+			status = "okay";
+			nr-ports = <2>;
+		};
+
+		serial@12000 {
+			status = "okay";
+		};
+
+		serial@12100 {
+			pinctrl-0 = <&pmx_uart1>;
+			pinctrl-names = "default";
+			status = "okay";
+		};
+	};
+
+	regulators {
+		compatible = "simple-bus";
+		#address-cells = <1>;
+		#size-cells = <0>;
+		pinctrl-0 = <&pmx_power_sata>;
+		pinctrl-names = "default";
+
+		sata_power: regulator@1 {
+			compatible = "regulator-fixed";
+			reg = <1>;
+			regulator-name = "SATA Power";
+			regulator-min-microvolt = <5000000>;
+			regulator-max-microvolt = <5000000>;
+			enable-active-high;
+			regulator-always-on;
+			regulator-boot-on;
+			gpio = <&gpio0 24 0>;
+		};
+	};
+};
+
+&nand {
+	pinctrl-0 = <&pmx_nand>;
+	pinctrl-names = "default";
+	chip-delay = <40>;
+	status = "okay";
+
+	partition@0 {
+		label = "u-boot";
+		reg = <0x0000000 0x100000>;
+	};
+
+	partition@100000 {
+		label = "ubootenv";
+		reg = <0x100000 0x20000>;
+	};
+
+	partition@120000 {
+		label = "ubi";
+		reg = <0x120000 0x6de0000>;
+	};
+
+	partition@6f00000 {
+		label = "mini firmware";
+		reg = <0x6f00000 0xa00000>;
+	};
+
+	partition@7900000 {
+		label = "config";
+		reg = <0x7900000 0x500000>;
+	};
+
+	partition@7e00000 {
+		label = "my-dlink";
+		reg = <0x7e00000 0x200000>;
+	};
+};
+
+&mdio {
+	status = "okay";
+
+	ethphy0: ethernet-phy@0 {
+		reg = <0>;
+	};
+};
+
+&pinctrl {
+	pmx_sata1: pmx-sata1 {
+		marvell,pins = "mpp20";
+		marvell,function = "sata1";
+	};
+
+	pmx_sata0: pmx-sata0 {
+		marvell,pins = "mpp21";
+		marvell,function = "sata0";
+	};
+
+	pmx_power_sata: pmx-power-sata {
+		marvell,pins = "mpp24";
+		marvell,function = "gpio";
+	};
+
+	pmx_leds: pmx-leds {
+		marvell,pins = "mpp22", "mpp23", "mpp25", "mpp26";
+		marvell,function = "gpio";
+	};
+
+	pmx_buttons: pmx-buttons {
+		marvell,pins = "mpp27", "mpp28", "mpp29";
+		marvell,function = "gpio";
+	};
+};
+
+&eth0 {
+	status = "okay";
+	ethernet0-port@0 {
+		phy-handle = <&ethphy0>;
+	};
+};
-- 
2.30.2

Re: [PATCH 2/3] dt-bindings: marvell: typo fix in SoC name

[Andrew Lunn]

On Sat, Jun 29, 2024 at 01:34:49PM +0200, Zoltan HERPAI wrote:
> Fix typo in the SoC name of Marvell Kirkwood. This change is in the
> description which shouldn't affect functionality.
> 
> Signed-off-by: Zoltan HERPAI <wigyori@uid0.hu>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew

Re: [PATCH 3/3] ARM: dts: marvell: add support for D-Link DNS-320L

[Andrew Lunn]

It has been a long time since i reviewed a kirkwood DT
description. Also, best practices have changed, so the example you
copied is probably doing things which today would be classed as wrong.

> +// SPDX-License-Identifier: GPL-2.0-or-later

It is typical to use a dual license now:

// SPDX-License-Identifier: (GPL-2.0+ OR MIT)

However, if this is mostly Sunke Schluters work, you probably cannot
change the license without his agreement.

> +/*
> + * Device Tree file for D-Link DNS-320L
> + *
> + * Copyright (C) 2024, Zoltan HERPAI <wigyori@uid0.hu>
> + * Copyright (C) 2015, Sunke Schluters <sunke-dev@schlueters.de>
> + *
> + * This file is based on the works of:
> + * - Sunke Schluters <sunke-dev@schlueters.de>
> + *   - https://github.com/scus1/dns320l/blob/master/kernel/dts/kirkwood-dns320l.dts
> + * - Andreas Bohler <dev@aboehler.at>:
> + *   - http://www.aboehler.at/doku/doku.php/projects:dns320l
> + *   - http://www.aboehler.at/hg/linux-dns320l/file/ba7a60ad7687/linux-3.12/kirkwood-dns320l.dts
> + */
> +
> +/dts-v1/;
> +
> +#include "kirkwood.dtsi"
> +#include "kirkwood-6281.dtsi"
> +
> +/ {
> +	model = "D-Link DNS-320L";
> +	compatible = "dlink,dns320l", "marvell,kirkwood-88f6702", "marvell,kirkwood";
> +
> +	memory {
> +		device_type = "memory";
> +		reg = <0x00000000 0x10000000>;
> +	};

> +
> +	chosen {
> +		bootargs = "console=ttyS0,115200n8 earlyprintk";
> +		stdout-path = &uart0;

I _think_ current best practice is to make the serial port speed part
of the stdout-path

	stdout-path = &uart0:115200n8;

Also, earlyprintk is a debug flag, it should not be needed for a
production DT file.

> +	};
> +
> +	gpio-keys {
> +		compatible = "gpio-keys";
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +		pinctrl-0 = <&pmx_buttons>;
> +		pinctrl-names = "default";
> +
> +		button@1 {
> +			label = "Reset push button";
> +			linux,code = <KEY_RESTART>;
> +			gpios = <&gpio0 28 1>;

Please use GPIO_ACTIVE_LOW rather than 1. Also for other places gpios
are used.

Thanks

	Andrew

Re: [PATCH 3/3] ARM: dts: marvell: add support for D-Link DNS-320L

[Krzysztof Kozlowski]

On 29/06/2024 13:34, Zoltan HERPAI wrote:
> Dual-slot NAS based on Marvell Kirkwood.
> 
> Specifications:
>  - Marvell 88F6702 @1GHz
>  - 256Mb RAM
>  - 128Mb NAND
>  - 1x GbE LAN (Marvell 88E1318R)
>  - 1x USB 2.0
>  - 2x SATA
>  - Weltrend WT69P3 ("supervisor" MCU chip)
>  - Serial on J2 (115200,8n1)
>  - Newer bootROM so kwboot-ing via serial is possible
> 
> Signed-off-by: Zoltan HERPAI <wigyori@uid0.hu>
> ---
>  arch/arm/boot/dts/marvell/Makefile            |   1 +
>  .../arm/boot/dts/marvell/kirkwood-dns320l.dts | 197 ++++++++++++++++++
>  2 files changed, 198 insertions(+)
>  create mode 100644 arch/arm/boot/dts/marvell/kirkwood-dns320l.dts
> 
> diff --git a/arch/arm/boot/dts/marvell/Makefile b/arch/arm/boot/dts/marvell/Makefile
> index 1e0f5ff492f7..cadd4039b783 100644
> --- a/arch/arm/boot/dts/marvell/Makefile
> +++ b/arch/arm/boot/dts/marvell/Makefile
> @@ -92,6 +92,7 @@ dtb-$(CONFIG_MACH_KIRKWOOD) += \
>  	kirkwood-db-88f6282.dtb \
>  	kirkwood-dir665.dtb \
>  	kirkwood-dns320.dtb \
> +	kirkwood-dns320l.dtb \
>  	kirkwood-dns325.dtb \
>  	kirkwood-dockstar.dtb \
>  	kirkwood-dreamplug.dtb \
> diff --git a/arch/arm/boot/dts/marvell/kirkwood-dns320l.dts b/arch/arm/boot/dts/marvell/kirkwood-dns320l.dts
> new file mode 100644
> index 000000000000..82a2d60376f7
> --- /dev/null
> +++ b/arch/arm/boot/dts/marvell/kirkwood-dns320l.dts
> @@ -0,0 +1,197 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +/*
> + * Device Tree file for D-Link DNS-320L
> + *
> + * Copyright (C) 2024, Zoltan HERPAI <wigyori@uid0.hu>
> + * Copyright (C) 2015, Sunke Schluters <sunke-dev@schlueters.de>
> + *
> + * This file is based on the works of:
> + * - Sunke Schluters <sunke-dev@schlueters.de>
> + *   - https://github.com/scus1/dns320l/blob/master/kernel/dts/kirkwood-dns320l.dts
> + * - Andreas Bohler <dev@aboehler.at>:
> + *   - http://www.aboehler.at/doku/doku.php/projects:dns320l
> + *   - http://www.aboehler.at/hg/linux-dns320l/file/ba7a60ad7687/linux-3.12/kirkwood-dns320l.dts
> + */
> +
> +/dts-v1/;
> +
> +#include "kirkwood.dtsi"
> +#include "kirkwood-6281.dtsi"
> +
> +/ {
> +	model = "D-Link DNS-320L";
> +	compatible = "dlink,dns320l", "marvell,kirkwood-88f6702", "marvell,kirkwood";
> +
> +	memory {
> +		device_type = "memory";
> +		reg = <0x00000000 0x10000000>;
> +	};
> +
> +	chosen {
> +		bootargs = "console=ttyS0,115200n8 earlyprintk";
> +		stdout-path = &uart0;
> +	};
> +
> +	gpio-keys {
> +		compatible = "gpio-keys";
> +		#address-cells = <1>;
> +		#size-cells = <0>;

Nope, these cannot be there.

It does not look like you tested the DTS against bindings. Please run
`make dtbs_check W=1` (see
Documentation/devicetree/bindings/writing-schema.rst or
https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
for instructions).

> +		pinctrl-0 = <&pmx_buttons>;
> +		pinctrl-names = "default";
> +
> +		button@1 {
> +			label = "Reset push button";
> +			linux,code = <KEY_RESTART>;
> +			gpios = <&gpio0 28 1>;
> +		};
> +
> +		button@2 {
> +			label = "USB unmount button";
> +			linux,code = <KEY_EJECTCD>;
> +			gpios = <&gpio0 27 1>;
> +		};
> +	};
> +
> +	gpio-leds {
> +		compatible = "gpio-leds";
> +		pinctrl-0 = <&pmx_leds>;
> +		pinctrl-names = "default";
> +
> +		blue-usb {

It does not look like you tested the DTS against bindings. Please run
`make dtbs_check W=1` (see
Documentation/devicetree/bindings/writing-schema.rst or
https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
for instructions).

> +			label = "dns320l:usb:blue";
> +			gpios = <&gpio0 25 GPIO_ACTIVE_HIGH>;
> +			linux,default-trigger = "usbport";
> +		};
> +
> +		orange-usb {
> +			label = "dns320l:usb:orange";
> +			gpios = <&gpio0 26 GPIO_ACTIVE_HIGH>;
> +		};
> +
> +		orange-l-hdd {
> +			label = "dns320l:orange:l_hdd";
> +			gpios = <&gpio0 23 GPIO_ACTIVE_HIGH>;
> +		};
> +
> +		orange-r-hdd {
> +			label = "dns320l:orange:r_hdd";
> +			gpios = <&gpio0 22 GPIO_ACTIVE_HIGH>;
> +		};
> +	};
> +
> +	ocp@f1000000 {

Why you are not overriding by label/phandle?

> +		sata@80000 {
> +			status = "okay";
> +			nr-ports = <2>;
> +		};
> +
> +		serial@12000 {
> +			status = "okay";
> +		};
> +
> +		serial@12100 {
> +			pinctrl-0 = <&pmx_uart1>;
> +			pinctrl-names = "default";
> +			status = "okay";
> +		};
> +	};
> +
> +	regulators {

No, there is no such node/bus. Drop

> +		compatible = "simple-bus";
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +		pinctrl-0 = <&pmx_power_sata>;
> +		pinctrl-names = "default";
> +
> +		sata_power: regulator@1 {
> +			compatible = "regulator-fixed";
> +			reg = <1>;
> +			regulator-name = "SATA Power";
> +			regulator-min-microvolt = <5000000>;
> +			regulator-max-microvolt = <5000000>;
> +			enable-active-high;
> +			regulator-always-on;
> +			regulator-boot-on;
> +			gpio = <&gpio0 24 0>;
> +		};
> +	};
> +};
> +


Best regards,
Krzysztof

Re: [PATCH 3/3] ARM: dts: marvell: add support for D-Link DNS-320L

[Zoltan Herpai]

Hi Andrew,

On 06/29/2024 16:34, Andrew Lunn wrote:
> It has been a long time since i reviewed a kirkwood DT
> description. Also, best practices have changed, so the example you
> copied is probably doing things which today would be classed as wrong.
>
>> +// SPDX-License-Identifier: GPL-2.0-or-later
> It is typical to use a dual license now:
>
> // SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>
> However, if this is mostly Sunke Schluters work, you probably cannot
> change the license without his agreement.
That's correct, a significant portion of the DTS was taken from Sunke, 
and there is at least one other DTS with this license, so I didn't want 
to change this.

>> +/*
>> + * Device Tree file for D-Link DNS-320L
>> + *
>> + * Copyright (C) 2024, Zoltan HERPAI<wigyori@uid0.hu>
>> + * Copyright (C) 2015, Sunke Schluters<sunke-dev@schlueters.de>
>> + *
>> + * This file is based on the works of:
>> + * - Sunke Schluters<sunke-dev@schlueters.de>
>> + *   -https://github.com/scus1/dns320l/blob/master/kernel/dts/kirkwood-dns320l.dts
>> + * - Andreas Bohler<dev@aboehler.at>:
>> + *   -http://www.aboehler.at/doku/doku.php/projects:dns320l
>> + *   -http://www.aboehler.at/hg/linux-dns320l/file/ba7a60ad7687/linux-3.12/kirkwood-dns320l.dts
>> + */
>> +
>> +/dts-v1/;
>> +
>> +#include "kirkwood.dtsi"
>> +#include "kirkwood-6281.dtsi"
>> +
>> +/ {
>> +	model = "D-Link DNS-320L";
>> +	compatible = "dlink,dns320l", "marvell,kirkwood-88f6702", "marvell,kirkwood";
>> +
>> +	memory {
>> +		device_type = "memory";
>> +		reg = <0x00000000 0x10000000>;
>> +	};
>> +
>> +	chosen {
>> +		bootargs = "console=ttyS0,115200n8 earlyprintk";
>> +		stdout-path = &uart0;
> I _think_ current best practice is to make the serial port speed part
> of the stdout-path
>
> 	stdout-path = &uart0:115200n8;
>
> Also, earlyprintk is a debug flag, it should not be needed for a
> production DT file.
Indeed, I'll update these.

>> +	};
>> +
>> +	gpio-keys {
>> +		compatible = "gpio-keys";
>> +		#address-cells = <1>;
>> +		#size-cells = <0>;
>> +		pinctrl-0 = <&pmx_buttons>;
>> +		pinctrl-names = "default";
>> +
>> +		button@1 {
>> +			label = "Reset push button";
>> +			linux,code = <KEY_RESTART>;
>> +			gpios = <&gpio0 28 1>;
> Please use GPIO_ACTIVE_LOW rather than 1. Also for other places gpios
> are used.
Ack, will send a v2 with these updates - thanks for reviewing.

Regards,
Zoltan Herpai

Re: [PATCH 3/3] ARM: dts: marvell: add support for D-Link DNS-320L

[Andrew Lunn]

On Mon, Jul 01, 2024 at 10:15:40AM +0200, Zoltan Herpai wrote:
> Hi Andrew,
> 
> On 06/29/2024 16:34, Andrew Lunn wrote:
> > It has been a long time since i reviewed a kirkwood DT
> > description. Also, best practices have changed, so the example you
> > copied is probably doing things which today would be classed as wrong.
> > 
> > > +// SPDX-License-Identifier: GPL-2.0-or-later
> > It is typical to use a dual license now:
> > 
> > // SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> > 
> > However, if this is mostly Sunke Schluters work, you probably cannot
> > change the license without his agreement.
> That's correct, a significant portion of the DTS was taken from Sunke, and
> there is at least one other DTS with this license, so I didn't want to
> change this.

O.K. Please at least Cc: Sunke on the next revision anyway.

     Andrew

Re: [PATCH 3/3] ARM: dts: marvell: add support for D-Link DNS-320L

[Andrew Lunn]

On Mon, Jul 01, 2024 at 08:01:46AM +0200, Krzysztof Kozlowski wrote:
> On 29/06/2024 13:34, Zoltan HERPAI wrote:

> > +++ b/arch/arm/boot/dts/marvell/kirkwood-dns320l.dts
> > @@ -0,0 +1,197 @@
> > +// SPDX-License-Identifier: GPL-2.0-or-later
> > +/*
> > + * Device Tree file for D-Link DNS-320L
> > + *
> > + * Copyright (C) 2024, Zoltan HERPAI <wigyori@uid0.hu>
> > + * Copyright (C) 2015, Sunke Schluters <sunke-dev@schlueters.de>
> > + *
> > + * This file is based on the works of:
> > + * - Sunke Schluters <sunke-dev@schlueters.de>
> > + *   - https://github.com/scus1/dns320l/blob/master/kernel/dts/kirkwood-dns320l.dts
> > + * - Andreas Bohler <dev@aboehler.at>:
> > + *   - http://www.aboehler.at/doku/doku.php/projects:dns320l
> > + *   - http://www.aboehler.at/hg/linux-dns320l/file/ba7a60ad7687/linux-3.12/kirkwood-dns320l.dts
> > + */
> > +
> > +/dts-v1/;
> > +
> > +#include "kirkwood.dtsi"
> > +#include "kirkwood-6281.dtsi"
> > +
> > +/ {
> > +	model = "D-Link DNS-320L";
> > +	compatible = "dlink,dns320l", "marvell,kirkwood-88f6702", "marvell,kirkwood";
> > +
> > +	memory {
> > +		device_type = "memory";
> > +		reg = <0x00000000 0x10000000>;
> > +	};
> > +
> > +	chosen {
> > +		bootargs = "console=ttyS0,115200n8 earlyprintk";
> > +		stdout-path = &uart0;
> > +	};
> > +
> > +	gpio-keys {
> > +		compatible = "gpio-keys";
> > +		#address-cells = <1>;
> > +		#size-cells = <0>;
> 
> Nope, these cannot be there.

Depends. The kernel, which is what really matters, is happy with them
there. Have a look at all the other kirkwood dts files.

> It does not look like you tested the DTS against bindings. Please run
> `make dtbs_check W=1` (see
> Documentation/devicetree/bindings/writing-schema.rst or
> https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
> for instructions).

I suspect that is not going to be easy to interpret. kirkwood is very
old, much older than the YAML descriptions. DT descriptions of this
age were considered correct if the kernel understood them, and the
kernel is much more flexible than the YAML bindings. As a result,
there are going to be a huge number of warnings, and it will take a
lot of skill to pick out real warning which can be fixed from the
noise. Also, nobody really cares, because these devices have been out
of production for a decade. Nobody is going to clean up the DT files.

> > +		pinctrl-0 = <&pmx_buttons>;
> > +		pinctrl-names = "default";
> > +
> > +		button@1 {
> > +			label = "Reset push button";
> > +			linux,code = <KEY_RESTART>;
> > +			gpios = <&gpio0 28 1>;
> > +		};
> > +
> > +		button@2 {
> > +			label = "USB unmount button";
> > +			linux,code = <KEY_EJECTCD>;
> > +			gpios = <&gpio0 27 1>;
> > +		};
> > +	};
> > +
> > +	gpio-leds {
> > +		compatible = "gpio-leds";
> > +		pinctrl-0 = <&pmx_leds>;
> > +		pinctrl-names = "default";
> > +
> > +		blue-usb {
> 
> It does not look like you tested the DTS against bindings. Please run
> `make dtbs_check W=1` (see
> Documentation/devicetree/bindings/writing-schema.rst or
> https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
> for instructions).

> > +	ocp@f1000000 {
> 
> Why you are not overriding by label/phandle?

Look at the old .dts files. That is the way it was done 10 years
ago. This is uniform with other kirkwood .dts files. There is
something to be said for being uniform with other files of the same
sort.

I think we need to find a balance here. I agree with some of your
comments, removing the regulator container, moving gpio-keys and
gpio-leds. But i think ocp@f1000000 and not using labels can stay.

	Andrew

Re: [PATCH 3/3] ARM: dts: marvell: add support for D-Link DNS-320L

[Krzysztof Kozlowski]

On 01/07/2024 15:25, Andrew Lunn wrote:
> On Mon, Jul 01, 2024 at 08:01:46AM +0200, Krzysztof Kozlowski wrote:
>> On 29/06/2024 13:34, Zoltan HERPAI wrote:
> 
>>> +++ b/arch/arm/boot/dts/marvell/kirkwood-dns320l.dts
>>> @@ -0,0 +1,197 @@
>>> +// SPDX-License-Identifier: GPL-2.0-or-later
>>> +/*
>>> + * Device Tree file for D-Link DNS-320L
>>> + *
>>> + * Copyright (C) 2024, Zoltan HERPAI <wigyori@uid0.hu>
>>> + * Copyright (C) 2015, Sunke Schluters <sunke-dev@schlueters.de>
>>> + *
>>> + * This file is based on the works of:
>>> + * - Sunke Schluters <sunke-dev@schlueters.de>
>>> + *   - https://github.com/scus1/dns320l/blob/master/kernel/dts/kirkwood-dns320l.dts
>>> + * - Andreas Bohler <dev@aboehler.at>:
>>> + *   - http://www.aboehler.at/doku/doku.php/projects:dns320l
>>> + *   - http://www.aboehler.at/hg/linux-dns320l/file/ba7a60ad7687/linux-3.12/kirkwood-dns320l.dts
>>> + */
>>> +
>>> +/dts-v1/;
>>> +
>>> +#include "kirkwood.dtsi"
>>> +#include "kirkwood-6281.dtsi"
>>> +
>>> +/ {
>>> +	model = "D-Link DNS-320L";
>>> +	compatible = "dlink,dns320l", "marvell,kirkwood-88f6702", "marvell,kirkwood";
>>> +
>>> +	memory {
>>> +		device_type = "memory";
>>> +		reg = <0x00000000 0x10000000>;
>>> +	};
>>> +
>>> +	chosen {
>>> +		bootargs = "console=ttyS0,115200n8 earlyprintk";
>>> +		stdout-path = &uart0;
>>> +	};
>>> +
>>> +	gpio-keys {
>>> +		compatible = "gpio-keys";
>>> +		#address-cells = <1>;
>>> +		#size-cells = <0>;
>>
>> Nope, these cannot be there.
> 
> Depends. The kernel, which is what really matters, is happy with them
> there. Have a look at all the other kirkwood dts files.

They cannot be here because the binding does not allow it. They are
redundant though, in a way that kernel will work.

> 
>> It does not look like you tested the DTS against bindings. Please run
>> `make dtbs_check W=1` (see
>> Documentation/devicetree/bindings/writing-schema.rst or
>> https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
>> for instructions).
> 
> I suspect that is not going to be easy to interpret. kirkwood is very
> old, much older than the YAML descriptions. DT descriptions of this
> age were considered correct if the kernel understood them, and the
> kernel is much more flexible than the YAML bindings. As a result,
> there are going to be a huge number of warnings, and it will take a
> lot of skill to pick out real warning which can be fixed from the
> noise. Also, nobody really cares, because these devices have been out
> of production for a decade. Nobody is going to clean up the DT files.

One can just read the binding. Is there address/size cells?

That's true that you need cleaned up platform to make efficient use of
the tools, but one can test one particular schema which would print just
limited amount of bindings.

> 
>>> +		pinctrl-0 = <&pmx_buttons>;
>>> +		pinctrl-names = "default";
>>> +
>>> +		button@1 {
>>> +			label = "Reset push button";
>>> +			linux,code = <KEY_RESTART>;
>>> +			gpios = <&gpio0 28 1>;
>>> +		};
>>> +
>>> +		button@2 {
>>> +			label = "USB unmount button";
>>> +			linux,code = <KEY_EJECTCD>;
>>> +			gpios = <&gpio0 27 1>;
>>> +		};
>>> +	};
>>> +
>>> +	gpio-leds {
>>> +		compatible = "gpio-leds";
>>> +		pinctrl-0 = <&pmx_leds>;
>>> +		pinctrl-names = "default";
>>> +
>>> +		blue-usb {
>>
>> It does not look like you tested the DTS against bindings. Please run
>> `make dtbs_check W=1` (see
>> Documentation/devicetree/bindings/writing-schema.rst or
>> https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
>> for instructions).
> 
>>> +	ocp@f1000000 {
>>
>> Why you are not overriding by label/phandle?
> 
> Look at the old .dts files. That is the way it was done 10 years
> ago. This is uniform with other kirkwood .dts files. There is
> something to be said for being uniform with other files of the same
> sort.

That's ok.

> 

Best regards,
Krzysztof

Re: [PATCH 3/3] ARM: dts: marvell: add support for D-Link DNS-320L

[Andrew Lunn]

> > I suspect that is not going to be easy to interpret. kirkwood is very
> > old, much older than the YAML descriptions. DT descriptions of this
> > age were considered correct if the kernel understood them, and the
> > kernel is much more flexible than the YAML bindings. As a result,
> > there are going to be a huge number of warnings, and it will take a
> > lot of skill to pick out real warning which can be fixed from the
> > noise. Also, nobody really cares, because these devices have been out
> > of production for a decade. Nobody is going to clean up the DT files.
> 
> One can just read the binding. Is there address/size cells?

And i suspect this developer also does not know that. This is a
typical cut/paste .dts file, copied from a 10 year old working
example. If the tools are not helpful, we Maintainers need to fill in
the gaps.

In this case, I would suggest looking in
arch/arm64/boot/dts/marvell/*.dts and follow what is done there. These
files are more modern, so are more likely to follow the bindings. Look
where the gpip-keys and gpip-led nodes are placed.

      Andrew

Re: [PATCH 3/3] ARM: dts: marvell: add support for D-Link DNS-320L

[Krzysztof Kozlowski]

On 01/07/2024 15:29, Krzysztof Kozlowski wrote:
> 
>>
>>> It does not look like you tested the DTS against bindings. Please run
>>> `make dtbs_check W=1` (see
>>> Documentation/devicetree/bindings/writing-schema.rst or
>>> https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
>>> for instructions).
>>
>> I suspect that is not going to be easy to interpret. kirkwood is very
>> old, much older than the YAML descriptions. DT descriptions of this
>> age were considered correct if the kernel understood them, and the
>> kernel is much more flexible than the YAML bindings. As a result,
>> there are going to be a huge number of warnings, and it will take a
>> lot of skill to pick out real warning which can be fixed from the
>> noise. Also, nobody really cares, because these devices have been out
>> of production for a decade. Nobody is going to clean up the DT files.
> 
> One can just read the binding. Is there address/size cells?
> 
> That's true that you need cleaned up platform to make efficient use of
> the tools, but one can test one particular schema which would print just
> limited amount of bindings.

BTW, I'll fix up few trivial things in Kirkwood and Orion files.

Best regards,
Krzysztof

Re: [PATCH 3/3] ARM: dts: marvell: add support for D-Link DNS-320L

[Rob Herring]

On Sat, Jun 29, 2024 at 04:34:25PM +0200, Andrew Lunn wrote:
> It has been a long time since i reviewed a kirkwood DT
> description. Also, best practices have changed, so the example you
> copied is probably doing things which today would be classed as wrong.
> 
> > +// SPDX-License-Identifier: GPL-2.0-or-later
> 
> It is typical to use a dual license now:
> 
> // SPDX-License-Identifier: (GPL-2.0+ OR MIT)

Yes, but more importantly, they should be consistent...

GPL-2.0+ in favor of GPL-2.0-or-later

> 
> However, if this is mostly Sunke Schluters work, you probably cannot
> change the license without his agreement.
> 
> > +/*
> > + * Device Tree file for D-Link DNS-320L
> > + *
> > + * Copyright (C) 2024, Zoltan HERPAI <wigyori@uid0.hu>
> > + * Copyright (C) 2015, Sunke Schluters <sunke-dev@schlueters.de>
> > + *
> > + * This file is based on the works of:
> > + * - Sunke Schluters <sunke-dev@schlueters.de>
> > + *   - https://github.com/scus1/dns320l/blob/master/kernel/dts/kirkwood-dns320l.dts
> > + * - Andreas Bohler <dev@aboehler.at>:
> > + *   - http://www.aboehler.at/doku/doku.php/projects:dns320l
> > + *   - http://www.aboehler.at/hg/linux-dns320l/file/ba7a60ad7687/linux-3.12/kirkwood-dns320l.dts
> > + */
> > +
> > +/dts-v1/;
> > +
> > +#include "kirkwood.dtsi"
> > +#include "kirkwood-6281.dtsi"

And these are GPL-2.0-only, so this one should match.

Rob

Re: [PATCH 1/3] dt-bindings: marvell: add D-Link DNS-320L

[Rob Herring (Arm)]

On Sat, 29 Jun 2024 13:34:48 +0200, Zoltan HERPAI wrote:
> Add DT-compatible for the D-Link DNS-320L NAS.
> 
> Signed-off-by: Zoltan HERPAI <wigyori@uid0.hu>
> ---
>  .../devicetree/bindings/arm/marvell/marvell,kirkwood.txt         | 1 +
>  1 file changed, 1 insertion(+)
> 


My bot found new DTB warnings on the .dts files added or changed in this
series.

Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.

If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:

  pip3 install dtschema --upgrade


New warnings running 'make CHECK_DTBS=y marvell/kirkwood-dns320l.dtb' for 20240629113450.127561-1-wigyori@uid0.hu:

arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /: memory: False schema does not allow {'device_type': ['memory'], 'reg': [[0, 268435456]]}
	from schema $id: http://devicetree.org/schemas/root-node.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /: failed to match any schema with compatible: ['dlink,dns320l', 'marvell,kirkwood-88f6702', 'marvell,kirkwood']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /: failed to match any schema with compatible: ['dlink,dns320l', 'marvell,kirkwood-88f6702', 'marvell,kirkwood']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /: failed to match any schema with compatible: ['dlink,dns320l', 'marvell,kirkwood-88f6702', 'marvell,kirkwood']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: mbus@f1000000: $nodename:0: 'mbus@f1000000' does not match '^([a-z][a-z0-9\-]+-bus|bus|localbus|soc|axi|ahb|apb)(@.+)?$'
	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /mbus@f1000000: failed to match any schema with compatible: ['marvell,kirkwood-mbus', 'simple-bus']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /mbus@f1000000/nand@12f: failed to match any schema with compatible: ['marvell,orion-nand']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: sa-sram@301: $nodename:0: 'sa-sram@301' does not match '^sram(@.*)?'
	from schema $id: http://devicetree.org/schemas/sram/sram.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: sa-sram@301: 'ranges' is a required property
	from schema $id: http://devicetree.org/schemas/sram/sram.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: pcie@82000000: pcie@1,0:interrupts:0:0: 9 is not one of [1, 2, 3, 4]
	from schema $id: http://devicetree.org/schemas/pci/pci-bus-common.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: pcie@82000000: pcie@1,0:interrupts: [[9], [44]] is too long
	from schema $id: http://devicetree.org/schemas/pci/pci-bus-common.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /mbus@f1000000/pcie@82000000: failed to match any schema with compatible: ['marvell,kirkwood-pcie']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: ocp@f1000000: $nodename:0: 'ocp@f1000000' does not match '^([a-z][a-z0-9\-]+-bus|bus|localbus|soc|axi|ahb|apb)(@.+)?$'
	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/pin-controller@10000: failed to match any schema with compatible: ['marvell,88f6281-pinctrl']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/core-clocks@10030: failed to match any schema with compatible: ['marvell,kirkwood-core-clock']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/spi@10600: failed to match any schema with compatible: ['marvell,orion-spi']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/mbus-controller@20000: failed to match any schema with compatible: ['marvell,mbus-controller']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/system-controller@20000: failed to match any schema with compatible: ['marvell,orion-system-controller']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/bridge-interrupt-ctrl@20110: failed to match any schema with compatible: ['marvell,orion-bridge-intc']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/clock-gating-control@2011c: failed to match any schema with compatible: ['marvell,kirkwood-gating-clock']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/l2-cache@20128: failed to match any schema with compatible: ['marvell,kirkwood-cache']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/timer@20300: failed to match any schema with compatible: ['marvell,orion-timer']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/watchdog-timer@20300: failed to match any schema with compatible: ['marvell,orion-wdt']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/crypto@30000: failed to match any schema with compatible: ['marvell,kirkwood-crypto']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: ehci@50000: $nodename:0: 'ehci@50000' does not match '^usb(@.*)?'
	from schema $id: http://devicetree.org/schemas/usb/generic-ehci.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/xor@60800: failed to match any schema with compatible: ['marvell,orion-xor']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/xor@60900: failed to match any schema with compatible: ['marvell,orion-xor']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/ethernet-controller@72000: failed to match any schema with compatible: ['marvell,kirkwood-eth']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/ethernet-controller@72000/ethernet0-port@0: failed to match any schema with compatible: ['marvell,kirkwood-eth-port']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/ethernet-controller@76000: failed to match any schema with compatible: ['marvell,kirkwood-eth']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/ethernet-controller@76000/ethernet1-port@0: failed to match any schema with compatible: ['marvell,kirkwood-eth-port']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/sata-phy@82000: failed to match any schema with compatible: ['marvell,mvebu-sata-phy']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/sata-phy@84000: failed to match any schema with compatible: ['marvell,mvebu-sata-phy']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/audio-controller@a0000: failed to match any schema with compatible: ['marvell,kirkwood-audio']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: rtc@10300: compatible:0: 'marvell,kirkwood-rtc' is not one of ['abracon,abb5zes3', 'abracon,abeoz9', 'aspeed,ast2400-rtc', 'aspeed,ast2500-rtc', 'aspeed,ast2600-rtc', 'cnxt,cx92755-rtc', 'dallas,ds1374', 'dallas,ds1672', 'dallas,ds3232', 'emmicro,em3027', 'epson,rx8010', 'epson,rx8025', 'epson,rx8035', 'epson,rx8111', 'epson,rx8571', 'epson,rx8581', 'google,goldfish-rtc', 'isil,isl1208', 'isil,isl1218', 'marvell,orion-rtc', 'maxim,ds1742', 'maxim,mcp795', 'microcrystal,rv3029', 'microcrystal,rv8523', 'nxp,lpc3220-rtc', 'pericom,pt7c4338', 'ricoh,r2025sd', 'ricoh,r2221tl', 'ricoh,rs5c372a', 'ricoh,rs5c372b', 'ricoh,rv5c386', 'ricoh,rv5c387a', 'sii,s35390a', 'st,spear600-rtc', 'via,vt8500-rtc', 'whwave,sd3078', 'xircom,x1205']
	from schema $id: http://devicetree.org/schemas/rtc/trivial-rtc.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: rtc@10300: compatible: ['marvell,kirkwood-rtc', 'marvell,orion-rtc'] is too long
	from schema $id: http://devicetree.org/schemas/rtc/trivial-rtc.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: rtc@10300: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+'
	from schema $id: http://devicetree.org/schemas/rtc/trivial-rtc.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/rtc@10300: failed to match any schema with compatible: ['marvell,kirkwood-rtc', 'marvell,orion-rtc']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: /ocp@f1000000/sata@80000: failed to match any schema with compatible: ['marvell,orion-sata']
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: mvsdio@90000: $nodename:0: 'mvsdio@90000' does not match '^mmc(@.*)?$'
	from schema $id: http://devicetree.org/schemas/mmc/marvell,orion-sdio.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: gpio-keys: '#address-cells', '#size-cells', 'button@1', 'button@2' do not match any of the regexes: '^(button|event|key|switch|(button|event|key|switch)-[a-z0-9-]+|[a-z0-9-]+-(button|event|key|switch))$', 'pinctrl-[0-9]+'
	from schema $id: http://devicetree.org/schemas/input/gpio-keys.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: gpio-leds: 'blue-usb', 'orange-l-hdd', 'orange-r-hdd', 'orange-usb' do not match any of the regexes: '(^led-[0-9a-f]$|led)', 'pinctrl-[0-9]+'
	from schema $id: http://devicetree.org/schemas/leds/leds-gpio.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: regulators: $nodename:0: 'regulators' does not match '^([a-z][a-z0-9\-]+-bus|bus|localbus|soc|axi|ahb|apb)(@.+)?$'
	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: regulators: #size-cells: 0 is not one of [1, 2]
	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: regulators: regulator@1:reg:0: [1] is too short
	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: regulators: 'ranges' is a required property
	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#
arch/arm/boot/dts/marvell/kirkwood-dns320l.dtb: regulator@1: Unevaluated properties are not allowed ('reg' was unexpected)
	from schema $id: http://devicetree.org/schemas/regulator/fixed-regulator.yaml#