Ver código fonte

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>
Gautham R Shenoy 9 anos atrás
pai
commit
2dadfd7564
1 arquivos alterados com 3 adições e 3 exclusões
  1. 3 3
      drivers/cpufreq/cpufreq.c

+ 3 - 3
drivers/cpufreq/cpufreq.c

@@ -48,11 +48,11 @@ static struct cpufreq_policy *next_policy(struct cpufreq_policy *policy,
 					  bool active)
 					  bool active)
 {
 {
 	do {
 	do {
-		policy = list_next_entry(policy, policy_list);
-
 		/* No more policies in the 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;
 			return NULL;
+
+		policy = list_next_entry(policy, policy_list);
 	} while (!suitable_policy(policy, active));
 	} while (!suitable_policy(policy, active));
 
 
 	return policy;
 	return policy;