[PATCH V2] video: remove set but not used variable 'mach64RefFreq'
Bartlomiej Zolnierkiewicz
b.zolnierkie at samsung.com
Mon Mar 2 15:42:38 UTC 2020
On 1/20/20 7:33 AM, yu kuai wrote:
> Fixes gcc '-Wunused-but-set-variable' warning:
>
> drivers/video/fbdev/aty/mach64_gx.c: In function ‘aty_var_to_pll_8398’:
> drivers/video/fbdev/aty/mach64_gx.c:621:36: warning: variable
> ‘mach64RefFreq’ set but not used [-Wunused-but-set-variable]
>
> It is never used, and so can be removed.
>
> Signed-off-by: yu kuai <yukuai3 at huawei.com>
Patch queued for v5.7, thanks.
Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
> ---
> changes in V2:
> Fix the mistake that definition of 'mach64MinFreq' and 'mach64MaxFreq'
> was removed.
>
> drivers/video/fbdev/aty/mach64_gx.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/video/fbdev/aty/mach64_gx.c b/drivers/video/fbdev/aty/mach64_gx.c
> index 27cb65fa2ba2..9c37e28fb78b 100644
> --- a/drivers/video/fbdev/aty/mach64_gx.c
> +++ b/drivers/video/fbdev/aty/mach64_gx.c
> @@ -618,14 +618,13 @@ static int aty_var_to_pll_8398(const struct fb_info *info, u32 vclk_per,
> u32 mhz100; /* in 0.01 MHz */
> u32 program_bits;
> /* u32 post_divider; */
> - u32 mach64MinFreq, mach64MaxFreq, mach64RefFreq;
> + u32 mach64MinFreq, mach64MaxFreq;
> u16 m, n, k = 0, save_m, save_n, twoToKth;
>
> /* Calculate the programming word */
> mhz100 = 100000000 / vclk_per;
> mach64MinFreq = MIN_FREQ_2595;
> mach64MaxFreq = MAX_FREQ_2595;
> - mach64RefFreq = REF_FREQ_2595; /* 14.32 MHz */
>
> save_m = 0;
> save_n = 0;
>
More information about the dri-devel
mailing list