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 9238b6124c)
This commit is contained in:
committed by
Aaron Whyte
parent
7b89a7b1f7
commit
c800ddc824
@@ -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;
|
||||
|
||||
Reference in New Issue
Block a user