power: supply: bq25890: Add support for setting user charge current and voltage limit

Let user set battery charge current and voltage limit via sysfs. This is
useful in case the user space needs to reduce charge current to keep the
system within thermal limits. The maximum charge current and voltage are
still limited to "ti,charge-current" and "ti,battery-regulation-voltage"
values to avoid damaging the hardware in case too high values are set by
user space.

Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Marek Vasut <marex@denx.de>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>

diff --git a/drivers/power/supply/bq25890_charger.c b/drivers/power/supply/bq25890_charger.c
index 050eef2..9580315 100644 (file)
--- a/drivers/power/supply/bq25890_charger.c
+++ b/drivers/power/supply/bq25890_charger.c
@@ -667,9 +667,18 @@ static int bq25890_power_supply_set_property(struct power_supply *psy,
                                             const union power_supply_propval *val)
 {
        struct bq25890_device *bq = power_supply_get_drvdata(psy);
+       int maxval;
        u8 lval;
 
        switch (psp) {
+       case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
+               maxval = bq25890_find_val(bq->init_data.ichg, TBL_ICHG);
+               lval = bq25890_find_idx(min(val->intval, maxval), TBL_ICHG);
+               return bq25890_field_write(bq, F_ICHG, lval);
+       case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:
+               maxval = bq25890_find_val(bq->init_data.vreg, TBL_VREG);
+               lval = bq25890_find_idx(min(val->intval, maxval), TBL_VREG);
+               return bq25890_field_write(bq, F_VREG, lval);
        case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:
                lval = bq25890_find_idx(val->intval, TBL_IINLIM);
                return bq25890_field_write(bq, F_IINLIM, lval);
@@ -682,6 +691,8 @@ static int bq25890_power_supply_property_is_writeable(struct power_supply *psy,
                                                      enum power_supply_property psp)
 {
        switch (psp) {
+       case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
+       case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:
        case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:
                return true;
        default: