From: Javier Martinez Canillas <javier@osg.samsung.com>
Date: Tue, 23 Feb 2016 18:48:08 +0000 (-0300)
Subject: [media] Revert "[media] tvp5150: document input connectors DT bindings"
X-Git-Tag: v4.14-rc1~3648^2^2~29
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=31e717dba1e18cc67f90124f326d8cf7a3725923;p=platform%2Fkernel%2Flinux-rpi.git

[media] Revert "[media] tvp5150: document input connectors DT bindings"

This reverts commit 82c2ffeb217a ("[media] tvp5150: document input
connectors DT bindings") since the DT binding is too device driver
specific and should instead be more generic and use the bindings
in Documentation/devicetree/bindings/display/connector/ and linked
to the tvp5150 using the OF graph port and endpoints.

There are still ongoing discussions about how the input connectors
will be supported by the Media Controller framework so until that
is settled, it is better to revert the connectors portion of the
bindings to avoid known to be broken bindings docs to hit mainline.

Suggested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
---

diff --git a/Documentation/devicetree/bindings/media/i2c/tvp5150.txt b/Documentation/devicetree/bindings/media/i2c/tvp5150.txt
index daa20e4..8c0fc1a 100644
--- a/Documentation/devicetree/bindings/media/i2c/tvp5150.txt
+++ b/Documentation/devicetree/bindings/media/i2c/tvp5150.txt
@@ -12,32 +12,6 @@ Optional Properties:
 - pdn-gpios: phandle for the GPIO connected to the PDN pin, if any.
 - reset-gpios: phandle for the GPIO connected to the RESETB pin, if any.
 
-Optional nodes:
-- connectors: The input connectors of tvp5150 have to be defined under
-  a subnode name "connectors" using the following format:
-
-	input-connector-name {
-		input connector properties
-	};
-
-Each input connector must contain the following properties:
-
-	- label: a name for the connector.
-	- input: the input connector.
-
-The possible values for the "input" property are:
-	0: Composite0
-	1: Composite1
-	2: S-Video
-
-and on a tvp5150am1 and tvp5151 there is another:
-	4: Signal generator
-
-The list of valid input connectors are defined in dt-bindings/media/tvp5150.h
-header file and can be included by device tree source files.
-
-Each input connector can be defined only once.
-
 The device node must contain one 'port' child node for its digital output
 video port, in accordance with the video interface bindings defined in
 Documentation/devicetree/bindings/media/video-interfaces.txt.
@@ -62,23 +36,6 @@ Example:
 		pdn-gpios = <&gpio4 30 GPIO_ACTIVE_LOW>;
 		reset-gpios = <&gpio6 7 GPIO_ACTIVE_LOW>;
 
-		connectors {
-			composite0 {
-				label = "Composite0";
-				input = <TVP5150_COMPOSITE0>;
-			};
-
-			composite1 {
-				label = "Composite1";
-				input = <TVP5150_COMPOSITE1>;
-			};
-
-			s-video {
-				label = "S-Video";
-				input = <TVP5150_SVIDEO>;
-			};
-		};
-
 		port {
 			tvp5150_1: endpoint {
 				remote-endpoint = <&ccdc_ep>;