On Sat, Dec 16, 2017 at 9:48 AM, Ognjen Galic <smclt30p@xxxxxxxxx> wrote: > On Sat, Dec 16, 2017 at 01:33:55AM +0100, Rafael J. Wysocki wrote: >> On Fri, Dec 15, 2017 at 5:57 PM, Ognjen Galic <smclt30p@xxxxxxxxx> wrote: >> > The EC/ACPI firmware on Lenovo ThinkPads used to report a status >> > of "Unknown" when the battery is between the charge start and >> > charge stop thresholds. On Windows, it reports "Not Charging" >> > so the quirk has been added to also report correctly. >> > >> > Now the "status" attribute returns "Not Charging" when the >> > battery on ThinkPads is not physicaly charging. >> > >> > Tested-by: Kevin Locke <kevin@xxxxxxxxxxxxxxx> >> > Tested-by: Christoph Böhmwalder <christoph@xxxxxxxxxxxxxx> >> > Signed-off-by: Ognjen Galic <smclt30p@xxxxxxxxx> >> >> It doesn't look like this is related to the [1/3] and [3/3], so why do >> you make it part of the series? >> > > I made it the same series because it is practically the same feature > set. Without this patch and with 1/3 and 3/3 applied, there is a bug > where the status attribute would show "Unknown" for a battery that is > between the start and stop thresholds while attached to AC. OK, fair enough. I would reorder it as the first patch in the series, then, because the other two are more related to each other. Thanks, Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
