[coreboot-gerrit] Patch merged into coreboot/master: ec/lenovo/h8: Guard against EC bugs in the battery status logic.

gerrit at coreboot.org gerrit at coreboot.org
Fri Feb 24 03:32:13 CET 2017


the following patch was just integrated into master:
commit 9b798d7904f50b8ad83fe43442a5a3da85789bdd
Author: Tobias Diedrich <ranma+openocd at tdiedrich.de>
Date:   Sun Feb 12 13:35:37 2017 +0100

    ec/lenovo/h8: Guard against EC bugs in the battery status logic.
    
    On my Thinkpad with an H8-compatible ENE KB9012 EC (GDHT92WW 1.52), when
    the battery is nearly full and we switch from battery to AC by plugging
    in the cable, the current rate will not drop to 0 immediately, but the
    discharging state is cleared immediately.
    
    This leads to the code trying to process an invalid rate value >0x8000,
    leading to a displayed rate of >1000W.
    
    This patch changes the logic to deal with these corner cases.
    
    Change-Id: Ideb588d00757f259792e5ae97729e371b63a096c
    Signed-off-by: Tobias Diedrich <ranma+coreboot at tdiedrich.de>
    Reviewed-on: https://review.coreboot.org/18349
    Tested-by: build bot (Jenkins)
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-by: Martin Roth <martinroth at google.com>


See https://review.coreboot.org/18349 for details.

-gerrit



More information about the coreboot-gerrit mailing list