staging: ft1000: ft1000-usb: Removed unnecessary else statement.
authorGulsah Kose <gulsah.1004@gmail.com>
Tue, 30 Sep 2014 17:32:08 +0000 (20:32 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 2 Oct 2014 17:13:37 +0000 (10:13 -0700)
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>
drivers/staging/ft1000/ft1000-usb/ft1000_download.c

index 65f980167130c3d70650bd0f14c1aa4fb840cf23..37707da09e9c3604ac6524279c1e0b26fe440c54 100644 (file)
@@ -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;