Commit a9b26931 authored Feb 25, 2022 by Maxime Ripard Committed by Stephen Boyd Mar 11, 2022

clk: Use clamp instead of open-coding our own



The code in clk_set_rate_range() will, if the current rate is outside of
the new range, force it to the minimum or maximum.

Since it's running under the condition that the rate is either lower
than the minimum, or higher than the maximum, this is equivalent to
using clamp, while being less readable. Let's switch to using clamp
instead.

Signed-off-by: Maxime Ripard <maxime@cerno.tech>
Link: https://lore.kernel.org/r/20220225143534.405820-6-maxime@cerno.tech


Signed-off-by: Stephen Boyd <sboyd@kernel.org>

parent 948fb096

drivers/clk/clk.c

+1 −5

Original line number	Diff line number	Diff line
		@@ -2388,11 +2388,7 @@ int clk_set_rate_range(struct clk *clk, unsigned long min, unsigned long max)
		* this corner case when determining the rate
		*/

		if (rate < min)
		rate = min;
		else
		rate = max;

		rate = clamp(clk->core->req_rate, min, max);
		ret = clk_core_set_rate_nolock(clk->core, rate);
		if (ret) {
		/* rollback the changes */