diff options
author | Spade Lee <spadelee@google.com> | 2023-11-13 04:04:27 +0000 |
---|---|---|
committer | AleX Pelosi <apelosi@google.com> | 2023-11-15 18:53:16 +0000 |
commit | 4c0fb5146ba032aad4a34819e25fcb7aaa926da8 (patch) | |
tree | 0b5a3b2430bc4fd12790ed861a8dfb7669c2ffc9 | |
parent | fed1edbd5beefcef79517ae243c1ddf897e5796f (diff) | |
download | bms-4c0fb5146ba032aad4a34819e25fcb7aaa926da8.tar.gz |
max1720x_battery: handle return value from gbms_storage_write()
Bug: 310501667
Test: run command 'echo 1 > /d/maxfg/gmsr_reset' on HK3 without error
Change-Id: I90c37f41aff56e89308cbdf7fbbc215b8c7d16d2
Signed-off-by: Spade Lee <spadelee@google.com>
(cherry picked from commit ff695294f6e08336c531b4e5b81ba4c9eabfe667)
-rw-r--r-- | max1720x_battery.c | 4 | ||||
-rw-r--r-- | max_m5.c | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/max1720x_battery.c b/max1720x_battery.c index 6c86d12..4df403f 100644 --- a/max1720x_battery.c +++ b/max1720x_battery.c @@ -2034,7 +2034,7 @@ static int max1720x_health_write_ai(u16 act_impedance, u16 act_timerh) if (ret < 0) return -EIO; - return ret; + return 0; } /* call holding chip->model_lock */ @@ -4013,7 +4013,7 @@ static int debug_cnhs_reset(void *data, u64 val) sizeof(reset_val)); dev_info(chip->dev, "reset CNHS to %d, (ret=%d)\n", reset_val, ret); - return ret; + return ret == sizeof(reset_val) ? 0 : ret; } DEFINE_SIMPLE_ATTRIBUTE(debug_reset_cnhs_fops, NULL, debug_cnhs_reset, "%llu\n"); @@ -419,7 +419,7 @@ int max_m5_reset_state_data(struct max_m5_data *m5_data) if (ret < 0) dev_warn(m5_data->dev, "Erase GMSR fail (%d)\n", ret); - return ret; + return ret == sizeof(data) ? 0 : ret; } int max_m5_needs_reset_model_data(const struct max_m5_data *m5_data) |