diff mbox series

regulator: vexpress: Use PTR_ERR_OR_ZERO() in vexpress_regulator_probe()

Message ID 1123a2ab-48f9-f41f-cabb-9b45310cb77e@web.de (mailing list archive)
State New, archived
Headers show
Series regulator: vexpress: Use PTR_ERR_OR_ZERO() in vexpress_regulator_probe() | expand

Commit Message

Markus Elfring Sept. 7, 2019, 11:23 a.m. UTC 
From: Markus Elfring <elfring@users.sourceforge.net>
Date: Sat, 7 Sep 2019 13:07:22 +0200

Simplify this function implementation by using a known function.

Generated by: scripts/coccinelle/api/ptr_ret.cocci

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
---
 drivers/regulator/vexpress-regulator.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

--
2.23.0

Comments

Sudeep Holla Sept. 18, 2019, 10:44 a.m. UTC | #1 
On Sat, Sep 07, 2019 at 01:23:08PM +0200, Markus Elfring wrote:
> From: Markus Elfring <elfring@users.sourceforge.net>
> Date: Sat, 7 Sep 2019 13:07:22 +0200
>
> Simplify this function implementation by using a known function.
>
> Generated by: scripts/coccinelle/api/ptr_ret.cocci
>

Acked-by: Sudeep Holla <sudeep.holla@arm.comi>

Hi Mark,

I assume you can take this via your tree.

--
Regards,
Sudeep
diff mbox series

Patch

diff --git a/drivers/regulator/vexpress-regulator.c b/drivers/regulator/vexpress-regulator.c
index 1235f46e633e..5d39663efcaa 100644
--- a/drivers/regulator/vexpress-regulator.c
+++ b/drivers/regulator/vexpress-regulator.c
@@ -75,10 +75,7 @@  static int vexpress_regulator_probe(struct platform_device *pdev)
 	config.of_node = pdev->dev.of_node;

 	rdev = devm_regulator_register(&pdev->dev, desc, &config);
-	if (IS_ERR(rdev))
-		return PTR_ERR(rdev);
-
-	return 0;
+	return PTR_ERR_OR_ZERO(rdev);
 }

 static const struct of_device_id vexpress_regulator_of_match[] = {