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 <[email protected]>

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;
> 
_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to