mwifiex: remove redundant "return" at end of void function
authorYogesh Ashok Powar <yogeshp@marvell.com>
Thu, 14 Apr 2011 00:27:07 +0000 (17:27 -0700)
committerJohn W. Linville <linville@tuxdriver.com>
Thu, 14 Apr 2011 19:35:12 +0000 (15:35 -0400)
The return statement at the last line of a void function
is not necessary.

Signed-off-by: Yogesh Ashok Powar <yogeshp@marvell.com>
Signed-off-by: Bing Zhao <bzhao@marvell.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/mwifiex/11n.c
drivers/net/wireless/mwifiex/11n_rxreorder.c
drivers/net/wireless/mwifiex/cfg80211.c
drivers/net/wireless/mwifiex/cmdevt.c
drivers/net/wireless/mwifiex/debugfs.c
drivers/net/wireless/mwifiex/init.c
drivers/net/wireless/mwifiex/main.c
drivers/net/wireless/mwifiex/sdio.c
drivers/net/wireless/mwifiex/sta_cmdresp.c
drivers/net/wireless/mwifiex/wmm.c

index edf4c274fa9b898d9011bafb503c85bcae2de05b..c57107a860d1c54bfc5c7077f07d0546e0619b9e 100644 (file)
@@ -543,8 +543,6 @@ mwifiex_cfg_tx_buf(struct mwifiex_private *priv,
        if (curr_tx_buf_size != tx_buf)
                mwifiex_send_cmd_async(priv, HostCmd_CMD_RECONFIGURE_TX_BUFF,
                                       HostCmd_ACT_GEN_SET, 0, &tx_buf);
-
-       return;
 }
 
 /*
@@ -582,8 +580,6 @@ void mwifiex_11n_delete_tx_ba_stream_tbl_entry(struct mwifiex_private *priv,
        list_del(&tx_ba_tsr_tbl->list);
 
        kfree(tx_ba_tsr_tbl);
-
-       return;
 }
 
 /*
@@ -662,8 +658,6 @@ void mwifiex_11n_create_tx_ba_stream_tbl(struct mwifiex_private *priv,
                list_add_tail(&new_node->list, &priv->tx_ba_stream_tbl_ptr);
                spin_unlock_irqrestore(&priv->tx_ba_stream_tbl_lock, flags);
        }
-
-       return;
 }
 
 /*
index ef46d0a8a6d0adca4716615fd00d7605a7e6a784..6736fc6048441876196f09bd76eb50527767eb1b 100644 (file)
@@ -309,8 +309,6 @@ mwifiex_11n_create_rx_reorder_tbl(struct mwifiex_private *priv, u8 *ta,
        spin_lock_irqsave(&priv->rx_reorder_tbl_lock, flags);
        list_add_tail(&new_node->list, &priv->rx_reorder_tbl_ptr);
        spin_unlock_irqrestore(&priv->rx_reorder_tbl_lock, flags);
-
-       return;
 }
 
 /*
@@ -610,8 +608,6 @@ void mwifiex_11n_ba_stream_timeout(struct mwifiex_private *priv,
                (u16) event->origninator << DELBA_INITIATOR_POS);
        delba.reason_code = cpu_to_le16(WLAN_REASON_QSTA_TIMEOUT);
        mwifiex_send_cmd_async(priv, HostCmd_CMD_11N_DELBA, 0, 0, &delba);
-
-       return;
 }
 
 /*
index a1ff490da836bd918f8b6f1059d71904138893c0..74b6cf20da043e0e19a2d2fb8639cb21a62daeba 100644 (file)
@@ -1428,6 +1428,4 @@ done:
                memset(priv->cfg_bssid, 0, ETH_ALEN);
                priv->disconnect = 0;
        }
-
-       return;
 }
index bb6fecd7761902fd9c95bb24559391f99b747e0b..776146a104ec966144adb504b7e244e65ef801ee 100644 (file)
@@ -94,8 +94,6 @@ mwifiex_clean_cmd_node(struct mwifiex_adapter *adapter,
                mwifiex_recv_complete(adapter, cmd_node->resp_skb, 0);
                cmd_node->resp_skb = NULL;
        }
-
-       return;
 }
 
 /*
@@ -536,7 +534,7 @@ mwifiex_insert_cmd_to_free_q(struct mwifiex_adapter *adapter,
 {
        unsigned long flags;
 
-       if (cmd_node == NULL)
+       if (!cmd_node)
                return;
 
        if (cmd_node->wait_q_enabled)
@@ -548,8 +546,6 @@ mwifiex_insert_cmd_to_free_q(struct mwifiex_adapter *adapter,
        spin_lock_irqsave(&adapter->cmd_free_q_lock, flags);
        list_add_tail(&cmd_node->list, &adapter->cmd_free_q);
        spin_unlock_irqrestore(&adapter->cmd_free_q_lock, flags);
-
-       return;
 }
 
 /*
@@ -594,8 +590,6 @@ mwifiex_insert_cmd_to_pending_q(struct mwifiex_adapter *adapter,
        spin_unlock_irqrestore(&adapter->cmd_pending_q_lock, flags);
 
        dev_dbg(adapter->dev, "cmd: QUEUE_CMD: cmd=%#x is queued\n", command);
-
-       return;
 }
 
 /*
@@ -871,8 +865,6 @@ mwifiex_cmd_timeout_func(unsigned long function_context)
        }
        if (adapter->hw_status == MWIFIEX_HW_STATUS_INITIALIZING)
                mwifiex_init_fw_complete(adapter);
-
-       return;
 }
 
 /*
@@ -989,8 +981,6 @@ mwifiex_cancel_pending_ioctl(struct mwifiex_adapter *adapter)
        }
        adapter->cmd_wait_q.status = -1;
        mwifiex_complete_cmd(adapter);
-
-       return;
 }
 
 /*
@@ -1094,7 +1084,6 @@ mwifiex_process_hs_config(struct mwifiex_adapter *adapter)
        adapter->is_hs_configured = false;
        mwifiex_hs_activated_event(mwifiex_get_priv(adapter,
                                   MWIFIEX_BSS_ROLE_ANY), false);
-       return;
 }
 
 /*
index 77d7c777ea6629a04c21a5f0acd1352a5e2e735a..7ddcb062f1037684fa9d3f176dba18d5c571e5e7 100644 (file)
@@ -735,8 +735,6 @@ mwifiex_dev_debugfs_init(struct mwifiex_private *priv)
        MWIFIEX_DFS_ADD_FILE(getlog);
        MWIFIEX_DFS_ADD_FILE(regrdwr);
        MWIFIEX_DFS_ADD_FILE(rdeeprom);
-
-       return;
 }
 
 /*
@@ -749,7 +747,6 @@ mwifiex_dev_debugfs_remove(struct mwifiex_private *priv)
                return;
 
        debugfs_remove_recursive(priv->dfs_dev_dir);
-       return;
 }
 
 /*
index 26931d5f950f2a14d57f12e933c4b71e82b9bb25..1b79a5ac9214f40756377dc42b2950ae462703c9 100644 (file)
@@ -299,8 +299,6 @@ static void mwifiex_init_adapter(struct mwifiex_adapter *adapter)
        adapter->adhoc_awake_period = 0;
        memset(&adapter->arp_filter, 0, sizeof(adapter->arp_filter));
        adapter->arp_filter_size = 0;
-
-       return;
 }
 
 /*
@@ -339,8 +337,6 @@ mwifiex_free_adapter(struct mwifiex_adapter *adapter)
        adapter->if_ops.cleanup_if(adapter);
 
        dev_kfree_skb_any(adapter->sleep_cfm);
-
-       return;
 }
 
 /*
@@ -428,8 +424,6 @@ void mwifiex_free_lock_list(struct mwifiex_adapter *adapter)
                        list_del(&priv->rx_reorder_tbl_ptr);
                }
        }
-
-       return;
 }
 
 /*
index df665db8c433dc407aae5e15335f839b6d056eb7..77abfc3d6c3262c6c248910a48c3f3e3b053e5e4 100644 (file)
@@ -505,7 +505,6 @@ mwifiex_fill_buffer(struct sk_buff *skb)
         */
        do_gettimeofday(&tv);
        skb->tstamp = timeval_to_ktime(tv);
