| Message ID | 1409157673-4154-4-git-send-email-laurent.pinchart+renesas@ideasonboard.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Wed, Aug 27, 2014 at 11:41 AM, Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> wrote: > 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.? > > Cc: devicetree@vger.kernel.org > Cc: linux-fbdev@vger.kernel.org > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > --- > .../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. > +- 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. > + > +Example > +------- > + > +vga0: connector@0 { > + compatible = "vga-connector"; > + label = "vga"; > + > + ddc-i2c-bus = <&i2c3>; > + > + port { > + vga_connector_in: endpoint { > + remote-endpoint = <&adv7123_out>; > + }; > + }; > +}; > -- > 1.8.5.5 > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
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: > > 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. > > Cc: devicetree@vger.kernel.org > > Cc: linux-fbdev@vger.kernel.org > > Signed-off-by: Laurent Pinchart > > <laurent.pinchart+renesas@ideasonboard.com> > > --- > > > > .../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. > > > +- 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. > > + > > +Example > > +------- > > + > > +vga0: connector@0 { > > + compatible = "vga-connector"; > > + label = "vga"; > > + > > + ddc-i2c-bus = <&i2c3>; > > + > > + port { > > + vga_connector_in: endpoint { > > + remote-endpoint = <&adv7123_out>; > > + }; > > + }; > > +};
Hi, On 27/08/14 19:41, Laurent Pinchart wrote: > The VGA connector is described by a single input port and an optional > DDC bus. > > Cc: devicetree@vger.kernel.org > Cc: linux-fbdev@vger.kernel.org > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > --- > .../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 > +- ddc-i2c-bus: phandle to the I2C bus that is connected to VGA DDC > + > +Required nodes: > +- Video port for VGA input > + > +Example > +------- > + > +vga0: connector@0 { > + compatible = "vga-connector"; > + label = "vga"; > + > + ddc-i2c-bus = <&i2c3>; > + > + port { > + vga_connector_in: endpoint { > + remote-endpoint = <&adv7123_out>; > + }; > + }; > +}; > Looks good to me. Tomi
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 +- ddc-i2c-bus: phandle to the I2C bus that is connected to VGA DDC + +Required nodes: +- Video port for VGA input + +Example +------- + +vga0: connector@0 { + compatible = "vga-connector"; + label = "vga"; + + ddc-i2c-bus = <&i2c3>; + + port { + vga_connector_in: endpoint { + remote-endpoint = <&adv7123_out>; + }; + }; +};
The VGA connector is described by a single input port and an optional DDC bus. Cc: devicetree@vger.kernel.org Cc: linux-fbdev@vger.kernel.org Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> --- .../devicetree/bindings/video/vga-connector.txt | 28 ++++++++++++++++++++++ 1 file changed, 28 insertions(+) create mode 100644 Documentation/devicetree/bindings/video/vga-connector.txt