power: supply: axp20x_ac_power: Allow offlining
authorSamuel Holland <samuel@sholland.org>
Mon, 13 Jan 2020 03:53:04 +0000 (21:53 -0600)
committerSebastian Reichel <sre@kernel.org>
Tue, 14 Jan 2020 00:00:41 +0000 (01:00 +0100)
AXP803/AXP813 have a flag that enables/disables the AC power supply
input. Allow control of this flag via the ONLINE property on those
variants.

Reviewed-by: Chen-Yu Tsai <wens@csie.org>
Signed-off-by: Samuel Holland <samuel@sholland.org>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
drivers/power/supply/axp20x_ac_power.c

index f74b0556bb6b701cee8b7782288aa593b1cf43c5..3ba38f2f281ce60f3e19b45589c40249ecc137d3 100644 (file)
@@ -24,6 +24,7 @@
 #define AXP20X_PWR_STATUS_ACIN_AVAIL   BIT(6)
 
 #define AXP813_ACIN_PATH_SEL           BIT(7)
+#define AXP813_ACIN_PATH_SEL_TO_BIT(x) (!!(x) << 7)
 
 #define AXP813_VHOLD_MASK              GENMASK(5, 3)
 #define AXP813_VHOLD_UV_TO_BIT(x)      ((((x) / 100000) - 40) << 3)
@@ -157,6 +158,11 @@ static int axp813_ac_power_set_property(struct power_supply *psy,
        struct axp20x_ac_power *power = power_supply_get_drvdata(psy);
 
        switch (psp) {
+       case POWER_SUPPLY_PROP_ONLINE:
+               return regmap_update_bits(power->regmap, AXP813_ACIN_PATH_CTRL,
+                                         AXP813_ACIN_PATH_SEL,
+                                         AXP813_ACIN_PATH_SEL_TO_BIT(val->intval));
+
        case POWER_SUPPLY_PROP_VOLTAGE_MIN:
                if (val->intval < 4000000 || val->intval > 4700000)
                        return -EINVAL;
@@ -183,7 +189,8 @@ static int axp813_ac_power_set_property(struct power_supply *psy,
 static int axp813_ac_power_prop_writeable(struct power_supply *psy,
                                          enum power_supply_property psp)
 {
-       return psp == POWER_SUPPLY_PROP_VOLTAGE_MIN ||
+       return psp == POWER_SUPPLY_PROP_ONLINE ||
+              psp == POWER_SUPPLY_PROP_VOLTAGE_MIN ||
               psp == POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT;
 }