On 09/05/2025 09:34, Chaoyi Chen wrote: > 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.
Uh, sorry, that was supposed to be under extcon. > > I think it could be phandle or phandle-array. Since the RK3399 DP has Here it is obviously not needed, that's a dtschema type. > 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. > > >> >>> + description: >>> + Phandle to the PHY device for DP output. You need to list the items with description iinstead. Best regards, Krzysztof
