From 631b4461de049bc7c70976a81ddeed3b600f6c1c Mon Sep 17 00:00:00 2001 From: Ting Shen Date: Mon, 19 Aug 2019 16:56:22 +0800 Subject: rt946x: raise current limit to 2A for HW ramp CL:1633930 raised DCP input current limit to 2A when SW ramp enabled, the same limit should also apply to HW ramp too. BUG=b:139012899 TEST=observe the charging curve (see issue link above for test result) BRANCH=master Change-Id: I22098bd81bb46c7d1e638e7b4db8b87cd3284533 Signed-off-by: Ting Shen Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1760655 Reviewed-by: Yilun Lin Commit-Queue: Ting Shen Tested-by: Ting Shen --- driver/charger/rt946x.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/driver/charger/rt946x.c b/driver/charger/rt946x.c index c935e27ec0..2615f9ff25 100644 --- a/driver/charger/rt946x.c +++ b/driver/charger/rt946x.c @@ -918,9 +918,11 @@ static int rt946x_get_bc12_ilim(int charge_supplier) { switch (charge_supplier) { case CHARGE_SUPPLIER_BC12_DCP: - if (IS_ENABLED(CONFIG_CHARGE_RAMP_SW)) + if (IS_ENABLED(CONFIG_CHARGE_RAMP_SW) || + IS_ENABLED(CONFIG_CHARGE_RAMP_HW)) /* A conservative value to prevent a bad charger. */ return 2000; + /* fallback */ case CHARGE_SUPPLIER_BC12_CDP: return 1500; case CHARGE_SUPPLIER_BC12_SDP: -- cgit v1.2.1