Re: [PATCH v2] dt-bindings: display: Convert connectors to DT schema

30 Jun 2020

On Tue, Jun 30, 2020 at 02:02:16PM -0600, Rob Herring wrote:
...
Convert the analog TV, DVI, HDMI, and VGA connector bindings to DT schema
format.
Cc: Sam Ravnborg sam@ravnborg.org
Cc: Laurent Pinchart Laurent.pinchart@ideasonboard.com
Cc: Maxime Ripard mripard@kernel.org
Signed-off-by: Rob Herring robh@kernel.org
Acked-by: Sam Ravnborg sam@ravnborg.org
...

v2:

Make Laurent maintainer
Add missing port and compatible required
Drop copy-n-paste 'type' from dvi-connector
Use 4 space indent on examples


.../display/connector/analog-tv-connector.txt | 31 --------
 .../connector/analog-tv-connector.yaml        | 52 ++++++++++++++
 .../display/connector/dvi-connector.txt       | 36 ----------
 .../display/connector/dvi-connector.yaml      | 70 +++++++++++++++++++
 .../display/connector/hdmi-connector.txt      | 31 --------
 .../display/connector/hdmi-connector.yaml     | 64 +++++++++++++++++
 .../display/connector/vga-connector.txt       | 36 ----------
 .../display/connector/vga-connector.yaml      | 46 ++++++++++++
 8 files changed, 232 insertions(+), 134 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt
 create mode 100644 Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml
 delete mode 100644 Documentation/devicetree/bindings/display/connector/dvi-connector.txt
 create mode 100644 Documentation/devicetree/bindings/display/connector/dvi-connector.yaml
 delete mode 100644 Documentation/devicetree/bindings/display/connector/hdmi-connector.txt
 create mode 100644 Documentation/devicetree/bindings/display/connector/hdmi-connector.yaml
 delete mode 100644 Documentation/devicetree/bindings/display/connector/vga-connector.txt
 create mode 100644 Documentation/devicetree/bindings/display/connector/vga-connector.yaml

diff --git a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt
deleted file mode 100644
index 883bcb2604c7..000000000000
--- a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt
+++ /dev/null
@@ -1,31 +0,0 @@
-Analog TV Connector



-Required properties:
-- compatible: "composite-video-connector" or "svideo-connector"



-Optional properties:
-- label: a symbolic name for the connector
-- sdtv-standards: limit the supported TV standards on a connector to the given

             ones. If not specified all TV standards are allowed.


             Possible TV standards are defined in


             include/dt-bindings/display/sdtv-standards.h.




-Required nodes:
-- Video port for TV input



-Example
-#include <dt-bindings/display/sdtv-standards.h>



-tv: connector {

compatible = "composite-video-connector";
label = "tv";
sdtv-standards = <(SDTV_STD_PAL | SDTV_STD_NTSC)>;

port {
tv_connector_in: endpoint {


	remote-endpoint = <&venc_out>;


};


};

-};
diff --git a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml
new file mode 100644
index 000000000000..eebe88fed999
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml
@@ -0,0 +1,52 @@
+# SPDX-License-Identifier: GPL-2.0-only
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/connector/analog-tv-connector.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#



+title: Analog TV Connector



+maintainers:


Laurent Pinchart Laurent.pinchart@ideasonboard.com




+properties:

compatible:
enum:
 - composite-video-connector


 - svideo-connector



label: true

sdtv-standards:
description:
 Limit the supported TV standards on a connector to the given ones. If


 not specified all TV standards are allowed. Possible TV standards are


 defined in include/dt-bindings/display/sdtv-standards.h.


$ref: /schemas/types.yaml#/definitions/uint32

port:
description: Connection to controller providing analog TV signals


+required:


compatible



port




+additionalProperties: false



+examples:


|


#include <dt-bindings/display/sdtv-standards.h>

