diff options
author | Tobin C. Harding <me@tobin.cc> | 2017-03-21 13:37:10 +1100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-21 08:56:07 +0100 |
commit | c133b9bd8ff18323f57859c5add3be9e872224c3 (patch) | |
tree | 6b3a440cf5b26e3b7ca7c4bc169e7d65739a2e58 /drivers/staging/ks7010/ks7010_sdio.c | |
parent | f283dd6901cad0d823cf469e71db5700d456d1f6 (diff) |
staging: ks7010: remove non-zero comparison
Comparison, does not equal zero, is redundant
'if (foo != 0)' is equal to 'if (foo)'
Typical kernel coding style is to use the shorter form.
Remove unnecessary non-zero comparison.
Signed-off-by: Tobin C. Harding <me@tobin.cc>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/ks7010/ks7010_sdio.c')
-rw-r--r-- | drivers/staging/ks7010/ks7010_sdio.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/staging/ks7010/ks7010_sdio.c b/drivers/staging/ks7010/ks7010_sdio.c index 40ec028de9ac..8823e934aff7 100644 --- a/drivers/staging/ks7010/ks7010_sdio.c +++ b/drivers/staging/ks7010/ks7010_sdio.c @@ -61,7 +61,7 @@ static int ks7010_sdio_read(struct ks_wlan_private *priv, unsigned int address, else /* CMD53 multi-block transfer */ rc = sdio_memcpy_fromio(card->func, buffer, address, length); - if (rc != 0) + if (rc) DPRINTK(1, "sdio error=%d size=%d\n", rc, length); return rc; @@ -80,7 +80,7 @@ static int ks7010_sdio_write(struct ks_wlan_private *priv, unsigned int address, else /* CMD53 */ rc = sdio_memcpy_toio(card->func, address, buffer, length); - if (rc != 0) + if (rc) DPRINTK(1, "sdio error=%d size=%d\n", rc, length); return rc; |