power: supply: bq24190_charger: Add power_supply_battery_info support
authorLiam Breck <kernel@networkimprov.net>
Mon, 28 Aug 2017 21:23:49 +0000 (14:23 -0700)
committerSebastian Reichel <sre@kernel.org>
Tue, 29 Aug 2017 21:08:39 +0000 (23:08 +0200)
Set pre-charge and charge-term current, obtained from power_supply_battery_info.
Add sysfs attributes precharge_current & charge_term_current.

Cc: Hans de Goede <hdegoede@redhat.com>
Acked-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Liam Breck <kernel@networkimprov.net>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk>
drivers/power/supply/bq24190_charger.c

index a12d263aea8a8c1345f4bd02954aedeefc025df6..fa711c106b6332b1781c315907e7bb41c44aa65d 100644 (file)
 #define BQ24190_REG_PCTCC_IPRECHG_MASK         (BIT(7) | BIT(6) | BIT(5) | \
                                                 BIT(4))
 #define BQ24190_REG_PCTCC_IPRECHG_SHIFT                4
+#define BQ24190_REG_PCTCC_IPRECHG_MIN                  128
+#define BQ24190_REG_PCTCC_IPRECHG_MAX                  2048
 #define BQ24190_REG_PCTCC_ITERM_MASK           (BIT(3) | BIT(2) | BIT(1) | \
                                                 BIT(0))
 #define BQ24190_REG_PCTCC_ITERM_SHIFT          0
+#define BQ24190_REG_PCTCC_ITERM_MIN                    128
+#define BQ24190_REG_PCTCC_ITERM_MAX                    2048
 
 #define BQ24190_REG_CVC                0x04 /* Charge Voltage Control */
 #define BQ24190_REG_CVC_VREG_MASK              (BIT(7) | BIT(6) | BIT(5) | \
@@ -162,6 +166,8 @@ struct bq24190_dev_info {
        bool                            initialized;
        bool                            irq_event;
        u16                             sys_min;
+       u16                             iprechg;
+       u16                             iterm;
        struct mutex                    f_reg_lock;
        u8                              f_reg;
        u8                              ss_reg;
@@ -543,6 +549,26 @@ static int bq24190_set_config(struct bq24190_dev_info *bdi)
                        return ret;
        }
 
+       if (bdi->iprechg) {
+               v = bdi->iprechg / 128 - 1; // manual section 9.5.1.4, table 11
+               ret = bq24190_write_mask(bdi, BQ24190_REG_PCTCC,
+                                        BQ24190_REG_PCTCC_IPRECHG_MASK,
+                                        BQ24190_REG_PCTCC_IPRECHG_SHIFT,
+                                        v);
+               if (ret < 0)
+                       return ret;
+       }
+
+       if (bdi->iterm) {
+               v = bdi->iterm / 128 - 1; // manual section 9.5.1.4, table 11
+               ret = bq24190_write_mask(bdi, BQ24190_REG_PCTCC,
+                                        BQ24190_REG_PCTCC_ITERM_MASK,
+                                        BQ24190_REG_PCTCC_ITERM_SHIFT,
+                                        v);
+               if (ret < 0)
+                       return ret;
+       }
+
        return 0;
 }
 
@@ -791,6 +817,38 @@ static int bq24190_charger_set_temp_alert_max(struct bq24190_dev_info *bdi,
        return bq24190_battery_set_temp_alert_max(bdi, val);
 }
 
+static int bq24190_charger_get_precharge(struct bq24190_dev_info *bdi,
+               union power_supply_propval *val)
+{
+       u8 v;
+       int ret;
+
+       ret = bq24190_read_mask(bdi, BQ24190_REG_PCTCC,
+                       BQ24190_REG_PCTCC_IPRECHG_MASK,
+                       BQ24190_REG_PCTCC_IPRECHG_SHIFT, &v);
+       if (ret < 0)
+               return ret;
+
+       val->intval = ++v * 128 * 1000;
+       return 0;
+}
+
+static int bq24190_charger_get_charge_term(struct bq24190_dev_info *bdi,
+               union power_supply_propval *val)
+{
+       u8 v;
+       int ret;
+
+       ret = bq24190_read_mask(bdi, BQ24190_REG_PCTCC,
+                       BQ24190_REG_PCTCC_ITERM_MASK,
+                       BQ24190_REG_PCTCC_ITERM_SHIFT, &v);
+       if (ret < 0)
+               return ret;
+
+       val->intval = ++v * 128 * 1000;
+       return 0;
+}
+
 static int bq24190_charger_get_current(struct bq24190_dev_info *bdi,
                union power_supply_propval *val)
 {
@@ -938,6 +996,12 @@ static int bq24190_charger_get_property(struct power_supply *psy,
        case POWER_SUPPLY_PROP_TEMP_ALERT_MAX:
                ret =  bq24190_charger_get_temp_alert_max(bdi, val);
                break;
+       case POWER_SUPPLY_PROP_PRECHARGE_CURRENT:
+               ret = bq24190_charger_get_precharge(bdi, val);
+               break;
+       case POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT:
+               ret = bq24190_charger_get_charge_term(bdi, val);
+               break;
        case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
                ret = bq24190_charger_get_current(bdi, val);
                break;
@@ -1044,6 +1108,8 @@ static enum power_supply_property bq24190_charger_properties[] = {
        POWER_SUPPLY_PROP_ONLINE,
        POWER_SUPPLY_PROP_STATUS,
        POWER_SUPPLY_PROP_TEMP_ALERT_MAX,
+       POWER_SUPPLY_PROP_PRECHARGE_CURRENT,
+       POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT,
        POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT,
        POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT_MAX,
        POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE,
@@ -1523,6 +1589,7 @@ static int bq24190_hw_init(struct bq24190_dev_info *bdi)
 static int bq24190_get_config(struct bq24190_dev_info *bdi)
 {
        const char * const s = "ti,system-minimum-microvolt";
+       struct power_supply_battery_info info = {};
        int v;
 
        if (device_property_read_u32(bdi->dev, s, &v) == 0) {
@@ -1534,6 +1601,25 @@ static int bq24190_get_config(struct bq24190_dev_info *bdi)
                        dev_warn(bdi->dev, "invalid value for %s: %u\n", s, v);
        }
 
+       if (bdi->dev->of_node &&
+           !power_supply_get_battery_info(bdi->charger, &info)) {
+               v = info.precharge_current_ua / 1000;
+               if (v >= BQ24190_REG_PCTCC_IPRECHG_MIN
+                && v <= BQ24190_REG_PCTCC_IPRECHG_MAX)
+                       bdi->iprechg = v;
+               else
+                       dev_warn(bdi->dev, "invalid value for battery:precharge-current-microamp: %d\n",
+                                v);
+
+               v = info.charge_term_current_ua / 1000;
+               if (v >= BQ24190_REG_PCTCC_ITERM_MIN
+                && v <= BQ24190_REG_PCTCC_ITERM_MAX)
+                       bdi->iterm = v;
+               else
+                       dev_warn(bdi->dev, "invalid value for battery:charge-term-current-microamp: %d\n",
+                                v);
+       }
+
        return 0;
 }
 
@@ -1601,6 +1687,7 @@ static int bq24190_probe(struct i2c_client *client,
        }
 
        charger_cfg.drv_data = bdi;
+       charger_cfg.of_node = dev->of_node;
        charger_cfg.supplied_to = bq24190_charger_supplied_to;
        charger_cfg.num_supplicants = ARRAY_SIZE(bq24190_charger_supplied_to),
        bdi->charger = power_supply_register(dev, &bq24190_charger_desc,