Re: [PATCH v13 4/4] battery: Add the ThinkPad "Not Charging" quirk

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 07 Feb 2018, Ognjen Galic 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.
> 
> Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
> Signed-off-by: Ognjen Galic <smclt30p@xxxxxxxxx>

AFAIK, This behavior goes back to the initial SBS implementation in the
IBM era ECs of the Thinkpads...  We've always called it "idle" in the
linux-thinkpad community.  The behavior comes from SBS
(http://smartbattery.org/specs/), the EC was reporting its status
(charging/not charging *THIS* battery) in one bit, and the battery's
status (discharging/not discharging *THIS* battery) in a different bit.

It was rather simple to observe the behavior of those bits in a
two-battery system.

Would that apply to these newer Lenovo models?  If so, you might want to
consider using "idle", instead. "not charging" does _not_ imply "neither
charging nor discharging", while "idle" does.

-- 
  Henrique Holschuh

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
ibm-acpi-devel mailing list
ibm-acpi-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/ibm-acpi-devel



[Index of Archives]     [Linux ACPI]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Photo]     [Yosemite Photos]     [Yosemite Advice]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]

  Powered by Linux