diff options
| author | Robert Zieba <robertzieba@google.com> | 2023-04-27 17:28:47 +0000 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-04-27 19:15:13 +0000 |
| commit | 04ee1f52a8996765b26c5f01ae8559026ac58bad (patch) | |
| tree | 0a1fb1b94eed4bdbfde468a6b88dd300621e403c /common/battery.c | |
| parent | 9dd6dabc8fcb85bc80bccd5c0f387b85e174ce61 (diff) | |
| download | chrome-ec-04ee1f52a8996765b26c5f01ae8559026ac58bad.tar.gz | |
Revert "Add workaround for erroneous battery temperatures"
This reverts commit 665509933ec7c3913daf73d0d5564b8a86054121.
BRANCH=skyrim
BUG=b:276000336
TEST=ODM successfully ran 5000 suspend cycles
Change-Id: I215385e9bd6379dc71d573a8a82750e4147f6f9b
Signed-off-by: Robert Zieba <robertzieba@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4484326
Commit-Queue: Diana Z <dzigterman@chromium.org>
Reviewed-by: Diana Z <dzigterman@chromium.org>
Diffstat (limited to 'common/battery.c')
| -rw-r--r-- | common/battery.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/common/battery.c b/common/battery.c index 4a47991738..9e46cd8bbf 100644 --- a/common/battery.c +++ b/common/battery.c @@ -695,12 +695,8 @@ void battery_validate_params(struct batt_params *batt) * anything above the boiling point of tungsten until this bug * is fixed. If the battery is really that warm, we probably * have more urgent problems. - * TODO(b/276000336). Some batteries occasionally give obviously - * incorrect results of 591.1K and 2151K, which are 318C and 1878C. - * Ignore these for now. */ - if (batt->temperature > CELSIUS_TO_DECI_KELVIN(5660) || - batt->temperature == 5911 || batt->temperature == 21510) { + if (batt->temperature > CELSIUS_TO_DECI_KELVIN(5660)) { CPRINTS("ignoring ridiculous batt.temp of %dC", DECI_KELVIN_TO_CELSIUS(batt->temperature)); batt->flags |= BATT_FLAG_BAD_TEMPERATURE; |