]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
Revert "ACPI / battery: fix wrong value of capacity_now reported when fully charged"
authorBjørn Mork <bjorn@mork.no>
Tue, 9 Sep 2014 08:45:18 +0000 (10:45 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 5 Oct 2014 20:40:59 +0000 (13:40 -0700)
commit7e6db41703b15e883cb4cd7eebfd5175e9adecc3
tree89f810134880a89ce64663a9c2adc628983de148
parentffaf87f6ee1cc7e0b0c6fa4f4f1132b53bd3589f
Revert "ACPI / battery: fix wrong value of capacity_now reported when fully charged"

commit 508b3c677601797f2d51df3df5caa436dd235cb9 upstream.

This reverts commit 232de5143790 ("ACPI / battery: fix wrong value of
capacity_now reported when fully charged")

There is nothing wrong or unexpected about 'capacity_now' increasing above
the last 'full_charge_capacity' value. Different charging cycles will cause
'full_charge_capacity' to vary, both up and down.  Good battery firmwares
will update 'full_charge_capacity' when the current charging cycle is
complete, increasing it if necessary. It might even go above
'design_capacity' on a fresh and healthy battery.

Capping 'capacity_now' to 'full_charge_capacity' is plain wrong, and
printing a warning if this doesn't happen to match the 'design_capacity'
is both annoying and terribly wrong.

This results in bogus warnings on perfectly working systems/firmwares:

 [Firmware Bug]: battery: reported current charge level (39800) is higher than reported maximum charge level (39800).

and wrong values being reported for 'capacity_now' and
'full_charge_capacity' after the warning has been triggered.

Fixes: 232de5143790 ("ACPI / battery: fix wrong value of capacity_now reported when fully charged")
Signed-off-by: Bjørn Mork <bjorn@mork.no>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/acpi/battery.c