staging: octeon: Removed unnecessary else expression.
authorDilek Uzulmez <dilekuzulmez@gmail.com>
Sat, 27 Sep 2014 11:58:31 +0000 (14:58 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 29 Sep 2014 03:30:54 +0000 (23:30 -0400)
This patch fixes "else is not generally useful after a break or return"
checkpatch.pl warning ethernet-util.h

Signed-off-by: Dilek Uzulmez <dilekuzulmez@gmail.com>
Acked-by: Daniel Baluta <daniel.baluta@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/octeon/ethernet-util.h

index 2da5ce1..0f9b4a1 100644 (file)
@@ -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;
 }