OSDN Git Service

mfd: stmfx: Simplify with dev_err_probe()
authorKrzysztof Kozlowski <krzk@kernel.org>
Wed, 26 Aug 2020 14:49:34 +0000 (16:49 +0200)
committerLee Jones <lee.jones@linaro.org>
Wed, 30 Sep 2020 15:24:59 +0000 (16:24 +0100)
Common pattern of handling deferred probe can be simplified with
dev_err_probe().  Less code and also it prints the error value.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
drivers/mfd/stmfx.c

index 711979a..5e680bf 100644 (file)
@@ -331,11 +331,9 @@ static int stmfx_chip_init(struct i2c_client *client)
        ret = PTR_ERR_OR_ZERO(stmfx->vdd);
        if (ret == -ENODEV) {
                stmfx->vdd = NULL;
-       } else if (ret == -EPROBE_DEFER) {
-               return ret;
-       } else if (ret) {
-               dev_err(&client->dev, "Failed to get VDD regulator: %d\n", ret);
-               return ret;
+       } else {
+               return dev_err_probe(&client->dev, ret,
+                                    "Failed to get VDD regulator\n");
        }
 
        if (stmfx->vdd) {