staging: unisys: visorbus: Remove useless else clause in visorutil_spar_detect.
authorDavid Kershner <david.kershner@unisys.com>
Wed, 30 Aug 2017 17:36:25 +0000 (13:36 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 31 Aug 2017 16:17:39 +0000 (18:17 +0200)
The function visorutil_spar_detect had an if clause that returns from the
function, no need to do the rest of the code in an else clause.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: Tim Sell <timothy.sell@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/unisys/visorbus/visorchipset.c

index d27e0e897ab92e519a5d404da5e0b70502b369f5..a602ba605bf51c1ab3d1f45e20694301d9027d5f 100644 (file)
@@ -1830,9 +1830,8 @@ static __init int visorutil_spar_detect(void)
                return  (ebx == UNISYS_VISOR_ID_EBX) &&
                        (ecx == UNISYS_VISOR_ID_ECX) &&
                        (edx == UNISYS_VISOR_ID_EDX);
-       } else {
-               return 0;
        }
+       return 0;
 }
 
 static int init_unisys(void)