diff options
author | Puneet Saxena <puneets@nvidia.com> | 2013-11-11 20:38:14 +0530 |
---|---|---|
committer | Bharat Nihalani <bnihalani@nvidia.com> | 2013-11-12 00:08:24 -0800 |
commit | 9859e528704cc31d5ad4642454f6e6aac4b51cc1 (patch) | |
tree | e7b3458962fa7cf23882876772b1c92bfd6eba3c /drivers/cpufreq | |
parent | 4d20a2cccade16bf9a40b423273ee2f657111a8d (diff) |
Cpufreq: interactive: remove duplicate min_sample_time
All of the global gov tunables are moved to "tunables" structure.
"DEFAULT_MIN_SAMPLE_TIME" is reinitialized wrongly, causing
"min_sample_time" set as very high value(80ms).
This causes cpu to take more time to ramp down freq.
It removes global "min_sample_time" and sets "min_sample_time"
as 30ms to improve ramping down period.
Bug 1402227
Change-Id: Ifae8db68c03a315474921028c39875bf1ab0b383
Signed-off-by: Puneet Saxena <puneets@nvidia.com>
Reviewed-on: http://git-master/r/328986
Reviewed-by: Prashant Gaikwad <pgaikwad@nvidia.com>
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>
Diffstat (limited to 'drivers/cpufreq')
-rw-r--r-- | drivers/cpufreq/cpufreq_interactive.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/drivers/cpufreq/cpufreq_interactive.c b/drivers/cpufreq/cpufreq_interactive.c index 9ec65e4fe0d0..f35d2dcdb3d3 100644 --- a/drivers/cpufreq/cpufreq_interactive.c +++ b/drivers/cpufreq/cpufreq_interactive.c @@ -66,13 +66,6 @@ static struct mutex gov_lock; static unsigned int default_target_loads[] = {DEFAULT_TARGET_LOAD}; /* - * The minimum amount of time to spend at a frequency before we can ramp down. - */ - -#define DEFAULT_MIN_SAMPLE_TIME (30 * USEC_PER_MSEC) -static unsigned long min_sample_time = DEFAULT_MIN_SAMPLE_TIME; - -/* * The sample rate of the timer used to increase frequency */ #define DEFAULT_TIMER_RATE (20 * USEC_PER_MSEC) @@ -100,7 +93,7 @@ struct cpufreq_interactive_tunables { * The minimum amount of time to spend at a frequency before we can ramp * down. */ -#define DEFAULT_MIN_SAMPLE_TIME (80 * USEC_PER_MSEC) +#define DEFAULT_MIN_SAMPLE_TIME (30 * USEC_PER_MSEC) unsigned long min_sample_time; /* |