Commit adff42b1 authored by Dave Jones's avatar Dave Jones

[CPUFREQ] Fix various oddities in the userspace governer.

parent 2e4dc88b
...@@ -498,9 +498,9 @@ static int cpufreq_governor_userspace(struct cpufreq_policy *policy, ...@@ -498,9 +498,9 @@ static int cpufreq_governor_userspace(struct cpufreq_policy *policy,
unsigned int cpu = policy->cpu; unsigned int cpu = policy->cpu;
switch (event) { switch (event) {
case CPUFREQ_GOV_START: case CPUFREQ_GOV_START:
if ((!cpu_online(cpu)) || (!try_module_get(THIS_MODULE)) || if ((!cpu_online(cpu)) || (!try_module_get(THIS_MODULE)))
!policy->cur)
return -EINVAL; return -EINVAL;
BUG_ON(!policy->cur);
down(&userspace_sem); down(&userspace_sem);
cpu_is_managed[cpu] = 1; cpu_is_managed[cpu] = 1;
cpu_min_freq[cpu] = policy->min; cpu_min_freq[cpu] = policy->min;
...@@ -551,7 +551,7 @@ static void cpufreq_sa11x0_compat(void) ...@@ -551,7 +551,7 @@ static void cpufreq_sa11x0_compat(void)
#endif #endif
static struct cpufreq_governor cpufreq_gov_userspace = { struct cpufreq_governor cpufreq_gov_userspace = {
.name = "userspace", .name = "userspace",
.governor = cpufreq_governor_userspace, .governor = cpufreq_governor_userspace,
.owner = THIS_MODULE, .owner = THIS_MODULE,
...@@ -587,5 +587,5 @@ MODULE_AUTHOR ("Dominik Brodowski <linux@brodo.de>, Russell King <rmk@arm.linux. ...@@ -587,5 +587,5 @@ MODULE_AUTHOR ("Dominik Brodowski <linux@brodo.de>, Russell King <rmk@arm.linux.
MODULE_DESCRIPTION ("CPUfreq policy governor 'userspace'"); MODULE_DESCRIPTION ("CPUfreq policy governor 'userspace'");
MODULE_LICENSE ("GPL"); MODULE_LICENSE ("GPL");
module_init(cpufreq_gov_userspace_init); fs_initcall(cpufreq_gov_userspace_init);
module_exit(cpufreq_gov_userspace_exit); module_exit(cpufreq_gov_userspace_exit);
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment