| Message ID | 20190601074338.64187-1-yuehaibing@huawei.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [-next] cpufreq: armada-37xx: Remove set but not used variable 'freq' | expand |
On 01-06-19, 07:43, YueHaibing wrote: > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/cpufreq/armada-37xx-cpufreq.c: In function 'armada37xx_cpufreq_avs_setup': > drivers/cpufreq/armada-37xx-cpufreq.c:260:28: warning: > variable 'freq' set but not used [-Wunused-but-set-variable] > > It's never used since introduction in commit 1c3528232f4b ("cpufreq: > armada-37xx: Add AVS support") > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > --- > drivers/cpufreq/armada-37xx-cpufreq.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/cpufreq/armada-37xx-cpufreq.c b/drivers/cpufreq/armada-37xx-cpufreq.c > index 0df16eb1eb3c..aa0f06dec959 100644 > --- a/drivers/cpufreq/armada-37xx-cpufreq.c > +++ b/drivers/cpufreq/armada-37xx-cpufreq.c > @@ -257,7 +257,7 @@ static void __init armada37xx_cpufreq_avs_configure(struct regmap *base, > static void __init armada37xx_cpufreq_avs_setup(struct regmap *base, > struct armada_37xx_dvfs *dvfs) > { > - unsigned int avs_val = 0, freq; > + unsigned int avs_val = 0; > int load_level = 0; > > if (base == NULL) > @@ -275,8 +275,6 @@ static void __init armada37xx_cpufreq_avs_setup(struct regmap *base, > > > for (load_level = 1; load_level < LOAD_LEVEL_NR; load_level++) { > - freq = dvfs->cpu_freq_max / dvfs->divider[load_level]; > - > avs_val = dvfs->avs[load_level]; > regmap_update_bits(base, ARMADA_37XX_AVS_VSET(load_level-1), > ARMADA_37XX_AVS_VDD_MASK << ARMADA_37XX_AVS_HIGH_VDD_LIMIT | > Applied. Thanks.
diff --git a/drivers/cpufreq/armada-37xx-cpufreq.c b/drivers/cpufreq/armada-37xx-cpufreq.c index 0df16eb1eb3c..aa0f06dec959 100644 --- a/drivers/cpufreq/armada-37xx-cpufreq.c +++ b/drivers/cpufreq/armada-37xx-cpufreq.c @@ -257,7 +257,7 @@ static void __init armada37xx_cpufreq_avs_configure(struct regmap *base, static void __init armada37xx_cpufreq_avs_setup(struct regmap *base, struct armada_37xx_dvfs *dvfs) { - unsigned int avs_val = 0, freq; + unsigned int avs_val = 0; int load_level = 0; if (base == NULL) @@ -275,8 +275,6 @@ static void __init armada37xx_cpufreq_avs_setup(struct regmap *base, for (load_level = 1; load_level < LOAD_LEVEL_NR; load_level++) { - freq = dvfs->cpu_freq_max / dvfs->divider[load_level]; - avs_val = dvfs->avs[load_level]; regmap_update_bits(base, ARMADA_37XX_AVS_VSET(load_level-1), ARMADA_37XX_AVS_VDD_MASK << ARMADA_37XX_AVS_HIGH_VDD_LIMIT |
Fixes gcc '-Wunused-but-set-variable' warning: drivers/cpufreq/armada-37xx-cpufreq.c: In function 'armada37xx_cpufreq_avs_setup': drivers/cpufreq/armada-37xx-cpufreq.c:260:28: warning: variable 'freq' set but not used [-Wunused-but-set-variable] It's never used since introduction in commit 1c3528232f4b ("cpufreq: armada-37xx: Add AVS support") Signed-off-by: YueHaibing <yuehaibing@huawei.com> --- drivers/cpufreq/armada-37xx-cpufreq.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)