Staging: vt6656: Combine "else { if" to "else if"
authorSimon Schuster <linux@rationality.eu>
Sat, 4 Jan 2014 15:25:13 +0000 (16:25 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 8 Jan 2014 01:17:58 +0000 (17:17 -0800)
This patch combines single ifs within the block of an else to a single
else if statement.

Therefore code that looks like that

else {
if (cond) {
statements;
} else {
other_statements;
}
}

is converted to code that looks like that

else if (cond) {
statements;
} else {
other_statements;
}

Signed-off-by: Sebastian Rachuj <sebastian.rachuj@studium.uni-erlangen.de>
Signed-off-by: Simon Schuster <linux@rationality.eu>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/vt6656/bssdb.c

index bd64d598c725fb5b1d7ceab3df3360889272dfb6..e281d3c5a8891c605e0151911d340c3d90cb9fbe 100644 (file)
@@ -137,14 +137,11 @@ PKnownBSS BSSpSearchBSSList(struct vnt_private *pDevice,
                                                                return pCurrBSS;
                                                        }
                                                }
-                                       } else {
-                                               if ((pMgmt->eConfigMode == WMAC_CONFIG_AUTO) ||
-                                                       ((pMgmt->eConfigMode == WMAC_CONFIG_IBSS_STA) && WLAN_GET_CAP_INFO_IBSS(pCurrBSS->wCapInfo)) ||
-                                                       ((pMgmt->eConfigMode == WMAC_CONFIG_ESS_STA) && WLAN_GET_CAP_INFO_ESS(pCurrBSS->wCapInfo))
-                                                       ) {
-                                                       pCurrBSS->bSelected = true;
-                                                       return pCurrBSS;
-                                               }
+                                       } else if ((pMgmt->eConfigMode == WMAC_CONFIG_AUTO) ||
+                                               ((pMgmt->eConfigMode == WMAC_CONFIG_IBSS_STA) && WLAN_GET_CAP_INFO_IBSS(pCurrBSS->wCapInfo)) ||
+                                               ((pMgmt->eConfigMode == WMAC_CONFIG_ESS_STA) && WLAN_GET_CAP_INFO_ESS(pCurrBSS->wCapInfo))) {
+                                               pCurrBSS->bSelected = true;
+                                               return pCurrBSS;
                                        }
                                }
                        }
@@ -196,11 +193,9 @@ PKnownBSS BSSpSearchBSSList(struct vnt_private *pDevice,
 
                                if (pSelect == NULL) {
                                        pSelect = pCurrBSS;
-                               } else {
-                                       /* compare RSSI, select the strongest signal */
-                                       if (pCurrBSS->uRSSI < pSelect->uRSSI) {
-                                               pSelect = pCurrBSS;
-                                       }
+                               /* compare RSSI, select the strongest signal */
+                               } else if (pCurrBSS->uRSSI < pSelect->uRSSI) {
+                                       pSelect = pCurrBSS;
                                }
                        }
                }
@@ -364,15 +359,12 @@ int BSSbInsertToBSSList(struct vnt_private *pDevice,
        pBSSList->sERP.bERPExist = psERP->bERPExist;
 
        /* Check if BSS is 802.11a/b/g */
-       if (pBSSList->uChannel > CB_MAX_CHANNEL_24G) {
+       if (pBSSList->uChannel > CB_MAX_CHANNEL_24G)
                pBSSList->eNetworkTypeInUse = PHY_TYPE_11A;
-       } else {
-               if (pBSSList->sERP.bERPExist == true) {
-                       pBSSList->eNetworkTypeInUse = PHY_TYPE_11G;
-               } else {
-                       pBSSList->eNetworkTypeInUse = PHY_TYPE_11B;
-               }
-       }
+       else if (pBSSList->sERP.bERPExist == true)
+               pBSSList->eNetworkTypeInUse = PHY_TYPE_11G;
+       else
+               pBSSList->eNetworkTypeInUse = PHY_TYPE_11B;
 
        pBSSList->byRxRate = pRxPacket->byRxRate;
        pBSSList->qwLocalTSF = pRxPacket->qwLocalTSF;
@@ -521,15 +513,12 @@ int BSSbUpdateToBSSList(struct vnt_private *pDevice,
        pBSSList->sERP.bERPExist = psERP->bERPExist;
 
        /* Check if BSS is 802.11a/b/g */
-       if (pBSSList->uChannel > CB_MAX_CHANNEL_24G) {
+       if (pBSSList->uChannel > CB_MAX_CHANNEL_24G)
                pBSSList->eNetworkTypeInUse = PHY_TYPE_11A;
-       } else {
-               if (pBSSList->sERP.bERPExist == true) {
-                       pBSSList->eNetworkTypeInUse = PHY_TYPE_11G;
-               } else {
-                       pBSSList->eNetworkTypeInUse = PHY_TYPE_11B;
-               }
-       }
+       else if (pBSSList->sERP.bERPExist == true)
+               pBSSList->eNetworkTypeInUse = PHY_TYPE_11G;
+       else
+               pBSSList->eNetworkTypeInUse = PHY_TYPE_11B;
 
        pBSSList->byRxRate = pRxPacket->byRxRate;
        pBSSList->qwLocalTSF = pRxPacket->qwLocalTSF;
@@ -891,11 +880,10 @@ void BSSvSecondCallBack(struct work_struct *work)
                                        /* ii = 0 for multicast node (AP & Adhoc) */
                                        RATEvTxRateFallBack((void *) pDevice,
                                                &(pMgmt->sNodeDBTable[ii]));
-                               } else {
+                               } else if (pMgmt->eCurrMode == WMAC_MODE_ESS_STA) {
                                        /* ii = 0 reserved for unicast AP node (Infra STA) */
-                                       if (pMgmt->eCurrMode == WMAC_MODE_ESS_STA)
-                                               RATEvTxRateFallBack((void *) pDevice,
-                                                       &(pMgmt->sNodeDBTable[ii]));
+                                       RATEvTxRateFallBack((void *) pDevice,
+                                               &(pMgmt->sNodeDBTable[ii]));
                                }
 
                        }
