From: Geert Uytterhoeven <geert+renesas@glider.be>
Date: Mon, 21 Feb 2022 16:28:16 +0000 (+0100)
Subject: pwm: rcar: Simplify multiplication/shift logic
X-Git-Tag: v6.1-rc5~1664^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ed14d36498c8d15be098df4af9ca324f96e9de74;p=platform%2Fkernel%2Flinux-starfive.git

pwm: rcar: Simplify multiplication/shift logic

- Remove the superfluous cast; the multiplication will yield a 64-bit
    result due to the "100ULL" anyway,
  - "a * (1 << b)" == "a << b".

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
---

diff --git a/drivers/pwm/pwm-rcar.c b/drivers/pwm/pwm-rcar.c
index b437192..55f46d0 100644
--- a/drivers/pwm/pwm-rcar.c
+++ b/drivers/pwm/pwm-rcar.c
@@ -110,7 +110,7 @@ static int rcar_pwm_set_counter(struct rcar_pwm_chip *rp, int div, int duty_ns,
 	unsigned long clk_rate = clk_get_rate(rp->clk);
 	u32 cyc, ph;
 
-	one_cycle = (unsigned long long)NSEC_PER_SEC * 100ULL * (1 << div);
+	one_cycle = NSEC_PER_SEC * 100ULL << div;
 	do_div(one_cycle, clk_rate);
 
 	tmp = period_ns * 100ULL;