power: remove POWER_SUPPLY_PROP_CAPACITY_LEVEL
authorAndres Salomon <dilinger@queued.net>
Wed, 12 Dec 2007 19:12:56 +0000 (14:12 -0500)
committerAnton Vorontsov <cbouatmailru@gmail.com>
Fri, 1 Feb 2008 23:42:59 +0000 (02:42 +0300)
commit8efe444038a205e79b38b7ad03878824901849a8
treeaf3fdf3d84059577ba86630103ae6adbb7d885a3
parent4d24473c435c7c3ad7b43e43b70cdb16aba25443
power: remove POWER_SUPPLY_PROP_CAPACITY_LEVEL

The CAPACITY_LEVEL stuff defines various levels of charge; however, what
is the difference between them?  What differentiates between HIGH and NORMAL,
LOW and CRITICAL, etc?

As it appears that these are fairly arbitrary, we end up making such policy
decisions in the kernel (or in hardware).  This is the sort of decision that
should be made in userspace, not in the kernel.

If the hardware does not support _CAPACITY and it cannot be easily calculated,
then perhaps the driver should register a custom CAPACITY_LEVEL attribute;
however, userspace should not become accustomed to looking for such a thing,
and we should certainly not encourage drivers to provide CAPACITY_LEVEL
stubs.

The following removes support for POWER_SUPPLY_PROP_CAPACITY_LEVEL.  The
OLPC battery driver is the only driver making use of this, so it's
removed from there as well.

Signed-off-by: Andres Salomon <dilinger@debian.org>
Signed-off-by: David Woodhouse <dwmw2@infradead.org>
Documentation/power_supply_class.txt
drivers/power/olpc_battery.c
drivers/power/power_supply_sysfs.c
include/linux/power_supply.h