connector {
   compatible = "composite-video-connector";


   label = "tv";


   sdtv-standards = <(SDTV_STD_PAL | SDTV_STD_NTSC)>;



   port {


       tv_connector_in: endpoint {


           remote-endpoint = <&venc_out>;


       };


   };


};


+...
diff --git a/Documentation/devicetree/bindings/display/connector/dvi-connector.txt b/Documentation/devicetree/bindings/display/connector/dvi-connector.txt
deleted file mode 100644
index 207e42e9eba0..000000000000
--- a/Documentation/devicetree/bindings/display/connector/dvi-connector.txt
+++ /dev/null
@@ -1,36 +0,0 @@
-DVI Connector



-Required properties:
-- compatible: "dvi-connector"



-Optional properties:
-- label: a symbolic name for the connector
-- ddc-i2c-bus: phandle to the i2c bus that is connected to DVI DDC
-- analog: the connector has DVI analog pins
-- digital: the connector has DVI digital pins
-- dual-link: the connector has pins for DVI dual-link
-- hpd-gpios: HPD GPIO number



-Required nodes:
-- Video port for DVI input



-Note: One (or both) of 'analog' or 'digital' must be set.



-Example



-dvi0: connector@0 {

compatible = "dvi-connector";
label = "dvi";

digital;

ddc-i2c-bus = <&i2c3>;

port {
dvi_connector_in: endpoint {


	remote-endpoint = <&tfp410_out>;


};


};

-};
diff --git a/Documentation/devicetree/bindings/display/connector/dvi-connector.yaml b/Documentation/devicetree/bindings/display/connector/dvi-connector.yaml
new file mode 100644
index 000000000000..71cb9220fa59
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/connector/dvi-connector.yaml
@@ -0,0 +1,70 @@
+# SPDX-License-Identifier: GPL-2.0-only
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/connector/dvi-connector.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#



+title: DVI Connector



+maintainers:


Laurent Pinchart Laurent.pinchart@ideasonboard.com




+properties:

compatible:
const: dvi-connector

label: true

hpd-gpios:
description: A GPIO line connected to HPD
maxItems: 1

ddc-i2c-bus:
description: phandle link to the I2C controller used for DDC EDID probing
$ref: /schemas/types.yaml#/definitions/phandle

analog:
type: boolean
description: the connector has DVI analog pins

digital:
type: boolean
description: the connector has DVI digital pins

dual-link:
type: boolean
description: the connector has pins for DVI dual-link

port:
description: Connection to controller providing DVI signals


+required:


compatible



port




+anyOf:


required:


 - analog



required:


 - digital




+additionalProperties: false



+examples:


|


connector {
   compatible = "dvi-connector";


   label = "dvi";



   digital;



   ddc-i2c-bus = <&i2c3>;



   port {


       dvi_connector_in: endpoint {


           remote-endpoint = <&tfp410_out>;


       };


   };


};


+...
diff --git a/Documentation/devicetree/bindings/display/connector/hdmi-connector.txt b/Documentation/devicetree/bindings/display/connector/hdmi-connector.txt
deleted file mode 100644
index aeb07c4bd703..000000000000
--- a/Documentation/devicetree/bindings/display/connector/hdmi-connector.txt
+++ /dev/null
@@ -1,31 +0,0 @@
-HDMI Connector



-Required properties:
-- compatible: "hdmi-connector"
-- type: the HDMI connector type: "a", "b", "c", "d" or "e"



-Optional properties:
-- label: a symbolic name for the connector
-- hpd-gpios: HPD GPIO number
-- ddc-i2c-bus: phandle link to the I2C controller used for DDC EDID probing
-- ddc-en-gpios: signal to enable DDC bus



-Required nodes:
-- Video port for HDMI input



-Example



