arm64: dts: mt8183-kukui: clean up regulator tree

Some regulators in the kukui device tree are modeled incorrectly. Some
are missing supplies and some switches are incorrectly modeled as
voltage regulators. A pass-through was incorrectly modeled as a
regulator.

Add supplies where missing, remove voltage constraints for "switches",
and drop the "bl_pp5000" pass-through.

This depends on commit 2a99858c172e ("arm64: dts: mediatek: mt8183-kukui:
Add PMIC regulator supplies") for reg_vsys.

Fixes: cd894e274b74 ("arm64: dts: mt8183: Add krane-sku176 board")
Fixes: f15722c0fef0 ("arm64: dts: mt8183: Add pwm and backlight node")
Signed-off-by: Chen-Yu Tsai <wenst@chromium.org>
Link: https://lore.kernel.org/r/20240625095441.3474194-1-wenst@chromium.org
Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>

diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi b/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi
index 7c8221fe62eb2e276631153d5adabc1b3f3c5b4f..22924f61ec9ed26536f4940cb4439eedeadf0f89 100644 (file)
--- a/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi
@@ -24,7 +24,7 @@
        backlight_lcd0: backlight_lcd0 {
                compatible = "pwm-backlight";
                pwms = <&pwm0 0 500000>;
-               power-supply = <&bl_pp5000>;
+               power-supply = <&reg_vsys>;
                enable-gpios = <&pio 176 0>;
                brightness-levels = <0 1023>;
                num-interpolated-steps = <1023>;
@@ -47,10 +47,9 @@
        it6505_pp18_reg: regulator0 {
                compatible = "regulator-fixed";
                regulator-name = "it6505_pp18";
-               regulator-min-microvolt = <1800000>;
-               regulator-max-microvolt = <1800000>;
                gpio = <&pio 178 0>;
                enable-active-high;
+               vin-supply = <&pp1800_alw>;
        };
 
        lcd_pp3300: regulator1 {
@@ -62,27 +61,16 @@
                regulator-boot-on;
        };
 
-       bl_pp5000: regulator2 {
-               compatible = "regulator-fixed";
-               regulator-name = "bl_pp5000";
-               regulator-min-microvolt = <5000000>;
-               regulator-max-microvolt = <5000000>;
-               regulator-always-on;
-               regulator-boot-on;
-       };
-
        mmc1_fixed_power: regulator3 {
                compatible = "regulator-fixed";
                regulator-name = "mmc1_power";
-               regulator-min-microvolt = <3300000>;
-               regulator-max-microvolt = <3300000>;
+               vin-supply = <&pp3300_alw>;
        };
 
        mmc1_fixed_io: regulator4 {
                compatible = "regulator-fixed";
                regulator-name = "mmc1_io";
-               regulator-min-microvolt = <1800000>;
-               regulator-max-microvolt = <1800000>;
+               vin-supply = <&pp1800_alw>;
        };
 
        pp1800_alw: regulator5 {
@@ -92,6 +80,7 @@
                regulator-boot-on;
                regulator-min-microvolt = <1800000>;
                regulator-max-microvolt = <1800000>;
+               vin-supply = <&reg_vsys>;
        };
 
        pp3300_alw: regulator6 {
@@ -101,6 +90,7 @@
                regulator-boot-on;
                regulator-min-microvolt = <3300000>;
                regulator-max-microvolt = <3300000>;
+               vin-supply = <&reg_vsys>;
        };
 
        /* system wide semi-regulated power rail from charger */