diff mbox series

clk: mxs: Use clamp() in clk_ref_round_rate() and clk_ref_set_rate()

Message ID 20240710143309.706135-2-thorsten.blum@toblux.com (mailing list archive)
State Accepted, archived
Headers show
Series clk: mxs: Use clamp() in clk_ref_round_rate() and clk_ref_set_rate() | expand

Commit Message

Thorsten Blum July 10, 2024, 2:33 p.m. UTC 
Use clamp() instead of duplicating its implementation.

Signed-off-by: Thorsten Blum <thorsten.blum@toblux.com>
---
 drivers/clk/mxs/clk-ref.c | 14 ++------------
 1 file changed, 2 insertions(+), 12 deletions(-)

Comments

Stephen Boyd July 10, 2024, 8:10 p.m. UTC | #1 
Quoting Thorsten Blum (2024-07-10 07:33:10)
> Use clamp() instead of duplicating its implementation.
> 
> Signed-off-by: Thorsten Blum <thorsten.blum@toblux.com>
> ---

Applied to clk-next
diff mbox series

Patch

diff --git a/drivers/clk/mxs/clk-ref.c b/drivers/clk/mxs/clk-ref.c
index 3161a0b021ee..2297259da89a 100644
--- a/drivers/clk/mxs/clk-ref.c
+++ b/drivers/clk/mxs/clk-ref.c
@@ -66,12 +66,7 @@  static long clk_ref_round_rate(struct clk_hw *hw, unsigned long rate,
 
 	tmp = tmp * 18 + rate / 2;
 	do_div(tmp, rate);
-	frac = tmp;
-
-	if (frac < 18)
-		frac = 18;
-	else if (frac > 35)
-		frac = 35;
+	frac = clamp(tmp, 18, 35);
 
 	tmp = parent_rate;
 	tmp *= 18;
@@ -91,12 +86,7 @@  static int clk_ref_set_rate(struct clk_hw *hw, unsigned long rate,
 
 	tmp = tmp * 18 + rate / 2;
 	do_div(tmp, rate);
-	frac = tmp;
-
-	if (frac < 18)
-		frac = 18;
-	else if (frac > 35)
-		frac = 35;
+	frac = clamp(tmp, 18, 35);
 
 	spin_lock_irqsave(&mxs_lock, flags);