# \[PATCHv2 2/8\] Doc/DT: Add DT binding documentation for Analog TV Connector

- Subsystems: OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, THE REST
- Maintainers: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Linus Torvalds
- Status: REVIEWED · 1 review, 0 by maintainers (Archit Taneja)
- Reviewed.
- 1 review trailer.
- List: linux-fbdev
- 3 messages, 3 authors
- Page: https://ratatoskr.run/linux-fbdev/2014/03/8087556/t

## \[PATCHv2 2/8\] Doc/DT: Add DT binding documentation for Analog TV Connector

- From: Tomi Valkeinen \<hidden\>
- Date: 2014-03-18 08:15:41 
- Message: https://ratatoskr.run/linux-fbdev/2014/03/8087556.md

```text
Add DT binding documentation for Analog TV Connector.

Signed-off-by: Tomi Valkeinen <redacted>
Reviewed-by: Archit Taneja <redacted>
---
 .../bindings/video/analog-tv-connector.txt         | 25 ++++++++++++++++++++++
 1 file changed, 25 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/video/analog-tv-connector.txt

diff --git a/Documentation/devicetree/bindings/video/analog-tv-connector.txt b/Documentation/devicetree/bindings/video/analog-tv-connector.txt
new file mode 100644
index 000000000000..0218fcdc1299
--- /dev/null
+++ b/Documentation/devicetree/bindings/video/analog-tv-connector.txt
@@ -0,0 +1,25 @@
+Analog TV Connector
+=========+
+Required properties:
+- compatible: "composite-connector" or "svideo-connector"
+
+Optional properties:
+- label: a symbolic name for the connector
+
+Required nodes:
+- Video port for TV input
+
+Example
+-------
+
+tv: connector {
+	compatible = "composite-connector";
+	label = "tv";
+
+	port {
+		tv_connector_in: endpoint {
+			remote-endpoint = <&venc_out>;
+		};
+	};
+};
-- 
1.8.3.2

```

## Re: \[PATCHv2 2/8\] Doc/DT: Add DT binding documentation for Analog TV Connector

- From: Geert Uytterhoeven \<geert@linux-m68k.org\>
- Date: 2014-03-18 08:40:52 
- Message: https://ratatoskr.run/linux-fbdev/2014/03/8087566.md

```text
Hi Tomi,

On Tue, Mar 18, 2014 at 9:15 AM, Tomi Valkeinen [ref] wrote:
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/video/analog-tv-connector.txt
> @@ -0,0 +1,25 @@
> +Analog TV Connector
> +=========> +
> +Required properties:
> +- compatible: "composite-connector" or "svideo-connector"
> +
> +Optional properties:
> +- label: a symbolic name for the connector
> +
> +Required nodes:
> +- Video port for TV input

Only input, not output?

Have you considered adding bindings for other analog connector types?
  - composite (YCbCr),
  - component with and without sync-on-green (RGB and RGBHV),
  - D-terminal (D1/D2/D3/D4/D5).

Am I missing some?

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: \[PATCHv2 2/8\] Doc/DT: Add DT binding documentation for Analog TV Connector

- From: Tomi Valkeinen \<hidden\>
- Date: 2014-03-18 08:52:59 
- Message: https://ratatoskr.run/linux-fbdev/2014/03/8087571.md

```text
On 18/03/14 10:40, Geert Uytterhoeven wrote:
> Hi Tomi,
> 
> On Tue, Mar 18, 2014 at 9:15 AM, Tomi Valkeinen [ref] wrote:
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/video/analog-tv-connector.txt
>> @@ -0,0 +1,25 @@
>> +Analog TV Connector
>> +===================
>> +
>> +Required properties:
>> +- compatible: "composite-connector" or "svideo-connector"
>> +
>> +Optional properties:
>> +- label: a symbolic name for the connector
>> +
>> +Required nodes:
>> +- Video port for TV input
> 
> Only input, not output?

Yes, I have only made the bindings for display. I have no experience on
the capture side. I can change the port bindings for the connectors to
say they may be input or output, but I have to say I have no idea
whether the bindings work for capture.

But this is also something that I think could be extended in the future,
when we have some code to actually use them on capture pipeline. For now
it may be safer to keep them only for display.

> Have you considered adding bindings for other analog connector types?
>   - composite (YCbCr),
>   - component with and without sync-on-green (RGB and RGBHV),
>   - D-terminal (D1/D2/D3/D4/D5).
> 
> Am I missing some?

No, I haven't. I have no experience on those, and generally very little
experience on analog video. I've added bindings only for the hardware I
have.

If there are lots of different analog TV connectors, should I just split
the current "analog-tv-connector" into separate composite and svideo,
instead of trying to manage them with a single one? The reason I
combined composite and svideo was that there didn't seem anything else
to specify than the connector type.

 Tomi


```