From aca130a133c7ddc9daab4cf06b247c2ac0015fc0 Mon Sep 17 00:00:00 2001 From: litong20 Date: Mon, 8 May 2023 16:10:41 +0800 Subject: [PATCH] use the battery voltage of fuel guage in charging logic. use the battery voltage estimated by the fuel gauge in charging logic, not the adc value of the charge pump. Change-Id: Ia001e41dc909ce3ec57ddbcfe888a7b566ad358a Reviewed-on: https://gerrit.mot.com/2599627 SME-Granted: SME Approvals Granted SLTApproved: Slta Waiver Tested-by: Jira Key Reviewed-by: Wentao Xu Reviewed-by: Guobin Zhang Submit-Approved: Jira Key --- drivers/power/sc760x_charger_mmi/sc760x_charger_mmi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/power/sc760x_charger_mmi/sc760x_charger_mmi.c b/drivers/power/sc760x_charger_mmi/sc760x_charger_mmi.c index e499563bca07..06e91aeb68cd 100644 --- a/drivers/power/sc760x_charger_mmi/sc760x_charger_mmi.c +++ b/drivers/power/sc760x_charger_mmi/sc760x_charger_mmi.c @@ -1580,11 +1580,11 @@ static int sc760x_charger_config_charge(void *data, struct mmi_charger_cfg *conf chg->chg_cfg.target_fv = config->target_fv; } - if (chg->chg_cfg.target_fv > 0 && (state.vbat_adc > chg->chg_cfg.target_fv * 1000)) { + if (chg->chg_cfg.target_fv > 0 && (chg->batt_info.batt_mv > chg->chg_cfg.target_fv)) { sc760x_ibat_limit_set -= IBAT_CHG_LIM_BASE; sc760x_set_ibat_limit(chg->sc, sc760x_ibat_limit_set); - pr_info("update new sc760x_ibat_limit_set %d, state.vbat_adc %d > chg->chg_cfg.target_fv %d\n", - sc760x_ibat_limit_set, state.vbat_adc, chg->chg_cfg.target_fv * 1000); + pr_info("update new sc760x_ibat_limit_set %d, state.vbat_adc %d, batt_mv %d > chg->chg_cfg.target_fv %d\n", + sc760x_ibat_limit_set, state.vbat_adc / 1000, chg->batt_info.batt_mv, chg->chg_cfg.target_fv); } else { ibat_limit_vote = MIN_VAL(chg->chg_cfg.target_fcc, chg->sc->thermal_fcc_ua / 1000);