diff options
| author | Tim Kryger <tkryger@google.com> | 2021-10-07 08:34:18 -0700 |
|---|---|---|
| committer | Tim Kryger <tkryger@google.com> | 2021-10-15 12:28:22 +0000 |
| commit | 6bcfaab6c794d431cda1eb9c4dcd01330efaa82a (patch) | |
| tree | 16b437432c7036a8652efe2f7ae7c295d10a5db9 | |
| parent | 9d7781fdf1ef6477c00fb57738b0fc7197dd8cfe (diff) | |
| download | devicetree-6bcfaab6c794d431cda1eb9c4dcd01330efaa82a.tar.gz | |
Adjust down charge_full_design to derated 279 mAh
Previously 300 mAh was used and while the pack itself is labeled with
a sticker saying 294 mAh, this number should be reduced to the derated
value of 279 mAh to be consistent with what we're using in modeling.
Note that within the Linux kernel battery capacity is discussed using
three different terms.
1. "charge" represents capacity in µAh only
2. "energy" represents capacity in µWh only
3. "capacity" represents capacity in *percents*, from 0 to 100
Thus the fuelgauge,capacity_max property should be in percents but it
is actually in tenths of a percent. Thus the 1000 number currently in
the device tree actually means 100%. More problematically, this value
is improperly sent to userspace as POWER_SUPPLY_PROP_ENERGY_FULL_DESIGN,
but luckily, Android does not appear to use this power supply property.
Bug: 202396388
Signed-off-by: Tim Kryger <tkryger@google.com>
Change-Id: Ia912e5ba668b16af1703bf8adf2113abf2693378
| -rw-r--r-- | r11_battery_root.dtsi | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/r11_battery_root.dtsi b/r11_battery_root.dtsi index bfec404..a713670 100644 --- a/r11_battery_root.dtsi +++ b/r11_battery_root.dtsi @@ -49,7 +49,7 @@ google_battery: battery { battery,temp_hysteresis = <10>; // 1 degree battery,voltage_hysteresis = <100>; // 100 mV battery,max_rawsoc = <10000>; /* In units of 0.01% */ - battery,charge_full_design = <300000>; /* 300000 uAh */ + battery,charge_full_design = <279000>; /* 279000 uAh */ }; wlc_cool_dev: wlc_cool_dev { |