-hdmi0: connector@1 {

compatible = "hdmi-connector";
label = "hdmi";

type = "a";

port {
hdmi_connector_in: endpoint {


	remote-endpoint = <&tpd12s015_out>;


};


};

-};
diff --git a/Documentation/devicetree/bindings/display/connector/hdmi-connector.yaml b/Documentation/devicetree/bindings/display/connector/hdmi-connector.yaml
new file mode 100644
index 000000000000..14d7128af592
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/connector/hdmi-connector.yaml
@@ -0,0 +1,64 @@
+# SPDX-License-Identifier: GPL-2.0-only
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/connector/hdmi-connector.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#



+title: HDMI Connector



+maintainers:


Laurent Pinchart Laurent.pinchart@ideasonboard.com




+properties:

compatible:
const: hdmi-connector

type:
description: The HDMI connector type
enum:
 - a   # Standard full size


 - b   # Never deployed?


 - c   # Mini


 - d   # Micro


 - e   # automotive



label: true

hpd-gpios:
description: A GPIO line connected to HPD
maxItems: 1

ddc-i2c-bus:
description: phandle link to the I2C controller used for DDC EDID probing
$ref: /schemas/types.yaml#/definitions/phandle

ddc-en-gpios:
description: GPIO signal to enable DDC bus
maxItems: 1

port:
description: Connection to controller providing HDMI signals


+required:


compatible



port



type




+additionalProperties: false



+examples:


|


connector {
   compatible = "hdmi-connector";


   label = "hdmi";



   type = "a";



   port {


       hdmi_connector_in: endpoint {


           remote-endpoint = <&tpd12s015_out>;


       };


   };


};


+...
diff --git a/Documentation/devicetree/bindings/display/connector/vga-connector.txt b/Documentation/devicetree/bindings/display/connector/vga-connector.txt
deleted file mode 100644
index c727f298e7ad..000000000000
--- a/Documentation/devicetree/bindings/display/connector/vga-connector.txt
+++ /dev/null
@@ -1,36 +0,0 @@
-VGA Connector



-Required properties:



-- compatible: "vga-connector"



-Optional properties:



-- label: a symbolic name for the connector corresponding to a hardware label
-- ddc-i2c-bus: phandle to the I2C bus that is connected to VGA DDC



-Required nodes:



-The VGA connector internal connections are modeled using the OF graph bindings
-specified in Documentation/devicetree/bindings/graph.txt.



-The VGA connector has a single port that must be connected to a video source
-port.




-Example



-vga0: connector@0 {

compatible = "vga-connector";
label = "vga";

ddc-i2c-bus = <&i2c3>;

port {
vga_connector_in: endpoint {


	remote-endpoint = <&adv7123_out>;


};


};

-};
diff --git a/Documentation/devicetree/bindings/display/connector/vga-connector.yaml b/Documentation/devicetree/bindings/display/connector/vga-connector.yaml
new file mode 100644
index 000000000000..5782c4bb3252
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/connector/vga-connector.yaml
@@ -0,0 +1,46 @@
+# SPDX-License-Identifier: GPL-2.0-only
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/connector/vga-connector.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#



+title: VGA Connector



+maintainers:


Laurent Pinchart Laurent.pinchart@ideasonboard.com




+properties:

compatible:
const: vga-connector

label: true

ddc-i2c-bus:
description: phandle link to the I2C controller used for DDC EDID probing
$ref: /schemas/types.yaml#/definitions/phandle

port:
description: Connection to controller providing VGA signals


+required:


compatible



port




+additionalProperties: false



+examples:


|


connector {
   compatible = "vga-connector";


   label = "vga";



   ddc-i2c-bus = <&i2c3>;



   port {


       vga_connector_in: endpoint {


           remote-endpoint = <&adv7123_out>;


       };


   };


};


+...
2.25.1

    

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

Re: [PATCH v2] dt-bindings: display: Convert connectors to DT schema

-Analog TV Connector

-Example

-DVI Connector

-Example

-HDMI Connector

-Example

-VGA Connector

-Example

+...