arm64: dts: allwinner: pinebook: Fix backlight regulator
authorSamuel Holland <samuel@sholland.org>
Sun, 19 Jan 2020 16:31:03 +0000 (10:31 -0600)
committerMaxime Ripard <maxime@cerno.tech>
Mon, 10 Feb 2020 07:57:42 +0000 (08:57 +0100)
The output from the backlight regulator is labeled as "VBKLT" in the
schematic. Using the equation and resistor values from the schematic,
the output is approximately 18V, not 3.3V. Since the regulator in use
(SS6640STR) is a boost regulator powered by PS (battery or AC input),
which are both >3.3V, the output could not be 3.3V anyway.

Signed-off-by: Samuel Holland <samuel@sholland.org>
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts

index 1ec39120323ff05cdd5b251e43142f94316dc012..313f4e6edc19576ce0289f65cacc35e6ca779274 100644 (file)
                ethernet0 = &rtl8723cs;
        };
 
-       vdd_bl: regulator@0 {
-               compatible = "regulator-fixed";
-               regulator-name = "bl-3v3";
-               regulator-min-microvolt = <3300000>;
-               regulator-max-microvolt = <3300000>;
-               gpio = <&pio 7 6 GPIO_ACTIVE_HIGH>; /* PH6 */
-               enable-active-high;
-       };
-
        backlight: backlight {
                compatible = "pwm-backlight";
                pwms = <&pwm 0 50000 0>;
                brightness-levels = <0 5 10 15 20 30 40 55 70 85 100>;
                default-brightness-level = <2>;
                enable-gpios = <&pio 3 23 GPIO_ACTIVE_HIGH>; /* PD23 */
-               power-supply = <&vdd_bl>;
+               power-supply = <&reg_vbklt>;
        };
 
        chosen {
                };
        };
 
+       reg_vbklt: vbklt {
+               compatible = "regulator-fixed";
+               regulator-name = "vbklt";
+               regulator-min-microvolt = <18000000>;
+               regulator-max-microvolt = <18000000>;
+               gpio = <&pio 7 6 GPIO_ACTIVE_HIGH>; /* PH6 */
+               enable-active-high;
+       };
+
        wifi_pwrseq: wifi_pwrseq {
                compatible = "mmc-pwrseq-simple";
                reset-gpios = <&r_pio 0 2 GPIO_ACTIVE_LOW>; /* PL2 */