diff options
author | Bhaktipriya Shridhar <bhaktipriya96@gmail.com> | 2016-08-13 12:21:23 -0400 |
---|---|---|
committer | Sebastian Reichel <sre@kernel.org> | 2016-08-15 18:54:37 -0400 |
commit | 9df82628265857e1c491c1ca0ace353e658457ea (patch) | |
tree | 0945463af02bbb09d92c93ac65055c373c74ae57 /drivers/power | |
parent | 87f818b35c3007d1014541c07688ab29443a22c2 (diff) |
power: ab8500_charger: Remove deprecated create_singlethread_workqueue
alloc_ordered_workqueue() with WQ_MEM_RECLAIM set replaces
deprecated create_singlethread_workqueue(). This is the identity
conversion.
The workqueue "charger_wq" is used for the IRQs and checking HW state of
the charger. It has been identity converted.
It has multiple work items viz usb_charger_attached_work, kick_wd_work,
check_vbat_work, check_hw_failure_work, usb_charger_attached_work,
ac_work, ac_charger_attached_work, attach_work and check_usbchgnotok_work,
which require execution ordering. Hence, a dedicated ordered workqueue
has been used here.
The WQ_MEM_RECLAIM flag has also been set to ensure
forward progress under memory pressure.
Signed-off-by: Bhaktipriya Shridhar <bhaktipriya96@gmail.com>
Signed-off-by: Sebastian Reichel <sre@kernel.org>
Diffstat (limited to 'drivers/power')
-rw-r--r-- | drivers/power/supply/ab8500_charger.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/power/supply/ab8500_charger.c b/drivers/power/supply/ab8500_charger.c index 30de5d42b26a..5cee9aa87aa3 100644 --- a/drivers/power/supply/ab8500_charger.c +++ b/drivers/power/supply/ab8500_charger.c | |||
@@ -3540,8 +3540,8 @@ static int ab8500_charger_probe(struct platform_device *pdev) | |||
3540 | di->usb_state.usb_current = -1; | 3540 | di->usb_state.usb_current = -1; |
3541 | 3541 | ||
3542 | /* Create a work queue for the charger */ | 3542 | /* Create a work queue for the charger */ |
3543 | di->charger_wq = | 3543 | di->charger_wq = alloc_ordered_workqueue("ab8500_charger_wq", |
3544 | create_singlethread_workqueue("ab8500_charger_wq"); | 3544 | WQ_MEM_RECLAIM); |
3545 | if (di->charger_wq == NULL) { | 3545 | if (di->charger_wq == NULL) { |
3546 | dev_err(di->dev, "failed to create work queue\n"); | 3546 | dev_err(di->dev, "failed to create work queue\n"); |
3547 | return -ENOMEM; | 3547 | return -ENOMEM; |