16de72b9f4
The performance and powersave cpufreq governors handle the CPUFREQ_GOV_START event in the same way as CPUFREQ_GOV_LIMITS. However, the cpufreq core always invokes cpufreq_governor() with the event argument equal to CPUFREQ_GOV_LIMITS right after invoking it with event equal to CPUFREQ_GOV_START. As a result, for both the governors in question, __cpufreq_driver_target() is executed twice in a row with the same arguments which is not useful. For this reason, simplify the performance and powersave governors to handle the CPUFREQ_GOV_LIMITS event only as that's going to be sufficient for the governor start too. Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
65 lines
1.5 KiB
C
65 lines
1.5 KiB
C
/*
|
|
* linux/drivers/cpufreq/cpufreq_powersave.c
|
|
*
|
|
* Copyright (C) 2002 - 2003 Dominik Brodowski <linux@brodo.de>
|
|
*
|
|
*
|
|
* This program is free software; you can redistribute it and/or modify
|
|
* it under the terms of the GNU General Public License version 2 as
|
|
* published by the Free Software Foundation.
|
|
*
|
|
*/
|
|
|
|
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
|
|
|
#include <linux/cpufreq.h>
|
|
#include <linux/init.h>
|
|
#include <linux/module.h>
|
|
|
|
static int cpufreq_governor_powersave(struct cpufreq_policy *policy,
|
|
unsigned int event)
|
|
{
|
|
switch (event) {
|
|
case CPUFREQ_GOV_LIMITS:
|
|
pr_debug("setting to %u kHz\n", policy->min);
|
|
__cpufreq_driver_target(policy, policy->min,
|
|
CPUFREQ_RELATION_L);
|
|
break;
|
|
default:
|
|
break;
|
|
}
|
|
return 0;
|
|
}
|
|
|
|
static struct cpufreq_governor cpufreq_gov_powersave = {
|
|
.name = "powersave",
|
|
.governor = cpufreq_governor_powersave,
|
|
.owner = THIS_MODULE,
|
|
};
|
|
|
|
static int __init cpufreq_gov_powersave_init(void)
|
|
{
|
|
return cpufreq_register_governor(&cpufreq_gov_powersave);
|
|
}
|
|
|
|
static void __exit cpufreq_gov_powersave_exit(void)
|
|
{
|
|
cpufreq_unregister_governor(&cpufreq_gov_powersave);
|
|
}
|
|
|
|
MODULE_AUTHOR("Dominik Brodowski <linux@brodo.de>");
|
|
MODULE_DESCRIPTION("CPUfreq policy governor 'powersave'");
|
|
MODULE_LICENSE("GPL");
|
|
|
|
#ifdef CONFIG_CPU_FREQ_DEFAULT_GOV_POWERSAVE
|
|
struct cpufreq_governor *cpufreq_default_governor(void)
|
|
{
|
|
return &cpufreq_gov_powersave;
|
|
}
|
|
|
|
fs_initcall(cpufreq_gov_powersave_init);
|
|
#else
|
|
module_init(cpufreq_gov_powersave_init);
|
|
#endif
|
|
module_exit(cpufreq_gov_powersave_exit);
|