staging: speakup: Replace del_timer with del_timer_sync
authorAmitoj Kaur Chawla <amitoj1606@gmail.com>
Sun, 6 Mar 2016 01:15:07 +0000 (06:45 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 12 Mar 2016 06:09:09 +0000 (22:09 -0800)
commitc943e13bd2bf4c344c3fe43ef39f07b57eff715e
treef0e98151b69e791ed9737c7055adf834d98f03a1
parentb33061fbcdb959a5cee024b052b6a64d7ccd0654
staging: speakup: Replace del_timer with del_timer_sync

Use del_timer_sync to ensure timer is stopped on all CPUs before
the driver exists and the timer should not run when the module is
being removed. Since the timer is not called from an interrupt
context, this change is safe and will not cause deadlock.

The Coccinelle semantic patch used to make this change is as
follows:
// <smpl>
@r@
declarer name module_exit;
identifier ex;
@@

module_exit(ex);

@@
identifier r.ex;
@@

ex(...) {
  <...
- del_timer
+ del_timer_sync
    (...)
  ...>
}
// </smpl>

Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/speakup/main.c