diff options
| author |   Shree Ramamoorthy <s-ramamoorthy@ti.com> | 2024-12-17 14:45:23 -0600 |
|---|---|---|
| committer |   Mark Brown <broonie@kernel.org> | 2024-12-17 21:19:59 +0000 |
| commit | e7ce0437709552a5f5e577c29f3dfde620153913 (patch) | |
| tree | d9b6c5b1815853e31940feaa606c74b1f8767cb9 | |
| parent | regulator: dt-bindings: mt6315: Drop regulator-compatible property (diff) | |
| download | wireguard-linux-e7ce0437709552a5f5e577c29f3dfde620153913.tar.xz wireguard-linux-e7ce0437709552a5f5e577c29f3dfde620153913.zip | |
regulator: tps65219: Use dev_err_probe() instead of dev_err()
Make the error message format unified by switching from dev_err() to
dev_err_probe() where there is a chance of -EPROBE_DEFER returned. This
helps simplify the error code where possible.
Signed-off-by: Shree Ramamoorthy <s-ramamoorthy@ti.com>
Link: https://patch.msgid.link/20241217204526.1010989-2-s-ramamoorthy@ti.com
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | drivers/regulator/tps65219-regulator.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/regulator/tps65219-regulator.c b/drivers/regulator/tps65219-regulator.c index b4065356392f..14d843a99d0f 100644 --- a/drivers/regulator/tps65219-regulator.c +++ b/drivers/regulator/tps65219-regulator.c @@ -322,11 +322,11 @@ static int tps65219_regulator_probe(struct platform_device *pdev) dev_dbg(tps->dev, "%s regul i= %d START", __func__, i); rdev = devm_regulator_register(&pdev->dev, ®ulators[i], &config); - if (IS_ERR(rdev)) { - dev_err(tps->dev, "failed to register %s regulator\n", - regulators[i].name); - return PTR_ERR(rdev); - } + if (IS_ERR(rdev)) + return dev_err_probe(tps->dev, PTR_ERR(rdev), + "Failed to register %s regulator\n", + regulators[i].name); + rdevtbl[i] = rdev; dev_dbg(tps->dev, "%s regul i= %d COMPLETED", __func__, i); } |