diff mbox

OMAP3: set the core dpll clk rate in its set_rate function.

Message ID 1304944180-20443-1-git-send-email-avinashhm@ti.com (mailing list archive)
State Accepted
Commit 5fd2a84ab3c8b87176e25db1d98c5cc34043a669
Headers show

Commit Message

Avinash H.M. May 9, 2011, 12:29 p.m. UTC 
The debug l3_ick/rate is not displaying the actual rate of the clock in
hardware. This is because, the core dpll set_rate function doesn't update the
clk.rate. After fixing, the l3_ick/rate is displaying proper values.

Signed-off-by: Shweta Gulati <shweta.gulati@ti.com>
Signed-off-by: Avinash.H.M <avinashhm@ti.com>
Cc: Rajendra Nayak <rnayak@ti.com>
Cc: Paul Wamsley <paul@pwsan.com>
---
* The patch is based on v2.6.39-rc6
* The patch is tested on zoom3.
* The patch is based on discussions from 
  http://www.mail-archive.com/linux-omap@vger.kernel.org/msg48518.html

 arch/arm/mach-omap2/clkt34xx_dpll3m2.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

Comments

Paul Walmsley May 13, 2011, 2:17 a.m. UTC | #1 
On Mon, 9 May 2011, Avinash.H.M wrote:

> The debug l3_ick/rate is not displaying the actual rate of the clock in
> hardware. This is because, the core dpll set_rate function doesn't update the
> clk.rate. After fixing, the l3_ick/rate is displaying proper values.
> 
> Signed-off-by: Shweta Gulati <shweta.gulati@ti.com>
> Signed-off-by: Avinash.H.M <avinashhm@ti.com>
> Cc: Rajendra Nayak <rnayak@ti.com>
> Cc: Paul Wamsley <paul@pwsan.com>

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

Thanks, let's see if we can get this into an -rc.


- Paul
--
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
Tony Lindgren May 13, 2011, 2:06 p.m. UTC | #2 
* Paul Walmsley <paul@pwsan.com> [110513 05:13]:
> On Mon, 9 May 2011, Avinash.H.M wrote:
> 
> > The debug l3_ick/rate is not displaying the actual rate of the clock in
> > hardware. This is because, the core dpll set_rate function doesn't update the
> > clk.rate. After fixing, the l3_ick/rate is displaying proper values.
> > 
> > Signed-off-by: Shweta Gulati <shweta.gulati@ti.com>
> > Signed-off-by: Avinash.H.M <avinashhm@ti.com>
> > Cc: Rajendra Nayak <rnayak@ti.com>
> > Cc: Paul Wamsley <paul@pwsan.com>
> 
> Acked-by: Paul Walmsley <paul@pwsan.com>
> 
> Thanks, let's see if we can get this into an -rc.

Might be out of time as we still should have this sit in
for-next too. Anyways, adding into devel-fixes.

Tony
--
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
diff mbox

Patch

diff --git a/arch/arm/mach-omap2/clkt34xx_dpll3m2.c b/arch/arm/mach-omap2/clkt34xx_dpll3m2.c
index b10d9ef..2a77faf 100644
--- a/arch/arm/mach-omap2/clkt34xx_dpll3m2.c
+++ b/arch/arm/mach-omap2/clkt34xx_dpll3m2.c
@@ -118,6 +118,7 @@  int omap3_core_dpll_m2_set_rate(struct clk *clk, unsigned long rate)
 				  sdrc_cs0->rfr_ctrl, sdrc_cs0->actim_ctrla,
 				  sdrc_cs0->actim_ctrlb, sdrc_cs0->mr,
 				  0, 0, 0, 0);
+	clk->rate = rate;
 
 	return 0;
 }