The AB8505 does not have the same address for USB link-status
as has ab8500. Add AB8505_USB_LINK_STATUS and code to switch
to correct constant.
Signed-off-by: NHakan Berg <hakan.berg@stericsson.com>
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Reviewed-by: NMian Yousaf KAUKAB <mian.yousaf.kaukab@stericsson.com>
Reviewed-by: NMarcus COOPER <marcus.xm.cooper@stericsson.com>
Reviewed-by: NRabin VINCENT <rabin.vincent@stericsson.com>

Do not set higher current in stepping functionality if VBUS is dropping.
After VBUS has dropped try to set current once again. If dropping again
then we have found the maximum capability of the charger.
Signed-off-by: NLee Jones <lee.jones@linaro.org>

Fix for charging not getting enabled in charge only mode by
external charger.
Signed-off-by: NLee Jones <lee.jones@linaro.org>

Due to a bug in some AB8500 ASICs charger removal cannot always
be detected if the removal and reinsertion is done to close in time.
This patch detects above described case and handles the situation
so that charging will be kept turned on.
Signed-off-by: NLee Jones <lee.jones@linaro.org>

Enforce the white space character after 'if'.
Signed-off-by: NLee Jones <lee.jones@linaro.org>

Add 2.7v, 2.9v, 3.0v, 3.2v and 3.3v charging voltages for backup
battery. Before that only 2.5v, 2.6v, 2.8v, 3.1v were available.
Signed-off-by: NYang QU <yang.qu@stericsson.com>
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Reviewed-by: NMaxime COQUELIN <maxime.coquelin@stericsson.com>
Reviewed-by: NMarcus COOPER <marcus.xm.cooper@stericsson.com>
Tested-by: NXiao Mei ZHANG <xiaomei.zhang@stericsson.com>

If charging is started before USB enumeration of an Accessory
Charger Adapter has finished, the AB8500 will generate a
VBUS_ERROR. This in turn results in timeouts and delays the
enumeration with around 15 seconds. This patch delays the
charging and then ramps currents slowly to avoid VBUS errors.
The delay allows the enumeration to have finished before
charging is turned on.
Signed-off-by: NMartin Sjoblom <martin.w.sjoblom@stericsson.com>
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Reviewed-by: NJonas ABERG <jonas.aberg@stericsson.com>
Tested-by: NJonas ABERG <jonas.aberg@stericsson.com>

None of the aforementioned components have their own dedicated
platform data structures anymore. Instead they have all been
merged into one big Battery Management container. Let's remove
them and place all the nice newly added attributes into the core
container.
Signed-off-by: NLee Jones <lee.jones@linaro.org>

Do not touch the VBUSOVV in USBCHTRL2 when running on AB8505.
Signed-off-by: NJonas Aaberg <jonas.aberg@stericsson.com>
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Reviewed-by: NMarcus COOPER <marcus.xm.cooper@stericsson.com>
Tested-by: NMian Yousaf KAUKAB <mian.yousaf.kaukab@stericsson.com>

Flush all workqueues at suspend time to avoid suspending during work.
Signed-off-by: NJonas Aaberg <jonas.aberg@stericsson.com>
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Reviewed-by: NMarcus COOPER <marcus.xm.cooper@stericsson.com>

Accessing autopower register fails on the AB8505 and ab9540 as
the fallback software control register has moved.
Signed-off-by: NMarcus Cooper <marcus.xm.cooper@stericsson.com>
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Reviewed-by: NMattias WALLIN <mattias.wallin@stericsson.com>
Reviewed-by: NNicolas GUION <nicolas.guion@stericsson.com>
Reviewed-by: NJonas ABERG <jonas.aberg@stericsson.com>
Tested-by: NJonas ABERG <jonas.aberg@stericsson.com>

Allow a user to check on AB8500 charging status from debugfs.
Signed-off-by: NLee Jones <lee.jones@linaro.org>

AB and PMxxx doesn't have same watchdog refresh period. Add watchdog
to refresh period parameters in x500 charger structure, this should
kick watchdog every 30sec. The AC charging should also kick both
pm2xxx and the AB charger watchdog.
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>
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>

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>

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>

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>

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>

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>

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>

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>

Now that we always pass platform specific battery management data
through platform_data instead of obtaining it via different means
depending the way be boot the system (DT or ATAGs); we need to
re-jiggle the way we acquire it in the driver start-up functions.
Now it is wrong for it to be missing, but we still allow Device
Tree code to fiddle with it once we've confirmed it's there.
Signed-off-by: NLee Jones <lee.jones@linaro.org>

Currently the AB8500 battery management subsystem receives platform
specific information via two different means depending on how the
platform is booted. If DT is not enabled, a reference to a *_bm_data
data structure containing each platform specific attribute is passed
though platform_data. However, if DT is enabled, then platform_data
is empty and the reference is gained though a DT specific probe
function. There are two issues here 1) the same reference is
being collected each time and 2) the DT way doesn't allow any
provisions to select different platform specific attributes, which
kind of defeats the object.

Cc: Samuel Ortiz <sameo@linux.intel.com>
Signed-off-by: NLee Jones <lee.jones@linaro.org>

The platform specific battery management configuration data structure
is currently called 'bat' short for 'battery'; however, it contains
information for all components of the battery management group, rather
than information pertaining to the battery itself - there are other
structures for that. So, in keeping with its structure namesake
'abx500_bm_data', we rename it to 'bm' here. Using similar logic,
we're also renaming 'bmdevs_of_probe' to the more device specific
'ab8500_bm_of_probe'.
Signed-off-by: NLee Jones <lee.jones@linaro.org>

