From: Gulsah Kose <gulsah.1004@gmail.com>
Date: Fri, 26 Sep 2014 19:49:26 +0000 (+0300)
Subject: staging: ft1000: ft1000-pcmcia: Removed unnecessary else expression.
X-Git-Tag: v5.15~17125^2~128
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=96bcbea0d59a5aeacf753bff6a2d67cfa558d4cd;p=platform%2Fkernel%2Flinux-starfive.git

staging: ft1000: ft1000-pcmcia: Removed unnecessary else expression.

This patch fixes "else is not generally useful after a break or return"
checkpatch.pl warning in ft1000_dnld.c

Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---

diff --git a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_dnld.c b/drivers/staging/ft1000/ft1000-pcmcia/ft1000_dnld.c
index 02c183d..070a1d1 100644
--- a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_dnld.c
+++ b/drivers/staging/ft1000/ft1000-pcmcia/ft1000_dnld.c
@@ -168,10 +168,9 @@ u16 get_handshake(struct net_device *dev, u16 expected_value)
 		if ((handshake == expected_value)
 			|| (handshake == HANDSHAKE_RESET_VALUE)) {
 			return handshake;
-		} else {
-			loopcnt++;
-			mdelay(DSP_WAIT_SLEEP_TIME);
 		}
+		loopcnt++;
+		mdelay(DSP_WAIT_SLEEP_TIME);
 
 	}