2020-08-17 23:23:49

by Tanmay Shah

[permalink] [raw]
Subject: [PATCH v2] arm64: dts: qcom: sc7180: Add DisplayPort HPD pin dt node

This node defines alternate DP HPD functionality of GPIO.

Signed-off-by: Tanmay Shah <[email protected]>
---
arch/arm64/boot/dts/qcom/sc7180.dtsi | 13 +++++++++++++
1 file changed, 13 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
index bf2f2bb1aa79..0eedf057acc1 100644
--- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
@@ -1457,6 +1457,19 @@ pinconf-sd-cd {
drive-strength = <2>;
};
};
+
+ dp_hot_plug_det: dp-hot-plug-det {
+ pinmux {
+ pins = "gpio117";
+ function = "dp_hot";
+ };
+
+ pinconf {
+ pins = "gpio117";
+ bias-disable;
+ input-enable;
+ };
+ };
};

gpu: gpu@5000000 {

base-commit: 62975d27d647a40c58d3b96c29b911fc4f33c310
prerequisite-patch-id: a4d3e51b6e1200ff7d4550f206db98a92c0a098f
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project


2020-08-18 00:24:09

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH v2] arm64: dts: qcom: sc7180: Add DisplayPort HPD pin dt node

Quoting Tanmay Shah (2020-08-17 15:59:12)
> This node defines alternate DP HPD functionality of GPIO.
>
> Signed-off-by: Tanmay Shah <[email protected]>
> ---
> arch/arm64/boot/dts/qcom/sc7180.dtsi | 13 +++++++++++++
> 1 file changed, 13 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> index bf2f2bb1aa79..0eedf057acc1 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> @@ -1457,6 +1457,19 @@ pinconf-sd-cd {
> drive-strength = <2>;
> };
> };
> +
> + dp_hot_plug_det: dp-hot-plug-det {

And this should be sorted alphabetically instead of put at the end of
the node.

> + pinmux {
> + pins = "gpio117";
> + function = "dp_hot";
> + };
> +
> + pinconf {
> + pins = "gpio117";
> + bias-disable;
> + input-enable;
> + };
> + };
> };
>
> gpu: gpu@5000000 {
>