From: Esra Altintas <es.altintas@gmail.com>
Date: Wed, 8 Oct 2014 18:21:50 +0000 (+0300)
Subject: staging: bcm: Removed unnecessary 'else' after 'return' statement in Qos.c
X-Git-Tag: firefly_0821_release~176^2~2665^2~1188
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=84995ca471ab409413b4938b8133a56ec36ff2f5;p=firefly-linux-kernel-4.4.55.git

staging: bcm: Removed unnecessary 'else' after 'return' statement in Qos.c

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>
---

diff --git a/drivers/staging/bcm/Qos.c b/drivers/staging/bcm/Qos.c
index b3ac614cd35f..0f16c6342799 100644
--- a/drivers/staging/bcm/Qos.c
+++ b/drivers/staging/bcm/Qos.c
@@ -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;
 
 	}