evie/android_kernel_oneplus_msm8998
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

qpnp-fg-gen3: show 0 time to full at 100% SOC

Browse source 
Currently the time to full is clamped to 0 when the charge status
reports full, but the full status is reported minutes after the SOC
reaches 100%. Furthermore, the CC estimates use SOC as a scaling
mechanism so using full status is inconsistent. Clamp the time to full
estimate to 0 when the SOC reaches 100% instead.

Change-Id: I71e3e0f0cddf3b75ae5e711c86df26fc6a79957d
Signed-off-by: Nicholas Troast <ntroast@codeaurora.org>
This commit is contained in:
Nicholas Troast 2016-12-14 16:12:04 -08:00
parent 2f817b68f0
commit fe9b8816f0
1 changed files with 9 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
drivers/power/qcom-charger/qpnp-fg-gen3.c
View file

@ -2148,7 +2148,14 @@ static int fg_get_time_to_full(struct fg_chip *chip, int *val)
return -ENODATA;
}
if (chip->charge_status == POWER_SUPPLY_STATUS_FULL) {
rc = fg_get_prop_capacity(chip, &msoc);
if (rc < 0) {
pr_err("failed to get msoc rc=%d\n", rc);
return rc;
}
fg_dbg(chip, FG_TTF, "msoc=%d\n", msoc);
if (msoc >= 100) {
*val = 0;
return 0;
}
@ -2211,13 +2218,6 @@ static int fg_get_time_to_full(struct fg_chip *chip, int *val)
act_cap_uah *= MILLI_UNIT;
fg_dbg(chip, FG_TTF, "actual_capacity_uah=%d\n", act_cap_uah);
rc = fg_get_prop_capacity(chip, &msoc);
if (rc < 0) {
pr_err("failed to get msoc rc=%d\n", rc);
return rc;
}
fg_dbg(chip, FG_TTF, "msoc=%d\n", msoc);
rc = fg_get_sram_prop(chip, FG_SRAM_FULL_SOC, &full_soc);
if (rc < 0) {
pr_err("failed to get full soc rc=%d\n", rc);
@ -2257,7 +2257,7 @@ skip_cc_estimate:
fg_dbg(chip, FG_TTF, "t_predicted_cc=%lld\n", t_predicted_cc);
/* CV estimate starts here */
if (chip->charge_type == POWER_SUPPLY_CHARGE_TYPE_TAPER)
if (chip->charge_type >= POWER_SUPPLY_CHARGE_TYPE_TAPER)
ln_val = ibatt_avg / abs(chip->dt.sys_term_curr_ma);
else
ln_val = i_cc2cv / abs(chip->dt.sys_term_curr_ma);