power: intel_mid_battery: Remove deprecated create_singlethread_workqueue
authorBhaktipriya Shridhar <bhaktipriya96@gmail.com>
Sat, 13 Aug 2016 16:20:11 +0000 (21:50 +0530)
committerSebastian Reichel <sre@kernel.org>
Mon, 15 Aug 2016 22:54:37 +0000 (00:54 +0200)
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>
drivers/power/supply/intel_mid_battery.c

index 9fa4acc107caf1a7bc37d47f4630e44e2c98d839..dc7feef1bea459c922aeadd2ab002b0603dcf90a 100644 (file)
@@ -689,8 +689,7 @@ static int probe(int irq, struct device *dev)
        /* initialize all required framework before enabling interrupts */
        INIT_WORK(&pbi->handler, pmic_battery_handle_intrpt);
        INIT_DELAYED_WORK(&pbi->monitor_battery, pmic_battery_monitor);
-       pbi->monitor_wqueue =
-                       create_singlethread_workqueue(dev_name(dev));
+       pbi->monitor_wqueue = alloc_workqueue(dev_name(dev), WQ_MEM_RECLAIM, 0);
        if (!pbi->monitor_wqueue) {
                dev_err(dev, "%s(): wqueue init failed\n", __func__);
                retval = -ESRCH;