Fix the way interrupts are handled within the PM2301 charging driver.
Signed-off-by: NRajkumar Kasirajan <rajkumar.kasirajan@stericsson.com>
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Reviewed-by: NOlivier CLERGEAUD <olivier.clergeaud@stericsson.com>
Reviewed-by: NMarcus COOPER <marcus.xm.cooper@stericsson.com>
Reviewed-by: NMichel JAOUEN <michel.jaouen@stericsson.com>
Tested-by: NMichel JAOUEN <michel.jaouen@stericsson.com>

Also shift the file description to a more suitable location.
Signed-off-by: NLee Jones <lee.jones@linaro.org>

When the charging safety timer elapses, the battery health is
shown as "Good". This is misleading and also makes it difficult
to distinguish issues relating to discharging despite the fact
that the charger is still connected.

When in actual fact a safety timer elapse is an indication of a
fault in the battery. Here we make this clearer by reporting
POWER_SUPPLY_HEALTH_UNSPEC_FAILURE instead.
Signed-off-by: NLee Jones <lee.jones@linaro.org>

Kicks the watchdog so charging will not stop.
Signed-off-by: NLee Jones <lee.jones@linaro.org>

Add AB9540 and AB8505 support to ABx500 BTEMP driver.
Signed-off-by: NRajkumar Kasirajan <rajkumar.kasirajan@stericsson.com>
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Reviewed-by: NMichel JAOUEN <michel.jaouen@stericsson.com>
Reviewed-by: NMarcus COOPER <marcus.xm.cooper@stericsson.com>
Reviewed-by: NJonas ABERG <jonas.aberg@stericsson.com>
Tested-by: NMichel JAOUEN <michel.jaouen@stericsson.com>
Tested-by: NJonas ABERG <jonas.aberg@stericsson.com>

Changed conditions for restarting low battery measurements counter
and adjusted the interval between measurements to avoid RF burst
induced voltage drops, and to shorten time to decide to shut down.
Signed-off-by: NHakan Berg <hakan.berg@stericsson.com>
Signed-off-by: NMartin Bergstrom <martin.bergstrom@stericsson.com>
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Reviewed-by: NJonas ABERG <jonas.aberg@stericsson.com>
Reviewed-by: NMarcus COOPER <marcus.xm.cooper@stericsson.com>

AB8500 refuses to start charging when some types of non standard
chargers are connected. This change force the AB to start charging.
Signed-off-by: NHenrik Sölver <henrik.solver@stericsson.com>
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Reviewed-by: NYvan FILLION <yvan.fillion@stericsson.com>
Reviewed-by: NJonas ABERG <jonas.aberg@stericsson.com>
Tested-by: NYvan FILLION <yvan.fillion@stericsson.com>

AC charger driver for the DB9540 based platforms.
Signed-off-by: NRajkumar Kasirajan <rajkumar.kasirajan@stericsson.com>
Signed-off-by: NLoic Pallardy <loic.pallardy@stericsson.com>
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Reviewed-by: NMichel JAOUEN <michel.jaouen@stericsson.com>
Tested-by: NMichel JAOUEN <michel.jaouen@stericsson.com>

GPLv2+ is not a valid license string. Replace it with one that is.
Signed-off-by: NAndrew Lunn <andrew@lunn.ch>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

As HZ is a full-second, there is little point in rounding it.
Let's save a few cycles by using HZ directly.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

When calculating the average current the nominator will
overflow when the charging current is high.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NHenrik Sölver <henrik.solver@stericsson.com>
Reviewed-by: NPar-Olof HAKANSSON <par-olof.hakansson@stericsson.com>
Reviewed-by: NJonas ABERG <jonas.aberg@stericsson.com>
Tested-by: NPar-Olof HAKANSSON <par-olof.hakansson@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

