leds: lp50xx: Get rid of redundant check in lp50xx_enable_disable()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Tue, 16 Feb 2021 15:50:48 +0000 (17:50 +0200)
committerPavel Machek <pavel@ucw.cz>
Fri, 19 Feb 2021 21:15:43 +0000 (22:15 +0100)
Since GPIO is optional the API is NULL aware and will check descriptor anyway.
Remove duplicate redundant check in lp50xx_enable_disable().

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Pavel Machek <pavel@ucw.cz>
drivers/leds/leds-lp50xx.c

index 0723b26..ae82d4d 100644 (file)
@@ -382,11 +382,9 @@ static int lp50xx_enable_disable(struct lp50xx *priv, int enable_disable)
 {
        int ret;
 
-       if (priv->enable_gpio) {
-               ret = gpiod_direction_output(priv->enable_gpio, enable_disable);
-               if (ret)
-                       return ret;
-       }
+       ret = gpiod_direction_output(priv->enable_gpio, enable_disable);
+       if (ret)
+               return ret;
 
        if (enable_disable)
                return regmap_write(priv->regmap, LP50XX_DEV_CFG0, LP50XX_CHIP_EN);