cpufreq: stats: time_in_state can't be NULL in cpufreq_stats_update()
authorViresh Kumar <viresh.kumar@linaro.org>
Tue, 6 Jan 2015 15:39:12 +0000 (21:09 +0530)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Fri, 23 Jan 2015 22:06:44 +0000 (23:06 +0100)
'time_in_state' can't be NULL if 'stats' is valid. These are allocated together
and only if time_in_state is allocated successfully, we update policy->stats.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/cpufreq/cpufreq_stats.c

index 5d7bf9b1dfc5219a8ff4676a834e57efb37e0117..a9a6dc479adfac9f54ccfbf4c88c45c4c1f689b6 100644 (file)
@@ -35,9 +35,7 @@ static int cpufreq_stats_update(struct cpufreq_stats *stats)
        unsigned long long cur_time = get_jiffies_64();
 
        spin_lock(&cpufreq_stats_lock);
-       if (stats->time_in_state)
-               stats->time_in_state[stats->last_index] +=
-                       cur_time - stats->last_time;
+       stats->time_in_state[stats->last_index] += cur_time - stats->last_time;
        stats->last_time = cur_time;
        spin_unlock(&cpufreq_stats_lock);
        return 0;