diff options
| author |  Viresh Kumar <viresh.kumar@linaro.org>
| 2015-01-06 21:09:13 +0530 |
|---|---|---|
| committer |  Rafael J. Wysocki <rafael.j.wysocki@intel.com>
| 2015-01-23 23:06:45 +0100 |
| commit | 9225913d386537f84f0eb72f022a8b4b9e684846 (patch) | |
| tree | 3017453546dee5c94d4934688b5c6f2dd00fe3a3 /drivers/cpufreq/cpufreq_stats.c | |
| parent | cpufreq: stats: time_in_state can't be NULL in cpufreq_stats_update() (diff) | |
| download | linux-9225913d386537f84f0eb72f022a8b4b9e684846.tar.gz linux-9225913d386537f84f0eb72f022a8b4b9e684846.tar.bz2 linux-9225913d386537f84f0eb72f022a8b4b9e684846.zip | |
cpufreq: stats: don't update stats from show_trans_table()
cpufreq_stats_update() updates time_in_state and nothing else. It should ideally
be updated only in two cases:
- User requested for the current value of time_in_state.
- We have switched states and so need to update time for the last state.
Currently, we are also doing this while user asks for the transition table of
frequencies. It wouldn't do any harm, but no good as well. Its useless here.
Remove it.
Reviewed-by: Prarit Bhargava <prarit@redhat.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/cpufreq/cpufreq_stats.c')
| -rw-r--r-- | drivers/cpufreq/cpufreq_stats.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/cpufreq/cpufreq_stats.c b/drivers/cpufreq/cpufreq_stats.c index a9a6dc479adf..4cd261156fff 100644 --- a/drivers/cpufreq/cpufreq_stats.c +++ b/drivers/cpufreq/cpufreq_stats.c @@ -68,7 +68,6 @@ static ssize_t show_trans_table(struct cpufreq_policy *policy, char *buf) ssize_t len = 0; int i, j; - cpufreq_stats_update(stats); len += snprintf(buf + len, PAGE_SIZE - len, " From : To\n"); len += snprintf(buf + len, PAGE_SIZE - len, " : "); for (i = 0; i < stats->state_num; i++) { |