diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2021-02-16 18:50:44 +0300 |
---|---|---|
committer | Pavel Machek <pavel@ucw.cz> | 2021-02-20 00:15:31 +0300 |
commit | 9a10def9ceb5fa341d96a8b731dc2dc492e48d11 (patch) | |
tree | 1803d6ae07fb5e91e9373d302e510a0150d0e054 /drivers/leds/leds-lp50xx.c | |
parent | 1b40faf7e4abe10db2f730cf66b2b47551110940 (diff) | |
download | linux-9a10def9ceb5fa341d96a8b731dc2dc492e48d11.tar.xz |
leds: lp50xx: Don't spam logs when probe is deferred
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().
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Pavel Machek <pavel@ucw.cz>
Diffstat (limited to 'drivers/leds/leds-lp50xx.c')
-rw-r--r-- | drivers/leds/leds-lp50xx.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/leds/leds-lp50xx.c b/drivers/leds/leds-lp50xx.c index f13117eed976..a2d18ec8fd2b 100644 --- a/drivers/leds/leds-lp50xx.c +++ b/drivers/leds/leds-lp50xx.c @@ -455,12 +455,9 @@ static int lp50xx_probe_dt(struct lp50xx *priv) int i = 0; priv->enable_gpio = devm_gpiod_get_optional(priv->dev, "enable", GPIOD_OUT_LOW); - if (IS_ERR(priv->enable_gpio)) { - ret = PTR_ERR(priv->enable_gpio); - dev_err(&priv->client->dev, "Failed to get enable gpio: %d\n", - ret); - return ret; - } + if (IS_ERR(priv->enable_gpio)) + return dev_err_probe(priv->dev, PTR_ERR(priv->enable_gpio), + "Failed to get enable GPIO\n"); priv->regulator = devm_regulator_get(priv->dev, "vled"); if (IS_ERR(priv->regulator)) |