diff options
| author | Geert Uytterhoeven <geert+renesas@glider.be> | 2021-03-12 14:02:41 +0100 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2021-03-15 16:42:11 +0100 |
| commit | 9cbc23f7d51fb0b1363bdfdd0b770aa7b5982f2f (patch) | |
| tree | 9a818ac6f9bbf485f21d07d67eca7d4431ead44c /drivers/regulator/bd9571mwv-regulator.c | |
| parent | regulator: bd9571mwv: Fix AVS and DVFS voltage range (diff) | |
| download | linux-9cbc23f7d51fb0b1363bdfdd0b770aa7b5982f2f.tar.xz linux-9cbc23f7d51fb0b1363bdfdd0b770aa7b5982f2f.zip | |
regulator: bd9571mwv: Fix regulator name printed on registration failure
If a regulator fails to register, the driver prints an error message
like:
bd9571mwv-regulator bd9571mwv-regulator.6.auto: failed to register bd9571mwv-regulator regulator
However, the platform device's name is already printed as part of
dev_err(), and does not allow the user to distinguish among the various
regulators that are part of the PMIC.
Fix this by printing regulator_desc.name instead, to change the message
like:
bd9571mwv-regulator bd9571mwv-regulator.6.auto: failed to register DVFS regulator
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
Link: https://lore.kernel.org/r/20210312130242.3390038-3-geert+renesas@glider.be
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/regulator/bd9571mwv-regulator.c')
| -rw-r--r-- | drivers/regulator/bd9571mwv-regulator.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/regulator/bd9571mwv-regulator.c b/drivers/regulator/bd9571mwv-regulator.c index 13393cdc6be1..a4d406022587 100644 --- a/drivers/regulator/bd9571mwv-regulator.c +++ b/drivers/regulator/bd9571mwv-regulator.c @@ -301,7 +301,7 @@ static int bd9571mwv_regulator_probe(struct platform_device *pdev) &config); if (IS_ERR(rdev)) { dev_err(&pdev->dev, "failed to register %s regulator\n", - pdev->name); + regulators[i].name); return PTR_ERR(rdev); } } |