On Thu, Feb 10, 2022 at 01:10:32AM +0100, Johan Jonker wrote:
> Hi Sascha,
>
> Something with port and endpoint gives notifications.
> Somehow with the conversion of rockchip,dw-hdmi.txt to YAML not all SoC
> options were checked/covered (see rk3328 and rk3568).
>
> Allow multiple vop:
> port or
> port@0
> 1x vop -> endpoint
>
> 2x vop -> endpoint@0
> -> endpoint@1
>
> Also allow for connector:
> (not all existing DT have this yet)
> port@1
> -> endpoint
>
> See also at graph.yaml
Ok, let me see if I get this right. The HDMI can either have one or two
ports. It has one when the HDMI connector is not described in the device
tree and two when it is. Also the first (or only) port can have one or
two endpoints. It has one endpoint when the SoC has a single VOP and two
endpoints when the SoC has two VOPs.
It's been a painful morning because my YAML knowledge is fairly limited,
but here's what I came up with:
ports:
$ref: /schemas/graph.yaml#/properties/ports
unevaluatedProperties: false
patternProperties:
"^port(@0)?$":
$ref: /schemas/graph.yaml#/properties/port
description: Input of the DWC HDMI TX
properties:
endpoint:
description: Connection to the VOP
endpoint@0:
description: Connection to the VOPB
endpoint@1:
description: Connection to the VOPL
properties:
port@1:
$ref: /schemas/graph.yaml#/properties/port
description: Output of the DWC HDMI TX
Does this look sane to you? With this a make dtbs_check on this binding
doesn't generate warnings anymore, and even better it does generate
warnings when I add some wrong nodes/properties to the dts files.
> ===
>
> dtc -I dtb -O dts rk3568-evb1-v10.dtb
> <stdout>: Warning (avoid_unnecessary_addr_size):
> /hdmi@fe0a0000/ports/port@1: unnecessary #address-cells/#size-cells
> without "ranges" or child "reg" property
> <stdout>: Warning (graph_child_address): /hdmi@fe0a0000/ports/port@0:
> graph node has single child node 'endpoint@0',
> #address-cells/#size-cells are not necessary
> <stdout>: Warning (graph_child_address): /hdmi@fe0a0000/ports/port@1:
> graph node has single child node 'endpoint', #address-cells/#size-cells
> are not necessary
That's easy. I dropped the @0 suffix and reg property from the node.
Sascha
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
