Skip to content

Commit d77e89b

Browse files
nfrapradoAngeloGioacchino Del Regno
authored andcommitted
arm64: dts: mediatek: mt6357: Drop regulator-fixed compatibles
Some of the regulators in the MT6357 PMIC dtsi have compatible set to regulator-fixed, even though they don't serve any purpose: all those regulators are handled as a whole by the mt6357-regulator driver. In fact this is the only dtsi in this family of chips where this is the case: mt6359 and mt6358 don't have any such compatibles. A side-effect caused by this is that the DT kselftest, which is supposed to identify nodes with compatibles that can be probed, but haven't, shows these nodes as failures. Remove the useless compatibles to move the dtsi in line with the others in its family and fix the DT kselftest failures. Fixes: 55749bb ("arm64: dts: mediatek: add mt6357 device-tree") Signed-off-by: Nícolas F. R. A. Prado <[email protected]> Link: https://lore.kernel.org/r/20250502-mt6357-regulator-fixed-compatibles-removal-v1-1-a582c16743fe@collabora.com Signed-off-by: AngeloGioacchino Del Regno <[email protected]>
1 parent ed34944 commit d77e89b

File tree

1 file changed

+0
-10
lines changed

1 file changed

+0
-10
lines changed

arch/arm64/boot/dts/mediatek/mt6357.dtsi

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,6 @@
6060
};
6161

6262
mt6357_vfe28_reg: ldo-vfe28 {
63-
compatible = "regulator-fixed";
6463
regulator-name = "vfe28";
6564
regulator-min-microvolt = <2800000>;
6665
regulator-max-microvolt = <2800000>;
@@ -75,15 +74,13 @@
7574
};
7675

7776
mt6357_vrf18_reg: ldo-vrf18 {
78-
compatible = "regulator-fixed";
7977
regulator-name = "vrf18";
8078
regulator-min-microvolt = <1800000>;
8179
regulator-max-microvolt = <1800000>;
8280
regulator-enable-ramp-delay = <110>;
8381
};
8482

8583
mt6357_vrf12_reg: ldo-vrf12 {
86-
compatible = "regulator-fixed";
8784
regulator-name = "vrf12";
8885
regulator-min-microvolt = <1200000>;
8986
regulator-max-microvolt = <1200000>;
@@ -112,15 +109,13 @@
112109
};
113110

114111
mt6357_vcn28_reg: ldo-vcn28 {
115-
compatible = "regulator-fixed";
116112
regulator-name = "vcn28";
117113
regulator-min-microvolt = <2800000>;
118114
regulator-max-microvolt = <2800000>;
119115
regulator-enable-ramp-delay = <264>;
120116
};
121117

122118
mt6357_vcn18_reg: ldo-vcn18 {
123-
compatible = "regulator-fixed";
124119
regulator-name = "vcn18";
125120
regulator-min-microvolt = <1800000>;
126121
regulator-max-microvolt = <1800000>;
@@ -142,7 +137,6 @@
142137
};
143138

144139
mt6357_vcamio_reg: ldo-vcamio18 {
145-
compatible = "regulator-fixed";
146140
regulator-name = "vcamio";
147141
regulator-min-microvolt = <1800000>;
148142
regulator-max-microvolt = <1800000>;
@@ -175,31 +169,27 @@
175169
};
176170

177171
mt6357_vaux18_reg: ldo-vaux18 {
178-
compatible = "regulator-fixed";
179172
regulator-name = "vaux18";
180173
regulator-min-microvolt = <1800000>;
181174
regulator-max-microvolt = <1800000>;
182175
regulator-enable-ramp-delay = <264>;
183176
};
184177

185178
mt6357_vaud28_reg: ldo-vaud28 {
186-
compatible = "regulator-fixed";
187179
regulator-name = "vaud28";
188180
regulator-min-microvolt = <2800000>;
189181
regulator-max-microvolt = <2800000>;
190182
regulator-enable-ramp-delay = <264>;
191183
};
192184

193185
mt6357_vio28_reg: ldo-vio28 {
194-
compatible = "regulator-fixed";
195186
regulator-name = "vio28";
196187
regulator-min-microvolt = <2800000>;
197188
regulator-max-microvolt = <2800000>;
198189
regulator-enable-ramp-delay = <264>;
199190
};
200191

201192
mt6357_vio18_reg: ldo-vio18 {
202-
compatible = "regulator-fixed";
203193
regulator-name = "vio18";
204194
regulator-min-microvolt = <1800000>;
205195
regulator-max-microvolt = <1800000>;

0 commit comments

Comments
 (0)