iwlagn: remove pointless return variables
authorJohannes Berg <johannes.berg@intel.com>
Tue, 5 Apr 2011 16:41:53 +0000 (09:41 -0700)
committerJohn W. Linville <linville@tuxdriver.com>
Thu, 7 Apr 2011 19:51:35 +0000 (15:51 -0400)
A number of places just use a variable to return
it right away, which is useless, so let's remove
the variables there.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/iwlwifi/iwl-agn-rs.c
drivers/net/wireless/iwlwifi/iwl-agn-ucode.c
drivers/net/wireless/iwlwifi/iwl-debugfs.c

index e394e49228ba0dacbae9d353f63547310283783f..97abc10f4f221a69e9000c9f5df02f572eac1f0c 100644 (file)
@@ -2771,16 +2771,13 @@ static void rs_get_rate(void *priv_r, struct ieee80211_sta *sta, void *priv_sta,
 static void *rs_alloc_sta(void *priv_rate, struct ieee80211_sta *sta,
                          gfp_t gfp)
 {
-       struct iwl_lq_sta *lq_sta;
        struct iwl_station_priv *sta_priv = (struct iwl_station_priv *) sta->drv_priv;
        struct iwl_priv *priv;
 
        priv = (struct iwl_priv *)priv_rate;
        IWL_DEBUG_RATE(priv, "create station rate scale window\n");
 
-       lq_sta = &sta_priv->lq_sta;
-
-       return lq_sta;
+       return &sta_priv->lq_sta;
 }
 
 /*
@@ -3259,7 +3256,6 @@ static ssize_t rs_sta_dbgfs_rate_scale_data_read(struct file *file,
 {
        char buff[120];
        int desc = 0;
-       ssize_t ret;
 
        struct iwl_lq_sta *lq_sta = file->private_data;
        struct iwl_priv *priv;
@@ -3276,8 +3272,7 @@ static ssize_t rs_sta_dbgfs_rate_scale_data_read(struct file *file,
                                "Bit Rate= %d Mb/s\n",
                                iwl_rates[lq_sta->last_txrate_idx].ieee >> 1);
 
-       ret = simple_read_from_buffer(user_buf, count, ppos, buff, desc);
-       return ret;
+       return simple_read_from_buffer(user_buf, count, ppos, buff, desc);
 }
 
 static const struct file_operations rs_sta_dbgfs_rate_scale_data_ops = {
index 5d35e3504516b1d053db1646bcb58c2527123e5e..bb262a60e816c812243c691acfd0834028cb1118 100644 (file)
@@ -653,7 +653,5 @@ int iwl_verify_ucode(struct iwl_priv *priv)
         * Selection of bootstrap image (vs. other images) is arbitrary. */
        image = (__le32 *)priv->ucode_boot.v_addr;
        len = priv->ucode_boot.len;
-       ret = iwl_verify_inst_full(priv, image, len);
-
-       return ret;
+       return iwl_verify_inst_full(priv, image, len);
 }
index 8842411f1cf312019c5edc75987efc198720760c..90f2b6464bc1d17bb5dd18da939ff3222c93153c 100644 (file)
@@ -1572,12 +1572,10 @@ static ssize_t iwl_dbgfs_bt_traffic_read(struct file *file,
        int pos = 0;
        char buf[200];
        const size_t bufsz = sizeof(buf);
-       ssize_t ret;
 
        if (!priv->bt_enable_flag) {
                pos += scnprintf(buf + pos, bufsz - pos, "BT coex disabled\n");
-               ret = simple_read_from_buffer(user_buf, count, ppos, buf, pos);
-               return ret;
+               return simple_read_from_buffer(user_buf, count, ppos, buf, pos);
        }
        pos += scnprintf(buf + pos, bufsz - pos, "BT enable flag: 0x%x\n",
                priv->bt_enable_flag);
@@ -1608,8 +1606,7 @@ static ssize_t iwl_dbgfs_bt_traffic_read(struct file *file,
                break;
        }
 
-       ret = simple_read_from_buffer(user_buf, count, ppos, buf, pos);
-       return ret;
+       return simple_read_from_buffer(user_buf, count, ppos, buf, pos);
 }
 
 static ssize_t iwl_dbgfs_protection_mode_read(struct file *file,