[-next] clk: ingenic: Remove set but not used variable 'enable'

Commit Message

Yue Haibing Feb. 26, 2019, 1:57 a.m. UTC

Fixes gcc '-Wunused-but-set-variable' warning:

drivers/clk/ingenic/cgu.c: In function 'ingenic_pll_recalc_rate':
drivers/clk/ingenic/cgu.c:86:15: warning:
 variable 'enable' set but not used [-Wunused-but-set-variable]

It's not used after commit ab27eb4bc365 ("clk: ingenic: Add code to
enable/disable PLLs")

Signed-off-by: YueHaibing <[e-mail]>
---
 drivers/clk/ingenic/cgu.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Stephen Boyd Feb. 26, 2019, 5:41 p.m. UTC | #1

Quoting YueHaibing (2019-02-25 17:57:32)
> Fixes gcc '-Wunused-but-set-variable' warning:
> 
> drivers/clk/ingenic/cgu.c: In function 'ingenic_pll_recalc_rate':
> drivers/clk/ingenic/cgu.c:86:15: warning:
>  variable 'enable' set but not used [-Wunused-but-set-variable]
> 
> It's not used after commit ab27eb4bc365 ("clk: ingenic: Add code to
> enable/disable PLLs")
> 
> Signed-off-by: YueHaibing <[e-mail]>
> ---

Applied to clk-next

Patch

diff --git a/drivers/clk/ingenic/cgu.c b/drivers/clk/ingenic/cgu.c
index b40160eb3372..510b685212d3 100644
--- a/drivers/clk/ingenic/cgu.c
+++ b/drivers/clk/ingenic/cgu.c
@@ -83,7 +83,7 @@  ingenic_pll_recalc_rate(struct clk_hw *hw, unsigned long parent_rate)
 	const struct ingenic_cgu_clk_info *clk_info;
 	const struct ingenic_cgu_pll_info *pll_info;
 	unsigned m, n, od_enc, od;
-	bool bypass, enable;
+	bool bypass;
 	unsigned long flags;
 	u32 ctl;
 
@@ -103,7 +103,6 @@  ingenic_pll_recalc_rate(struct clk_hw *hw, unsigned long parent_rate)
 	od_enc &= GENMASK(pll_info->od_bits - 1, 0);
 	bypass = !pll_info->no_bypass_bit &&
 		 !!(ctl & BIT(pll_info->bypass_bit));
-	enable = !!(ctl & BIT(pll_info->enable_bit));
 
 	if (bypass)
 		return parent_rate;