staging: bcm: Removed unnecessary braces after return
authorSarah Khan <sarahjmi07@gmail.com>
Tue, 7 Oct 2014 12:29:24 +0000 (17:59 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 20 Oct 2014 02:29:14 +0000 (10:29 +0800)
WARNING: else is generally not useful after return
checkpatch.pl warning in PHSModule.c

Signed-off-by: Sarah Khan <sarahjmi07@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/bcm/PHSModule.c

index 5f4e503..2626134 100644 (file)
@@ -1244,19 +1244,15 @@ static UINT CreateClassifierPHSRule(IN B_UINT16  uiClsId,
 
                if (eClsContext == eActiveClassifierRuleContext)
                        return ERR_CLSASSIFIER_TABLE_FULL;
-               else {
-                       /* Lets replace the oldest rule if we are looking in
-                        * old Rule table */
-                       if (psaClassifiertable->uiOldestPhsRuleIndex >= MAX_PHSRULE_PER_SF)
-                               psaClassifiertable->uiOldestPhsRuleIndex = 0;
-
-                       iClassifierIndex =
-                               psaClassifiertable->uiOldestPhsRuleIndex;
-                       psClassifierRules =
-                               &psaClassifiertable->stOldPhsRulesList[iClassifierIndex];
+               /* Lets replace the oldest rule if we are looking in
+               * old Rule table */
+               if (psaClassifiertable->uiOldestPhsRuleIndex >= MAX_PHSRULE_PER_SF)
+                       psaClassifiertable->uiOldestPhsRuleIndex = 0;
 
-                       (psaClassifiertable->uiOldestPhsRuleIndex)++;
-               }
+               iClassifierIndex = psaClassifiertable->uiOldestPhsRuleIndex;
+               psClassifierRules = &psaClassifiertable->stOldPhsRulesList[iClassifierIndex];
+
+               psaClassifiertable->uiOldestPhsRuleIndex++;
        }
 
        if (eClsContext == eOldClassifierRuleContext) {