Re: [PATCH 1/1] arm64: dts: ti: k3-am69-aquila: Add Toradex DSI to LVDS adapter with the 10.1" V2 display

[Kumar, Udit]

Hi Leonardo,

On 9/9/2026 6:46 PM, Leonardo Costa wrote:
> Hi Udit,
> 
> On Sun, Sep 06, 2026 at 10:49:52AM +0530, Kumar, Udit wrote:
>> Hi Leonardo
>>
>> On 8/3/2026 11:04 PM, Leonardo Costa wrote:
>>> From: Leonardo Costa <leonardo.costa@xxxxxxxxxxx>
>>>
>>> Add a device tree overlay for the Toradex DSI to LVDS Adapter with the
>>> Toradex Capacitive Touch Display 10.1" LVDS V2. The adapter connects to the
>>> Aquila DSI_1 interface. It is based on the Texas Instruments SN65DSI84
>>> DSI-to-LVDS bridge and drives an Opto Logic SCX1001511GGC49 10.1" WXGA TFT
>>> LCD LVDS panel. Touch input is provided by an ILITEK ILI251x 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
>>> Signed-off-by: Leonardo Costa <leonardo.costa@xxxxxxxxxxx>
>>> ---
>>>  arch/arm64/boot/dts/ti/Makefile               |   5 +
>>>  ...dsi-to-lvds-v2-panel-cap-touch-10inch.dtso | 153 ++++++++++++++++++
>>>  2 files changed, 158 insertions(+)
>>>  create mode 100644 arch/arm64/boot/dts/ti/k3-am69-aquila-dsi-to-lvds-v2-panel-cap-touch-10inch.dtso
>>>
>>> diff --git a/arch/arm64/boot/dts/ti/Makefile b/arch/arm64/boot/dts/ti/Makefile
>>> index f4d109ef6798a..8a59fc597aede 100644
>>> --- a/arch/arm64/boot/dts/ti/Makefile
>>> +++ b/arch/arm64/boot/dts/ti/Makefile
>>> @@ -218,6 +218,7 @@ dtb-$(CONFIG_ARCH_K3) += k3-am69-aquila-dev.dtb
>>> [..]
>>> +
>>> +	panel-lvds-bridge {
>>> +		compatible = "optologic,scx1001511ggc49", "panel-lvds";
>>
>>> +		backlight = <&backlight_pwm3>;
>>> +		data-mapping = "vesa-24";
>>> +		height-mm = <136>;
>>> +		width-mm = <217>;
>>> +
>>> +		panel-timing {
>>> +			clock-frequency = <70000000>;
>>
>> Looking at
>> https://lore.kernel.org/linux-arm-kernel/20260623195741.495734-1-leoreis.costa@xxxxxxxxx/T/#mc4ed25e5253da6dc635fd3eddc317dac8f447f3e
>>
>>
>> you are using three different clock-freq 69500000, 64500000 and
>> 70000000 here.
>> Could add some comment here, why freq is adjusted.
> 
> Just to confirm it, considering my last reply, do I understand correctly
> that you want a v2 of this patch that adds this comment? In the end, these
> are just values that we found to work consistently with the bridge, and
> we're not sure if this comment would add much value.

Still adding comments will be helpful.
If someone taking this panel code as reference on other SOC/Board, then
they will be aware of clock-freq needs to adjusted.

Thanks
Udit



> Kind regards,
> Leonardo