# Re: \[PATCH\] drivers: video: mxsfb: clean use of devm\_ioremap\_resource()

- From: Laurent Navet \<hidden\>
- Date: 2013-05-16 09:46:23 
- List: linux-fbdev
- Page: https://ratatoskr.run/linux-fbdev/2013/05/4050291
- Thread: https://ratatoskr.run/linux-fbdev/2013/05/4050271/t.md

```text
2013/5/16, Jingoo Han [ref]:
> It makes build warning as below:
>
> drivers/video/mxsfb.c:887:13: warning: 'host' is used uninitialized in this
> function [-Wuninitialized]
> drivers/video/mxsfb.c:965:21: warning: 'fb_info' may be used uninitialized
> in this function [-Wuninitialized]
>
> It breaks the assignment.
> 	host = to_imxfb_host(fb_info);
>
> Also, 'goto fb_release;' is not good.
> Please use ' return PTR_ERR(host->base);' as below:
> +       host->base = devm_ioremap_resource(&pdev->dev, res);
> +       if (IS_ERR(host->base))
> +               return PTR_ERR(host->base);
>
>
> Best regards,
> Jingoo Han

Thank's for reviewing, I'll look at and resend.

Laurent,
```