mfd: madera: Simplify with dev_err_probe()
authorKrzysztof Kozlowski <krzk@kernel.org>
Wed, 26 Aug 2020 14:49:33 +0000 (16:49 +0200)
committerLee Jones <lee.jones@linaro.org>
Wed, 30 Sep 2020 15:24:57 +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>
Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
drivers/mfd/madera-core.c

index 8a8d733..4ed6ad8 100644 (file)
@@ -369,19 +369,14 @@ EXPORT_SYMBOL_GPL(madera_of_match);
 static int madera_get_reset_gpio(struct madera *madera)
 {
        struct gpio_desc *reset;
-       int ret;
 
        if (madera->pdata.reset)
                return 0;
 
        reset = devm_gpiod_get_optional(madera->dev, "reset", GPIOD_OUT_LOW);
-       if (IS_ERR(reset)) {
-               ret = PTR_ERR(reset);
-               if (ret != -EPROBE_DEFER)
-                       dev_err(madera->dev, "Failed to request /RESET: %d\n",
-                               ret);
-               return ret;
-       }
+       if (IS_ERR(reset))
+               return dev_err_probe(madera->dev, PTR_ERR(reset),
+                               "Failed to request /RESET");
 
        /*
         * A hard reset is needed for full reset of the chip. We allow running