On 03/04/2019 at 09:24, YueHaibing wrote:
> External E-Mail
> 
> 
> Fixes gcc '-Wunused-but-set-variable' warning:
> 
> drivers/video/fbdev/atmel_lcdfb.c: In function 'atmel_lcdfb_remove':
> drivers/video/fbdev/atmel_lcdfb.c:1255:28: warning:
>   variable 'pdata' set but not used [-Wunused-but-set-variable]
> 
> It's not used since commit 42110e91de7f ("video: atmel_lcdfb: introduce
> atmel_lcdfb_power_control")
> 
> Signed-off-by: YueHaibing <yuehaib...@huawei.com>

Indeed:
Acked-by: Nicolas Ferre <nicolas.fe...@microchip.com>

Thanks!

> ---
>   drivers/video/fbdev/atmel_lcdfb.c | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/video/fbdev/atmel_lcdfb.c 
> b/drivers/video/fbdev/atmel_lcdfb.c
> index b8786ff62b87..e67dfd94bf1d 100644
> --- a/drivers/video/fbdev/atmel_lcdfb.c
> +++ b/drivers/video/fbdev/atmel_lcdfb.c
> @@ -1252,12 +1252,10 @@ static int __exit atmel_lcdfb_remove(struct 
> platform_device *pdev)
>       struct device *dev = &pdev->dev;
>       struct fb_info *info = dev_get_drvdata(dev);
>       struct atmel_lcdfb_info *sinfo;
> -     struct atmel_lcdfb_pdata *pdata;
>   
>       if (!info || !info->par)
>               return 0;
>       sinfo = info->par;
> -     pdata = &sinfo->pdata;
>   
>       cancel_work_sync(&sinfo->task);
>       exit_backlight(sinfo);
> 
> 
> 
> 
> 


-- 
Nicolas Ferre
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to