[PATCH 4/9] Doc/DT: Add DT binding documentation for HDMI Connector

Subsystems: open firmware and flattened device tree bindings, the rest

STALE4608d REVIEWED: 1 (0M)

1 review trailer.

8 messages, 5 authors, 2014-03-05 · open the first message on its own page

[PATCH 4/9] Doc/DT: Add DT binding documentation for HDMI Connector

From: Tomi Valkeinen <hidden>
Date: 2014-02-28 12:20:11

Add DT binding documentation for HDMI Connector.

Signed-off-by: Tomi Valkeinen <redacted>
Reviewed-by: Archit Taneja <redacted>
---
 .../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>;
+	};
+};
-- 
1.8.3.2

Re: [PATCH 4/9] Doc/DT: Add DT binding documentation for HDMI Connector

From: Russell King - ARM Linux <hidden>
Date: 2014-02-28 16:06:12

On Fri, Feb 28, 2014 at 02:20:11PM +0200, Tomi Valkeinen wrote:
quoted hunk
Add DT binding documentation for HDMI Connector.

Signed-off-by: Tomi Valkeinen <redacted>
Reviewed-by: Archit Taneja <redacted>
---
 .../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.

Re: [PATCH 4/9] Doc/DT: Add DT binding documentation for HDMI Connector

From: Tomi Valkeinen <hidden>
Date: 2014-02-28 16:15:05

On 28/02/14 18:06, Russell King - ARM Linux wrote:
quoted
+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 have added only the properties that I have used. I did think about the
i2c for HDMI also, but thought that I don't use it, and so can't test
it, and so I could well create a bad binding.

And, as I don't see any issue in adding it later, when someone uses it,
I decided to leave it out.
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.
True, but isn't that just a form factor? No functional differences? But
I agree, we can add the two types here also to the compatible string.

 Tomi


Re: [PATCH 4/9] Doc/DT: Add DT binding documentation for HDMI Connector

From: Russell King - ARM Linux <hidden>
Date: 2014-02-28 16:34:31

On Fri, Feb 28, 2014 at 06:15:05PM +0200, Tomi Valkeinen wrote:
On 28/02/14 18:06, Russell King - ARM Linux wrote:
quoted
quoted
+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 have added only the properties that I have used. I did think about the
i2c for HDMI also, but thought that I don't use it, and so can't test
it, and so I could well create a bad binding.

And, as I don't see any issue in adding it later, when someone uses it,
I decided to leave it out.
quoted
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.
True, but isn't that just a form factor? No functional differences? But
I agree, we can add the two types here also to the compatible string.
HDMI A is 19 pins carrying TMDS channels 0-2.  HDMI B is 29 pins
carrying TMDS channels 0-5.  So, the difference is the same as the
single vs dual link on DVI-D/I connectors.

There's actually three HDMI connectors:

  All three connectors carry all required HDMI signals, including a TMDS
  link. The Type B connector is slightly larger and carries a second TMDS
  link, which is necessary to support very high resolution displays using
  dual link. The Type C connector carries the same signals as the Type A
  but is more compact and intended for mobile applications.

So, Type C and Type A are electrically the same.

-- 
FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
improving, and getting towards what was expected from it.

Re: [PATCH 4/9] Doc/DT: Add DT binding documentation for HDMI Connector

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2014-03-01 18:58:30

On Fri, Feb 28, 2014 at 5:34 PM, Russell King - ARM Linux
[off-list ref] wrote:
There's actually three HDMI connectors:

  All three connectors carry all required HDMI signals, including a TMDS
  link. The Type B connector is slightly larger and carries a second TMDS
  link, which is necessary to support very high resolution displays using
  dual link. The Type C connector carries the same signals as the Type A
  but is more compact and intended for mobile applications.

So, Type C and Type A are electrically the same.
There's also D (e.g. on BeagleBone Black) and E:

http://en.wikipedia.org/wiki/HDMI#Connectors

