[PATCH] arm64: dts: ti: k3-am62p-verdin: fix PWM_3_DSI GPIO direction

Parth Pancholi posted 1 patch 3 months ago
arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] arm64: dts: ti: k3-am62p-verdin: fix PWM_3_DSI GPIO direction
Posted by Parth Pancholi 3 months ago
From: Parth Pancholi <parth.pancholi@toradex.com>

PWM_3_DSI is used as the HDMI Hot-Plug Detect (HPD) GPIO for the Verdin
DSI-to-HDMI adapter. After the commit 33bab9d84e52 ("arm64: dts: ti:
k3-am62p: fix pinctrl settings"), the pin was incorrectly set as output
without RXACTIVE, breaking HPD detection and display functionality.
The issue was previously hidden and worked by chance before the mentioned
pinctrl fix.

Fix the pinmux configuration to correctly set PWM_3_DSI GPIO as an input.

Fixes: 87f95ea316ac ("arm64: dts: ti: Add Toradex Verdin AM62P")
Signed-off-by: Parth Pancholi <parth.pancholi@toradex.com>
---
 arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi b/arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi
index d90d13287076..1a6490572453 100644
--- a/arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi
@@ -426,7 +426,7 @@ AM62PX_IOPAD(0x00f4, PIN_INPUT, 7) /* (Y20) VOUT0_DATA15.GPIO0_60 */ /* WIFI_SPI
 	/* Verdin PWM_3_DSI as GPIO */
 	pinctrl_pwm3_dsi_gpio: main-gpio1-16-default-pins {
 		pinctrl-single,pins = <
-			AM62PX_IOPAD(0x01b8, PIN_OUTPUT, 7) /* (E20) SPI0_CS1.GPIO1_16 */ /* SODIMM 19 */
+			AM62PX_IOPAD(0x01b8, PIN_INPUT, 7) /* (E20) SPI0_CS1.GPIO1_16 */ /* SODIMM 19 */
 		>;
 	};
 
-- 
2.34.1
Re: [PATCH] arm64: dts: ti: k3-am62p-verdin: fix PWM_3_DSI GPIO direction
Posted by Vignesh Raghavendra 3 months ago
Hi Parth Pancholi,

On Thu, 03 Jul 2025 10:45:34 +0200, Parth Pancholi wrote:
> PWM_3_DSI is used as the HDMI Hot-Plug Detect (HPD) GPIO for the Verdin
> DSI-to-HDMI adapter. After the commit 33bab9d84e52 ("arm64: dts: ti:
> k3-am62p: fix pinctrl settings"), the pin was incorrectly set as output
> without RXACTIVE, breaking HPD detection and display functionality.
> The issue was previously hidden and worked by chance before the mentioned
> pinctrl fix.
> 
> [...]

I have applied the following to branch ti-k3-dts-next on [1].
Thank you!

[1/1] arm64: dts: ti: k3-am62p-verdin: fix PWM_3_DSI GPIO direction
      commit: b1a8daa7cf2650637f6cca6aaf014bee89672120

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
--
Vignesh
Re: [PATCH] arm64: dts: ti: k3-am62p-verdin: fix PWM_3_DSI GPIO direction
Posted by Francesco Dolcini 3 months ago
On Thu, Jul 03, 2025 at 10:45:34AM +0200, Parth Pancholi wrote:
> From: Parth Pancholi <parth.pancholi@toradex.com>
> 
> PWM_3_DSI is used as the HDMI Hot-Plug Detect (HPD) GPIO for the Verdin
> DSI-to-HDMI adapter. After the commit 33bab9d84e52 ("arm64: dts: ti:
> k3-am62p: fix pinctrl settings"), the pin was incorrectly set as output
> without RXACTIVE, breaking HPD detection and display functionality.
> The issue was previously hidden and worked by chance before the mentioned
> pinctrl fix.
> 
> Fix the pinmux configuration to correctly set PWM_3_DSI GPIO as an input.
> 
> Fixes: 87f95ea316ac ("arm64: dts: ti: Add Toradex Verdin AM62P")
> Signed-off-by: Parth Pancholi <parth.pancholi@toradex.com>

Reviewed-by: Francesco Dolcini <francesco.dolcini@toradex.com>