power_status: Removed of the scaling of charge_full_design
Initial assumptions about battery health were flawed. Multiple
errors have been found in the process used to identify the
typical battery health on deployed devices. Further investigation
has not shown any devices that need the scaling factors.
The battery FSI targets specified a target of 97% within 50
cycles as measured by the smart batteries self reported full
charge capacity (FCC) / design capacity (DC). ECs on older
platforms were scaling down the FCC ratio to 98% of it's value
so any platform with a typical battery usage of .97*.98=95% was
actually meeting our targets. All platforms are hitting this
threshold on the typical values and the remaining regressions
can be attributed to storage and user behavior of dogfood
devices.
BUG=b:181639085, b:174433637
TEST=None
Change-Id: Ib73653412995caec416978fd8ad2363d2bd285ae
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/3123555
Reviewed-by: Puthikorn Voravootivat <puthik@chromium.org>
Tested-by: Brian J. Nemec <bnemec@chromium.org>
Auto-Submit: Brian J. Nemec <bnemec@chromium.org>
Commit-Queue: Brian J. Nemec <bnemec@chromium.org>
diff --git a/client/cros/power/power_status.py b/client/cros/power/power_status.py
index aeca571..5674d64 100644
--- a/client/cros/power/power_status.py
+++ b/client/cros/power/power_status.py
@@ -30,13 +30,6 @@
BatteryDataReportType = autotest_enum.AutotestEnum('CHARGE', 'ENERGY')
-# For devices whose full capacity is significantly lower than design full
-# capacity, scale down their design full capacity.
-BATTERY_DESIGN_FULL_SCALE = {
- 'berknip': 0.96, # b/172625511 (0.94), b/194162890 (0.96)
- 'delbin': 0.96, # b/177076373
- 'jinlon': 0.95, # b/161307060
-}
# battery data reported at 1e6 scale
BATTERY_DATA_SCALE = 1e6
# number of times to retry reading the battery in the case of bad data
@@ -333,12 +326,6 @@
raise error.TestError('Failed to determine battery voltage')
battery_design_full_scale = 1
- model = utils.get_platform()
- if model in BATTERY_DESIGN_FULL_SCALE:
- battery_design_full_scale = BATTERY_DESIGN_FULL_SCALE.get(model)
- logging.info(
- 'Apply %f scale to design full battery capacity for model '
- '%s', battery_design_full_scale, model)
# Since charge data is present, calculate parameters based upon
# reported charge data.
