| Message ID | 1456253288-397-1-git-send-email-javier@osg.samsung.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Javier, Thank you for the patch. On Tuesday 23 February 2016 15:48:08 Javier Martinez Canillas wrote: > 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> > > --- > > .../devicetree/bindings/media/i2c/tvp5150.txt | 43 ------------------ > 1 file changed, 43 deletions(-) > > diff --git a/Documentation/devicetree/bindings/media/i2c/tvp5150.txt > b/Documentation/devicetree/bindings/media/i2c/tvp5150.txt index > daa20e43a8e3..8c0fc1a26bf0 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>;
On Tue, Feb 23, 2016 at 03:48:08PM -0300, Javier Martinez Canillas wrote: > 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> > > --- > > .../devicetree/bindings/media/i2c/tvp5150.txt | 43 ---------------------- > 1 file changed, 43 deletions(-) Acked-by: Rob Herring <robh@kernel.org> -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/media/i2c/tvp5150.txt b/Documentation/devicetree/bindings/media/i2c/tvp5150.txt index daa20e43a8e3..8c0fc1a26bf0 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>;
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> --- .../devicetree/bindings/media/i2c/tvp5150.txt | 43 ---------------------- 1 file changed, 43 deletions(-)