Re: [PATCH v2 3/6] dt-bindings: display: imx: hdmi: Convert binding to YAML

On Sun, 2020-12-20 at 21:50 +0200, Laurent Pinchart wrote:

Convert the i.MX6 HDMI TX text binding to YAML.

Signed-off-by: Laurent Pinchart laurent.pinchart+renesas@ideasonboard.com

Changes since v1:

• Only specify maxItems for clocks
• Drop reg and interrupts as they're checked in the base schema
• Rebase on top of OF graph schema, dropped redundant properties
• Fix identation for enum entries
• Drop clock-names items, use maxItems only

---

.../bindings/display/imx/fsl,imx6-hdmi.yaml | 130 ++++++++++++++++++ .../devicetree/bindings/display/imx/hdmi.txt | 65 --------- 2 files changed, 130 insertions(+), 65 deletions(-) create mode 100644 Documentation/devicetree/bindings/display/imx/fsl,imx6-hdmi.yaml delete mode 100644 Documentation/devicetree/bindings/display/imx/hdmi.txt

diff --git a/Documentation/devicetree/bindings/display/imx/fsl,imx6-hdmi.yaml b/Documentation/devicetree/bindings/display/imx/fsl,imx6-hdmi.yaml new file mode 100644 index 000000000000..f9b131bb3339 --- /dev/null +++ b/Documentation/devicetree/bindings/display/imx/fsl,imx6-hdmi.yaml @@ -0,0 +1,130 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/display/imx/fsl,imx6-hdmi.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml#

+title: Freescale i.MX6 DWC HDMI TX Encoder

+maintainers:

• • Philipp Zabel p.zabel@pengutronix.de

+description: |

• The HDMI transmitter is a Synopsys DesignWare HDMI 1.4 TX controller IP
• with a companion PHY IP.

+allOf:

• • $ref: ../bridge/synopsys,dw-hdmi.yaml#

+properties:

• compatible:
• enum:

•  - fsl,imx6dl-hdmi
  

•  - fsl,imx6q-hdmi
  

• reg-io-width:
• const: 1
• clocks:
• maxItems: 2
• clock-names:
• maxItems: 2
• ddc-i2c-bus:
• $ref: /schemas/types.yaml#/definitions/phandle
• description:

•  The HDMI DDC bus can be connected to either a system I2C master or the
  

•  functionally-reduced I2C master contained in the DWC HDMI. When connected
  

•  to a system I2C master this property contains a phandle to that I2C
  

•  master controller.
  

• gpr:
• $ref: /schemas/types.yaml#/definitions/phandle
• description:

•  phandle to the iomuxc-gpr region containing the HDMI multiplexer control
  

•  register.
  

• ports:
• $ref: /schemas/graph.yaml#/properties/ports
• description: |

•  This device has four video ports, corresponding to the four inputs of the
  

•  HDMI multiplexer. Each port shall have a single endpoint.
  

• properties:

•  port@0:
  

•    $ref: /schemas/graph.yaml#/$defs/port-base
  

•    unevaluatedProperties: false
  

•    description: First input of the HDMI multiplexer
  

•  port@1:
  

•    $ref: /schemas/graph.yaml#/$defs/port-base
  

•    unevaluatedProperties: false
  

•    description: Second input of the HDMI multiplexer
  

•  port@2:
  

•    $ref: /schemas/graph.yaml#/$defs/port-base
  

•    unevaluatedProperties: false
  

•    description: Third input of the HDMI multiplexer
  

•  port@3:
  

•    $ref: /schemas/graph.yaml#/$defs/port-base
  

•    unevaluatedProperties: false
  

•    description: Fourth input of the HDMI multiplexer
  

• anyOf:

•  - required:
  

•      - port@0
  

•  - required:
  

•      - port@1
  

•  - required:
  

•      - port@2
  

•  - required:
  

•      - port@3
  

+required:

• • compatible
• • reg
• • clocks
• • clock-names
• • gpr
• • interrupts
• • ports

Won't this break validation of current i.MX6 DTs, which don't use the ports node (in imx6q(dl).dtsi)?

regards Philipp