[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <cdeb1f80-a959-423d-9cba-dc1af760fe75@ti.com>
Date: Mon, 21 Apr 2025 16:40:46 +0530
From: Jayesh Choudhary <j-choudhary@...com>
To: "Kumar, Udit" <u-kumar1@...com>, <nm@...com>, <vigneshr@...com>,
<devarsht@...com>, <linux-kernel@...r.kernel.org>
CC: <robh@...nel.org>, <krzk+dt@...nel.org>, <conor+dt@...nel.org>,
<kristo@...nel.org>, <devicetree@...r.kernel.org>,
<linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH 7/7] arm64: dts: ti: k3-am68-sk: Enable DSI on
DisplayPort-0
Hello Udit,
On 19/04/25 15:04, Kumar, Udit wrote:
>
> On 4/11/2025 4:21 PM, Jayesh Choudhary wrote:
>> Enable DSI support for AM68-SK platform.
>>
>> Add DT node for DSI2eDP bridge. The DSI to eDP bridge is sn65dsi86
>> on the board.
>>
>> Add the endpoint nodes to describe connection from:
>> DSS => DSI => SN65DSI86 bridge => DisplayPort-0
>>
>> Set status for all required nodes for DisplayPort-0 as 'okay'.
>>
>> Signed-off-by: Jayesh Choudhary <j-choudhary@...com>
>> ---
>> .../boot/dts/ti/k3-am68-sk-base-board.dts | 96 +++++++++++++++++++
>> 1 file changed, 96 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
>> b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
>> index 11522b36e0ce..df54de2d8236 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
>> @@ -124,6 +124,35 @@ transceiver4: can-phy3 {
>> max-bitrate = <5000000>;
>> };
>> + edp0_refclk: clock-edp0-refclk {
>> + #clock-cells = <0>;
>> + compatible = "fixed-clock";
>> + clock-frequency = <19200000>;
>> + };
>
> I assume on this board and through out series, display is DP not eDP.
>
> do you see possibility to change wording from eDP to DP
>
Yes, the display is DP.
But this is the refclk consumed by SN65DSI86 bridge (DSI-to-eDP bridge)
Hence the naming.
>
>> +
>> + dp0_pwr_3v3: fixedregulator-dp0-pwr {
>> + compatible = "regulator-fixed";
>> + regulator-name = "dp0-pwr";
>> + regulator-min-microvolt = <3300000>;
>> + regulator-max-microvolt = <3300000>;
>> + gpio = <&exp2 2 GPIO_ACTIVE_HIGH>; /*P0 - DP0_3V3 _EN */
>> + enable-active-high;
>> + regulator-always-on;
>> + };
>> +
>> [..]
Thanks,
Jayesh
Powered by blists - more mailing lists