* [PATCH v2] arm64: dts: qcom: x1-dell-thena: Add missing pinctrl for eDP HPD
@ 2025-10-12 22:40 Val Packett
2025-10-13 8:17 ` Konrad Dybcio
2025-10-27 22:37 ` Bjorn Andersson
0 siblings, 2 replies; 3+ messages in thread
From: Val Packett @ 2025-10-12 22:40 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Bryan O'Donoghue, Val Packett, Laurentiu Tudor
Cc: Konrad Dybcio, linux-arm-msm, devicetree, linux-kernel
The commit a41d23142d87 ("arm64: dts: qcom: x1e80100-dell-xps13-9345:
Add missing pinctrl for eDP HPD") has applied this change to a very
similar machine, so apply it here too.
This allows us not to rely on the boot firmware to set up the pinctrl
for the eDP HPD line of the internal display.
Fixes: e7733b42111c ("arm64: dts: qcom: Add support for Dell Inspiron 7441 / Latitude 7455")
Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Signed-off-by: Val Packett <val@packett.cool>
---
v2: fixed commit msg style, pulled R-b
v1: https://lore.kernel.org/all/20250927032330.21091-1-val@packett.cool/
---
arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi b/arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi
index ef83e87e1b7a..0aad80a1159c 100644
--- a/arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi
+++ b/arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi
@@ -1104,6 +1104,9 @@ &mdss_dp1_out {
&mdss_dp3 {
/delete-property/ #sound-dai-cells;
+ pinctrl-0 = <&edp0_hpd_default>;
+ pinctrl-names = "default";
+
status = "okay";
aux-bus {
--
2.51.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] arm64: dts: qcom: x1-dell-thena: Add missing pinctrl for eDP HPD
2025-10-12 22:40 [PATCH v2] arm64: dts: qcom: x1-dell-thena: Add missing pinctrl for eDP HPD Val Packett
@ 2025-10-13 8:17 ` Konrad Dybcio
2025-10-27 22:37 ` Bjorn Andersson
1 sibling, 0 replies; 3+ messages in thread
From: Konrad Dybcio @ 2025-10-13 8:17 UTC (permalink / raw)
To: Val Packett, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bryan O'Donoghue,
Laurentiu Tudor
Cc: linux-arm-msm, devicetree, linux-kernel
On 10/13/25 12:40 AM, Val Packett wrote:
> The commit a41d23142d87 ("arm64: dts: qcom: x1e80100-dell-xps13-9345:
> Add missing pinctrl for eDP HPD") has applied this change to a very
> similar machine, so apply it here too.
This is somewhat redundant and not very convincing as an opening
argument
> This allows us not to rely on the boot firmware to set up the pinctrl
> for the eDP HPD line of the internal display.
This in turn describes what the problem is and how the patch
alleviates it
Konrad
>
> Fixes: e7733b42111c ("arm64: dts: qcom: Add support for Dell Inspiron 7441 / Latitude 7455")
> Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> Signed-off-by: Val Packett <val@packett.cool>
> ---
> v2: fixed commit msg style, pulled R-b
> v1: https://lore.kernel.org/all/20250927032330.21091-1-val@packett.cool/
> ---
> arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi b/arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi
> index ef83e87e1b7a..0aad80a1159c 100644
> --- a/arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi
> +++ b/arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi
> @@ -1104,6 +1104,9 @@ &mdss_dp1_out {
> &mdss_dp3 {
> /delete-property/ #sound-dai-cells;
>
> + pinctrl-0 = <&edp0_hpd_default>;
> + pinctrl-names = "default";
> +
> status = "okay";
>
> aux-bus {
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] arm64: dts: qcom: x1-dell-thena: Add missing pinctrl for eDP HPD
2025-10-12 22:40 [PATCH v2] arm64: dts: qcom: x1-dell-thena: Add missing pinctrl for eDP HPD Val Packett
2025-10-13 8:17 ` Konrad Dybcio
@ 2025-10-27 22:37 ` Bjorn Andersson
1 sibling, 0 replies; 3+ messages in thread
From: Bjorn Andersson @ 2025-10-27 22:37 UTC (permalink / raw)
To: Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Bryan O'Donoghue, Laurentiu Tudor, Val Packett
Cc: Konrad Dybcio, linux-arm-msm, devicetree, linux-kernel
On Sun, 12 Oct 2025 19:40:08 -0300, Val Packett wrote:
> The commit a41d23142d87 ("arm64: dts: qcom: x1e80100-dell-xps13-9345:
> Add missing pinctrl for eDP HPD") has applied this change to a very
> similar machine, so apply it here too.
>
> This allows us not to rely on the boot firmware to set up the pinctrl
> for the eDP HPD line of the internal display.
>
> [...]
Applied, thanks!
[1/1] arm64: dts: qcom: x1-dell-thena: Add missing pinctrl for eDP HPD
commit: 1bdfe3edd4c862f97ac65b60da1db999981fc52a
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-10-27 22:35 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-12 22:40 [PATCH v2] arm64: dts: qcom: x1-dell-thena: Add missing pinctrl for eDP HPD Val Packett
2025-10-13 8:17 ` Konrad Dybcio
2025-10-27 22:37 ` Bjorn Andersson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).