[Patch v2 2/6] dt-bindings: display/ti: Move common dispc bindings to omap-dss.txt

Laurent Pinchart laurent.pinchart at ideasonboard.com
Wed Apr 4 14:29:59 UTC 2018


Hi Benoit,

Thank you for the patch.

On Monday, 26 March 2018 19:21:24 EEST Benoit Parrot wrote:
> Add common DISPC bindings into the top level bindings file.
> Move common bindings here instead of having multiple copies of
> the same information in all of the variant specific files.
> 
> Signed-off-by: Benoit Parrot <bparrot at ti.com>
> Reviewed-by: Rob Herring <robh at kernel.org>
> ---
>  Documentation/devicetree/bindings/display/ti/ti,dra7-dss.txt  | 5 -----
>  Documentation/devicetree/bindings/display/ti/ti,omap-dss.txt  | 7 +++++++
>  Documentation/devicetree/bindings/display/ti/ti,omap2-dss.txt | 4 ----
>  Documentation/devicetree/bindings/display/ti/ti,omap3-dss.txt | 4 ----
>  Documentation/devicetree/bindings/display/ti/ti,omap4-dss.txt | 4 ----
>  Documentation/devicetree/bindings/display/ti/ti,omap5-dss.txt | 4 ----
>  6 files changed, 7 insertions(+), 21 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/display/ti/ti,dra7-dss.txt
> b/Documentation/devicetree/bindings/display/ti/ti,dra7-dss.txt index
> 91279f1060fe..c30f9ec189ed 100644
> --- a/Documentation/devicetree/bindings/display/ti/ti,dra7-dss.txt
> +++ b/Documentation/devicetree/bindings/display/ti/ti,dra7-dss.txt
> @@ -47,11 +47,6 @@ Required properties:
>  - clocks: handle to fclk
>  - clock-names: "fck"
> 
> -Optional properties:
> -- max-memory-bandwidth: Input memory (from main memory to dispc) bandwidth
> limit -			in bytes per second
> -
> -
>  HDMI
>  ----
> 
> diff --git a/Documentation/devicetree/bindings/display/ti/ti,omap-dss.txt
> b/Documentation/devicetree/bindings/display/ti/ti,omap-dss.txt index
> e1ef29569338..249e588d7865 100644
> --- a/Documentation/devicetree/bindings/display/ti/ti,omap-dss.txt
> +++ b/Documentation/devicetree/bindings/display/ti/ti,omap-dss.txt
> @@ -21,6 +21,13 @@ a RGB pixel stream to encoders.
>  The encoder modules encode the received RGB pixel stream to a video output
> like HDMI, MIPI DPI, etc.
> 
> +DISPC
> +-----
> +
> +Optional properties:
> +- max-memory-bandwidth: Input memory (from main memory to dispc) bandwidth
> limit
> +			in bytes per second
> +

I like the idea, but I think you need to expand the description a bit. 
ti,omap-dss.txt should explain how each module should be represented by a 
child DT node in the DSS DT node, and this section should briefly explain what 
the DISPC is (you can simply move the 4th paragraph of the file here) and that 
it describes common properties. You could also move the description of other 
DISPC properties here.

>  Video Ports
>  -----------
> 
> diff --git a/Documentation/devicetree/bindings/display/ti/ti,omap2-dss.txt
> b/Documentation/devicetree/bindings/display/ti/ti,omap2-dss.txt index
> ee867c4d1152..afcd5a86c6a4 100644
> --- a/Documentation/devicetree/bindings/display/ti/ti,omap2-dss.txt
> +++ b/Documentation/devicetree/bindings/display/ti/ti,omap2-dss.txt
> @@ -28,10 +28,6 @@ Required properties:
>  - ti,hwmods: "dss_dispc"
>  - interrupts: the DISPC interrupt
> 
> -Optional properties:
> -- max-memory-bandwidth: Input memory (from main memory to dispc) bandwidth
> limit -			in bytes per second
> -
> 
>  RFBI
>  ----
> diff --git a/Documentation/devicetree/bindings/display/ti/ti,omap3-dss.txt
> b/Documentation/devicetree/bindings/display/ti/ti,omap3-dss.txt index
> cd02516a40b6..dc66e1447c31 100644
> --- a/Documentation/devicetree/bindings/display/ti/ti,omap3-dss.txt
> +++ b/Documentation/devicetree/bindings/display/ti/ti,omap3-dss.txt
> @@ -37,10 +37,6 @@ Required properties:
>  - clocks: handle to fclk
>  - clock-names: "fck"
> 
> -Optional properties:
> -- max-memory-bandwidth: Input memory (from main memory to dispc) bandwidth
> limit -			in bytes per second
> -
> 
>  RFBI
>  ----
> diff --git a/Documentation/devicetree/bindings/display/ti/ti,omap4-dss.txt
> b/Documentation/devicetree/bindings/display/ti/ti,omap4-dss.txt index
> 0f85f6b3a5a8..bc624db8888d 100644
> --- a/Documentation/devicetree/bindings/display/ti/ti,omap4-dss.txt
> +++ b/Documentation/devicetree/bindings/display/ti/ti,omap4-dss.txt
> @@ -36,10 +36,6 @@ Required properties:
>  - clocks: handle to fclk
>  - clock-names: "fck"
> 
> -Optional properties:
> -- max-memory-bandwidth: Input memory (from main memory to dispc) bandwidth
> limit -			in bytes per second
> -
> 
>  RFBI
>  ----
> diff --git a/Documentation/devicetree/bindings/display/ti/ti,omap5-dss.txt
> b/Documentation/devicetree/bindings/display/ti/ti,omap5-dss.txt index
> 20861218649f..118a486c47bb 100644
> --- a/Documentation/devicetree/bindings/display/ti/ti,omap5-dss.txt
> +++ b/Documentation/devicetree/bindings/display/ti/ti,omap5-dss.txt
> @@ -36,10 +36,6 @@ Required properties:
>  - clocks: handle to fclk
>  - clock-names: "fck"
> 
> -Optional properties:
> -- max-memory-bandwidth: Input memory (from main memory to dispc) bandwidth
> limit -			in bytes per second
> -
> 
>  RFBI
>  ----

-- 
Regards,

Laurent Pinchart





More information about the dri-devel mailing list