From d1ffa894cd35c402d41c4a12d7eaaaf69d7bfb3e Mon Sep 17 00:00:00 2001 From: Gulsah Kose Date: Tue, 30 Sep 2014 20:32:08 +0300 Subject: [PATCH] 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 Signed-off-by: Greg Kroah-Hartman --- drivers/staging/ft1000/ft1000-usb/ft1000_download.c | 10 ++++------ 1 file 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; -- 2.11.0