Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 34b08705 authored by Rafael J. Wysocki's avatar Rafael J. Wysocki
Browse files

cpufreq: Simplify the cpufreq_for_each_valid_entry() 



That macro uses an internal static inline function that is first
totally unnecessary and second hard to read, so simplify it and
get rid of that monster.

No functional changes.

Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
Acked-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
parent 9a909a14

include/linux/cpufreq.h

+5 −12
Original line number Diff line number Diff line
@@ -504,16 +504,6 @@ static inline void dev_pm_opp_free_cpufreq_table(struct device *dev, 
}

#endif



static inline bool cpufreq_next_valid(struct cpufreq_frequency_table **pos)

{

	while ((*pos)->frequency != CPUFREQ_TABLE_END)

		if ((*pos)->frequency != CPUFREQ_ENTRY_INVALID)

			return true;

		else

			(*pos)++;

	return false;

}



/*

 * cpufreq_for_each_entry -	iterate over a cpufreq_frequency_table

 * @pos:	the cpufreq_frequency_table * to use as a loop cursor.
@@ -531,7 +521,10 @@ static inline bool cpufreq_next_valid(struct cpufreq_frequency_table **pos) 
 */



#define cpufreq_for_each_valid_entry(pos, table)			\

	for (pos = table; cpufreq_next_valid(&pos); pos++)

	for (pos = table; pos->frequency != CPUFREQ_TABLE_END; pos++)	\

		if (pos->frequency == CPUFREQ_ENTRY_INVALID)		\

			continue;					\

		else



int cpufreq_frequency_table_cpuinfo(struct cpufreq_policy *policy,

				    struct cpufreq_frequency_table *table);