From 6e8d68414cddd1418333d56ab54ca93cd629ba61 Mon Sep 17 00:00:00 2001 From: Jacky Wang Date: Thu, 6 May 2021 16:24:04 +0800 Subject: collis: Config the EC battery setting. Config the EC battery setting depend on battery spec. BUG=b:187366003, b:186609352 BRANCH=firmware-volteer-13672.B TEST=make BOARD=collis 1. Check battery found on EC log. 2. Check battery cutoff function on EC console. 3. Check battery charging FET status when battery full. Signed-off-by: Jacky Wang Change-Id: I76f24e2ca0d0f6806e597fdcafaa9a39f8bd84d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2875912 Reviewed-by: Michael5 Chen Reviewed-by: Keith Short Reviewed-by: Zhuohao Lee Commit-Queue: Zhuohao Lee --- board/collis/battery.c | 36 ++++++++++++++++++------------------ board/collis/board.h | 2 +- 2 files changed, 19 insertions(+), 19 deletions(-) diff --git a/board/collis/battery.c b/board/collis/battery.c index dc99d3b834..5da46e9caa 100644 --- a/board/collis/battery.c +++ b/board/collis/battery.c @@ -32,35 +32,35 @@ * address, mask, and disconnect value need to be provided. */ const struct board_batt_params board_battery_info[] = { - [BATTERY_C433] = { + [BATTERY_C490] = { .fuel_gauge = { - .manuf_name = "AS1GVPc3KB", - .device_name = "C433-41", + .manuf_name = "AS3GWQd3jB", .ship_mode = { .reg_addr = 0x00, .reg_data = { 0x10, 0x10 }, }, .fet = { - .mfgacc_support = 1, - .reg_addr = 0x44, - .reg_mask = 0x6000, - .disconnect_val = 0x6000, + .reg_addr = 0x99, + .reg_mask = 0x000c, + .disconnect_val = 0x000c, + .cfet_mask = 0x0004, + .cfet_off_val = 0x0004, } }, .batt_info = { - .voltage_max = 13200, - .voltage_normal = 11550, /* mV */ - .voltage_min = 9000, /* mV */ - .precharge_current = 256, /* mA */ - .start_charging_min_c = 0, - .start_charging_max_c = 45, - .charging_min_c = 0, - .charging_max_c = 60, - .discharging_min_c = -20, - .discharging_max_c = 60, + .voltage_max = 13200, /* mV */ + .voltage_normal = 11550, /* mV */ + .voltage_min = 9000, /* mV */ + .precharge_current = 256, /* mA */ + .start_charging_min_c = 0, + .start_charging_max_c = 45, + .charging_min_c = 0, + .charging_max_c = 60, + .discharging_min_c = -20, + .discharging_max_c = 60, }, }, }; BUILD_ASSERT(ARRAY_SIZE(board_battery_info) == BATTERY_TYPE_COUNT); -const enum battery_type DEFAULT_BATTERY_TYPE = BATTERY_C433; +const enum battery_type DEFAULT_BATTERY_TYPE = BATTERY_C490; diff --git a/board/collis/board.h b/board/collis/board.h index 90b904a4f6..1f5ca0e154 100644 --- a/board/collis/board.h +++ b/board/collis/board.h @@ -160,7 +160,7 @@ #include "registers.h" enum battery_type { - BATTERY_C433, + BATTERY_C490, BATTERY_TYPE_COUNT, }; -- cgit v1.2.1