On Mon, 10 Aug 2009, Sanjeev Premi wrote:

> The argument 'mpurate' had no effect on the MPU
> frequency. This patch fixes the same.

Thanks, I'll plan to squeeze this into the .32 merge window, with a couple 
of minor changes described below.  Please review the changes:

> 
> Signed-off-by: Sanjeev Premi <[email protected]>
> ---
>  arch/arm/mach-omap2/clock34xx.c |   14 ++++++++++----
>  1 files changed, 10 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/clock34xx.c b/arch/arm/mach-omap2/clock34xx.c
> index 045da92..0c62aad 100644
> --- a/arch/arm/mach-omap2/clock34xx.c
> +++ b/arch/arm/mach-omap2/clock34xx.c
> @@ -27,6 +27,7 @@
>  #include <linux/limits.h>
>  #include <linux/bitops.h>
>  
> +#include <mach/cpu.h>
>  #include <mach/clock.h>
>  #include <mach/sram.h>
>  #include <asm/div64.h>
> @@ -948,13 +949,18 @@ static int __init omap2_clk_arch_init(void)
>       if (clk_set_rate(&virt_prcm_set, mpurate))
>               printk(KERN_ERR "Could not find matching MPU rate\n");
>  #endif

The previous four lines (including the three above) have been removed from 
clock34xx.c.

> +     if (clk_set_rate(&dpll1_ck, mpurate))
> +             printk(KERN_ERR "*** Unable to set MPU rate\n");
> +     omap3_dpll_recalc(&dpll1_ck);

I dropped the omap3_dpll_recalc(&dpll1_ck); since the following 
recalculate_root_clocks() should handle this.

>       recalculate_root_clocks();
>  
> -     printk(KERN_INFO "Switched to new clocking rate (Crystal/DPLL3/MPU): "
> +     printk(KERN_INFO "Switched to new clocking rate (Crystal/Core/MPU): "
>              "%ld.%01ld/%ld/%ld MHz\n",
> -            (osc_sys_ck.rate / 1000000), (osc_sys_ck.rate / 100000) % 10,
> -            (core_ck.rate / 1000000), (dpll1_fck.rate / 1000000)) ;
> +            (osc_sys_ck.rate / 1000000), ((osc_sys_ck.rate / 100000) % 10),
> +            (core_ck.rate / 1000000), (arm_fck.rate / 1000000)) ;
> +
> +     calibrate_delay();
>  
>       return 0;
>  }
> @@ -1013,7 +1019,7 @@ int __init omap2_clk_init(void)
>  
>       recalculate_root_clocks();
>  
> -     printk(KERN_INFO "Clocking rate (Crystal/DPLL/ARM core): "
> +     printk(KERN_INFO "Clocking rate (Crystal/Core/MPU): "
>              "%ld.%01ld/%ld/%ld MHz\n",
>              (osc_sys_ck.rate / 1000000), (osc_sys_ck.rate / 100000) % 10,
>              (core_ck.rate / 1000000), (arm_fck.rate / 1000000));
> -- 
> 1.6.2.2
> 


- Paul
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to