Commit 5d80a86a authored by ChiYuan Huang's avatar ChiYuan Huang Committed by Sebastian Reichel
Browse files

power: supply: rt9467: Make charger-enable control as logic level 



The current coding make 'charger-enable-gpio' control as real hardware
level. This conflicts with the default binding example. For driver
behavior, no need to use real hardware level, just logic level is
enough. This change can make this flexibility keep in dts gpio active
level about this pin.

Fixes: 6f7f70e3 ("power: supply: rt9467: Add Richtek RT9467 charger driver")
Signed-off-by: default avatarChiYuan Huang <cy_huang@richtek.com>
Reviewed-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: default avatarSebastian Reichel <sebastian.reichel@collabora.com>
parent fe20b1dc

drivers/power/supply/rt9467-charger.c

+1 −1
Original line number Diff line number Diff line
@@ -1192,7 +1192,7 @@ static int rt9467_charger_probe(struct i2c_client *i2c) 
	i2c_set_clientdata(i2c, data);



	/* Default pull charge enable gpio to make 'CHG_EN' by SW control only */

	ceb_gpio = devm_gpiod_get_optional(dev, "charge-enable", GPIOD_OUT_LOW);

	ceb_gpio = devm_gpiod_get_optional(dev, "charge-enable", GPIOD_OUT_HIGH);

	if (IS_ERR(ceb_gpio))

		return dev_err_probe(dev, PTR_ERR(ceb_gpio),

				     "Failed to config charge enable gpio\n");