summaryrefslogtreecommitdiffstats
path: root/drivers/usb/serial/mos7840.c
diff options
context:
space:
mode:
authorGeyslan G. Bem <geyslan@gmail.com>2015-12-11 10:46:42 +0100
committerJohan Hovold <johan@kernel.org>2015-12-11 11:05:49 +0100
commit365a0442f6def11b0b3012778d6a838c7a8e6912 (patch)
treeef2a10d79da345741addadb4d0f706fb850f2e34 /drivers/usb/serial/mos7840.c
parentUSB: io_edgeport: remove redundant conditions (diff)
downloadlinux-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/mos7840.c')
-rw-r--r--drivers/usb/serial/mos7840.c2
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;
}