On 11/04/2025 13:49, Johan Hovold wrote:
On Wed, Apr 02, 2025 at 03:36:32PM +0100, Christopher Obbard wrote:
Add edp_hpd_active pinctrl to the X1E80100 device tree.
Please squash this one with the patch adding the user.
Signed-off-by: Christopher Obbard <christopher.obb...@linaro.org>
---
arch/arm64/boot/dts/qcom/x1e80100.dtsi | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/x1e80100.dtsi
b/arch/arm64/boot/dts/qcom/x1e80100.dtsi
index
46b79fce92c90d969e3de48bc88e27915d1592bb..5b1b80c445404fd02e9f6e5dab207610b03ff9c5
100644
--- a/arch/arm64/boot/dts/qcom/x1e80100.dtsi
+++ b/arch/arm64/boot/dts/qcom/x1e80100.dtsi
@@ -6389,6 +6389,11 @@ data-pins {
bias-pull-up;
};
};
+
+ edp_hpd_active: edp-hpd-active-state {
Please keep the nodes sorted by name.
+ pins = "gpio119";
+ function = "edp_hot";
There is no "edp_hot" function on x1e so I wonder how this has been
tested.
As I mentioned in a comment to an earlier revision this pin has been
configured by the firmware as "edp0_hot".
With edp_hot, screen stays black with UI comes up, with edp0_hot it works again.
Since there is also an "edp1_hot" pin, this needs to be reflected in the
node name and label.
Pin configurations really do belong in board files, but unfortunately
this was not followed for x1e. You should still include the bias
configuration (somewhere).
Which bias ? do this pin config needs an additional bias config ?
Neil
+ };
};
stm@10002000 {
Johan
