On Mon, 30 Dec 2013, Julian Andres Klode wrote: > I think that a more generic approach is a good idea, but > I don't think creating a new power supply device would be the right ... > We could extend the existing battery devices and then get paths like > /sys/class/power_supply/BAT0/start_charge_thresh > which looks `natural'. There is one problem with that approach, though: I'm fine with extending battery devices, in fact I agree that it would be a better place to add these attributes to. > A ThinkPad supports multiple batteries: In my case, I could add a slice > battery. But the ACPI driver only exposes a BAT0 device without a slice > being attached (I assume a BAT1 will pop-up if you add a slice battery, Thinkpad batteries are static, BAT1 is always BAT1 for the firmware, regardless of whether BAT0 is present or not. Their parameters can always be set as well, since they're stored in the EC and not in the battery pack uC. And the EC doesn't care whether the battery is present or not. > but can't test it, as I don't own one). This means that you can only > configure the slice battery when it's plugged in. I don't know if its > a good idea to work like this. We could change the ACPI battery driver to optionally allow a platform driver (or a dmi table) to order it to register batteries that are not present at boot, and set that flag for thinkpads. > Another problem is that I don't know any other systems supporting charge > thresholds, and I don't want to dictate interfaces built for ThinkPads > as a generic way. All SBS batteries support it, which means most notebooks do. Whether they expose it or not, and whether their platform drivers know about it or not in order to expose these parameters to the kernel and to userspace, is a separate issue. Refer to: http://www.sbs-forum.org/ So yes, we can and should add these as generic *optional* parameters for generic batteries. How to best add it to the ACPI battery driver is something to think about. -- "One disk to rule them all, One disk to find them. One disk to bring them all and in the darkness grind them. In the Land of Redmond where the shadows lie." -- The Silicon Valley Tarot Henrique Holschuh ------------------------------------------------------------------------------ Rapidly troubleshoot problems before they affect your business. Most IT organizations don't have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk _______________________________________________ ibm-acpi-devel mailing list ibm-acpi-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.sourceforge.net/lists/listinfo/ibm-acpi-devel
