[PATCH v3] arm64: dts: mt8173: Fix elm panel DT node

From: Pin-Yen Lin
Date: Tue Jun 21 2022 - 05:19:11 EST


Move the panel DT node under `aux-bus` node, so it won't use the buggy
`ps8640_bridge_get_edid` to get the display modes. Also, change the
compatible string from "lg,lp120up1" to "edp-panel", so the panel-edp
driver gets the displays modes from the EDID instead of using the
predefined display mode (`lg_lp129qe_mode`) in the driver, which is
actually not working.

Signed-off-by: Pin-Yen Lin <treapking@xxxxxxxxxxxx>
---

Changes in v3:
- Include more details in the commit message.

Changes in v2:
- Remove the Fixes tag because this change is not compatible with the
old kernel versions.

arch/arm64/boot/dts/mediatek/mt8173-elm.dtsi | 26 +++++++++++---------
1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/arch/arm64/boot/dts/mediatek/mt8173-elm.dtsi b/arch/arm64/boot/dts/mediatek/mt8173-elm.dtsi
index 9c75fbb31f98..1e0802a6f6d2 100644
--- a/arch/arm64/boot/dts/mediatek/mt8173-elm.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8173-elm.dtsi
@@ -90,18 +90,6 @@ volume_up {
};
};

- panel: panel {
- compatible = "lg,lp120up1";
- power-supply = <&panel_fixed_3v3>;
- backlight = <&backlight>;
-
- port {
- panel_in: endpoint {
- remote-endpoint = <&ps8640_out>;
- };
- };
- };
-
panel_fixed_3v3: regulator1 {
compatible = "regulator-fixed";
regulator-name = "PANEL_3V3";
@@ -282,6 +270,20 @@ ps8640_out: endpoint {
};
};
};
+
+ aux-bus {
+ panel {
+ compatible = "edp-panel";
+ power-supply = <&panel_fixed_3v3>;
+ backlight = <&backlight>;
+
+ port {
+ panel_in: endpoint {
+ remote-endpoint = <&ps8640_out>;
+ };
+ };
+ };
+ };
};
};

--
2.37.0.rc0.104.g0611611a94-goog