[PATCH 1/1] drm/panfrost: Remove redundant error printing in panfrost_device_init()

Steven Price steven.price at arm.com
Wed May 12 15:30:39 UTC 2021


On 11/05/2021 10:04, Zhen Lei wrote:
> When devm_ioremap_resource() fails, a clear enough error message will be
> printed by its subfunction __devm_ioremap_resource(). The error
> information contains the device name, failure cause, and possibly resource
> information.
> 
> Therefore, remove the error printing here to simplify code and reduce the
> binary size.
> 
> Reported-by: Hulk Robot <hulkci at huawei.com>
> Signed-off-by: Zhen Lei <thunder.leizhen at huawei.com>

Reviewed-by: Steven Price <steven.price at arm.com>

I'll push to drm-misc-next.

Thanks,

Steve

> ---
>   drivers/gpu/drm/panfrost/panfrost_device.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/panfrost/panfrost_device.c b/drivers/gpu/drm/panfrost/panfrost_device.c
> index fbcf5edbe367521..125ed973feaad0a 100644
> --- a/drivers/gpu/drm/panfrost/panfrost_device.c
> +++ b/drivers/gpu/drm/panfrost/panfrost_device.c
> @@ -238,7 +238,6 @@ int panfrost_device_init(struct panfrost_device *pfdev)
>   	res = platform_get_resource(pfdev->pdev, IORESOURCE_MEM, 0);
>   	pfdev->iomem = devm_ioremap_resource(pfdev->dev, res);
>   	if (IS_ERR(pfdev->iomem)) {
> -		dev_err(pfdev->dev, "failed to ioremap iomem\n");
>   		err = PTR_ERR(pfdev->iomem);
>   		goto out_pm_domain;
>   	}
> 



More information about the dri-devel mailing list