From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: [PATCH 4/9] Doc/DT: Add DT binding documentation for HDMI Connector Date: Fri, 28 Feb 2014 16:06:12 +0000 Message-ID: <20140228160612.GR21483@n2100.arm.linux.org.uk> References: <1393590016-9361-1-git-send-email-tomi.valkeinen@ti.com> <1393590016-9361-5-git-send-email-tomi.valkeinen@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1393590016-9361-5-git-send-email-tomi.valkeinen@ti.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces@lists.freedesktop.org Errors-To: dri-devel-bounces@lists.freedesktop.org To: Tomi Valkeinen Cc: devicetree@vger.kernel.org, linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, Andrzej Hajda , Laurent Pinchart , linux-arm-kernel@lists.infradead.org, Sebastian Hesselbarth List-Id: devicetree@vger.kernel.org On Fri, Feb 28, 2014 at 02:20:11PM +0200, Tomi Valkeinen wrote: > Add DT binding documentation for HDMI Connector. > > Signed-off-by: Tomi Valkeinen > Reviewed-by: Archit Taneja > --- > .../devicetree/bindings/video/hdmi-connector.txt | 23 ++++++++++++++++++++++ > 1 file changed, 23 insertions(+) > create mode 100644 Documentation/devicetree/bindings/video/hdmi-connector.txt > > diff --git a/Documentation/devicetree/bindings/video/hdmi-connector.txt b/Documentation/devicetree/bindings/video/hdmi-connector.txt > new file mode 100644 > index 000000000000..5d25f6a432bb > --- /dev/null > +++ b/Documentation/devicetree/bindings/video/hdmi-connector.txt > @@ -0,0 +1,23 @@ > +HDMI Connector > +============== > + > +Required properties: > +- compatible: "hdmi-connector" > + > +Optional properties: > +- label: a symbolic name for the connector > + > +Required nodes: > +- Video port for HDMI input > + > +Example > +------- > + > +hdmi0: connector@1 { > + compatible = "hdmi-connector"; > + label = "hdmi"; > + > + hdmi_connector_in: endpoint { > + remote-endpoint = <&tpd12s015_out>; > + }; > +}; It seems rather weird to have DVI connectors having an optional I2C property, but HDMI (which augments DVI) not having that as at least an optional property. I can quite well see the iMX HDMI support needing an i2c bus here. Also, HDMI has two connector standards - type A and type B, much like the single vs dual link of DVI. Again, DRM exposes this to userspace. -- FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly improving, and getting towards what was expected from it.