[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1824349.VLH7GnMWUR@diego>
Date: Fri, 09 May 2025 09:46:57 +0200
From: Heiko StĂĽbner <heiko@...ech.de>
To: Krzysztof Kozlowski <krzk@...nel.org>, Chaoyi Chen <kernel@...kyi.com>,
Sandy Huang <hjc@...k-chips.com>, Andy Yan <andy.yan@...k-chips.com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Chaoyi Chen <chaoyi.chen@...k-chips.com>
Cc: Dragan Simic <dsimic@...jaro.org>, dri-devel@...ts.freedesktop.org,
devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-rockchip@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject:
Re: [PATCH v2 2/2] dt-bindings: display: rockchip: Convert
cdn-dp-rockchip.txt to yaml
Am Freitag, 9. Mai 2025, 09:34:47 Mitteleuropäische Sommerzeit schrieb Chaoyi Chen:
> Hi Krzysztof,
>
> On 2025/5/9 15:11, Krzysztof Kozlowski wrote:
> > On 09/05/2025 09:02, Chaoyi Chen wrote:
> >> +
> >> + clock-names:
> >> + items:
> >> + - const: core-clk
> >> + - const: pclk
> >> + - const: spdif
> >> + - const: grf
> >> +
> >> + extcon:
> >> + $ref: /schemas/types.yaml#/definitions/phandle-array
> >> + description:
> >> + Phandle to the extcon device providing the cable state for the DP PHY.
> >> +
> >> + interrupts:
> >> + maxItems: 1
> >> +
> >> + phys:
> >> + $ref: /schemas/types.yaml#/definitions/phandle-array
> > Just phandle. If this is an array (but why?), you need maxItems for
> > outer and inner dimensions.
>
> I think it could be phandle or phandle-array. Since the RK3399 DP has
> two PHYs, if we put in two PHYs here, the driver will pick one PHY port
> that is already plugged into the DP for output. If we fill in only one
> PHY here, then output is only allowed on the corresponding PHY.
>
> Will add restrictions and add more descriptions in v3.
Just looked that up in the documentation.
Only the rk3399 has used this controller at all so far. There is one
cdn-dp controller on it, but two phys for the two type-c blocks on
the soc.
The controller can then select which one to drive. So yep, there
are two phys.
Heiko
Powered by blists - more mailing lists