| Message ID | 1526923663-8179-2-git-send-email-jacopo+renesas@jmondi.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Jacopo, Thanks for your patch. On 2018-05-21 19:27:40 +0200, Jacopo Mondi wrote: > Describe the optional properties for endpoint nodes of port@0 > and port@1 of the R-Car VIN driver device tree bindings documentation. > > Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org> > Acked-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > --- > Documentation/devicetree/bindings/media/rcar_vin.txt | 13 ++++++++++++- > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/media/rcar_vin.txt b/Documentation/devicetree/bindings/media/rcar_vin.txt > index 5c6f2a7..dab3118 100644 > --- a/Documentation/devicetree/bindings/media/rcar_vin.txt > +++ b/Documentation/devicetree/bindings/media/rcar_vin.txt > @@ -54,6 +54,16 @@ from local SoC CSI-2 receivers (port1) depending on SoC. > from external SoC pins described in video-interfaces.txt[1]. > Describing more then one endpoint in port 0 is invalid. Only VIN > instances that are connected to external pins should have port 0. > + > + - Optional properties for endpoint nodes of port@0: > + - hsync-active: active state of the HSYNC signal, 0/1 for LOW/HIGH > + respectively. Default is active high. > + - vsync-active: active state of the VSYNC signal, 0/1 for LOW/HIGH > + respectively. Default is active high. > + > + If both HSYNC and VSYNC polarities are not specified, embedded > + synchronization is selected. > + > - port 1 - sub-nodes describing one or more endpoints connected to > the VIN from local SoC CSI-2 receivers. The endpoint numbers must > use the following schema. > @@ -63,6 +73,8 @@ from local SoC CSI-2 receivers (port1) depending on SoC. > - Endpoint 2 - sub-node describing the endpoint connected to CSI40 > - Endpoint 3 - sub-node describing the endpoint connected to CSI41 > > + Endpoint nodes of port@1 do not support any optional endpoint property. > + > Device node example for Gen2 platforms > -------------------------------------- > > @@ -113,7 +125,6 @@ Board setup example for Gen2 platforms (vin1 composite video input) > > vin1ep0: endpoint { > remote-endpoint = <&adv7180>; > - bus-width = <8>; Until we have figured out if we should document or remove the bus-width parameter maybe move this to 4/4 ? > }; > }; > }; > -- > 2.7.4 >
diff --git a/Documentation/devicetree/bindings/media/rcar_vin.txt b/Documentation/devicetree/bindings/media/rcar_vin.txt index 5c6f2a7..dab3118 100644 --- a/Documentation/devicetree/bindings/media/rcar_vin.txt +++ b/Documentation/devicetree/bindings/media/rcar_vin.txt @@ -54,6 +54,16 @@ from local SoC CSI-2 receivers (port1) depending on SoC. from external SoC pins described in video-interfaces.txt[1]. Describing more then one endpoint in port 0 is invalid. Only VIN instances that are connected to external pins should have port 0. + + - Optional properties for endpoint nodes of port@0: + - hsync-active: active state of the HSYNC signal, 0/1 for LOW/HIGH + respectively. Default is active high. + - vsync-active: active state of the VSYNC signal, 0/1 for LOW/HIGH + respectively. Default is active high. + + If both HSYNC and VSYNC polarities are not specified, embedded + synchronization is selected. + - port 1 - sub-nodes describing one or more endpoints connected to the VIN from local SoC CSI-2 receivers. The endpoint numbers must use the following schema. @@ -63,6 +73,8 @@ from local SoC CSI-2 receivers (port1) depending on SoC. - Endpoint 2 - sub-node describing the endpoint connected to CSI40 - Endpoint 3 - sub-node describing the endpoint connected to CSI41 + Endpoint nodes of port@1 do not support any optional endpoint property. + Device node example for Gen2 platforms -------------------------------------- @@ -113,7 +125,6 @@ Board setup example for Gen2 platforms (vin1 composite video input) vin1ep0: endpoint { remote-endpoint = <&adv7180>; - bus-width = <8>; }; }; };