diff options
author | Daisuke Nojiri <dnojiri@chromium.org> | 2022-03-10 02:25:26 +0000 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2022-03-16 03:02:43 +0000 |
commit | fdb7b2381c54bae5e71984d9689f148e44bd9f6c (patch) | |
tree | cfcbd9dcb0dae55c376f66fe501aa3f8b9ed0508 /include | |
parent | 6362dde219dd00f7fc5adcc3457f1d90cfc6c5a4 (diff) | |
download | chrome-ec-fdb7b2381c54bae5e71984d9689f148e44bd9f6c.tar.gz |
Revert "battery: Do not return display_soc if data is bad"
This reverts commit 6362dde219dd00f7fc5adcc3457f1d90cfc6c5a4.
Reason for revert: Will be fixed by https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3514489, which doesn't have a side effect.
Original change's description:
> battery: Do not return display_soc if data is bad
>
> If BATT_FLAG_BAD_FULL_CAPACITY or BATT_FLAG_BAD_REMAINING_CAPACITY
> flags are set, then display_charge value is also bad.
> Return UNAVALIABLE error rather than 0%.
>
> BUG=b:217401040
> BRANCH=guybrush
> TEST=Force battery glitch, do not observe 0% battery in OS
>
> Change-Id: Iea861582dde1f7db3d6a4c40e88169d7f74ca159
> Signed-off-by: Rob Barnes <robbarnes@google.com>
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3496464
> Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
> Reviewed-by: Diana Z <dzigterman@chromium.org>
> (cherry picked from commit b135e45447ba42f276b6f5232e2573d166320b31)
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3514623
> Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
> Auto-Submit: Daisuke Nojiri <dnojiri@chromium.org>
> Reviewed-by: caveh jalali <caveh@chromium.org>
> Commit-Queue: caveh jalali <caveh@chromium.org>
Bug: b:217401040
Change-Id: I32d91fb223c796da9af2979d3ce1ff42ce7b2213
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3515274
Reviewed-by: Rob Barnes <robbarnes@google.com>
Commit-Queue: Daisuke Nojiri <dnojiri@chromium.org>
Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
Auto-Submit: Daisuke Nojiri <dnojiri@chromium.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/ec_commands.h | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/include/ec_commands.h b/include/ec_commands.h index 2be578c1fb..3d98585b06 100644 --- a/include/ec_commands.h +++ b/include/ec_commands.h @@ -5267,14 +5267,6 @@ struct ec_response_rollback_info { #define EC_CMD_AP_RESET 0x0125 -/***************************************************************************** - * Get displayable charge percent - * - * Return - * EC_RES_SUCCESS : Values successfully read - * EC_RES_UNAVAILABLE : Values are currently unavailable, - * e.g. unresponsive battery. - */ #define EC_CMD_DISPLAY_SOC 0x0137 struct ec_response_display_soc { |