]> Pileus Git - ~andy/linux/commitdiff
cpufreq: OMAP: put clk if cpu_init failed
authorNishanth Menon <nm@ti.com>
Fri, 27 May 2011 02:39:19 +0000 (19:39 -0700)
committerKevin Hilman <khilman@ti.com>
Tue, 8 Nov 2011 19:42:17 +0000 (11:42 -0800)
Release the mpu_clk in fail paths.

Reported-by: Todd Poynor <toddpoynor@google.com>
Signed-off-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Kevin Hilman <khilman@ti.com>
drivers/cpufreq/omap-cpufreq.c

index 8f778b9dbb46bd8e7da5443521404adfe08565ba..8c5419201ac55b69a1d9543bbbb6dc9f6b3aac08 100644 (file)
@@ -158,8 +158,10 @@ static int __cpuinit omap_cpu_init(struct cpufreq_policy *policy)
        if (IS_ERR(mpu_clk))
                return PTR_ERR(mpu_clk);
 
-       if (policy->cpu >= NR_CPUS)
-               return -EINVAL;
+       if (policy->cpu >= NR_CPUS) {
+               result = -EINVAL;
+               goto fail_ck;
+       }
 
        policy->cur = policy->min = policy->max = omap_getspeed(policy->cpu);
        result = opp_init_cpufreq_table(mpu_dev, &freq_table);
@@ -167,12 +169,14 @@ static int __cpuinit omap_cpu_init(struct cpufreq_policy *policy)
        if (result) {
                dev_err(mpu_dev, "%s: cpu%d: failed creating freq table[%d]\n",
                                __func__, policy->cpu, result);
-               return result;
+               goto fail_ck;
        }
 
        result = cpufreq_frequency_table_cpuinfo(policy, freq_table);
        if (!result)
                cpufreq_frequency_table_get_attr(freq_table, policy->cpu);
+       else
+               goto fail_ck;
 
        policy->min = policy->cpuinfo.min_freq;
        policy->max = policy->cpuinfo.max_freq;
@@ -194,6 +198,10 @@ static int __cpuinit omap_cpu_init(struct cpufreq_policy *policy)
        policy->cpuinfo.transition_latency = 300 * 1000;
 
        return 0;
+
+fail_ck:
+       clk_put(mpu_clk);
+       return result;
 }
 
 static int omap_cpu_exit(struct cpufreq_policy *policy)