diff mbox series

[1/1] pinctrl: ocelot: Remove redundant error printing in ocelot_pinctrl_probe()

Message ID 20210511090936.4452-1-thunder.leizhen@huawei.com
State Accepted
Commit 0f9facdbd9d432442e5655425e8702ed8e28f1b8
Headers show
Series [1/1] pinctrl: ocelot: Remove redundant error printing in ocelot_pinctrl_probe() | expand

Commit Message

Leizhen (ThunderTown) May 11, 2021, 9:09 a.m. UTC
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>
---
 drivers/pinctrl/pinctrl-ocelot.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

-- 
2.26.0.106.g9fadedd

Comments

Linus Walleij May 19, 2021, 11:34 p.m. UTC | #1
On Tue, May 11, 2021 at 11:09 AM Zhen Lei <thunder.leizhen@huawei.com> 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@huawei.com>

> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>


Patch applied!
Yours,
Linus Walleij
diff mbox series

Patch

diff --git a/drivers/pinctrl/pinctrl-ocelot.c b/drivers/pinctrl/pinctrl-ocelot.c
index 2fd18e356d0c6f2..e470c16718dea20 100644
--- a/drivers/pinctrl/pinctrl-ocelot.c
+++ b/drivers/pinctrl/pinctrl-ocelot.c
@@ -1362,10 +1362,8 @@  static int ocelot_pinctrl_probe(struct platform_device *pdev)
 
 	base = devm_ioremap_resource(dev,
 			platform_get_resource(pdev, IORESOURCE_MEM, 0));
-	if (IS_ERR(base)) {
-		dev_err(dev, "Failed to ioremap registers\n");
+	if (IS_ERR(base))
 		return PTR_ERR(base);
-	}
 
 	info->stride = 1 + (info->desc->npins - 1) / 32;