-       return;
 }
 
 /*
@@ -820,8 +819,6 @@ mwifiex_remove_interface(struct mwifiex_adapter *adapter, u8 bss_index)
        wiphy_unregister(priv->wdev->wiphy);
        wiphy_free(priv->wdev->wiphy);
        kfree(priv->wdev);
-
-       return;
 }
 
 /*
index f207756cbb796fa6fc12da6cee08b2303e6e418d..fa46df509757a674ae103b4763636c1c6fc40bd1 100644 (file)
@@ -859,8 +859,6 @@ static void mwifiex_interrupt_status(struct mwifiex_adapter *adapter)
                adapter->int_status |= sdio_ireg;
                spin_unlock_irqrestore(&adapter->int_lock, flags);
        }
-
-       return;
 }
 
 /*
@@ -891,8 +889,6 @@ mwifiex_sdio_interrupt(struct sdio_func *func)
 
        mwifiex_interrupt_status(adapter);
        queue_work(adapter->workqueue, &adapter->main_work);
-
-       return;
 }
 
 /*
index 8743c116bee539026619c71bd7077a0ff343406f..20ce8cb39186175b7c3b96a9dbe09fbf0ee2e39b 100644 (file)
@@ -103,8 +103,6 @@ mwifiex_process_cmdresp_error(struct mwifiex_private *priv,
        spin_lock_irqsave(&adapter->mwifiex_cmd_lock, flags);
        adapter->curr_cmd = NULL;
        spin_unlock_irqrestore(&adapter->mwifiex_cmd_lock, flags);
-
-       return;
 }
 
 /*
index 1cfbc6bed6929eccdf6d819ba45aaaa2fe3c5ee9..6ce6f94e222b165a868dee68b153e1cc071733d3 100644 (file)
@@ -1232,6 +1232,4 @@ mwifiex_wmm_process_tx(struct mwifiex_adapter *adapter)
                if (mwifiex_dequeue_tx_packet(adapter))
                        break;
        } while (true);
-
-       return;
 }