arm64: dts: qcom: sc7280: Hook up the touchscreen IO rail on villager
authorDouglas Anderson <dianders@chromium.org>
Tue, 7 Feb 2023 02:48:12 +0000 (18:48 -0800)
committerBjorn Andersson <andersson@kernel.org>
Thu, 9 Feb 2023 04:09:36 +0000 (20:09 -0800)
On never revs of sc7280-herobrine-villager (rev2+) the L3C rail is
provided to the touchscreen as the IO voltage rail. Let's add it in
the device tree.

NOTE: Even though this is only really needed on rev2+ villagers (-rev0
had non-functioning touchscreen and -rev1 had some hacky hardware
magic), it doesn't actually hurt to do this for old villager revs. As
talked about in the patch ("arm64: dts: qcom: sc7280: On QCard,
regulator L3C should be 1.8V") the L3C regulator didn't go anywhere at
all on older revs. That means that turning it on for older revs
doesn't hurt other than drawing a tiny bit of extra power. Since -rev0
and -rev1 villagers will never make it to real customers and it's nice
not to have too many old device trees, the better tradeoff seems to be
to enable it everywhere.

Signed-off-by: Douglas Anderson <dianders@chromium.org>
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20230206184744.3.I740d409bc5bb69bf4a7b3c4568ea6e7a92f16ccd@changeid
arch/arm64/boot/dts/qcom/sc7280-herobrine-villager.dtsi

index 17553e0..818d404 100644 (file)
@@ -55,6 +55,7 @@ ts_i2c: &i2c13 {
                reset-gpios = <&tlmm 54 GPIO_ACTIVE_LOW>;
 
                vcc33-supply = <&ts_avdd>;
+               vccio-supply = <&ts_avccio>;
        };
 };