leds: lm3697: Don't spam logs when probe is deferred
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Mon, 10 May 2021 09:50:39 +0000 (12:50 +0300)
committerPavel Machek <pavel@ucw.cz>
Fri, 28 May 2021 10:00:15 +0000 (12:00 +0200)
When requesting GPIO line the probe can be deferred.
In such case don't spam logs with an error message.
This can be achieved by switching to dev_err_probe().

Fixes: 5c1d824cda9f ("leds: lm3697: Introduce the lm3697 driver")
Cc: Dan Murphy <dmurphy@ti.com>
Signed-off-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Pavel Machek <pavel@ucw.cz>
drivers/leds/leds-lm3697.c

index 0d10d12..970a4f3 100644 (file)
@@ -207,11 +207,9 @@ static int lm3697_probe_dt(struct lm3697 *priv)
 
        priv->enable_gpio = devm_gpiod_get_optional(dev, "enable",
                                                    GPIOD_OUT_LOW);
-       if (IS_ERR(priv->enable_gpio)) {
-               ret = PTR_ERR(priv->enable_gpio);
-               dev_err(dev, "Failed to get enable gpio: %d\n", ret);
-               return ret;
-       }
+       if (IS_ERR(priv->enable_gpio))
+               return dev_err_probe(dev, PTR_ERR(priv->enable_gpio),
+                                         "Failed to get enable GPIO\n");
 
        priv->regulator = devm_regulator_get(dev, "vled");
        if (IS_ERR(priv->regulator))