Re: [PATCH 3/9] Doc/DT: Add DT binding documentation for DVI Connector
From: Tomi Valkeinen <hidden>
Date: 2014-03-05 08:41:03
On 28/02/14 18:23, Russell King - ARM Linux wrote:
That's rather a lot of compatible strings. Another possibility is: compatible = "dvi-connector"; analog; digital; single-link; dual-link;
I made the following changes compared to the posted version. I decided to leave the "single-link" out, as it's implied if "digital" is set. Tomi
@@ -6,11 +6,16 @@ Required properties: Optional properties: - label: a symbolic name for the connector -- i2c-bus: phandle to the i2c bus that is connected to DVI DDC +- 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 Required nodes: - Video port for DVI input +Note: One (or both) of 'analog' or 'digital' must be set. + Example -------
@@ -18,7 +23,9 @@ dvi0: connector@0 { compatible = "dvi-connector"; label = "dvi"; - i2c-bus = <&i2c3>; + digital; + + ddc-i2c-bus = <&i2c3>; dvi_connector_in: endpoint { remote-endpoint = <&tfp410_out>;