diff mbox

[1/1] of/documentation: Fix a typo in exynos5250-clock.txt

Message ID 1373012281-26433-1-git-send-email-sachin.kamat@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Sachin Kamat July 5, 2013, 8:18 a.m. UTC
The clocks should be numbered sequentially as they are represented as
enum values in the clock driver.

Typo introduced by commit 17d4caccef ("clk: exynos5250: register
display block gate clocks to common clock framework")

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 .../devicetree/bindings/clock/exynos5250-clock.txt |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Kim Kukjin July 22, 2013, 1:51 a.m. UTC | #1
Sachin Kamat wrote:
> 
> The clocks should be numbered sequentially as they are represented as
> enum values in the clock driver.
> 
> Typo introduced by commit 17d4caccef ("clk: exynos5250: register
> display block gate clocks to common clock framework")
> 
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> ---
>  .../devicetree/bindings/clock/exynos5250-clock.txt |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
> b/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
> index 781a627..1a05761 100644
> --- a/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
> +++ b/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
> @@ -154,7 +154,7 @@ clock which they consume.
>    dsim0			341
>    dp			342
>    mixer			343
> -  hdmi			345
> +  hdmi			344
> 
>  Example 1: An example of a clock controller node is listed below.
> 
> --
> 1.7.9.5

Looks OK to me.

Mike, do you want me to take into the samsung tree?

Thanks,
Kukjin
Mike Turquette July 22, 2013, 4:42 a.m. UTC | #2
Quoting Kukjin Kim (2013-07-21 18:51:54)
> Sachin Kamat wrote:
> > 
> > The clocks should be numbered sequentially as they are represented as
> > enum values in the clock driver.
> > 
> > Typo introduced by commit 17d4caccef ("clk: exynos5250: register
> > display block gate clocks to common clock framework")
> > 
> > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> > ---
> >  .../devicetree/bindings/clock/exynos5250-clock.txt |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
> > b/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
> > index 781a627..1a05761 100644
> > --- a/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
> > +++ b/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
> > @@ -154,7 +154,7 @@ clock which they consume.
> >    dsim0                      341
> >    dp                 342
> >    mixer                      343
> > -  hdmi                       345
> > +  hdmi                       344
> > 
> >  Example 1: An example of a clock controller node is listed below.
> > 
> > --
> > 1.7.9.5
> 
> Looks OK to me.
> 
> Mike, do you want me to take into the samsung tree?

I've pulled this into clk-next.

Thanks!
Mike

> 
> Thanks,
> Kukjin
Sachin Kamat July 22, 2013, 4:54 a.m. UTC | #3
On 22 July 2013 10:12, Mike Turquette <mturquette@linaro.org> wrote:
> Quoting Kukjin Kim (2013-07-21 18:51:54)
>> Sachin Kamat wrote:
>> >
>> > The clocks should be numbered sequentially as they are represented as
>> > enum values in the clock driver.
>> >
>> > Typo introduced by commit 17d4caccef ("clk: exynos5250: register
>> > display block gate clocks to common clock framework")
>> >
>> > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
>> > ---
>> >  .../devicetree/bindings/clock/exynos5250-clock.txt |    2 +-
>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>> > diff --git a/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
>> > b/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
>> > index 781a627..1a05761 100644
>> > --- a/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
>> > +++ b/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
>> > @@ -154,7 +154,7 @@ clock which they consume.
>> >    dsim0                      341
>> >    dp                 342
>> >    mixer                      343
>> > -  hdmi                       345
>> > +  hdmi                       344
>> >
>> >  Example 1: An example of a clock controller node is listed below.
>> >
>> > --
>> > 1.7.9.5
>>
>> Looks OK to me.
>>
>> Mike, do you want me to take into the samsung tree?
>
> I've pulled this into clk-next.

Thanks Mike.
Please also consider the following patch series:

http://comments.gmane.org/gmane.linux.kernel.samsung-soc/20829
http://comments.gmane.org/gmane.linux.kernel.samsung-soc/20582
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/clock/exynos5250-clock.txt b/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
index 781a627..1a05761 100644
--- a/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
+++ b/Documentation/devicetree/bindings/clock/exynos5250-clock.txt
@@ -154,7 +154,7 @@  clock which they consume.
   dsim0			341
   dp			342
   mixer			343
-  hdmi			345
+  hdmi			344
 
 Example 1: An example of a clock controller node is listed below.