* [PATCH] clk: bcm2835: Remove unused variable
@ 2022-02-22 14:07 Maxime Ripard
2022-02-22 17:31 ` Florian Fainelli
2022-03-12 3:22 ` Stephen Boyd
0 siblings, 2 replies; 3+ messages in thread
From: Maxime Ripard @ 2022-02-22 14:07 UTC (permalink / raw)
To: linux-clk, Mike Turquette, Stephen Boyd, Florian Fainelli,
Ray Jui, Scott Branden
Cc: bcm-kernel-feedback-list, linux-rpi-kernel, Maxime Ripard,
kernel test robot
Since commit 8ca011ef4af4 ("clk: bcm-2835: Remove rounding up the
dividers"), the rem variable is still set but no longer used. Remove it.
Fixes: 8ca011ef4af4 ("clk: bcm-2835: Remove rounding up the dividers")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
---
drivers/clk/bcm/clk-bcm2835.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/clk/bcm/clk-bcm2835.c b/drivers/clk/bcm/clk-bcm2835.c
index 3667b4d731e7..3ad20e75fd23 100644
--- a/drivers/clk/bcm/clk-bcm2835.c
+++ b/drivers/clk/bcm/clk-bcm2835.c
@@ -939,10 +939,8 @@ static u32 bcm2835_clock_choose_div(struct clk_hw *hw,
u32 unused_frac_mask =
GENMASK(CM_DIV_FRAC_BITS - data->frac_bits, 0) >> 1;
u64 temp = (u64)parent_rate << CM_DIV_FRAC_BITS;
- u64 rem;
u32 div, mindiv, maxdiv;
- rem = do_div(temp, rate);
div = temp;
div &= ~unused_frac_mask;
--
2.35.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] clk: bcm2835: Remove unused variable
2022-02-22 14:07 [PATCH] clk: bcm2835: Remove unused variable Maxime Ripard
@ 2022-02-22 17:31 ` Florian Fainelli
2022-03-12 3:22 ` Stephen Boyd
1 sibling, 0 replies; 3+ messages in thread
From: Florian Fainelli @ 2022-02-22 17:31 UTC (permalink / raw)
To: Maxime Ripard, linux-clk, Mike Turquette, Stephen Boyd,
Florian Fainelli, Ray Jui, Scott Branden
Cc: bcm-kernel-feedback-list, linux-rpi-kernel, kernel test robot
On 2/22/2022 6:07 AM, Maxime Ripard wrote:
> Since commit 8ca011ef4af4 ("clk: bcm-2835: Remove rounding up the
> dividers"), the rem variable is still set but no longer used. Remove it.
>
> Fixes: 8ca011ef4af4 ("clk: bcm-2835: Remove rounding up the dividers")
> Reported-by: kernel test robot <lkp@intel.com>
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
--
Florian
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] clk: bcm2835: Remove unused variable
2022-02-22 14:07 [PATCH] clk: bcm2835: Remove unused variable Maxime Ripard
2022-02-22 17:31 ` Florian Fainelli
@ 2022-03-12 3:22 ` Stephen Boyd
1 sibling, 0 replies; 3+ messages in thread
From: Stephen Boyd @ 2022-03-12 3:22 UTC (permalink / raw)
To: Florian Fainelli, Maxime Ripard, Mike Turquette, Ray Jui,
Scott Branden, linux-clk
Cc: bcm-kernel-feedback-list, linux-rpi-kernel, Maxime Ripard,
kernel test robot
Quoting Maxime Ripard (2022-02-22 06:07:32)
> Since commit 8ca011ef4af4 ("clk: bcm-2835: Remove rounding up the
> dividers"), the rem variable is still set but no longer used. Remove it.
>
> Fixes: 8ca011ef4af4 ("clk: bcm-2835: Remove rounding up the dividers")
> Reported-by: kernel test robot <lkp@intel.com>
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-03-12 3:22 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-22 14:07 [PATCH] clk: bcm2835: Remove unused variable Maxime Ripard
2022-02-22 17:31 ` Florian Fainelli
2022-03-12 3:22 ` Stephen Boyd
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.