| Message ID | 1395130547-18633-3-git-send-email-tomi.valkeinen@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Tomi, On Tue, Mar 18, 2014 at 9:15 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> 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
On 18/03/14 10:40, Geert Uytterhoeven wrote: > Hi Tomi, > > On Tue, Mar 18, 2014 at 9:15 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> 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
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>; + }; + }; +};