From ec819d6ccc2575f2b600b2570bb20ffde91e1bbf Mon Sep 17 00:00:00 2001 From: Elyes HAOUAS Date: Sun, 9 Jun 2019 17:50:51 +0200 Subject: serprog.c: Remove unneeded 'else' 'else' is not needed after a 'break' or 'return'. Change-Id: I9015020889c25ecbd391a18f56f99affc8ea307d Signed-off-by: Elyes HAOUAS Reviewed-on: https://review.coreboot.org/c/flashrom/+/33348 Tested-by: build bot (Jenkins) Reviewed-by: Angel Pons --- serprog.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/serprog.c b/serprog.c index c3d48a4d9..b7ef35ce7 100644 --- a/serprog.c +++ b/serprog.c @@ -937,9 +937,8 @@ void *serprog_map(const char *descr, uintptr_t phys_addr, size_t len) * needed for non-SPI chips). Below we make sure that the requested range is within this window. */ if ((phys_addr & 0xFF000000) == 0xFF000000) { return (void*)phys_addr; - } else { - msg_pwarn(MSGHEADER "requested mapping %s is incompatible: 0x%zx bytes at 0x%0*" PRIxPTR ".\n", - descr, len, PRIxPTR_WIDTH, phys_addr); - return NULL; } + msg_pwarn(MSGHEADER "requested mapping %s is incompatible: 0x%zx bytes at 0x%0*" PRIxPTR ".\n", + descr, len, PRIxPTR_WIDTH, phys_addr); + return NULL; } -- cgit v1.2.3