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. > + 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. > Best regards, Krzysztof
