Staging: rtl8192e: Fix else is not useful style warning
authorMahati Chamarthy <mahati.chamarthy@gmail.com>
Fri, 19 Sep 2014 17:42:53 +0000 (23:12 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 20 Sep 2014 00:14:30 +0000 (17:14 -0700)
This fixes the following checkpatch.pl warnings:
WARNING: else is not generally useful after a break or return

Signed-off-by: Mahati Chamarthy <mahati.chamarthy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8192e/rtllib_rx.c

index 9ded5417f5b6884f4455ac52f2b8d5cd93862fdf..8f4f5893303b35982a993912dc444a8cdf9c8317 100644 (file)
@@ -917,12 +917,10 @@ static int rtllib_rx_check_duplicate(struct rtllib_device *ieee,
                if (GetTs(ieee, (struct ts_common_info **) &pRxTS, hdr->addr2,
                        (u8)Frame_QoSTID((u8 *)(skb->data)), RX_DIR, true)) {
                        if ((fc & (1<<11)) && (frag == pRxTS->RxLastFragNum) &&
-                           (WLAN_GET_SEQ_SEQ(sc) == pRxTS->RxLastSeqNum)) {
+                           (WLAN_GET_SEQ_SEQ(sc) == pRxTS->RxLastSeqNum))
                                return -1;
-                       } else {
-                               pRxTS->RxLastFragNum = frag;
-                               pRxTS->RxLastSeqNum = WLAN_GET_SEQ_SEQ(sc);
-                       }
+                       pRxTS->RxLastFragNum = frag;
+                       pRxTS->RxLastSeqNum = WLAN_GET_SEQ_SEQ(sc);
                } else {
                        RTLLIB_DEBUG(RTLLIB_DL_ERR, "ERR!!%s(): No TS!! Skip"
                                     " the check!!\n", __func__);