Re: [PATCH 03/16] video: Add DT binding documentation for VGA connector

Hi Rob,

On Wednesday 27 August 2014 12:12:36 Rob Herring wrote:

On Wed, Aug 27, 2014 at 11:41 AM, Laurent Pinchart wrote:

quoted

The VGA connector is described by a single input port and an optional
DDC bus.

Wasn't there a generic connector binding for DVI, HDMI, etc.?

As far as I know, there are three separate generic bindings for DVI 
connectors, HDMI connectors, and analog TV connectors. The VGA connector 
doesn't seem to really fit into one of those categories.

quoted

Cc: devicetree@vger.kernel.org
Cc: linux-fbdev@vger.kernel.org
Signed-off-by: Laurent Pinchart
[off-list ref]
---

 .../devicetree/bindings/video/vga-connector.txt    | 28
 ++++++++++++++++++++++ 1 file changed, 28 insertions(+)
 create mode 100644
 Documentation/devicetree/bindings/video/vga-connector.txt> 

diff --git a/Documentation/devicetree/bindings/video/vga-connector.txt

b/Documentation/devicetree/bindings/video/vga-connector.txt new file mode
100644
index 0000000..9a45ec1

--- /dev/null
+++ b/Documentation/devicetree/bindings/video/vga-connector.txt

@@ -0,0 +1,28 @@
+VGA Connector
+=======
+
+Required properties:
+- compatible: "vga-connector"
+
+Optional properties:
+- label: a symbolic name for the connector

...which corresponds to hardware labels.

quoted

+- ddc-i2c-bus: phandle to the I2C bus that is connected to VGA DDC
+
+Required nodes:
+- Video port for VGA input

A reference to the relevant video graph bindings should be added here.

I'll fix that.

quoted

+
+Example
+-------
+
+vga0: connector@0 {
+       compatible = "vga-connector";
+       label = "vga";
+
+       ddc-i2c-bus = <&i2c3>;
+
+       port {
+               vga_connector_in: endpoint {
+                       remote-endpoint = <&adv7123_out>;
+               };
+       };
+};

-- 
Regards,

Laurent Pinchart