Commit 9e9fd801 authored Aug 20, 2013 by Viresh Kumar Committed by Rafael J. Wysocki Aug 20, 2013

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>

parent 9515f4d6

drivers/cpufreq/cpufreq.c

+0 −2

Original line number	Diff line number	Diff line
		@@ -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;
		}

Admin message