Commit 30023876 authored Jan 03, 2025 by Frank Wunderlich Committed by Gregory CLEMENT Jan 08, 2025

arm64: dts: marvell: only enable complete sata nodes



The ahci-platform binding requires phys/target-supply property.

After converting the binding to yaml the following files
reporting "'anyOf' conditional failed" on

sata@540000: sata-port@0

armada-7040-db.dts
armada-8040-clearfog-gt-8k.dts
armada-8040-mcbin.dts
armada-8040-mcbin-singleshot.dts
cn9130-db.dts
cn9130-db-B.dts
cn9131-db.dts
cn9131-db-B.dts
cn9132-db.dts
cn9132-db-B.dts

the following files reporting 'anyOf' conditional failed on

sata@540000: sata-port@1

cn9132-db.dts
cn9132-db-B.dts
cn9130-crb-B.dts

'phys' is a required property
'target-supply' is a required property
>From schema: Documentation/devicetree/bindings/ata/ahci-platform.yaml

This is caused by defining sata-ports incomplete in armada-cp11x.dtsi
and overriding only a subset of ports with the needed
phys/target-supply property.

Fix this by disabling the node-templates and enabling the needed nodes.

Signed-off-by: Frank Wunderlich <frank-w@public-files.de>
Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com>

parent 09cdb973

arch/arm64/boot/dts/marvell/armada-7040-db.dts

+1 −0

Original line number	Diff line number	Diff line
		@@ -214,6 +214,7 @@ &cp0_sata0 {

		sata-port@1 {
		phys = <&cp0_comphy3 1>;
		status = "okay";
		};
		};

arch/arm64/boot/dts/marvell/armada-7040-mochabin.dts

+2 −0

Original line number	Diff line number	Diff line
		@@ -433,11 +433,13 @@ &cp0_sata0 {
		/* 7 + 12 SATA connector (J24) */
		sata-port@0 {
		phys = <&cp0_comphy2 0>;
		status = "okay";
		};

		/* M.2-2250 B-key (J39) */
		sata-port@1 {
		phys = <&cp0_comphy3 1>;
		status = "okay";
		};
		};

arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts

+1 −0

Original line number	Diff line number	Diff line
		@@ -475,6 +475,7 @@ &cp1_sata0 {

		sata-port@1 {
		phys = <&cp1_comphy0 1>;
		status = "okay";
		};
		};

arch/arm64/boot/dts/marvell/armada-8040-db.dts

+3 −0

Original line number	Diff line number	Diff line
		@@ -145,9 +145,12 @@ &cp0_sata0 {

		sata-port@0 {
		phys = <&cp0_comphy1 0>;
		status = "okay";
		};

		sata-port@1 {
		phys = <&cp0_comphy3 1>;
		status = "okay";
		};
		};

arch/arm64/boot/dts/marvell/armada-8040-mcbin.dtsi

+1 −0

Original line number	Diff line number	Diff line
		@@ -245,6 +245,7 @@ &cp0_sata0 {
		/* CPM Lane 5 - U29 */
		sata-port@1 {
		phys = <&cp0_comphy5 1>;
		status = "okay";
		};
		};