Staging: rt2860: Hoist assign from if
authorJoe Perches <joe@perches.com>
Thu, 25 Mar 2010 05:16:59 +0000 (22:16 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 11 May 2010 18:35:44 +0000 (11:35 -0700)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/rt2860/common/cmm_data.c
drivers/staging/rt2860/common/cmm_wpa.c
drivers/staging/rt2860/common/spectrum.c
drivers/staging/rt2860/rt_linux.c

index 68263cee79521ef607703554894e1d58ddd867e6..65b00e685eb25534506758de7e2139fc6466acec 100644 (file)
@@ -773,7 +773,8 @@ void RTMPDeQueuePacket(struct rt_rtmp_adapter *pAd, IN BOOLEAN bIntContext, u8 Q
 
                        /* probe the Queue Head */
                        pQueue = &pAd->TxSwQueue[QueIdx];
-                       if ((pEntry = pQueue->Head) == NULL) {
+                       pEntry = pQueue->Head;
+                       if (pEntry == NULL) {
                                DEQUEUE_UNLOCK(&pAd->irq_lock, bIntContext,
                                               IrqFlags);
                                break;
@@ -824,7 +825,8 @@ void RTMPDeQueuePacket(struct rt_rtmp_adapter *pAd, IN BOOLEAN bIntContext, u8 Q
                                }
 
                                do {
-                                       if ((pEntry = pQueue->Head) == NULL)
+                                       pEntry = pQueue->Head;
+                                       if (pEntry == NULL)
                                                break;
 
                                        /* For TX_AMSDU_FRAME/TX_RALINK_FRAME, Need to check if next pakcet can do aggregation. */
index 94e119faaa71b01c04cd7477cc2d7ff7c4c0c0c8..c16f3763cca66ec4e4b57966be08c4cced987152 100644 (file)
@@ -2928,25 +2928,23 @@ void WpaShowAllsuite(u8 *rsnie, u32 rsnie_len)
        hex_dump("RSNIE", rsnie, rsnie_len);
 
        /* group cipher */
-       if ((pSuite =
-            GetSuiteFromRSNIE(rsnie, rsnie_len, GROUP_SUITE,
-                              &count)) != NULL) {
+       pSuite = GetSuiteFromRSNIE(rsnie, rsnie_len, GROUP_SUITE, &count);
+       if (pSuite != NULL) {
                hex_dump("group cipher", pSuite, 4 * count);
        }
        /* pairwise cipher */
-       if ((pSuite =
-            GetSuiteFromRSNIE(rsnie, rsnie_len, PAIRWISE_SUITE,
-                              &count)) != NULL) {
+       pSuite = GetSuiteFromRSNIE(rsnie, rsnie_len, PAIRWISE_SUITE, &count);
+       if (pSuite != NULL) {
                hex_dump("pairwise cipher", pSuite, 4 * count);
        }
        /* AKM */
-       if ((pSuite =
-            GetSuiteFromRSNIE(rsnie, rsnie_len, AKM_SUITE, &count)) != NULL) {
+       pSuite = GetSuiteFromRSNIE(rsnie, rsnie_len, AKM_SUITE, &count);
+       if (pSuite != NULL) {
                hex_dump("AKM suite", pSuite, 4 * count);
        }
        /* PMKID */
-       if ((pSuite =
-            GetSuiteFromRSNIE(rsnie, rsnie_len, PMKID_LIST, &count)) != NULL) {
+       pSuite = GetSuiteFromRSNIE(rsnie, rsnie_len, PMKID_LIST, &count);
+       if (pSuite != NULL) {
                hex_dump("PMKID", pSuite, LEN_PMKID);
        }
 
index 51e38d80933349161ed147e739e36902ba9ef61b..2d5f847e6cc6ce73631379c62432fdbbc619930d 100644 (file)
@@ -1900,8 +1900,8 @@ static void PeerMeasureReportAction(struct rt_rtmp_adapter *pAd,
 /*      if (pAd->CommonCfg.bIEEE80211H != TRUE) */
 /*              return; */
 
-       if ((pMeasureReportInfo =
-            kmalloc(sizeof(struct rt_measure_rpi_report), GFP_ATOMIC)) == NULL) {
+       pMeasureReportInfo = kmalloc(sizeof(struct rt_measure_rpi_report), GFP_ATOMIC);
+       if (pMeasureReportInfo == NULL) {
                DBGPRINT(RT_DEBUG_ERROR,
                         ("%s unable to alloc memory for measure report buffer (size=%zu).\n",
                          __func__, sizeof(struct rt_measure_rpi_report)));
@@ -2016,7 +2016,8 @@ static void PeerTpcRepAction(struct rt_rtmp_adapter *pAd, struct rt_mlme_queue_e
        NdisZeroMemory(&TpcRepInfo, sizeof(struct rt_tpc_report_info));
        if (PeerTpcRepSanity
            (pAd, Elem->Msg, Elem->MsgLen, &DialogToken, &TpcRepInfo)) {
-               if ((pEntry = TpcReqLookUp(pAd, DialogToken)) != NULL) {
+               pEntry = TpcReqLookUp(pAd, DialogToken);
+               if (pEntry != NULL) {
                        TpcReqDelete(pAd, pEntry->DialogToken);
                        DBGPRINT(RT_DEBUG_TRACE,
                                 ("%s: DialogToken=%x, TxPwr=%d, LinkMargin=%d\n",
index bfda187d2fbb95b069b1151a40e170b35341d282..27b78102f4bd472a6990f5de0ebaf5cd9b1df407 100644 (file)
@@ -463,8 +463,8 @@ void *duplicate_pkt(struct rt_rtmp_adapter *pAd,
        struct sk_buff *skb;
        void *pPacket = NULL;
 
-       if ((skb =
-            __dev_alloc_skb(HdrLen + DataSize + 2, MEM_ALLOC_FLAG)) != NULL) {
+       skb = __dev_alloc_skb(HdrLen + DataSize + 2, MEM_ALLOC_FLAG);
+       if (skb != NULL) {
                skb_reserve(skb, 2);
                NdisMoveMemory(skb_tail_pointer(skb), pHeader802_3, HdrLen);
                skb_put(skb, HdrLen);
@@ -673,7 +673,8 @@ void RTMPSendWirelessEvent(struct rt_rtmp_adapter *pAd,
                return;
        }
        /*Allocate memory and copy the msg. */
-       if ((pBuf = kmalloc(IW_CUSTOM_MAX_LEN, GFP_ATOMIC)) != NULL) {
+       pBuf = kmalloc(IW_CUSTOM_MAX_LEN, GFP_ATOMIC);
+       if (pBuf != NULL) {
                /*Prepare the payload */
                memset(pBuf, 0, IW_CUSTOM_MAX_LEN);