iio: ep93xx: Remove redundant error printing in ep93xx_adc_probe()
authorZhen Lei <thunder.leizhen@huawei.com>
Tue, 11 May 2021 11:39:12 +0000 (19:39 +0800)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Mon, 17 May 2021 12:54:24 +0000 (13:54 +0100)
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@huawei.com>
Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/adc/ep93xx_adc.c

index c08ab3c..a10a4e8 100644 (file)
@@ -165,10 +165,8 @@ static int ep93xx_adc_probe(struct platform_device *pdev)
 
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        priv->base = devm_ioremap_resource(&pdev->dev, res);
-       if (IS_ERR(priv->base)) {
-               dev_err(&pdev->dev, "Cannot map memory resource\n");
+       if (IS_ERR(priv->base))
                return PTR_ERR(priv->base);
-       }
 
        iiodev->name = dev_name(&pdev->dev);
        iiodev->modes = INDIO_DIRECT_MODE;