[PATCH 1/2] dt-bindings: display: tfp410: Add bus-width parameter property

[Laurent Pinchart]

Hi Peter,

Thank you for the patch.

On Mon, Apr 01, 2019 at 03:41:42PM +0300, Peter Ujfalusi wrote:
> tfp410 can be connect to host processor in 24bit, single-edge (24 lines) or
> 12bit, dual-edge (12 lines).
> 
> Add bus-width to the documentation so it can be used to select between the
> two connection scheme.
> 
> Signed-off-by: Peter Ujfalusi <peter.ujfalusi at ti.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

> ---
>  .../devicetree/bindings/display/bridge/ti,tfp410.txt   | 10 +++++++++-
>  1 file changed, 9 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/display/bridge/ti,tfp410.txt b/Documentation/devicetree/bindings/display/bridge/ti,tfp410.txt
> index 3f903af93949..5ff4f64ef8e8 100644
> --- a/Documentation/devicetree/bindings/display/bridge/ti,tfp410.txt
> +++ b/Documentation/devicetree/bindings/display/bridge/ti,tfp410.txt
> @@ -18,7 +18,14 @@ This device has two video ports. Their connections are modeled using the OF
>  graph bindings specified in [1]. Each port node shall have a single endpoint.
>  
>  - Port 0 is the DPI input port. Its endpoint subnode shall contain a
> -  pclk-sample property and a remote-endpoint property as specified in [1].
> +  pclk-sample and bus-width property and a remote-endpoint property as specified
> +  in [1].
> +  - If pclk-sample is not defined, pclk-sample = 0 should be assumed for
> +    backward compatibility.
> +  - If bus-width is not defined then bus-width = 24 should be assumed for
> +    backward compatibility.
> +    bus-width = 24: 24 data lines are connected and single-edge mode
> +    bus-width = 12: 12 data lines are connected and dual-edge mode
>  
>  - Port 1 is the DVI output port. Its endpoint subnode shall contain a
>    remote-endpoint property is specified in [1].
> @@ -43,6 +50,7 @@ tfp410: encoder at 0 {
>  
>  			tfp410_in: endpoint at 0 {
>  				pclk-sample = <1>;
> +				bus-width = <24>;
>  				remote-endpoint = <&dpi_out>;
>  			};
>  		};

-- 
Regards,

Laurent Pinchart