diff options
| author | Jacky Wang <jacky5_wang@pegatron.corp-partner.google.com> | 2020-12-17 10:18:58 +0800 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-12-18 03:54:18 +0000 |
| commit | a4445b15c9aba20f9078779adb1a0cb2e307c36a (patch) | |
| tree | d53475a76427c4b522b99b0c4a79ddc7162489aa | |
| parent | 26aab7af80c4f131a0c3708810ead3361ddab4f6 (diff) | |
| download | chrome-ec-a4445b15c9aba20f9078779adb1a0cb2e307c36a.tar.gz | |
copano: Config the EC battery setting.
Config the EC battery setting depend on battery spec.
BUG=b:174969897
BRANCH=firmware-volteer-13521.B
TEST=BOARD=copano
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 <jacky5_wang@pegatron.corp-partner.google.com>
Change-Id: I188d4c560cedd0d34e9ef6a95b1946a6aa9389ec
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2596562
Reviewed-by: Michael5 Chen <michael5_chen1@pegatron.corp-partner.google.com>
Reviewed-by: Keith Short <keithshort@chromium.org>
| -rw-r--r-- | board/copano/battery.c | 48 | ||||
| -rw-r--r-- | board/copano/board.h | 3 |
2 files changed, 10 insertions, 41 deletions
diff --git a/board/copano/battery.c b/board/copano/battery.c index d129ede528..add2094b52 100644 --- a/board/copano/battery.c +++ b/board/copano/battery.c @@ -32,25 +32,23 @@ * address, mask, and disconnect value need to be provided. */ const struct board_batt_params board_battery_info[] = { - /* LGC\011 L17L3PB0 Battery Information */ - /* - * Battery info provided by ODM on b/143477210, comment #11 - */ - [BATTERY_LGC011] = { + [BATTERY_C433] = { .fuel_gauge = { - .manuf_name = "LGC", + .manuf_name = "AS1GVPc3KB", + .device_name = "C433-41", .ship_mode = { .reg_addr = 0x00, .reg_data = { 0x10, 0x10 }, }, .fet = { - .reg_addr = 0x0, + .mfgacc_support = 1, + .reg_addr = 0x44, .reg_mask = 0x6000, .disconnect_val = 0x6000, } }, .batt_info = { - .voltage_max = TARGET_WITH_MARGIN(13200, 5), + .voltage_max = 13200, .voltage_normal = 11550, /* mV */ .voltage_min = 9000, /* mV */ .precharge_current = 256, /* mA */ @@ -58,39 +56,11 @@ const struct board_batt_params board_battery_info[] = { .start_charging_max_c = 45, .charging_min_c = 0, .charging_max_c = 60, - .discharging_min_c = 0, - .discharging_max_c = 75, - }, - }, - /* LGC AP18C8K Battery Information */ - [BATTERY_LGC_AP18C8K] = { - .fuel_gauge = { - .manuf_name = "LGC KT0030G020", - .device_name = "AP18C8K", - .ship_mode = { - .reg_addr = 0x3A, - .reg_data = { 0xC574, 0xC574 }, - }, - .fet = { - .reg_addr = 0x43, - .reg_mask = 0x0001, - .disconnect_val = 0x0, - }, - }, - .batt_info = { - .voltage_max = 13050, - .voltage_normal = 11250, - .voltage_min = 9000, - .precharge_current = 256, - .start_charging_min_c = 0, - .start_charging_max_c = 50, - .charging_min_c = 0, - .charging_max_c = 60, - .discharging_min_c = -20, - .discharging_max_c = 75, + .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_LGC011; +const enum battery_type DEFAULT_BATTERY_TYPE = BATTERY_C433; diff --git a/board/copano/board.h b/board/copano/board.h index 1a917c4119..41c0fc4ef0 100644 --- a/board/copano/board.h +++ b/board/copano/board.h @@ -163,8 +163,7 @@ #include "registers.h" enum battery_type { - BATTERY_LGC011, - BATTERY_LGC_AP18C8K, + BATTERY_C433, BATTERY_TYPE_COUNT, }; |