diff options
| author |   Amitoj Kaur Chawla <amitoj1606@gmail.com> | 2016-02-24 20:54:28 +0530 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-03-05 14:48:04 -0800 |
| commit | d6e29ca1f6cf566f67012dce269b5f9ba72a1659 (patch) | |
| tree | 38b90be386580dc896be623a1cd69b07f992e211 /drivers/staging/speakup/synth.c | |
| parent | staging: speakup: remove multiple assignments (diff) | |
| download | linux-dev-d6e29ca1f6cf566f67012dce269b5f9ba72a1659.tar.xz linux-dev-d6e29ca1f6cf566f67012dce269b5f9ba72a1659.zip | |
staging: speakup: Remove unnecessary test in if condition
Remove unnecessary test on synth->alive since it has already been
tested previously.
This fixes the following smatch warning:
drivers/staging/speakup/synth.c:182 spk_synth_is_alive_restart() warn:
we tested 'synth->alive' before and it was 'false'
Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/speakup/synth.c')
| -rw-r--r-- | drivers/staging/speakup/synth.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/speakup/synth.c b/drivers/staging/speakup/synth.c index 01eddab93c66..4f462c35fdd9 100644 --- a/drivers/staging/speakup/synth.c +++ b/drivers/staging/speakup/synth.c @@ -179,7 +179,7 @@ int spk_synth_is_alive_restart(struct spk_synth *synth) { if (synth->alive) return 1; - if (!synth->alive && spk_wait_for_xmitr() > 0) { + if (spk_wait_for_xmitr() > 0) { /* restart */ synth->alive = 1; synth_printf("%s", synth->init); |