diff options
author | Viresh Kumar <viresh.kumar@linaro.org> | 2013-08-20 02:38:24 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-08-20 09:43:50 -0400 |
commit | 9e9fd801676a946b759a8669baa24ba327c8c903 (patch) | |
tree | 9d091f9b64a55eaf68ff7158e7d4a0e68de4ce21 /drivers/cpufreq | |
parent | 9515f4d69b92feafe37581047a1bb41e41602faa (diff) |
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')
-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, | |||
1719 | if ((!policy->governor_enabled && (event == CPUFREQ_GOV_STOP)) || | 1719 | if ((!policy->governor_enabled && (event == CPUFREQ_GOV_STOP)) || |
1720 | (policy->governor_enabled && (event == CPUFREQ_GOV_START))) { | 1720 | (policy->governor_enabled && (event == CPUFREQ_GOV_START))) { |
1721 | mutex_unlock(&cpufreq_governor_lock); | 1721 | mutex_unlock(&cpufreq_governor_lock); |
1722 | if (event == CPUFREQ_GOV_POLICY_INIT) | ||
1723 | module_put(policy->governor->owner); | ||
1724 | return -EBUSY; | 1722 | return -EBUSY; |
1725 | } | 1723 | } |
1726 | 1724 | ||