Input: mc13783_ts - remove deprecated create_singletheread_workqueue
authorBhaktipriya Shridhar <bhaktipriya96@gmail.com>
Tue, 23 Aug 2016 20:47:03 +0000 (13:47 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Thu, 25 Aug 2016 17:11:00 +0000 (10:11 -0700)
commit62e5147568968e0679ab413c7cb2ef35c8a0a4e3
tree3f840b28a599839cdd5031c7d7e715fcc003d154
parent24dde60f564b1e2eb588761ba797c7b99a374860
Input: mc13783_ts - remove deprecated create_singletheread_workqueue

The workqueue "workqueue" has a single workitem(&priv->work) and hence
doesn't require ordering. Also, it is not being used on a memory reclaim
path. Hence, the singlethreaded workqueue has been replaced with the use
of system_wq.

System workqueues have been able to handle high level of concurrency
for a long time now and hence it's not required to have a singlethreaded
workqueue just to gain concurrency. Unlike a dedicated per-cpu workqueue
created with create_singlethread_workqueue(), system_wq allows multiple
work items to overlap executions even on the same CPU; however, a
per-cpu workqueue doesn't have any CPU locality or global ordering
guarantee unless the target CPU is explicitly specified and thus the
increase of local concurrency shouldn't make any difference.

Workitem is sync cancelled in mc13783_ts_remove() to ensure that there
are no workitems pending when the driver is disconnected.

Signed-off-by: Bhaktipriya Shridhar <bhaktipriya96@gmail.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/touchscreen/mc13783_ts.c