The tvp5150 and tvp5151 decoders support different video input source connections to their AIP1A and AIP1B pins. Either two Composite or a S-Video input signals are supported. The possible configurations are as follows: - Analog Composite signal connected to AIP1A. - Analog Composite signal connected to AIP1B. - Analog S-Video Y (luminance) and C (chrominance) signals connected to AIP1A and AIP1B respectively. This patch extends the Device Tree binding documentation to describe how the input connectors for these devices should be defined in a DT. Suggested-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> Signed-off-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx> --- Hello, The DT binding assumes that there is a 1:1 map between physical connectors and connections, so there will be a connector described in the DT for each connection. There is also the question about how the DT bindings will be extended to support other attributes (color/position/group) using the properties API. But I believe that can be done as a follow-up, once the properties API is in mainline. Best regards, Javier Changes in v2: - Remove from the changelog a mention of devices that multiplex the physical RCA connectors to be used for the S-Video Y and C signals since it's a special case and it doesn't really work on the IGEPv2. .../devicetree/bindings/media/i2c/tvp5150.txt | 59 ++++++++++++++++++++++ 1 file changed, 59 insertions(+) diff --git a/Documentation/devicetree/bindings/media/i2c/tvp5150.txt b/Documentation/devicetree/bindings/media/i2c/tvp5150.txt index 8c0fc1a26bf0..df555650b0b4 100644 --- a/Documentation/devicetree/bindings/media/i2c/tvp5150.txt +++ b/Documentation/devicetree/bindings/media/i2c/tvp5150.txt @@ -26,8 +26,46 @@ Required Endpoint Properties for parallel synchronization: If none of hsync-active, vsync-active and field-even-active is specified, the endpoint is assumed to use embedded BT.656 synchronization. +-Optional nodes: +- connectors: The list of tvp5150 input connectors available on a given + board. The node should contain a child 'port' node for each connector. + + The tvp5150 has support for three possible connectors: 2 Composite and + 1 S-video. The "reg" property is used to specify which input connector + is associated with each 'port', using the following possible values: + + 0: Composite0 + 1: Composite1 + 2: S-Video + + The ports should have an endpoint subnode that is linked to a connector + node defined in Documentation/devicetree/bindings/display/connector/. + The linked connector compatible string should match the connector type. + Example: +composite0: connector@0 { + compatible = "composite-video-connector"; + label = "Composite0"; + + port { + comp0_out: endpoint { + remote-endpoint = <&tvp5150_comp0_in>; + }; + }; +}; + +svideo: connector@1 { + compatible = "composite-video-connector"; + label = "S-Video"; + + port { + svideo_out: endpoint { + remote-endpoint = <&tvp5150_svideo_in>; + }; + }; +}; + &i2c2 { ... tvp5150@5c { @@ -36,6 +74,27 @@ Example: pdn-gpios = <&gpio4 30 GPIO_ACTIVE_LOW>; reset-gpios = <&gpio6 7 GPIO_ACTIVE_LOW>; + connectors { + #address-cells = <1>; + #size-cells = <0>; + + /* Composite0 input */ + port@0 { + reg = <0>; + tvp5150_comp0_in: endpoint { + remote-endpoint = <&comp0_out>; + }; + }; + + /* S-Video input */ + port@2 { + reg = <2>; + tvp5150_svideo_in: endpoint { + remote-endpoint = <&svideo_out>; + }; + }; + }; + port { tvp5150_1: endpoint { remote-endpoint = <&ccdc_ep>; -- 2.5.5 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html