diff options
| author | Dino Li <Dino.Li@ite.com.tw> | 2021-08-23 18:05:04 +0800 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-08-24 07:32:36 +0000 |
| commit | ffc3742a73c7124c62890210f9136908e7b669c1 (patch) | |
| tree | 16d44a866bbbdc84448a8f8dc638d81f948fb3a5 | |
| parent | c97daf56d5d5d57594bf61a7a316b6e26c143179 (diff) | |
| download | chrome-ec-ffc3742a73c7124c62890210f9136908e7b669c1.tar.gz | |
mt6360: Don't clear negative number supplier's charge port info
Because we didn't create space for it to store current/voltage
info. Clear its charge port info will override other global variables.
BRANCH=asurada
BUG=b:179206540
TEST=No panic on asurada when plug AC only on port 0.
Signed-off-by: Dino Li <Dino.Li@ite.com.tw>
Change-Id: Ia7a9a058d65aca9b5a84963d1e08f3e2541bc8da
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3113487
Reviewed-by: Eric Yilun Lin <yllin@google.com>
Commit-Queue: Eric Yilun Lin <yllin@google.com>
| -rw-r--r-- | driver/bc12/mt6360.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/driver/bc12/mt6360.c b/driver/bc12/mt6360.c index 41b5b0e244..50aa4d0e45 100644 --- a/driver/bc12/mt6360.c +++ b/driver/bc12/mt6360.c @@ -118,7 +118,9 @@ static void mt6360_update_charge_manager(int port, static enum charge_supplier current_bc12_type = CHARGE_SUPPLIER_NONE; if (new_bc12_type != current_bc12_type) { - charge_manager_update_charge(current_bc12_type, port, NULL); + if (current_bc12_type >= 0) + charge_manager_update_charge(current_bc12_type, port, + NULL); if (new_bc12_type != CHARGE_SUPPLIER_NONE) { struct charge_port_info chg = { |