diff options
| author | Liao Chang <liaochang1@huawei.com> | 2023-08-16 01:58:53 +0000 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2023-08-22 22:10:10 +0200 |
| commit | a436ae9434ec491eefb4ed4c77bdb9c762925976 (patch) | |
| tree | 9a910ed2e666ff99c207deea55f91456449cf32b | |
| parent | d51847acb018d83186e4af67bc93f9a00a8644f7 (diff) | |
| download | linux-stable-a436ae9434ec491eefb4ed4c77bdb9c762925976.tar.gz linux-stable-a436ae9434ec491eefb4ed4c77bdb9c762925976.tar.bz2 linux-stable-a436ae9434ec491eefb4ed4c77bdb9c762925976.zip | |
cpufreq: Use clamp() helper macro to improve the code readability
The valid values of policy.{min, max} should be between 'min' and 'max',
so use clamp() helper macro to makes cpufreq_verify_within_limits() easier
to follow.
Signed-off-by: Liao Chang <liaochang1@huawei.com>
[ rjw: Subject and changelog edits ]
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
| -rw-r--r-- | include/linux/cpufreq.h | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h index 172ff51c1b2a..9bf94ae08158 100644 --- a/include/linux/cpufreq.h +++ b/include/linux/cpufreq.h @@ -19,6 +19,7 @@ #include <linux/pm_qos.h> #include <linux/spinlock.h> #include <linux/sysfs.h> +#include <linux/minmax.h> /********************************************************************* * CPUFREQ INTERFACE * @@ -467,17 +468,8 @@ static inline void cpufreq_verify_within_limits(struct cpufreq_policy_data *poli unsigned int min, unsigned int max) { - if (policy->min < min) - policy->min = min; - if (policy->max < min) - policy->max = min; - if (policy->min > max) - policy->min = max; - if (policy->max > max) - policy->max = max; - if (policy->min > policy->max) - policy->min = policy->max; - return; + policy->max = clamp(policy->max, min, max); + policy->min = clamp(policy->min, min, policy->max); } static inline void |