diff options
author | Colin Ian King <colin.king@canonical.com> | 2020-06-11 17:27:08 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2020-07-10 14:50:51 +0200 |
commit | 7ca78630a1ae082e6d4488884cd771ee4bb266ea (patch) | |
tree | 7016dcd62bff00bb3cd8fbd1e022628fdfb4f050 /drivers/char/mwave | |
parent | ttyprintk: remove redundant initialization of variable ret (diff) | |
download | linux-7ca78630a1ae082e6d4488884cd771ee4bb266ea.tar.xz linux-7ca78630a1ae082e6d4488884cd771ee4bb266ea.zip |
char/mwave: remove redundant initialization of variable bRC
The variable bRC is being initialized with a value that is never read
and it is being updated later with a new value. The initialization is
redundant and can be removed.
Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Link: https://lore.kernel.org/r/20200611152708.927344-1-colin.king@canonical.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/char/mwave')
-rw-r--r-- | drivers/char/mwave/smapi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/char/mwave/smapi.c b/drivers/char/mwave/smapi.c index 691f5898bb32..f8d79d393b69 100644 --- a/drivers/char/mwave/smapi.c +++ b/drivers/char/mwave/smapi.c @@ -126,7 +126,7 @@ static int smapi_request(unsigned short inBX, unsigned short inCX, int smapi_query_DSP_cfg(SMAPI_DSP_SETTINGS * pSettings) { - int bRC = -EIO; + int bRC; unsigned short usAX, usBX, usCX, usDX, usDI, usSI; static const unsigned short ausDspBases[] = { 0x0030, 0x4E30, 0x8E30, 0xCE30, @@ -497,7 +497,7 @@ exit_smapi_request_error: int smapi_set_DSP_power_state(bool bOn) { - int bRC = -EIO; + int bRC; unsigned short usAX, usBX, usCX, usDX, usDI, usSI; unsigned short usPowerFunction; |