diff options
| author |   Gulsah Kose <gulsah.1004@gmail.com> | 2014-09-30 20:32:08 +0300 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-10-02 10:13:37 -0700 |
| commit | d1ffa894cd35c402d41c4a12d7eaaaf69d7bfb3e (patch) | |
| tree | 66a9b9cd61e66b07aa1d4587dc498e05eef767d2 /drivers/staging | |
| parent | staging: ft1000: ft1000-usb: Removed unnecessary else statement. (diff) | |
| download | linux-dev-d1ffa894cd35c402d41c4a12d7eaaaf69d7bfb3e.tar.xz linux-dev-d1ffa894cd35c402d41c4a12d7eaaaf69d7bfb3e.zip | |
staging: ft1000: ft1000-usb: Removed unnecessary else statement.
This patch fixes "else is not generally useful after a break or return"
checkpatch.pl warning in ft1000_download.c
Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging')
| -rw-r--r-- | drivers/staging/ft1000/ft1000-usb/ft1000_download.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_download.c b/drivers/staging/ft1000/ft1000-usb/ft1000_download.c index 65f980167130..37707da09e9c 100644 --- a/drivers/staging/ft1000/ft1000-usb/ft1000_download.c +++ b/drivers/staging/ft1000/ft1000-usb/ft1000_download.c @@ -129,10 +129,9 @@ static int check_usb_db(struct ft1000_usb *ft1000dev) status = ft1000_write_register(ft1000dev, 0x8000, FT1000_REG_DOORBELL); break; - } else { - loopcnt++; - msleep(10); } + loopcnt++; + msleep(10); } @@ -190,10 +189,9 @@ static u16 get_handshake(struct ft1000_usb *ft1000dev, u16 expected_value) if ((handshake == expected_value) || (handshake == HANDSHAKE_RESET_VALUE_USB)) { return handshake; - } else { - loopcnt++; - msleep(10); } + loopcnt++; + msleep(10); } return HANDSHAKE_TIMEOUT_VALUE; |