diff options
| author |   Gautham R Shenoy <ego@linux.vnet.ibm.com> | 2016-01-27 12:02:26 +0530 |
|---|---|---|
| committer |   Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2016-01-27 23:13:59 +0100 |
| commit | 2dadfd7564ef28391609c7ef896fd85218799012 (patch) | |
| tree | 08dc2311bca1e793dc9fdb9c8d3a346348bf0e78 /drivers/cpufreq/cpufreq.c | |
| parent | cpufreq: Fix NULL reference crash while accessing policy->governor_data (diff) | |
| download | linux-dev-2dadfd7564ef28391609c7ef896fd85218799012.tar.xz linux-dev-2dadfd7564ef28391609c7ef896fd85218799012.zip | |
cpufreq: Use list_is_last() to check last entry of the policy list
Currently next_policy() explicitly checks if a policy is the last
policy in the cpufreq_policy_list. Use the standard list_is_last
primitive instead.
Signed-off-by: Gautham R. Shenoy <ego@linux.vnet.ibm.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/cpufreq/cpufreq.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index c35e7da1ed7a..e979ec78b695 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -48,11 +48,11 @@ static struct cpufreq_policy *next_policy(struct cpufreq_policy *policy, bool active) { do { - policy = list_next_entry(policy, policy_list); - /* No more policies in the list */ - if (&policy->policy_list == &cpufreq_policy_list) + if (list_is_last(&policy->policy_list, &cpufreq_policy_list)) return NULL; + + policy = list_next_entry(policy, policy_list); } while (!suitable_policy(policy, active)); return policy; |