Subject: Re: [PATCH v2 01/11] arm64: dts: ti: k3-am62-verdin: Add Toradex DSI to LVDS adapter with 10.1" display

From: Vitor Soares <[email-masked]>
Date: Fri, 22 May 2026 16:51:31 +0100

Hi,

On Fri, 2026-05-22 at 14:20 +0100, Vitor Soares wrote:
> From: Vitor Soares <[email-masked]>
> 
> Add a device tree overlay for the Toradex DSI to LVDS Adapter with the
> Toradex Capacitive Touch Display 10.1" LVDS. The adapter connects to the
> Verdin DSI_1 interface. It is based on the Texas Instruments SN65DSI84
> DSI-to-LVDS bridge and drives a LogicTechno LT170410-2WHC 10.1" WXGA LVDS
> panel. Touch input is provided by an Atmel MaxTouch capacitive touch
> controller.
> 
> Link:
> https://developer.toradex.com/hardware/accessories/add-ons/dsi-lvds-adapter
> Link:
> https://developer.toradex.com/hardware/accessories/displays/capacitive-touch-display-101inch-lvds
> Assisted-by: Claude:claude-sonnet-4.6
> Signed-off-by: Vitor Soares <[email-masked]>
> 
[...]
> diff --git a/arch/arm64/boot/dts/ti/k3-am625-verdin-dsi-to-lvds-panel-cap-
> touch-10inch.dtso b/arch/arm64/boot/dts/ti/k3-am625-verdin-dsi-to-lvds-panel-
> cap-touch-10inch.dtso
> new file mode 100644
> index 000000000000..0e873f2ccf65
> --- /dev/null
> +++ b/arch/arm64/boot/dts/ti/k3-am625-verdin-dsi-to-lvds-panel-cap-touch-
> 10inch.dtso
> 
[...]
> +
> +       reg_3v3_lvds_bridge: regulator-3v3-lvds-bridge {
> +               compatible = "regulator-fixed";
> +               regulator-max-microvolt = <3300000>;
> +               regulator-min-microvolt = <3300000>;

Sashiko flagged the missing 'regulator-name' property here, 

> +       };
> +};
> +

[...]
> +
> +       touch@4a {

and to use the generic node name 'touchscreen@4a' here instead of
'touch@4a'.

> +               compatible = "atmel,maxtouch";
> +               reg = <0x4a>;
> +               pinctrl-names = "default";
> +               pinctrl-0 = <&pinctrl_dsi1_int>, <&pinctrl_i2s_2_bclk_gpio>;
> +               /* Verdin GPIO_9_DSI (SODIMM 17) - TOUCH_INT# */
> +               interrupt-parent = <&main_gpio1>;
> +               interrupts = <49 IRQ_TYPE_EDGE_FALLING>;
> +               /* Verdin I2S_2_BCLK (SODIMM 42) - TOUCH_RESET# */
> +               reset-gpios = <&main_gpio0 35 GPIO_ACTIVE_LOW>;
> +       };
> +};

I will send a v3 addressing these issues here and where it applies.

Thanks,
Vitor Soares