From c800ddc824256cea4142ac2530b0489e4f816201 Mon Sep 17 00:00:00 2001 From: "Ying Wai (Daniel) Fan" Date: Wed, 18 Jan 2017 15:50:19 -0800 Subject: [PATCH] Correct the formula of battery capacity estimation. Instead of (current charge) * (current battery level percentage) the formula should be (current charge) / (current battery level percentage) to estimate the charge at 100% battery level. Bug: 34255953 Fixes: 34255953 Test: formula change CL. No existing unit tests found. Change-Id: I56ef7af3fb097a3082e0ef24329f522d2a0229cc (cherry picked from commit 9238b6124c72a216ccbbaa82f5f75828571d5841) --- core/java/com/android/internal/os/BatteryStatsImpl.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/java/com/android/internal/os/BatteryStatsImpl.java b/core/java/com/android/internal/os/BatteryStatsImpl.java index 3b3344e7d9346..c0bf825ec3f04 100644 --- a/core/java/com/android/internal/os/BatteryStatsImpl.java +++ b/core/java/com/android/internal/os/BatteryStatsImpl.java @@ -9596,7 +9596,7 @@ public class BatteryStatsImpl extends BatteryStats { resetAllStatsLocked(); if (chargeUAh > 0) { // Only use the reported coulomb charge value if it is supported and reported. - mEstimatedBatteryCapacity = (int) ((level / 100.0) * (chargeUAh / 1000)); + mEstimatedBatteryCapacity = (int) ((chargeUAh / 1000) / (level / 100.0)); } mDischargeStartLevel = level; reset = true;