ath6kl: Do not hide ath6kl_wmi_addkey_cmd() error values
authorJouni Malinen <jouni@qca.qualcomm.com>
Wed, 2 Nov 2011 21:46:47 +0000 (23:46 +0200)
committerKalle Valo <kvalo@qca.qualcomm.com>
Fri, 11 Nov 2011 10:59:59 +0000 (12:59 +0200)
Instead of converting any error to EIO, just return the real error
value to upper layers.

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
drivers/net/wireless/ath/ath6kl/cfg80211.c

index d7e0a8c75001e6337615872441a112545c1f2f7b..44e2c76f92468bd736955ecb1c76b36a99c44287 100644 (file)
@@ -908,7 +908,6 @@ static int ath6kl_cfg80211_add_key(struct wiphy *wiphy, struct net_device *ndev,
        struct ath6kl_key *key = NULL;
        u8 key_usage;
        u8 key_type;
-       int status = 0;
 
        if (!ath6kl_cfg80211_ready(vif))
                return -EIO;
@@ -1011,17 +1010,12 @@ static int ath6kl_cfg80211_add_key(struct wiphy *wiphy, struct net_device *ndev,
                return 0;
        }
 
-       status = ath6kl_wmi_addkey_cmd(ar->wmi, vif->fw_vif_idx,
-                                      vif->def_txkey_index,
-                                      key_type, key_usage, key->key_len,
-                                      key->seq, key->seq_len, key->key,
-                                      KEY_OP_INIT_VAL,
-                                      (u8 *) mac_addr, SYNC_BOTH_WMIFLAG);
-
-       if (status)
-               return -EIO;
-
-       return 0;
+       return ath6kl_wmi_addkey_cmd(ar->wmi, vif->fw_vif_idx,
+                                    vif->def_txkey_index,
+                                    key_type, key_usage, key->key_len,
+                                    key->seq, key->seq_len, key->key,
+                                    KEY_OP_INIT_VAL,
+                                    (u8 *) mac_addr, SYNC_BOTH_WMIFLAG);
 }
 
 static int ath6kl_cfg80211_del_key(struct wiphy *wiphy, struct net_device *ndev,
@@ -1097,7 +1091,6 @@ static int ath6kl_cfg80211_set_default_key(struct wiphy *wiphy,
        struct ath6kl *ar = (struct ath6kl *)ath6kl_priv(ndev);
        struct ath6kl_vif *vif = netdev_priv(ndev);
        struct ath6kl_key *key = NULL;
-       int status = 0;
        u8 key_usage;
        enum crypto_type key_type = NONE_CRYPT;
 
@@ -1132,17 +1125,13 @@ static int ath6kl_cfg80211_set_default_key(struct wiphy *wiphy,
        if (vif->next_mode == AP_NETWORK && !test_bit(CONNECTED, &vif->flags))
                return 0; /* Delay until AP mode has been started */
 
-       status = ath6kl_wmi_addkey_cmd(ar->wmi, vif->fw_vif_idx,
-                                      vif->def_txkey_index,
-                                      key_type, key_usage,
-                                      key->key_len, key->seq, key->seq_len,
-                                      key->key,
-                                      KEY_OP_INIT_VAL, NULL,
-                                      SYNC_BOTH_WMIFLAG);
-       if (status)
-               return -EIO;
-
-       return 0;
+       return ath6kl_wmi_addkey_cmd(ar->wmi, vif->fw_vif_idx,
+                                    vif->def_txkey_index,
+                                    key_type, key_usage,
+                                    key->key_len, key->seq, key->seq_len,
+                                    key->key,
+                                    KEY_OP_INIT_VAL, NULL,
+                                    SYNC_BOTH_WMIFLAG);
 }
 
 void ath6kl_cfg80211_tkip_micerr_event(struct ath6kl_vif *vif, u8 keyid,