@@ -922,11 +910,9 @@ void BSSvSecondCallBack(struct work_struct *work)
                                MACvEnableProtectMD(pDevice);
                                pDevice->bProtectMode = true;
                        }
-               } else {
-                       if (pDevice->bProtectMode) {
-                               MACvDisableProtectMD(pDevice);
-                               pDevice->bProtectMode = false;
-                       }
+               } else if (pDevice->bProtectMode) {
+                       MACvDisableProtectMD(pDevice);
+                       pDevice->bProtectMode = false;
                }
                /* on/off short slot time */
 
@@ -936,12 +922,10 @@ void BSSvSecondCallBack(struct work_struct *work)
                                BBvSetShortSlotTime(pDevice);
                                vUpdateIFS((void *) pDevice);
                        }
-               } else {
-                       if (!pDevice->bShortSlotTime) {
+               } else if (!pDevice->bShortSlotTime) {
                                pDevice->bShortSlotTime = true;
                                BBvSetShortSlotTime(pDevice);
                                vUpdateIFS((void *) pDevice);
-                       }
                }
 
                /* on/off barker long preamble mode */
@@ -951,11 +935,9 @@ void BSSvSecondCallBack(struct work_struct *work)
                                MACvEnableBarkerPreambleMd(pDevice);
                                pDevice->bBarkerPreambleMd = true;
                        }
-               } else {
-                       if (pDevice->bBarkerPreambleMd) {
+               } else if (pDevice->bBarkerPreambleMd) {
                                MACvDisableBarkerPreambleMd(pDevice);
                                pDevice->bBarkerPreambleMd = false;
-                       }
                }
 
        }
@@ -1031,28 +1013,26 @@ void BSSvSecondCallBack(struct work_struct *work)
                                        if (pDevice->uIsroamingTime >= 20)
                                                pDevice->bIsRoaming = false;
                                }
+                       } else if (pDevice->uAutoReConnectTime < 10) {
+                               pDevice->uAutoReConnectTime++;
+                               /* network manager support need not do Roaming scan??? */
+                               if (pDevice->bWPASuppWextEnabled == true)
+                                       pDevice->uAutoReConnectTime = 0;
                        } else {
-                               if (pDevice->uAutoReConnectTime < 10) {
-                                       pDevice->uAutoReConnectTime++;
-                                       /* network manager support need not do Roaming scan??? */
-                                       if (pDevice->bWPASuppWextEnabled == true)
-                                               pDevice->uAutoReConnectTime = 0;
-                               } else {
-                                       /* mike use old encryption status for wpa reauthen */
-                                       if (pDevice->bWPADEVUp)
-                                               pDevice->eEncryptionStatus = pDevice->eOldEncryptionStatus;
+                               /* mike use old encryption status for wpa reauthen */
+                               if (pDevice->bWPADEVUp)
+                                       pDevice->eEncryptionStatus = pDevice->eOldEncryptionStatus;
 
-                                       DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO "Roaming ...\n");
-                                       BSSvClearBSSList((void *) pDevice, pDevice->bLinkPass);
-                                       pMgmt->eScanType = WMAC_SCAN_ACTIVE;
-                                       bScheduleCommand((void *) pDevice,
-                                                        WLAN_CMD_BSSID_SCAN,
-                                                        pMgmt->abyDesireSSID);
-                                       bScheduleCommand((void *) pDevice,
-                                                        WLAN_CMD_SSID,
-                                                        pMgmt->abyDesireSSID);
-                                       pDevice->uAutoReConnectTime = 0;
-                               }
+                               DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO "Roaming ...\n");
+                               BSSvClearBSSList((void *) pDevice, pDevice->bLinkPass);
+                               pMgmt->eScanType = WMAC_SCAN_ACTIVE;
+                               bScheduleCommand((void *) pDevice,
+                                                WLAN_CMD_BSSID_SCAN,
+                                                pMgmt->abyDesireSSID);
+                               bScheduleCommand((void *) pDevice,
+                                                WLAN_CMD_SSID,
+                                                pMgmt->abyDesireSSID);
+                               pDevice->uAutoReConnectTime = 0;
                        }
                }
        }
@@ -1203,11 +1183,11 @@ void BSSvUpdateNodeTxCounter(struct vnt_private *pDevice, u8 byTSR, u8 byPktNO)
                }
 
                if ((pMgmt->eCurrMode == WMAC_MODE_IBSS_STA) ||
-                       (pMgmt->eCurrMode == WMAC_MODE_ESS_AP)) {
+                    (pMgmt->eCurrMode == WMAC_MODE_ESS_AP)) {
 
                        if (BSSbIsSTAInNodeDB((void *) pDevice,
-                                                 pbyDestAddr,
-                                                 &uNodeIndex)) {
+                                             pbyDestAddr,
+                                             &uNodeIndex)) {
                                pMgmt->sNodeDBTable[uNodeIndex].uTxAttempts += 1;
                                if (!(byTSR & (TSR_TMO | TSR_RETRYTMO))) {
                                        /* transmit success, TxAttempts at least plus one */