diff options
author | Bhaktipriya Shridhar <bhaktipriya96@gmail.com> | 2016-08-13 12:20:11 -0400 |
---|---|---|
committer | Sebastian Reichel <sre@kernel.org> | 2016-08-15 18:54:37 -0400 |
commit | 87f818b35c3007d1014541c07688ab29443a22c2 (patch) | |
tree | 5f4fa364acceae0fb086514f6b886c6a843e60b9 /drivers/power | |
parent | d8a69251fb58a756e2dd00cc5c7f1b54d383e203 (diff) |
power: intel_mid_battery: Remove deprecated create_singlethread_workqueue
The workqueue "monitor_wqueue" is used to monitor the PMIC battery status.
It queues a single work item (pbi->monitor_battery) and hence doesn't
require ordering. Hence, alloc_workqueue has been used to replace the
deprecated create_singlethread_workqueue instance.
Since PMIC battery status needs to be monitored for any change, the
WQ_MEM_RECLAIM flag has been set to ensure forward progress under memory
pressure.
Since there is a single work item, explicit concurrency
limit is unnecessary here.
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/intel_mid_battery.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/power/supply/intel_mid_battery.c b/drivers/power/supply/intel_mid_battery.c index 9fa4acc107ca..dc7feef1bea4 100644 --- a/drivers/power/supply/intel_mid_battery.c +++ b/drivers/power/supply/intel_mid_battery.c | |||
@@ -689,8 +689,7 @@ static int probe(int irq, struct device *dev) | |||
689 | /* initialize all required framework before enabling interrupts */ | 689 | /* initialize all required framework before enabling interrupts */ |
690 | INIT_WORK(&pbi->handler, pmic_battery_handle_intrpt); | 690 | INIT_WORK(&pbi->handler, pmic_battery_handle_intrpt); |
691 | INIT_DELAYED_WORK(&pbi->monitor_battery, pmic_battery_monitor); | 691 | INIT_DELAYED_WORK(&pbi->monitor_battery, pmic_battery_monitor); |
692 | pbi->monitor_wqueue = | 692 | pbi->monitor_wqueue = alloc_workqueue(dev_name(dev), WQ_MEM_RECLAIM, 0); |
693 | create_singlethread_workqueue(dev_name(dev)); | ||
694 | if (!pbi->monitor_wqueue) { | 693 | if (!pbi->monitor_wqueue) { |
695 | dev_err(dev, "%s(): wqueue init failed\n", __func__); | 694 | dev_err(dev, "%s(): wqueue init failed\n", __func__); |
696 | retval = -ESRCH; | 695 | retval = -ESRCH; |