public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] The clockdomain of clkout2_src_ck fixed
@ 2008-08-29 14:48 Tomi Valkeinen
  2008-09-04  1:44 ` Tony Lindgren
  0 siblings, 1 reply; 5+ messages in thread
From: Tomi Valkeinen @ 2008-08-29 14:48 UTC (permalink / raw)
  To: linux-omap; +Cc: Tomi Valkeinen

The clockdomain of clkout2_src_ck was set to "core_clkdm", which does not
exist.
---
 arch/arm/mach-omap2/clock34xx.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/clock34xx.h b/arch/arm/mach-omap2/clock34xx.h
index d7d6ffd..3504e48 100644
--- a/arch/arm/mach-omap2/clock34xx.h
+++ b/arch/arm/mach-omap2/clock34xx.h
@@ -1010,7 +1010,7 @@ static struct clk clkout2_src_ck = {
 	.clksel_mask	= OMAP3430_CLKOUT2SOURCE_MASK,
 	.clksel		= clkout2_src_clksel,
 	.flags		= CLOCK_IN_OMAP343X | RATE_PROPAGATES,
-	.clkdm		= { .name = "core_clkdm" },
+	.clkdm		= { .name = "core_l4_clkdm" },
 	.recalc		= &omap2_clksel_recalc,
 };
 
-- 
1.5.4.3


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] The clockdomain of clkout2_src_ck fixed
  2008-08-29 14:48 [PATCH] The clockdomain of clkout2_src_ck fixed Tomi Valkeinen
@ 2008-09-04  1:44 ` Tony Lindgren
  2008-09-04  8:54   ` Tomi Valkeinen
  0 siblings, 1 reply; 5+ messages in thread
From: Tony Lindgren @ 2008-09-04  1:44 UTC (permalink / raw)
  To: Tomi Valkeinen; +Cc: linux-omap

* Tomi Valkeinen <tomi.valkeinen@nokia.com> [080829 07:50]:
> The clockdomain of clkout2_src_ck was set to "core_clkdm", which does not
> exist.

Can you please reply with your Signed-off-by?

Thanks,

Tony

> ---
>  arch/arm/mach-omap2/clock34xx.h |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/clock34xx.h b/arch/arm/mach-omap2/clock34xx.h
> index d7d6ffd..3504e48 100644
> --- a/arch/arm/mach-omap2/clock34xx.h
> +++ b/arch/arm/mach-omap2/clock34xx.h
> @@ -1010,7 +1010,7 @@ static struct clk clkout2_src_ck = {
>  	.clksel_mask	= OMAP3430_CLKOUT2SOURCE_MASK,
>  	.clksel		= clkout2_src_clksel,
>  	.flags		= CLOCK_IN_OMAP343X | RATE_PROPAGATES,
> -	.clkdm		= { .name = "core_clkdm" },
> +	.clkdm		= { .name = "core_l4_clkdm" },
>  	.recalc		= &omap2_clksel_recalc,
>  };
>  
> -- 
> 1.5.4.3
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH] The clockdomain of clkout2_src_ck fixed
  2008-09-04  1:44 ` Tony Lindgren
@ 2008-09-04  8:54   ` Tomi Valkeinen
  2008-09-04 17:14     ` Paul Walmsley
  0 siblings, 1 reply; 5+ messages in thread
From: Tomi Valkeinen @ 2008-09-04  8:54 UTC (permalink / raw)
  To: tony; +Cc: linux-omap, Tomi Valkeinen

The clockdomain of clkout2_src_ck was set to "core_clkdm", which does not
exist.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@nokia.com>
---
 arch/arm/mach-omap2/clock34xx.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/clock34xx.h b/arch/arm/mach-omap2/clock34xx.h
index 5d855e9..8da0097 100644
--- a/arch/arm/mach-omap2/clock34xx.h
+++ b/arch/arm/mach-omap2/clock34xx.h
@@ -1010,7 +1010,7 @@ static struct clk clkout2_src_ck = {
 	.clksel_mask	= OMAP3430_CLKOUT2SOURCE_MASK,
 	.clksel		= clkout2_src_clksel,
 	.flags		= CLOCK_IN_OMAP343X | RATE_PROPAGATES,
-	.clkdm		= { .name = "core_clkdm" },
+	.clkdm		= { .name = "core_l4_clkdm" },
 	.recalc		= &omap2_clksel_recalc,
 };
 
-- 
1.6.0.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] The clockdomain of clkout2_src_ck fixed
  2008-09-04  8:54   ` Tomi Valkeinen
@ 2008-09-04 17:14     ` Paul Walmsley
  2008-09-05 16:53       ` Tony Lindgren
  0 siblings, 1 reply; 5+ messages in thread
From: Paul Walmsley @ 2008-09-04 17:14 UTC (permalink / raw)
  To: Tomi Valkeinen; +Cc: tony, linux-omap

On Thu, 4 Sep 2008, Tomi Valkeinen wrote:

> The clockdomain of clkout2_src_ck was set to "core_clkdm", which does not
> exist.

This is fine for now.  Technically, clkout2_src_ck is in the CM 
clockdomain, and there are some patches here that will add that; but until 
those are posted, this is good enough.

Acked-by: Paul Walmsley <paul@pwsan.com>

> 
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@nokia.com>
> ---
>  arch/arm/mach-omap2/clock34xx.h |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/clock34xx.h b/arch/arm/mach-omap2/clock34xx.h
> index 5d855e9..8da0097 100644
> --- a/arch/arm/mach-omap2/clock34xx.h
> +++ b/arch/arm/mach-omap2/clock34xx.h
> @@ -1010,7 +1010,7 @@ static struct clk clkout2_src_ck = {
>  	.clksel_mask	= OMAP3430_CLKOUT2SOURCE_MASK,
>  	.clksel		= clkout2_src_clksel,
>  	.flags		= CLOCK_IN_OMAP343X | RATE_PROPAGATES,
> -	.clkdm		= { .name = "core_clkdm" },
> +	.clkdm		= { .name = "core_l4_clkdm" },
>  	.recalc		= &omap2_clksel_recalc,
>  };
>  
> -- 
> 1.6.0.1
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 


- Paul

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] The clockdomain of clkout2_src_ck fixed
  2008-09-04 17:14     ` Paul Walmsley
@ 2008-09-05 16:53       ` Tony Lindgren
  0 siblings, 0 replies; 5+ messages in thread
From: Tony Lindgren @ 2008-09-05 16:53 UTC (permalink / raw)
  To: Paul Walmsley; +Cc: Tomi Valkeinen, linux-omap

* Paul Walmsley <paul@pwsan.com> [080904 10:14]:
> On Thu, 4 Sep 2008, Tomi Valkeinen wrote:
> 
> > The clockdomain of clkout2_src_ck was set to "core_clkdm", which does not
> > exist.
> 
> This is fine for now.  Technically, clkout2_src_ck is in the CM 
> clockdomain, and there are some patches here that will add that; but until 
> those are posted, this is good enough.

Pushing today.

Tony

> 
> Acked-by: Paul Walmsley <paul@pwsan.com>
> 
> > 
> > Signed-off-by: Tomi Valkeinen <tomi.valkeinen@nokia.com>
> > ---
> >  arch/arm/mach-omap2/clock34xx.h |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/arch/arm/mach-omap2/clock34xx.h b/arch/arm/mach-omap2/clock34xx.h
> > index 5d855e9..8da0097 100644
> > --- a/arch/arm/mach-omap2/clock34xx.h
> > +++ b/arch/arm/mach-omap2/clock34xx.h
> > @@ -1010,7 +1010,7 @@ static struct clk clkout2_src_ck = {
> >  	.clksel_mask	= OMAP3430_CLKOUT2SOURCE_MASK,
> >  	.clksel		= clkout2_src_clksel,
> >  	.flags		= CLOCK_IN_OMAP343X | RATE_PROPAGATES,
> > -	.clkdm		= { .name = "core_clkdm" },
> > +	.clkdm		= { .name = "core_l4_clkdm" },
> >  	.recalc		= &omap2_clksel_recalc,
> >  };
> >  
> > -- 
> > 1.6.0.1
> > 
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > 
> 
> 
> - Paul

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2008-09-05 16:53 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-08-29 14:48 [PATCH] The clockdomain of clkout2_src_ck fixed Tomi Valkeinen
2008-09-04  1:44 ` Tony Lindgren
2008-09-04  8:54   ` Tomi Valkeinen
2008-09-04 17:14     ` Paul Walmsley
2008-09-05 16:53       ` Tony Lindgren

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox