diff options
| author | Douglas Anderson <dianders@chromium.org> | 2022-08-10 00:27:45 +0300 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2022-08-10 16:52:07 +0300 |
| commit | d511e8a7e850db567cd7f633288aa96a19508e5b (patch) | |
| tree | df07f513f37dd49bdee276c22075744fb07d79bd /drivers/regulator | |
| parent | b0de7fa706506bf0591037908376351beda8c5d6 (diff) | |
| download | linux-d511e8a7e850db567cd7f633288aa96a19508e5b.tar.xz | |
regulator: core: Fix missing error return from regulator_bulk_get()
In commit 6eabfc018e8d ("regulator: core: Allow specifying an initial
load w/ the bulk API") I changed the error handling but had a subtle
that caused us to always return no error even if there was an
error. Fix it.
Fixes: 6eabfc018e8d ("regulator: core: Allow specifying an initial load w/ the bulk API")
Signed-off-by: Douglas Anderson <dianders@chromium.org>
Link: https://lore.kernel.org/r/20220809142738.1.I91625242f137c707bb345c51c80c5ecee02eeff3@changeid
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/regulator')
| -rw-r--r-- | drivers/regulator/core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c index 7150b1d0159e..d8373cb04f90 100644 --- a/drivers/regulator/core.c +++ b/drivers/regulator/core.c @@ -4784,10 +4784,10 @@ int regulator_bulk_get(struct device *dev, int num_consumers, consumers[i].consumer = regulator_get(dev, consumers[i].supply); if (IS_ERR(consumers[i].consumer)) { - consumers[i].consumer = NULL; ret = dev_err_probe(dev, PTR_ERR(consumers[i].consumer), "Failed to get supply '%s'", consumers[i].supply); + consumers[i].consumer = NULL; goto err; } |