Re: [PATCH 2/2] ARM: dts: Update MFC documentation for clock entries

[Sylwester Nawrocki <s.nawrocki@samsung.com>]

On 04/04/2013 08:26 AM, Sachin Kamat wrote:
> Added clock entry definitions to MFC bindings document.
> 
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> ---
>  .../devicetree/bindings/media/s5p-mfc.txt          |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/media/s5p-mfc.txt b/Documentation/devicetree/bindings/media/s5p-mfc.txt
> index bf0182d..d7b991a 100644
> --- a/Documentation/devicetree/bindings/media/s5p-mfc.txt
> +++ b/Documentation/devicetree/bindings/media/s5p-mfc.txt
> @@ -15,6 +15,9 @@ Required properties:
>  	  mapped region.
>  
>    - interrupts : MFC interrupt number to the CPU.
> +  - clocks: from common clock binding: handle to mfc clocks.
> +  - clock-names: from common clock binding: Shall be "sclk_mfc" and "mfc"
> +		 for "samsung,mfc-v5".

Not sure if we shouldn't be more specific about the fact the entries in
the clocks property must be in same order as names in clock-names. Perhaps
it's sufficient you mentioned the common clock binding which defines that.

Also we should keep the clock names same, regardless which MFC IP version
those apply to. At least as far as same driver is used for them and the
number of clocks is same. Thus perhaps this could be:

- clock-names : from common clock binding: must contain "sclk_mfc" and "mfc",
	corresponding to entries in the clocks property.

Please note there is a white space before the first colon, that makes
it really an important improvement...

Please feel free to add my

Reviewed-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

with those issues addressed.

>    - samsung,mfc-r : Base address of the first memory bank used by MFC
>  		    for DMA contiguous memory allocation and its size.
> @@ -34,6 +37,8 @@ mfc: codec@13400000 {
>  	reg = <0x13400000 0x10000>;
>  	interrupts = <0 94 0>;
>  	samsung,power-domain = <&pd_mfc>;
> +	clocks = <&clock 170>, <&clock 273>;
> +	clock-names = "sclk_mfc", "mfc";
>  };
>  
>  Board specific DT entry: