diff options
author | matt_wang <matt_wang@compal.corp-partner.google.com> | 2019-02-13 20:59:23 +0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2019-02-26 16:49:14 -0800 |
commit | c2c2776e3a5ee68a1d2a2bab09635692418db998 (patch) | |
tree | 6c8b4676216febe6fa2185a3b5c5e38e3f2d599f /common | |
parent | 219b09e87f61ffb410fe1a1dbaed4983460fefb4 (diff) | |
download | chrome-ec-c2c2776e3a5ee68a1d2a2bab09635692418db998.tar.gz |
Battery: add support battery-cutoff SMBus block write function
Implements battery-cutoff SMBus write block function
BUG=b:122944526
BRANCH=None
TEST=Verify battery cuff on fleex via SMBus block write.
Change-Id: Ib52146cd3042c4a6d2dbafadd430591936230891
Signed-off-by: matt_wang <matt_wang@compal.corp-partner.google.com>
Reviewed-on: https://chromium-review.googlesource.com/1470462
Commit-Ready: Justin TerAvest <teravest@chromium.org>
Tested-by: Justin TerAvest <teravest@chromium.org>
Tested-by: Elmo Lan <elmo_lan@compal.corp-partner.google.com>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Jett Rink <jettrink@chromium.org>
Reviewed-by: Justin TerAvest <teravest@chromium.org>
Reviewed-by: Elmo Lan <elmo_lan@compal.corp-partner.google.com>
Diffstat (limited to 'common')
-rw-r--r-- | common/battery_fuel_gauge.c | 57 |
1 files changed, 46 insertions, 11 deletions
diff --git a/common/battery_fuel_gauge.c b/common/battery_fuel_gauge.c index 476ca52570..a0dd250899 100644 --- a/common/battery_fuel_gauge.c +++ b/common/battery_fuel_gauge.c @@ -9,6 +9,7 @@ #include "battery_smart.h" #include "console.h" #include "hooks.h" +#include "i2c.h" #include "util.h" #define CPRINTS(format, args...) cprints(CC_CHARGER, format, ## args) @@ -89,26 +90,60 @@ const struct battery_info *battery_get_info(void) return &get_batt_params()->batt_info; } +int cut_off_battery_block_write(const struct ship_mode_info *ship_mode) +{ + int rv; + + uint8_t cutdata[3] = { + 0x02, + ship_mode->reg_data[0] & 0xFF, + ship_mode->reg_data[0] >> 8, + }; + + /* SMBus protocols are block write, which include byte count + * byte. Byte count segments are required to communicate + * required action and the number of data bytes. + * Due to ship mode command requires writing data values twice + * to cutoff the battery, so byte count is 0x02. + */ + rv = sb_write_block(ship_mode->reg_addr, cutdata, sizeof(cutdata)); + if (rv) + return rv; + + /* Use the next set of values */ + cutdata[1] = ship_mode->reg_data[1] & 0xFF; + cutdata[2] = ship_mode->reg_data[1] >> 8; + + return sb_write_block(ship_mode->reg_addr, cutdata, sizeof(cutdata)); +} + +int cut_off_battery_sb_write(const struct ship_mode_info *ship_mode) +{ + int rv; + + /* Ship mode command requires writing 2 data values */ + rv = sb_write(ship_mode->reg_addr, ship_mode->reg_data[0]); + if (rv) + return rv; + + return sb_write(ship_mode->reg_addr, ship_mode->reg_data[1]); +} + int board_cut_off_battery(void) { int rv; - int cmd; - int data; int type = get_battery_type(); /* If battery type is unknown can't send ship mode command */ if (type == BATTERY_TYPE_COUNT) return EC_RES_ERROR; - /* Ship mode command requires writing 2 data values */ - cmd = board_battery_info[type].fuel_gauge.ship_mode.reg_addr; - data = board_battery_info[type].fuel_gauge.ship_mode.reg_data[0]; - rv = sb_write(cmd, data); - if (rv != EC_SUCCESS) - return EC_RES_ERROR; - - data = board_battery_info[type].fuel_gauge.ship_mode.reg_data[1]; - rv = sb_write(cmd, data); + if (board_battery_info[type].fuel_gauge.ship_mode.wb_support) + rv = cut_off_battery_block_write( + &board_battery_info[type].fuel_gauge.ship_mode); + else + rv = cut_off_battery_sb_write( + &board_battery_info[type].fuel_gauge.ship_mode); return rv ? EC_RES_ERROR : EC_RES_SUCCESS; } |