From aa66d88d0bf8d098794cf9923be7e773454d1806 Mon Sep 17 00:00:00 2001 From: Dilek Uzulmez Date: Sat, 27 Sep 2014 14:58:31 +0300 Subject: [PATCH] staging: octeon: Removed unnecessary else expression. This patch fixes "else is not generally useful after a break or return" checkpatch.pl warning ethernet-util.h Signed-off-by: Dilek Uzulmez Acked-by: Daniel Baluta Signed-off-by: Greg Kroah-Hartman --- drivers/staging/octeon/ethernet-util.h | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/octeon/ethernet-util.h b/drivers/staging/octeon/ethernet-util.h index 2da5ce1..0f9b4a1 100644 --- a/drivers/staging/octeon/ethernet-util.h +++ b/drivers/staging/octeon/ethernet-util.h @@ -53,8 +53,7 @@ static inline int INTERFACE(int ipd_port) return 3; else if (ipd_port == 40) /* Non existent interface for POW0 */ return 4; - else - panic("Illegal ipd_port %d passed to INTERFACE\n", ipd_port); + panic("Illegal ipd_port %d passed to INTERFACE\n", ipd_port); } /** @@ -67,6 +66,5 @@ static inline int INDEX(int ipd_port) { if (ipd_port < 32) return ipd_port & 15; - else - return ipd_port & 3; + return ipd_port & 3; } -- 2.7.4