diff options
author | Diana Z <dzigterman@chromium.org> | 2021-04-28 14:28:23 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-04-30 00:32:29 +0000 |
commit | 21ea0ade8cc30d3b798c4e9184de06b29443f0af (patch) | |
tree | 69524440eb2e0b0de33b1a69679373577c079666 /baseboard/guybrush/baseboard.c | |
parent | f406a9065d0750621375c7198cfea459a7a78b95 (diff) | |
download | chrome-ec-21ea0ade8cc30d3b798c4e9184de06b29443f0af.tar.gz |
Guybrush: Move batteries to board level
Move battery information into the board level in anticipation of them
probably varying with each board.
BRANCH=None
BUG=b:186577625
TEST=make -j buildall, load on guybrush with trembyle battery and ensure
it's identified
Signed-off-by: Diana Z <dzigterman@chromium.org>
Change-Id: I1780949746cd30c3033b2c9f36d1ba8f5d331166
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2861055
Reviewed-by: Rob Barnes <robbarnes@google.com>
Commit-Queue: Rob Barnes <robbarnes@google.com>
Diffstat (limited to 'baseboard/guybrush/baseboard.c')
-rw-r--r-- | baseboard/guybrush/baseboard.c | 55 |
1 files changed, 0 insertions, 55 deletions
diff --git a/baseboard/guybrush/baseboard.c b/baseboard/guybrush/baseboard.c index fa48959478..353018142a 100644 --- a/baseboard/guybrush/baseboard.c +++ b/baseboard/guybrush/baseboard.c @@ -257,61 +257,6 @@ struct ec_thermal_config thermal_params[TEMP_SENSOR_COUNT] = { }; BUILD_ASSERT(ARRAY_SIZE(thermal_params) == TEMP_SENSOR_COUNT); - -/* - * Battery info for all Guybrush battery types. Note that the fields - * start_charging_min/max and charging_min/max are not used for the charger. - * The effective temperature limits are given by discharging_min/max_c. - * - * Fuel Gauge (FG) parameters which are used for determining if the battery - * is connected, the appropriate ship mode (battery cutoff) command, and the - * charge/discharge FETs status. - * - * Ship mode (battery cutoff) requires 2 writes to the appropriate smart battery - * register. For some batteries, the charge/discharge FET bits are set when - * charging/discharging is active, in other types, these bits set mean that - * charging/discharging is disabled. Therefore, in addition to the mask for - * these bits, a disconnect value must be specified. Note that for TI fuel - * gauge, the charge/discharge FET status is found in Operation Status (0x54), - * but a read of Manufacturer Access (0x00) will return the lower 16 bits of - * Operation status which contains the FET status bits. - * - * The assumption for battery types supported is that the charge/discharge FET - * status can be read with a sb_read() command and therefore, only the register - * address, mask, and disconnect value need to be provided. - */ -const struct board_batt_params board_battery_info[] = { - /* AP18F4M / LIS4163ACPC */ - [BATTERY_AP18F4M] = { - .fuel_gauge = { - .manuf_name = "Murata KT00404001", - .ship_mode = { - .reg_addr = 0x3A, - .reg_data = { 0xC574, 0xC574 }, - }, - .fet = { - .reg_addr = 0x0, - .reg_mask = 0x2000, - .disconnect_val = 0x2000, - } - }, - .batt_info = { - .voltage_max = 8700, - .voltage_normal = 7600, - .voltage_min = 5500, - .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, - }, - }, -}; -BUILD_ASSERT(ARRAY_SIZE(board_battery_info) == BATTERY_TYPE_COUNT); -const enum battery_type DEFAULT_BATTERY_TYPE = BATTERY_AP18F4M; - const struct charger_config_t chg_chips[] = { { .i2c_port = I2C_PORT_CHARGER, |