mmc: sh_mmcif: Simplify division/shift logic
authorGeert Uytterhoeven <geert+renesas@glider.be>
Mon, 21 Feb 2022 16:27:20 +0000 (17:27 +0100)
committerUlf Hansson <ulf.hansson@linaro.org>
Mon, 28 Feb 2022 15:11:47 +0000 (16:11 +0100)
"a / (1 << b)" == "a >> b".

No change in generated code.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Link: https://lore.kernel.org/r/68d689c39c769d298b53ee8cb9de0e594a2999b2.1645460780.git.geert+renesas@glider.be
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/sh_mmcif.c

index 104dcd702870c423b9b5f2a7fb1c20e186adb580..5f9ebf045b1cb22b59c142105e2d04600deb8160 100644 (file)
@@ -521,8 +521,7 @@ static void sh_mmcif_clock_control(struct sh_mmcif_host *host, unsigned int clk)
                }
 
                dev_dbg(dev, "clk %u/%u (%u, 0x%x)\n",
-                       (best_freq / (1 << (clkdiv + 1))), clk,
-                       best_freq, clkdiv);
+                       (best_freq >> (clkdiv + 1)), clk, best_freq, clkdiv);
 
                clk_set_rate(host->clk, best_freq);
                clkdiv = clkdiv << 16;
@@ -1012,8 +1011,8 @@ static void sh_mmcif_clk_setup(struct sh_mmcif_host *host)
                 */
                host->clkdiv_map = 0x3ff;
 
-               host->mmc->f_max = f_max / (1 << ffs(host->clkdiv_map));
-               host->mmc->f_min = f_min / (1 << fls(host->clkdiv_map));
+               host->mmc->f_max = f_max >> ffs(host->clkdiv_map);
+               host->mmc->f_min = f_min >> fls(host->clkdiv_map);
        } else {
                unsigned int clk = clk_get_rate(host->clk);