diff mbox series

leds: lp55xx: Initialize enable GPIO direction to output

Message ID 20210510204000.GA3714758@odessy
State New
Headers show
Series leds: lp55xx: Initialize enable GPIO direction to output | expand

Commit Message

Doug Zobel May 10, 2021, 8:40 p.m. UTC
The "Convert to use GPIO descriptors" commit changed the
initialization of the enable GPIO from GPIOF_DIR_OUT to
GPIOD_ASIS.  This breaks systems where the GPIO does not
default to output.  Changing the enable initialization
to GPIOD_OUT_LOW.

Signed-off-by: Doug Zobel <dougdev334@gmail.com>
---
 drivers/leds/leds-lp55xx-common.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Pavel Machek June 23, 2021, 8:10 p.m. UTC | #1
On Mon 2021-05-10 15:40:00, Doug Zobel wrote:
> The "Convert to use GPIO descriptors" commit changed the

> initialization of the enable GPIO from GPIOF_DIR_OUT to

> GPIOD_ASIS.  This breaks systems where the GPIO does not

> default to output.  Changing the enable initialization

> to GPIOD_OUT_LOW.


Thanks, applied.
							Pavel

-- 
http://www.livejournal.com/~pavelmachek
diff mbox series

Patch

diff --git a/drivers/leds/leds-lp55xx-common.c b/drivers/leds/leds-lp55xx-common.c
index 81de1346bf5d..d1657c46ee2f 100644
--- a/drivers/leds/leds-lp55xx-common.c
+++ b/drivers/leds/leds-lp55xx-common.c
@@ -694,7 +694,7 @@  struct lp55xx_platform_data *lp55xx_of_populate_pdata(struct device *dev,
 	of_property_read_u8(np, "clock-mode", &pdata->clock_mode);
 
 	pdata->enable_gpiod = devm_gpiod_get_optional(dev, "enable",
-						      GPIOD_ASIS);
+						      GPIOD_OUT_LOW);
 	if (IS_ERR(pdata->enable_gpiod))
 		return ERR_CAST(pdata->enable_gpiod);