Commit ae3a1b46 authored by Carlos Corbacho's avatar Carlos Corbacho Committed by Len Brown

acer-wmi: Remove private workqueue

As per Dmitry Torokhov's suggestion, acer-wmi doesn't need a private
workqueue, so remove it.
Signed-off-by: default avatarCarlos Corbacho <carlos@strangeworlds.co.uk>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: default avatarLen Brown <len.brown@intel.com>
parent a66bfa7a
...@@ -930,8 +930,6 @@ static void acer_backlight_exit(void) ...@@ -930,8 +930,6 @@ static void acer_backlight_exit(void)
/* /*
* Rfkill devices * Rfkill devices
*/ */
static struct workqueue_struct *rfkill_workqueue;
static void acer_rfkill_update(struct work_struct *ignored); static void acer_rfkill_update(struct work_struct *ignored);
static DECLARE_DELAYED_WORK(acer_rfkill_work, acer_rfkill_update); static DECLARE_DELAYED_WORK(acer_rfkill_work, acer_rfkill_update);
static void acer_rfkill_update(struct work_struct *ignored) static void acer_rfkill_update(struct work_struct *ignored)
...@@ -952,8 +950,7 @@ static void acer_rfkill_update(struct work_struct *ignored) ...@@ -952,8 +950,7 @@ static void acer_rfkill_update(struct work_struct *ignored)
RFKILL_STATE_SOFT_BLOCKED); RFKILL_STATE_SOFT_BLOCKED);
} }
queue_delayed_work(rfkill_workqueue, &acer_rfkill_work, schedule_delayed_work(&acer_rfkill_work, round_jiffies_relative(HZ));
round_jiffies_relative(HZ));
} }
static int acer_rfkill_set(void *data, enum rfkill_state state) static int acer_rfkill_set(void *data, enum rfkill_state state)
...@@ -1018,17 +1015,7 @@ static int acer_rfkill_init(struct device *dev) ...@@ -1018,17 +1015,7 @@ static int acer_rfkill_init(struct device *dev)
} }
} }
rfkill_workqueue = create_singlethread_workqueue("rfkill_workqueue"); schedule_delayed_work(&acer_rfkill_work, round_jiffies_relative(HZ));
if (!rfkill_workqueue) {
if (has_cap(ACER_CAP_BLUETOOTH)) {
kfree(bluetooth_rfkill->data);
rfkill_unregister(bluetooth_rfkill);
}
kfree(wireless_rfkill->data);
rfkill_unregister(wireless_rfkill);
return -ENOMEM;
}
queue_delayed_work(rfkill_workqueue, &acer_rfkill_work, HZ);
return 0; return 0;
} }
...@@ -1036,7 +1023,6 @@ static int acer_rfkill_init(struct device *dev) ...@@ -1036,7 +1023,6 @@ static int acer_rfkill_init(struct device *dev)
static void acer_rfkill_exit(void) static void acer_rfkill_exit(void)
{ {
cancel_delayed_work_sync(&acer_rfkill_work); cancel_delayed_work_sync(&acer_rfkill_work);
destroy_workqueue(rfkill_workqueue);
kfree(wireless_rfkill->data); kfree(wireless_rfkill->data);
rfkill_unregister(wireless_rfkill); rfkill_unregister(wireless_rfkill);
if (has_cap(ACER_CAP_BLUETOOTH)) { if (has_cap(ACER_CAP_BLUETOOTH)) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment