diff options
author | Geyslan G. Bem <geyslan@gmail.com> | 2015-12-11 04:46:42 -0500 |
---|---|---|
committer | Johan Hovold <johan@kernel.org> | 2015-12-11 05:05:49 -0500 |
commit | 365a0442f6def11b0b3012778d6a838c7a8e6912 (patch) | |
tree | ef2a10d79da345741addadb4d0f706fb850f2e34 | |
parent | 232dce89b5b00078fcb633322edadf8badae326a (diff) |
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>
-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) | |||
635 | * Byte 4 IIR Port 4 (port.number is 3) | 635 | * Byte 4 IIR Port 4 (port.number is 3) |
636 | * Byte 5 FIFO status for both */ | 636 | * Byte 5 FIFO status for both */ |
637 | 637 | ||
638 | if (length && length > 5) { | 638 | if (length > 5) { |
639 | dev_dbg(&urb->dev->dev, "%s", "Wrong data !!!\n"); | 639 | dev_dbg(&urb->dev->dev, "%s", "Wrong data !!!\n"); |
640 | return; | 640 | return; |
641 | } | 641 | } |