[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ccf4b15a-8399-4a7f-ae40-d01d0975921c@kernel.org>
Date: Fri, 9 May 2025 09:11:07 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Chaoyi Chen <kernel@...kyi.com>, Sandy Huang <hjc@...k-chips.com>,
Heiko Stuebner <heiko@...ech.de>, 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>
Cc: Chaoyi Chen <chaoyi.chen@...k-chips.com>,
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
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
Powered by blists - more mailing lists