A Legacy USB charger should be handled directly by the charger
driver.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NMarcus Cooper <marcus.xm.cooper@stericsson.com>
Reviewed-by: NKarl KOMIEROWSKI <karl.komierowski@stericsson.com>
Reviewed-by: NJonas ABERG <jonas.aberg@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Cleanup of the ab8500_charger driver.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NMarcus Cooper <marcus.xm.cooper@stericsson.com>
Reviewed-by: NKarl KOMIEROWSKI <karl.komierowski@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Add support for the battery over-voltage situation
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NHakan Berg <hakan.berg@stericsson.com>
Reviewed-by: NKarl KOMIEROWSKI <karl.komierowski@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

BTEMP thermal compensation factor times 10 is applied in two places, probe
and get_property. Removed from probe.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NHakan Berg <hakan.berg@stericsson.com>
Reviewed-by: NKarl KOMIEROWSKI <karl.komierowski@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Round the capacity values for better enduser experience.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: Npender01 <peter.enderborg@stericsson.com>
Reviewed-by: NJonas ABERG <jonas.aberg@stericsson.com>
Tested-by: NMarcus COOPER <marcus.xm.cooper@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

The Travel and Carkit adapter should be handled directly by
the charger driver.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NMarcus Cooper <marcus.xm.cooper@stericsson.com>
Reviewed-by: NJonas ABERG <jonas.aberg@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Ignore the low btemp interrupts for ab8500 3.0 and 3.3
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NHakan Berg <hakan.berg@stericsson.com>
Reviewed-by: NJonas ABERG <jonas.aberg@stericsson.com>
Tested-by: NMarcus COOPER <marcus.xm.cooper@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

The Linux Power Supply framework expects battery temperatures to have
a resolution of 0.1 degree Celsius; however, the AB8500 btemp driver
supplies a battery temperature resolution of 1 degree Celsius. We
therefore have to use a factor 10 on the measured values.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

In case of time out error IRQ needs to be disabled
otherwise we will get unbalanced enable/disable pairs.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NRickard Andersson <rickard.andersson@stericsson.com>
Reviewed-by: NJonas ABERG <jonas.aberg@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Today the battery recharge is determined with a voltage threshold. This
voltage threshold is only valid when the battery is relaxed. In charging
algorithm the voltage read is the loaded battery voltage and no
compensation is done to get the relaxed voltage. When maintenance
charging is not selected, this makes the recharging condition to almost
immediately activate when there is a discharge present on the battery.

Depending on which vendor the battery comes from this behavior can wear
out the battery much faster than normal.

The fuelgauge driver is responsible to monitor the actual battery
capacity and is able to estimate the remaining capacity. It is better to
use the remaining capacity as a limit to determine when battery should
be recharged.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NMarcus Cooper <marcus.xm.cooper@stericsson.com>
Reviewed-by: NHakan BERG <hakan.berg@stericsson.com>
Reviewed-by: NJonas ABERG <jonas.aberg@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Gracefully handle gpadc conversion errors.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NJonas Aaberg <jonas.aberg@stericsson.com>
Reviewed-by: NJohan BJORNSTEDT <johan.bjornstedt@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Doing so provides a greater degree of accuracy when dealing with
time-frames between 1us and 20ms. msleep() is only accurate for
wake-ups greater than 20ms.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NJonas ABERG <jonas.aberg@stericsson.com>
Reviewed-by: NJohan BJORNSTEDT <johan.bjornstedt@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Add two new work queues to provide USB and AC charger disconnect
detection.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

When enabling the CCEOC irq we might get false interrupt
from ab8500-driver due to the latched value will be saved
and interpreted as an IRQ when enabled
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NJohan Bjornstedt <johan.bjornstedt@stericsson.com>
Signed-off-by: NHenrik Solver <henrik.solver@stericsson.com>
Reviewed-by: NKarl KOMIEROWSKI <karl.komierowski@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Rename the ab8500_power_loss_handling function to a more
descriptive name ab8500_enable_disable_sw_fallback
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NRobert Marklund <robert.marklund@stericsson.com>
Reviewed-by: NPar-Olof HAKANSSON <par-olof.hakansson@stericsson.com>
Reviewed-by: NKarl KOMIEROWSKI <karl.komierowski@stericsson.com>
Tested-by: NPar-Olof HAKANSSON <par-olof.hakansson@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Avoid accessing vc_tbl_ref[3], which is one past the end of that array, in
da9052_determine_vc_tbl_index(), by adjusting the loop bound.

