Re: [PATCH v2 2/2] drm/ofdrm: Cast error pointers to void __iomem *

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Thomas,

On 10/28/22 14:22, Thomas Zimmermann wrote:
> Cast error pointers when returning them as void __iomem *. Fixes
> a number of Sparse warnings, such as the ones shown below.
> 
> ../drivers/gpu/drm/tiny/ofdrm.c:439:31: warning: incorrect type in return expression (different address spaces)
> ../drivers/gpu/drm/tiny/ofdrm.c:439:31:    expected void [noderef] __iomem *
> ../drivers/gpu/drm/tiny/ofdrm.c:439:31:    got void *
> ../drivers/gpu/drm/tiny/ofdrm.c:442:31: warning: incorrect type in return expression (different address spaces)
> ../drivers/gpu/drm/tiny/ofdrm.c:442:31:    expected void [noderef] __iomem *
> ../drivers/gpu/drm/tiny/ofdrm.c:442:31:    got void *
> 
> See [1] for the bug report.
> 
> Reported-by: kernel test robot <lkp@xxxxxxxxx>
> Signed-off-by: Thomas Zimmermann <tzimmermann@xxxxxxx>
> Link: https://lore.kernel.org/dri-devel/202210200016.yiQzPIy0-lkp@xxxxxxxxx/ # [1]
> ---
>  drivers/gpu/drm/tiny/ofdrm.c | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/gpu/drm/tiny/ofdrm.c b/drivers/gpu/drm/tiny/ofdrm.c
> index 44f13a2b372be..f1c301820d54b 100644
> --- a/drivers/gpu/drm/tiny/ofdrm.c
> +++ b/drivers/gpu/drm/tiny/ofdrm.c
> @@ -438,21 +438,21 @@ static void __iomem *get_cmap_address_of(struct ofdrm_device *odev, struct devic
>  	if (!addr_p)
>  		addr_p = of_get_address(of_node, bar_no, &max_size, &flags);
>  	if (!addr_p)
> -		return ERR_PTR(-ENODEV);
> +		return (void __iomem *)ERR_PTR(-ENODEV);
>  

There's an IOMEM_ERR_PTR() macro already for these cases. If you use
that instead, feel free to add my r-b when posting v3.

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux