diff options
author | Geyslan G. Bem <geyslan@gmail.com> | 2015-12-11 10:46:42 +0100 |
---|---|---|
committer | Johan Hovold <johan@kernel.org> | 2015-12-11 11:05:49 +0100 |
commit | 365a0442f6def11b0b3012778d6a838c7a8e6912 (patch) | |
tree | ef2a10d79da345741addadb4d0f706fb850f2e34 /drivers/usb/serial | |
parent | USB: io_edgeport: remove redundant conditions (diff) | |
download | linux-365a0442f6def11b0b3012778d6a838c7a8e6912.tar.xz linux-365a0442f6def11b0b3012778d6a838c7a8e6912.zip |
USB: mos7840: remove redundant condition
This patch removes redundant condition.
(length && length > 5) can be reduced to a single evaluation.
Tested by compilation only.
Caught by cppcheck.
Signed-off-by: Geyslan G. Bem <geyslan@gmail.com>
Signed-off-by: Johan Hovold <johan@kernel.org>
Diffstat (limited to 'drivers/usb/serial')
-rw-r--r-- | drivers/usb/serial/mos7840.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/serial/mos7840.c b/drivers/usb/serial/mos7840.c index 8ac9b55f05af..2c69bfcdacc6 100644 --- a/drivers/usb/serial/mos7840.c +++ b/drivers/usb/serial/mos7840.c @@ -635,7 +635,7 @@ static void mos7840_interrupt_callback(struct urb *urb) * Byte 4 IIR Port 4 (port.number is 3) * Byte 5 FIFO status for both */ - if (length && length > 5) { + if (length > 5) { dev_dbg(&urb->dev->dev, "%s", "Wrong data !!!\n"); return; } |