diff options
author | Logan_Liao <Logan_Liao@compal.corp-partner.google.com> | 2023-01-13 14:21:17 +0800 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-01-31 04:05:47 +0000 |
commit | 3bc906f4a74aca8f3edbb336582d7c8a9de26d3e (patch) | |
tree | 6dd9184d99721756d98572f01f02289d0fc7cfca | |
parent | 49d7b78d0a966384d11cef87be87d5b71a2d9af3 (diff) | |
download | chrome-ec-3bc906f4a74aca8f3edbb336582d7c8a9de26d3e.tar.gz |
Omnigul : Enable Battery parameter.
According to the power team, modify the battery parameter for Omnigul.
BUG=b:263691095
BRANCH=none
TEST=make BOARD=omnigul
Change-Id: I85c85803bcf1140c32cf12ad4cab56614b1304cb
Signed-off-by: Logan_Liao <Logan_Liao@compal.corp-partner.google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4160821
Reviewed-by: Logan Liao <logan_liao@compal.corp-partner.google.com>
Commit-Queue: Kyle Lin <kylelinck@google.com>
Reviewed-by: Kyle Lin <kylelinck@google.com>
Code-Coverage: Kyle Lin <kylelinck@google.com>
Tested-by: Logan Liao <logan_liao@compal.corp-partner.google.com>
Reviewed-by: Elthan Huang <elthan_huang@compal.corp-partner.google.com>
-rw-r--r-- | board/omnigul/battery.c | 54 | ||||
-rw-r--r-- | board/omnigul/board.h | 2 |
2 files changed, 29 insertions, 27 deletions
diff --git a/board/omnigul/battery.c b/board/omnigul/battery.c index 5d2eff0722..39087f72a3 100644 --- a/board/omnigul/battery.c +++ b/board/omnigul/battery.c @@ -33,70 +33,72 @@ * address, mask, and disconnect value need to be provided. */ const struct board_batt_params board_battery_info[] = { - /* POW-TECH GQA05 Battery Information */ - [BATTERY_POWER_TECH] = { - /* BQ40Z50 Fuel Gauge */ + /* LGC AP19B8M Battery Information */ + /* + * Battery info provided by ODM on b/263691095, comment #2 + */ + [BATTERY_AP19B8M] = { .fuel_gauge = { - .manuf_name = "POW-TECH", - .device_name = "BATGQA05L22", + .manuf_name = "LGC KT0030G024", + .device_name = "AP19B8M", .ship_mode = { .reg_addr = 0x00, .reg_data = { 0x0010, 0x0010 }, }, .fet = { - .mfgacc_support = 1, - .reg_addr = 0x00, - .reg_mask = 0x2000, /* XDSG */ - .disconnect_val = 0x2000, + .reg_addr = 0x43, + .reg_mask = 0x0003, /* D-FET C-FET */ + .disconnect_val = 0x0000, } }, .batt_info = { - .voltage_max = TARGET_WITH_MARGIN(13050, 5), - .voltage_normal = 11400, /* mV */ + .voltage_max = 13350, + .voltage_normal = 11610, /* mV */ .voltage_min = 9000, /* mV */ - .precharge_current = 280, /* mA */ + .precharge_current = 256, /* mA */ .start_charging_min_c = 0, - .start_charging_max_c = 45, + .start_charging_max_c = 50, .charging_min_c = 0, - .charging_max_c = 45, - .discharging_min_c = -10, - .discharging_max_c = 60, + .charging_max_c = 60, + .discharging_min_c = -20, + .discharging_max_c = 75, }, }, - /* LGC L17L3PB0 Battery Information */ + /* COSMX AP20CBL Battery Information */ /* - * Battery info provided by ODM on b/143477210, comment #11 + * Battery info provided by ODM on b/263691095, comment #2 */ - [BATTERY_LGC011] = { + [BATTERY_AP20CBL] = { .fuel_gauge = { - .manuf_name = "LGC", + .manuf_name = "COSMX KT0030B004", .ship_mode = { .reg_addr = 0x00, .reg_data = { 0x0010, 0x0010 }, }, .fet = { + .mfgacc_support = 1, .reg_addr = 0x0, - .reg_mask = 0x6000, - .disconnect_val = 0x6000, + .reg_mask = 0x0006, /* D-FET C-FET */ + .disconnect_val = 0x0000, } }, .batt_info = { - .voltage_max = TARGET_WITH_MARGIN(13200, 5), + .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, + .start_charging_max_c = 50, .charging_min_c = 0, .charging_max_c = 60, - .discharging_min_c = 0, + .discharging_min_c = -20, .discharging_max_c = 75, }, }, }; BUILD_ASSERT(ARRAY_SIZE(board_battery_info) == BATTERY_TYPE_COUNT); -const enum battery_type DEFAULT_BATTERY_TYPE = BATTERY_POWER_TECH; +const enum battery_type DEFAULT_BATTERY_TYPE = BATTERY_AP19B8M; enum battery_present battery_hw_present(void) { diff --git a/board/omnigul/board.h b/board/omnigul/board.h index f637b0c6fa..2f260f88f8 100644 --- a/board/omnigul/board.h +++ b/board/omnigul/board.h @@ -245,7 +245,7 @@ enum sensor_id { enum ioex_port { IOEX_C0_NCT38XX = 0, IOEX_C2_NCT38XX, IOEX_PORT_COUNT }; -enum battery_type { BATTERY_POWER_TECH, BATTERY_LGC011, BATTERY_TYPE_COUNT }; +enum battery_type { BATTERY_AP19B8M, BATTERY_AP20CBL, BATTERY_TYPE_COUNT }; enum pwm_channel { PWM_CH_LED2 = 0, /* PWM0 (white charger) */ |