diff mbox

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

Message ID 1365056815-28000-2-git-send-email-sachin.kamat@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Sachin Kamat April 4, 2013, 6:26 a.m. UTC
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(+)

Comments

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:

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Sachin Kamat April 8, 2013, 6:14 a.m. UTC | #2
Hi Sylwester,

Thanks for your review.

On 4 April 2013 16:38, Sylwester Nawrocki <s.nawrocki@samsung.com> wrote:
> 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.

Right.

> 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:

Right. Since mfc-v6 currently took aclk_333 as second clk instead of
sclk_mfc, I made the description specific to mfc-v5.

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

I will update as above.

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

Ok.

>
> Please feel free to add my
>
> Reviewed-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
>
> with those issues addressed.

Thanks.
diff mbox

Patch

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".
 
   - 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: