Cpufreq: interactive: remove duplicate min_sample_time
Puneet Saxena [Mon, 11 Nov 2013 15:08:14 +0000 (20:08 +0530)]
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>

drivers/cpufreq/cpufreq_interactive.c

index 9ec65e4..f35d2dc 100644 (file)
@@ -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;
 
        /*