OMAP2 clock: fix recursive spinlock attempt when CONFIG_CPU_FREQ=y
authorPaul Walmsley <paul@pwsan.com>
Wed, 19 May 2010 02:24:06 +0000 (20:24 -0600)
committerPaul Walmsley <paul@pwsan.com>
Thu, 20 May 2010 18:31:14 +0000 (12:31 -0600)
The OMAP2 MPU virtual clock node code attempted to call clk_get_rate()
while the clockfw_lock spinlock was held.  Fix by reading the sys_ck
rate directly.

Signed-off-by: Paul Walmsley <paul@pwsan.com>
arch/arm/mach-omap2/clkt2xxx_virt_prcm_set.c

index e60ca4e47bbd8a1927e5c11e3e533d3ad5700faf..aef62918aaf0eefee25d71077b18d036e2720d91 100644 (file)
@@ -68,16 +68,13 @@ long omap2_round_to_table_rate(struct clk *clk, unsigned long rate)
 {
        const struct prcm_config *ptr;
        long highest_rate;
-       long sys_ck_rate;
-
-       sys_ck_rate = clk_get_rate(sclk);
 
        highest_rate = -EINVAL;
 
        for (ptr = rate_table; ptr->mpu_speed; ptr++) {
                if (!(ptr->flags & cpu_mask))
                        continue;
-               if (ptr->xtal_speed != sys_ck_rate)
+               if (ptr->xtal_speed != sclk->rate)
                        continue;
 
                highest_rate = ptr->mpu_speed;
@@ -96,15 +93,12 @@ int omap2_select_table_rate(struct clk *clk, unsigned long rate)
        const struct prcm_config *prcm;
        unsigned long found_speed = 0;
        unsigned long flags;
-       long sys_ck_rate;
-
-       sys_ck_rate = clk_get_rate(sclk);
 
        for (prcm = rate_table; prcm->mpu_speed; prcm++) {
                if (!(prcm->flags & cpu_mask))
                        continue;
 
-               if (prcm->xtal_speed != sys_ck_rate)
+               if (prcm->xtal_speed != sclk->rate)
                        continue;
 
                if (prcm->mpu_speed <= rate) {
@@ -181,19 +175,16 @@ static struct cpufreq_frequency_table *freq_table;
 void omap2_clk_init_cpufreq_table(struct cpufreq_frequency_table **table)
 {
        const struct prcm_config *prcm;
-       long sys_ck_rate;
        int i = 0;
        int tbl_sz = 0;
 
        if (!cpu_is_omap24xx())
                return;
 
-       sys_ck_rate = clk_get_rate(sclk);
-
        for (prcm = rate_table; prcm->mpu_speed; prcm++) {
                if (!(prcm->flags & cpu_mask))
                        continue;
-               if (prcm->xtal_speed != sys_ck_rate)
+               if (prcm->xtal_speed != sclk->rate)
                        continue;
 
                /* don't put bypass rates in table */
@@ -226,7 +217,7 @@ void omap2_clk_init_cpufreq_table(struct cpufreq_frequency_table **table)
        for (prcm = rate_table; prcm->mpu_speed; prcm++) {
                if (!(prcm->flags & cpu_mask))
                        continue;
-               if (prcm->xtal_speed != sys_ck_rate)
+               if (prcm->xtal_speed != sclk->rate)
                        continue;
 
                /* don't put bypass rates in table */