staging: bcm: Removed unnecessary 'else' after 'return' statement in Qos.c
authorEsra Altintas <es.altintas@gmail.com>
Wed, 8 Oct 2014 18:21:50 +0000 (21:21 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 20 Oct 2014 02:29:20 +0000 (10:29 +0800)
The following patch fixes the checkpatch.pl warning:
WARNING: else is not generally useful after a break or return

Signed-off-by: Esra Altintas <es.altintas@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/bcm/Qos.c

index b3ac614cd35f4284d01038a530fc3d17a9adbe06..0f16c634279900ddd8f18a2db422ffc41419cad8 100644 (file)
@@ -978,8 +978,7 @@ static bool EthCSMatchEThTypeSAP(struct bcm_classifier_rule *pstClassifierRule,
                if (pstEthCsPktInfo->ucDSAP ==
                                pstClassifierRule->au8EthCSEtherType[2])
                        return TRUE;
-               else
-                       return false;
+               return false;
 
        }