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>;