There is no state machine in the AB to step up/down the charger
current to avoid dips and spikes on VBUS and VBAT when charging
is started. Instead this is implemented in SW.
Signed-off-by: NJohan Bjornstedt <johan.bjornstedt@stericsson.com>
Signed-off-by: NMattias Wallin <mattias.wallin@stericsson.com>
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Reviewed-by: NKarl KOMIEROWSKI <karl.komierowski@stericsson.com>

CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: NBill Pemberton <wfp5p@virginia.edu>
Cc: Anton Vorontsov <cbou@mail.ru>
Cc: David Woodhouse <dwmw2@infradead.org>
Acked-by: NMark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: NBill Pemberton <wfp5p@virginia.edu>
Cc: Anton Vorontsov <cbou@mail.ru>
Cc: David Woodhouse <dwmw2@infradead.org>
Acked-by: NMark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: NBill Pemberton <wfp5p@virginia.edu>
Cc: Anton Vorontsov <cbou@mail.ru>
Cc: David Woodhouse <dwmw2@infradead.org>
Acked-by: NMark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch adds device tree support for ab8500-charger driver
Signed-off-by: NRajanikanth H.V <rajanikanth.hv@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton.vorontsov@linaro.org>

- This patch adds device tree support for fuelgauge driver
- optimize bm devices platform_data usage and of_probe(...)
  Note: of_probe() routine for battery managed devices is made
  common across all bm drivers.
- test status:
  - interrupt numbers assigned differs between legacy and FDT mode.
Signed-off-by: NRajanikanth H.V <rajanikanth.hv@stericsson.com>
Signed-off-by: NAnton Vorontsov <anton.vorontsov@linaro.org>

Initialize return variable before exiting on an error path.

A simplified version of the semantic match that finds this problem is as
follows: (http://coccinelle.lip6.fr/)

// <smpl>
(
if@p1 (\(ret < 0\|ret != 0\))
 { ... return ret; }
|
ret@p1 = 0
)
... when != ret = e1
    when != &ret
*if(...)
{
  ... when != ret = e2
      when forall
 return ret;
}

// </smpl>
Signed-off-by: NJulia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: NAnton Vorontsov <anton.vorontsov@linaro.org>

Initalizers for deferrable delayed_work are confused.

* __DEFERRED_WORK_INITIALIZER()
* DECLARE_DEFERRED_WORK()
* INIT_DELAYED_WORK_DEFERRABLE()

Rename them to

* __DEFERRABLE_WORK_INITIALIZER()
* DECLARE_DEFERRABLE_WORK()
* INIT_DEFERRABLE_WORK()

This patch doesn't cause any functional changes.
Signed-off-by: NTejun Heo <tj@kernel.org>

usb_get_phy will return -ENODEV if it's not able to find the phy. Hence
fixed all the callers of usb_get_phy to check for this error condition
instead of relying on a non-zero value as success condition.
Signed-off-by: NKishon Vijay Abraham I <kishon@ti.com>
Signed-off-by: NFelipe Balbi <balbi@ti.com>

Add a linked list for keeping multiple PHY instances with different
types so that we can have separate USB2 and USB3 PHYs on one single
board. _get_phy_ has been changed so that the controller gets
the transceiver by type. _remove_phy_ has been added to let the phy
be removed from the phy list.
Signed-off-by: NKishon Vijay Abraham I <kishon@ti.com>
Signed-off-by: NFelipe Balbi <balbi@ti.com>

_transceiver() in otg.c is replaced with _phy. usb_set_transceiver is
replaced with usb_add_phy to make it similar to other usb standard
function names like usb_add_hcd.
Signed-off-by: NKishon Vijay Abraham I <kishon@ti.com>
Signed-off-by: NFelipe Balbi <balbi@ti.com>

These patches clean up some ugliness and brings the variable
initialisation formatting more into line with other drivers.
Signed-off-by: NLee Jones <lee.jones@linaro.org>
Signed-off-by: NAnton Vorontsov <anton.vorontsov@linaro.org>

If no platform data at all is supplied the driver crashes,
extend the checks to be more careful so we can compile in the
driver and boot also without platform data present.
Acked-by: NArun Murthy <arun.murthy@stericsson.com>
Signed-off-by: NLinus Walleij <linus.walleij@linaro.org>
Signed-off-by: NAnton Vorontsov <anton.vorontsov@linaro.org>

The patch fixes the following sparse warning:

drivers/power/ab8500_charger.c:1619:6: warning: symbol 'ab8500_charger_detect_usb_type_work' was not declared. Should it be static?
drivers/power/abx500_chargalg.c:1709:24: warning: symbol 'abx500_chargalg_sysfs_ops' was not declared. Should it be static?
drivers/power/ab8500_fg.c:2328:24: warning: symbol 'ab8500_fg_sysfs_ops' was not declared. Should it be static?
Signed-off-by: NAnton Vorontsov <anton.vorontsov@linaro.org>

This patch fixes the following build errors:

ab8500_charger.c: In function 'ab8500_charger_remove':
ab8500_charger.c:2519:2: error: implicit declaration of function 'otg_unregister_notifier' [-Werror=implicit-function-declaration]
ab8500_charger.c:2520:2: error: implicit declaration of function 'otg_put_transceiver' [-Werror=implicit-function-declaration]
ab8500_charger.c: In function 'ab8500_charger_probe':
ab8500_charger.c:2688:2: error: implicit declaration of function 'otg_get_transceiver' [-Werror=implicit-function-declaration]
ab8500_charger.c:2688:10: warning: assignment makes pointer from integer without a cast [enabled by default]
ab8500_charger.c:2695:2: error: implicit declaration of function 'otg_register_notifier' [-Werror=implicit-function-declaration]
Signed-off-by: NAnton Vorontsov <anton.vorontsov@linaro.org>