HAOUAS Elyes has uploaded this change for review.

View Change

ft2232_spi.c: Remove unneeded 'else'

'else' is not needed after a 'break' or 'return'.

Change-Id: Ia4762d0c0601d56528de56658b869b62fbe5b263
Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr>
---
M ft2232_spi.c
1 file changed, 4 insertions(+), 6 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/46/33346/1

diff --git a/ft2232_spi.c b/ft2232_spi.c
index 95584aa..5c4d06b 100644
--- a/ft2232_spi.c
+++ b/ft2232_spi.c
@@ -321,9 +321,8 @@
 				 "Valid are even values between 2 and 131072.\n", arg);
 			free(arg);
 			return -2;
-		} else {
-			divisor = (uint32_t)temp;
 		}
+		divisor = (uint32_t)temp;
 	}
 	free(arg);
 
@@ -336,11 +335,10 @@
 				 "Valid values are between 0 and 3.\n", arg);
 			free(arg);
 			return -2;
-		} else {
-			unsigned int pin = temp + 4;
-			cs_bits |= 1 << pin;
-			pindir |= 1 << pin;
 		}
+		unsigned int pin = temp + 4;
+		cs_bits |= 1 << pin;
+		pindir |= 1 << pin;
 	}
 	free(arg);
 

To view, visit change 33346. To unsubscribe, or for help writing mail filters, visit settings.