linux-toradex.git/drivers/power/88pm860x_battery.c, branch v4.0.2

power: drop owner assignment from platform_drivers

2014-10-20T14:21:27+00:00

A platform_driver does not need to set an owner, it will be populated by the
driver core.

Signed-off-by: Wolfram Sang

88pm860x_battery: Remove unnecessary platform_set_drvdata()

2013-06-07T00:12:37+00:00

The driver core clears the driver data to NULL after device_release or on
probe failure, since commit 0998d0631001288a5974afc0b2a5f568bcdecb4d
(device-core: Ensure drvdata = NULL when no driver is bound). Thus, it is
not needed to manually clear the device driver data to NULL.

Signed-off-by: Jingoo Han 
Signed-off-by: Anton Vorontsov

88pm860x_battery: Eliminate possible references to released resources

2013-01-06T20:46:23+00:00

devm_kzalloc should not be followed by kfree, as this results in a double
free.  The problem was found using the following semantic match
(http://coccinelle.lip6.fr/):

// 
@@
expression x,e;
@@
x = devm_kzalloc(...)
... when != x = e
?-kfree(x,...);
// 

Furthermore, in the remove function, the calls to free_irq are moved up to
prevent a possible reference in the interrupt handler to resources freed by
power_supply_unregister.

Signed-off-by: Julia Lawall 
Signed-off-by: Anton Vorontsov

power: remove use of __devexit

2012-11-28T19:43:22+00:00

CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.

Signed-off-by: Bill Pemberton 
Cc: Anton Vorontsov 
Cc: David Woodhouse 
Acked-by: Mark Brown 
Signed-off-by: Greg Kroah-Hartman

power: remove use of __devinit

2012-11-28T19:43:22+00:00

CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.

Signed-off-by: Bill Pemberton 
Cc: Anton Vorontsov 
Cc: David Woodhouse 
Acked-by: Mark Brown 
Signed-off-by: Greg Kroah-Hartman

power: remove use of __devexit_p

2012-11-28T19:43:22+00:00

CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.

Signed-off-by: Bill Pemberton 
Cc: Anton Vorontsov 
Cc: David Woodhouse 
Acked-by: Mark Brown 
Signed-off-by: Greg Kroah-Hartman

88pm860x_battery and charger: Fix a few post-merge issues

2012-09-20T22:33:12+00:00

Sparse complains:

  CHECK   drivers/power/88pm860x_battery.c
drivers/power/88pm860x_battery.c:128:5: warning: symbol 'array_soc' was not declared. Should it be static?
  CHECK   drivers/power/88pm860x_charger.c
drivers/power/88pm860x_charger.c:640:3: warning: symbol 'pm860x_irq_descs' was not declared. Should it be static?
  CHECK   drivers/mfd/88pm860x-core.c
drivers/mfd/88pm860x-core.c:803:53: warning: incorrect type in assignment (different base types)
drivers/mfd/88pm860x-core.c:803:53:    expected struct charger_regulator *charger_regulators
drivers/mfd/88pm860x-core.c:803:53:    got struct regulator_bulk_data static [toplevel] *

The issues are minor, except for the last one. We seemed to use
'regulator_bulk_data' struct (just as charger manager documentation
wrongly tells us), but in real it should have been
'struct charger_regulator'. The only reason that it worked is
because both 'supply' and 'regulator_name' struct members are the
first in these structs. :-)

Signed-off-by: Anton Vorontsov

power_supply: Enable battery-charger for 88pm860x

2012-09-20T22:32:55+00:00

There are charger and battery measurement feature for 88pm860x PMIC.

For charger, it can support pre-charge with small current when battery is
nearly exausted and then changed into fast-charge with CC&CV mode.

For battery monitor, it can support battery measurement such as
vbat,vsys,vchg and ibat etc,it can aslo accumulating the Coulomb value
charged or discharged from battery based on Conlomb Counter, we use it
to estimate battery capacity.

Signed-off-by: Jett.Zhou 
Signed-off-by: Anton Vorontsov