summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorViresh Kumar <viresh.kumar@linaro.org>2013-08-20 08:38:24 +0200
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2013-08-20 15:43:50 +0200
commit9e9fd801676a946b759a8669baa24ba327c8c903 (patch)
tree9d091f9b64a55eaf68ff7158e7d4a0e68de4ce21
parentcpufreq: remove policy from cpufreq_policy_list during suspend (diff)
downloadlinux-9e9fd801676a946b759a8669baa24ba327c8c903.tar.xz
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>
-rw-r--r--drivers/cpufreq/cpufreq.c2
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;
}