Electrically they seem to be the same as A/C.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Re: [PATCH 4/9] Doc/DT: Add DT binding documentation for HDMI Connector

From: Tomi Valkeinen <hidden>
Date: 2014-03-03 07:23:05

On 01/03/14 20:58, Geert Uytterhoeven wrote:
On Fri, Feb 28, 2014 at 5:34 PM, Russell King - ARM Linux
[off-list ref] wrote:
quoted
There's actually three HDMI connectors:

  All three connectors carry all required HDMI signals, including a TMDS
  link. The Type B connector is slightly larger and carries a second TMDS
  link, which is necessary to support very high resolution displays using
  dual link. The Type C connector carries the same signals as the Type A
  but is more compact and intended for mobile applications.

So, Type C and Type A are electrically the same.
There's also D (e.g. on BeagleBone Black) and E:

http://en.wikipedia.org/wiki/HDMI#Connectors

Electrically they seem to be the same as A/C.
Right. And then there are the HDMI versions, and things like HDMI
Ethernet Channel. After looking at these a bit, I don't think the HDMI
connector needs any of those (hdmi version, eth) defined.

So...

	compatible = "hdmi-connector";
	type = "a";

Or

	compatible = "hdmi-connector";
	type-a;

I don't right away see any big pro with either one compared to the other.

 Tomi

Re: [PATCH 4/9] Doc/DT: Add DT binding documentation for HDMI Connector

From: Daniel Vetter <hidden>
Date: 2014-03-04 12:59:48

On Fri, Feb 28, 2014 at 04:06:12PM +0000, Russell King - ARM Linux wrote:
On Fri, Feb 28, 2014 at 02:20:11PM +0200, Tomi Valkeinen wrote:
quoted
Add DT binding documentation for HDMI Connector.

Signed-off-by: Tomi Valkeinen <redacted>
Reviewed-by: Archit Taneja <redacted>
---
 .../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.
Yeah, there's a dual-link hdmi standard somewhere. I've never seen it in
the wild anywhere though, and newer hdmi all just increase the max link
clock to push higher res modes over the wire. Imo can't hurt to just
enumerate all type of physical connectors standardized.

HDMI revisions themselves are only relevant for the sink (as advertised
capabilities in the EDID) and for your hdmi encoder chip (where pretty
much just the max dotclock is interesting really). So I don't think we
need a hdmi revision in dt for the physical connector.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

Re: [PATCH 4/9] Doc/DT: Add DT binding documentation for HDMI Connector

From: Tomi Valkeinen <hidden>
Date: 2014-03-05 08:42:36

On 01/03/14 20:58, Geert Uytterhoeven wrote:
On Fri, Feb 28, 2014 at 5:34 PM, Russell King - ARM Linux
[off-list ref] wrote:
quoted
There's actually three HDMI connectors:

  All three connectors carry all required HDMI signals, including a TMDS
  link. The Type B connector is slightly larger and carries a second TMDS
  link, which is necessary to support very high resolution displays using
  dual link. The Type C connector carries the same signals as the Type A
  but is more compact and intended for mobile applications.

So, Type C and Type A are electrically the same.
There's also D (e.g. on BeagleBone Black) and E:

http://en.wikipedia.org/wiki/HDMI#Connectors

Electrically they seem to be the same as A/C.
I made the following change compared to the posted version.

 Tomi
@@ -3,6 +3,7 @@ HDMI Connector

 Required properties:
 - compatible: "hdmi-connector"
+- type: the HDMI connector type: "a", "b", "c", "d" or "e"

 Optional properties:
 - label: a symbolic name for the connector
@@ -17,6 +18,8 @@ hdmi0: connector@1 {
        compatible = "hdmi-connector";
        label = "hdmi";

+       type = "a";
+
        hdmi_connector_in: endpoint {
                remote-endpoint = <&tpd12s015_out>;
        };

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help