(Hint: there is 'i + 1' inside the loop.)
Signed-off-by: NNickolai Zeldovich <nickolai@csail.mit.edu>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

use devm_kzalloc and no need of error path and unload frees.
Signed-off-by: NDevendra Naga <devendra.aaru@gmail.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Use 'platform_device' in kernel messages rather than i2c client device
node. lp8788_update_charger_params() needs additional argument, 'pdev'.
Then, remove unnecessary lp8788 private data in lp8788_irq_register().
Signed-off-by: NMilo(Woogyom) Kim <milo.kim@ti.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

The lp8788-charger is a platform driver of lp8788-mfd. The platform device
is allocated when mfd_add_devices() is called in lp8788-mfd. On the other
hand, 'lp->dev' is the i2c client device.

Therefore, this 'platform_device' is a proper parent device in case of
resource managed mem alloc and device kernel message.
Signed-off-by: NMilo(Woogyom) Kim <milo.kim@ti.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Some devices, Buffalo Linkstation LS-XHL and LS-CHLv2 for example,
power-off by restarting to letting u-boot hold the SoC until the user
presses a key. Add a generic driver to implement this. It binds a function
to pm_power_off, which calls arm_pm_restart.
Signed-off-by: NAndrew Lunn <andrew@lunn.ch>
Tested-by: NMichael Walle <michael@walle.cc>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

The QNAP NAS boxes have a microcontroller attached to the SoCs second
serial port. By sending it a simple command, it will turn the power for
the board off. This driver registers a function for pm_power_off to send
such a command.
Signed-off-by: NAndrew Lunn <andrew@lunn.ch>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Cc: Anton Vorontsov <cbou@mail.ru>
Cc: Bryan Wu <cooloney@gmail.com>
Cc: Jonathan Cameron <jic23@cam.ac.uk>
Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: Samuel Ortiz <sameo@linux.intel.com>
Signed-off-by: NMilo(Woogyom) Kim <milo.kim@ti.com>
Acked-by: NBryan Wu <cooloney@gmail.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

Cc: Anton Vorontsov <cbou@mail.ru>
Cc: David Woodhouse <dwmw2@infradead.org>
Signed-off-by: NMilo(Woogyom) Kim <milo.kim@ti.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

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/):

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

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: NJulia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

devm_regulator_get() is device managed and makes error handling and code
cleanup simpler.
Signed-off-by: NSachin Kamat <sachin.kamat@linaro.org>
Acked-by: NArun Murthy <arun.murthy@stericsson.com>
Acked-by: NLinus Walleij <linus.walleij@linaro.org>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

commit a66f59ba

    bq27x00_battery: Add support for BQ27425 chip

introduced 2 bugs.

1/ 'chip' was set to BQ27425 unconditionally - breaking support for
   other devices;

2/ BQ27425 does not support cycle count, how the code still tries to
   get the cycle count for BQ27425, and now does it twice for other chips.
Signed-off-by: NNeilBrown <neilb@suse.de>
Cc: Saranya Gopal <saranya.gopal@intel.com>
Cc: stable@vger.kernel.org
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

As most of the charger chips come with two kinds of safety features
related to timing:

1. Watchdog Timer (interms of seconds/mins)
2. Safety Timer (interms of hours)

This patch adds these to fault causes in POWER_SUPPLY_PROP_HEALTH_* enums
so that whenever there is either watchdog timeout or safety timer timeout
driver could notify the user space accurately about the fault and will
also be helpful for debug.
Signed-off-by: NRamakrishna Pallala <ramakrishna.pallala@intel.com>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

devm_kzalloc is device managed and makes error handling and code cleanup a
bit simpler.
Signed-off-by: NSachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>

module_i2c_driver() makes the code simpler by eliminating module_init and
module_exit calls.
Signed-off-by: NSachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: NAnton Vorontsov <anton@enomsg.org>