diff options
| author | Tejun Heo <tj@kernel.org> | 2012-12-22 02:57:11 +0100 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2012-12-28 22:40:16 +0100 |
| commit | 7998005bf7a706a1d6bb3cf2e9783b6437e4419d (patch) | |
| tree | eb257cf0fe17700582e7e9f300cc37422f84fd6c /drivers/tty/serial/max3100.c | |
| parent | x86/mce: don't use [delayed_]work_pending() (diff) | |
| download | linux-7998005bf7a706a1d6bb3cf2e9783b6437e4419d.tar.xz linux-7998005bf7a706a1d6bb3cf2e9783b6437e4419d.zip | |
tty/max3100: don't use [delayed_]work_pending()
There's no need to test whether a (delayed) work item in pending
before queueing, flushing or cancelling it. Most uses are unnecessary
and quite a few of them are buggy.
Remove unnecessary pending tests from max3100. Only compile tested.
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Jiri Slaby <jslaby@suse.cz>
Diffstat (limited to 'drivers/tty/serial/max3100.c')
| -rw-r--r-- | drivers/tty/serial/max3100.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/tty/serial/max3100.c b/drivers/tty/serial/max3100.c index 7ce3197087bb..dd6277eb5a38 100644 --- a/drivers/tty/serial/max3100.c +++ b/drivers/tty/serial/max3100.c @@ -179,8 +179,7 @@ static void max3100_work(struct work_struct *w); static void max3100_dowork(struct max3100_port *s) { - if (!s->force_end_work && !work_pending(&s->work) && - !freezing(current) && !s->suspending) + if (!s->force_end_work && !freezing(current) && !s->suspending) queue_work(s->workqueue, &s->work); } |