Commit 6d8d840b authored by Rishi Gupta's avatar Rishi Gupta Committed by Mark Brown

regulator: da9063: remove redundant return statement

The devm_request_threaded_irq() already returns 0 on success
and negative error code on failure. So return from this itself
can be used while preserving error log in case of failure.
Signed-off-by: default avatarRishi Gupta <gupt21@gmail.com>
Link: https://lore.kernel.org/r/1580996996-28798-1-git-send-email-gupt21@gmail.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent a33b25f5
...@@ -877,12 +877,10 @@ static int da9063_regulator_probe(struct platform_device *pdev) ...@@ -877,12 +877,10 @@ static int da9063_regulator_probe(struct platform_device *pdev)
NULL, da9063_ldo_lim_event, NULL, da9063_ldo_lim_event,
IRQF_TRIGGER_LOW | IRQF_ONESHOT, IRQF_TRIGGER_LOW | IRQF_ONESHOT,
"LDO_LIM", regulators); "LDO_LIM", regulators);
if (ret) { if (ret)
dev_err(&pdev->dev, "Failed to request LDO_LIM IRQ.\n"); dev_err(&pdev->dev, "Failed to request LDO_LIM IRQ.\n");
return ret;
}
return 0; return ret;
} }
static struct platform_driver da9063_regulator_driver = { static struct platform_driver da9063_regulator_driver = {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment