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>;
Attachment:
signature.asc
Description: OpenPGP digital signature
