Tero Kristo <[EMAIL PROTECTED]> writes:

> Noncore dpll can enter autoidle state, in which case the rate calculation
> fails. Fixed by checking dpll mode instead of idle status.
>
> Signed-off-by: Tero Kristo <[EMAIL PROTECTED]>

Signed-off-by: Kevin Hilman <[EMAIL PROTECTED]>

> ---
>  arch/arm/mach-omap2/clock.c           |   16 +++++++++++-----
>  arch/arm/mach-omap2/cm-regbits-34xx.h |    2 ++
>  2 files changed, 13 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/clock.c b/arch/arm/mach-omap2/clock.c
> index 522ce6f..bbd3e82 100644
> --- a/arch/arm/mach-omap2/clock.c
> +++ b/arch/arm/mach-omap2/clock.c
> @@ -212,9 +212,16 @@ u32 omap2_get_dpll_rate(struct clk *clk)
>               return 0;
>  
>       /* Return bypass rate if DPLL is bypassed */
> -     v = cm_read_mod_reg(clk->prcm_mod, dd->idlest_reg);
> -     v &= dd->idlest_mask;
> -     v >>= __ffs(dd->idlest_mask);
> +     if (cpu_is_omap34xx()) {
> +             v = cm_read_mod_reg(clk->prcm_mod, dd->control_reg);
> +             v &= dd->enable_mask;
> +             v >>= __ffs(dd->enable_mask);
> +     } else {
> +             v = cm_read_mod_reg(clk->prcm_mod, dd->idlest_reg);
> +             v &= dd->idlest_mask;
> +             v >>= __ffs(dd->idlest_mask);
> +     }
> +
>       if (cpu_is_omap24xx()) {
>  
>               if (v == ST_CORE_CLK_REF)
> @@ -224,9 +231,8 @@ u32 omap2_get_dpll_rate(struct clk *clk)
>  
>       } else if (cpu_is_omap34xx()) {
>  
> -             if (!v)
> +             if (v == OMAP3430_EN_MPU_DPLL_BYPASS)
>                       return dd->bypass_clk->rate;
> -
>       }
>  
>       v = cm_read_mod_reg(clk->prcm_mod, dd->mult_div1_reg);
> diff --git a/arch/arm/mach-omap2/cm-regbits-34xx.h 
> b/arch/arm/mach-omap2/cm-regbits-34xx.h
> index 6f3f5a3..9995353 100644
> --- a/arch/arm/mach-omap2/cm-regbits-34xx.h
> +++ b/arch/arm/mach-omap2/cm-regbits-34xx.h
> @@ -112,6 +112,8 @@
>  #define OMAP3430_EN_MPU_DPLL_DRIFTGUARD_MASK         (1 << 3)
>  #define OMAP3430_EN_MPU_DPLL_SHIFT                   0
>  #define OMAP3430_EN_MPU_DPLL_MASK                    (0x7 << 0)
> +#define OMAP3430_EN_MPU_DPLL_LOCK                    (0x7 << 0)
> +#define OMAP3430_EN_MPU_DPLL_BYPASS                  (0x5 << 0)
>  
>  /* CM_IDLEST_MPU */
>  #define OMAP3430_ST_MPU                                      (1 << 0)
> -- 
> 1.5.4.3
>
> --
> 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
--
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