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.
+ description:
+ Phandle to the PHY device for DP output.
+
+ ports:
+ $ref: /schemas/graph.yaml#/properties/ports
+
+ properties:
+ port@0:
+ $ref: /schemas/graph.yaml#/properties/port
+ description: Input of the CDN DP
+ properties:
+ endpoint@0:
+ description: Connection to the VOPB
+ endpoint@1:
+ description: Connection to the VOPL
+ port@1:
+ $ref: /schemas/graph.yaml#/properties/port
+ description: Output of the CDN DP
+
+ required:
+ - port@0
+ - port@1
+
+ power-domains:
+ maxItems: 1
+
+ resets:
+ maxItems: 4
+
+ reset-names:
+ items:
+ - const: spdif
+ - const: dptx
+ - const: apb
+ - const: core
+
+ rockchip,grf:
+ $ref: /schemas/types.yaml#/definitions/phandle
+ description:
+ Phandle to GRF register to control HPD.
+
+ "#sound-dai-cells":
+ const: 1
+
+required:
+ - compatible
+ - reg
+ - clocks
+ - clock-names
+ - interrupts
+ - phys
+ - ports
+ - resets
+ - reset-names
+ - rockchip,grf
+
+additionalProperties: false
Well, if you added dai-common $ref then this could have stayed as
unevaluatedProperties, so you will allow names for the DAI as well.
Thanks for your clarification. Will fix in v3.
Best regards,
Krzysztof
