diff options
author | Viresh Kumar <viresh.kumar@linaro.org> | 2013-08-20 12:08:24 +0530 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-08-20 15:43:50 +0200 |
commit | 9e9fd801676a946b759a8669baa24ba327c8c903 (patch) | |
tree | 9d091f9b64a55eaf68ff7158e7d4a0e68de4ce21 /drivers/cpufreq/cpufreq.c | |
parent | 9515f4d69b92feafe37581047a1bb41e41602faa (diff) | |
download | talos-obmc-linux-9e9fd801676a946b759a8669baa24ba327c8c903.tar.gz talos-obmc-linux-9e9fd801676a946b759a8669baa24ba327c8c903.zip |
cpufreq: remove unnecessary check in __cpufreq_governor()
We don't need to check if event is CPUFREQ_GOV_POLICY_INIT and put
governor module as we are sure event can only be START/STOP here.
Remove the useless check.
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/cpufreq/cpufreq.c')
-rw-r--r-- | drivers/cpufreq/cpufreq.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 030212104fe6..b03a8514b574 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -1719,8 +1719,6 @@ static int __cpufreq_governor(struct cpufreq_policy *policy, if ((!policy->governor_enabled && (event == CPUFREQ_GOV_STOP)) || (policy->governor_enabled && (event == CPUFREQ_GOV_START))) { mutex_unlock(&cpufreq_governor_lock); - if (event == CPUFREQ_GOV_POLICY_INIT) - module_put(policy->governor->owner); return -EBUSY; } |