admin/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2025-12-07 20:06:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

arm64: dts: marvell: only enable complete sata nodes

Browse Source 
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>
This commit is contained in:
Frank Wunderlich
2025-01-03 11:37:21 +01:00
committed by Gregory CLEMENT
parent 09cdb973af
commit 30023876ae
10 changed files with 15 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
arch/arm64/boot/dts/marvell/armada-7040-db.dts
View File

@@ -214,6 +214,7 @@
sata-port@1 {
phys = <&cp0_comphy3 1>;
status = "okay";
};
};

2
arch/arm64/boot/dts/marvell/armada-7040-mochabin.dts
View File

@@ -433,11 +433,13 @@
/* 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";
};
};

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

@@ -475,6 +475,7 @@
sata-port@1 {
phys = <&cp1_comphy0 1>;
status = "okay";
};
};

3
arch/arm64/boot/dts/marvell/armada-8040-db.dts
View File

@@ -145,9 +145,12 @@
sata-port@0 {
phys = <&cp0_comphy1 0>;
status = "okay";
};
sata-port@1 {
phys = <&cp0_comphy3 1>;
status = "okay";
};
};

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

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

2
arch/arm64/boot/dts/marvell/armada-8040-puzzle-m801.dts
View File

@@ -408,10 +408,12 @@
sata-port@0 {
phys = <&cp0_comphy2 0>;
status = "okay";
};
sata-port@1 {
phys = <&cp0_comphy5 1>;
status = "okay";
};
};

2
arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
View File

@@ -347,10 +347,12 @@
sata-port@0 {
reg = <0>;
status = "disabled";
};
sata-port@1 {
reg = <1>;
status = "disabled";
};
};

1
arch/arm64/boot/dts/marvell/cn9130-crb-B.dts
View File

@@ -28,6 +28,7 @@
status = "okay";
/* Generic PHY, providing serdes lanes */
phys = <&cp0_comphy2 0>;
status = "okay";
};
};

1
arch/arm64/boot/dts/marvell/cn9131-db.dtsi
View File

@@ -127,6 +127,7 @@
sata-port@1 {
/* Generic PHY, providing serdes lanes */
phys = <&cp1_comphy5 1>;
status = "okay";
};
};

1
arch/arm64/boot/dts/marvell/cn9132-db.dtsi
View File

@@ -175,6 +175,7 @@
sata-port@0 {
/* Generic PHY, providing serdes lanes */
phys = <&cp2_comphy2 0>;
status = "okay";
};
};