diff options
| author | Rishi Gupta <gupt21@gmail.com> | 2020-02-06 19:19:56 +0530 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2020-02-11 11:33:54 +0000 |
| commit | 6d8d840b214e12be6556ed7bee803d9280c54f3b (patch) | |
| tree | 6f94f110f2808e6f6d3afd0fb40772bf509ea6c4 /drivers/regulator/da9063-regulator.c | |
| parent | a33b25f57ddeb8545dab6ffee6be0f38d89d42c6 (diff) | |
| download | linux-6d8d840b214e12be6556ed7bee803d9280c54f3b.tar.bz2 | |
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: Rishi Gupta <gupt21@gmail.com>
Link: https://lore.kernel.org/r/1580996996-28798-1-git-send-email-gupt21@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/regulator/da9063-regulator.c')
| -rw-r--r-- | drivers/regulator/da9063-regulator.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/regulator/da9063-regulator.c b/drivers/regulator/da9063-regulator.c index ae54c76a8580..aaa994293e9b 100644 --- a/drivers/regulator/da9063-regulator.c +++ b/drivers/regulator/da9063-regulator.c @@ -877,12 +877,10 @@ static int da9063_regulator_probe(struct platform_device *pdev) NULL, da9063_ldo_lim_event, IRQF_TRIGGER_LOW | IRQF_ONESHOT, "LDO_LIM", regulators); - if (ret) { + if (ret) dev_err(&pdev->dev, "Failed to request LDO_LIM IRQ.\n"); - return ret; - } - return 0; + return ret; } static struct platform_driver da9063_regulator_driver = { |