mwifiex: remove unnecessary _set_auth functions

mwifiex_set_encrypt_mode()
mwifiex_set_auth_mode()
mwifiex_set_auth()
These functions are confusing and misleading.
And they are really not needed at all.

Some unused definitions are also removed.

Signed-off-by: Marc Yang <yangyang@marvell.com>
Signed-off-by: Bing Zhao <bzhao@marvell.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>

diff --git a/drivers/net/wireless/mwifiex/cfg80211.c b/drivers/net/wireless/mwifiex/cfg80211.c
index 80f367f27efca96c15b52a3e53d7385656030de7..84e33f1f0ffe96a66fd77a562f66b72340c40453 100644 (file)
--- a/drivers/net/wireless/mwifiex/cfg80211.c
+++ b/drivers/net/wireless/mwifiex/cfg80211.c
@@ -1055,11 +1055,10 @@ mwifiex_cfg80211_assoc(struct mwifiex_private *priv, size_t ssid_len, u8 *ssid,
                         * scan. The cfg80211 does not give us the encryption
                         * mode at this stage so just setting it to WEP here.
                         */
-                       wpa_enabled = 0;
-                       auth_type = MWIFIEX_AUTH_MODE_OPEN;
-                       ret = mwifiex_set_auth(priv,
-                                               MWIFIEX_ENCRYPTION_MODE_WEP104,
-                                               auth_type, wpa_enabled);
+                       priv->sec_info.encryption_mode =
+                                       MWIFIEX_ENCRYPTION_MODE_WEP104;
+                       priv->sec_info.authentication_mode =
+                                       MWIFIEX_AUTH_MODE_OPEN;
                }
 
                goto done;
@@ -1075,15 +1074,15 @@ mwifiex_cfg80211_assoc(struct mwifiex_private *priv, size_t ssid_len, u8 *ssid,
        if (sme->crypto.n_ciphers_pairwise) {
                pairwise_encrypt_mode = mwifiex_get_mwifiex_cipher(sme->crypto.
                                        ciphers_pairwise[0], &wpa_enabled);
-               ret = mwifiex_set_auth(priv, pairwise_encrypt_mode, auth_type,
-                                                               wpa_enabled);
+               priv->sec_info.encryption_mode = pairwise_encrypt_mode;
+               priv->sec_info.authentication_mode = auth_type;
        }
 
        if (sme->crypto.cipher_group) {
                group_encrypt_mode = mwifiex_get_mwifiex_cipher(sme->crypto.
                                                   cipher_group, &wpa_enabled);
-               ret = mwifiex_set_auth(priv, group_encrypt_mode, auth_type,
-                                                               wpa_enabled);
+               priv->sec_info.encryption_mode = group_encrypt_mode;
+               priv->sec_info.authentication_mode = auth_type;
        }
        if (sme->ie)
                ret = mwifiex_set_gen_ie(priv, sme->ie, sme->ie_len);

diff --git a/drivers/net/wireless/mwifiex/ioctl.h b/drivers/net/wireless/mwifiex/ioctl.h
index d6babfb1495c81c5d30c7a578c77c8e145a0ce07..b7e457110b4f497021ee8c956e993b0c77fb034e 100644 (file)
--- a/drivers/net/wireless/mwifiex/ioctl.h
+++ b/drivers/net/wireless/mwifiex/ioctl.h
@@ -277,8 +277,6 @@ struct mwifiex_debug_info {
 enum {
        MWIFIEX_AUTH_MODE_OPEN = 0x00,
        MWIFIEX_AUTH_MODE_SHARED = 0x01,
-       MWIFIEX_AUTH_MODE_NETWORKEAP = 0x80,
-       MWIFIEX_AUTH_MODE_AUTO = 0xFF,
 };
 
 enum {

diff --git a/drivers/net/wireless/mwifiex/join.c b/drivers/net/wireless/mwifiex/join.c
index d06f4c2d1d30ce0a3c6a17dd982e2840bf2d940d..98d76d8c26563ad6a332eb3855a751d7b7944665 100644 (file)
--- a/drivers/net/wireless/mwifiex/join.c
+++ b/drivers/net/wireless/mwifiex/join.c
@@ -441,20 +441,17 @@ int mwifiex_cmd_802_11_associate(struct mwifiex_private *priv,
        dev_dbg(priv->adapter->dev, "info: ASSOC_CMD: rates size = %d\n",
                                        rates_size);
 
-       /* Add the Authentication type to be used for Auth frames if needed */
-       if (priv->sec_info.authentication_mode != MWIFIEX_AUTH_MODE_AUTO) {
-               auth_tlv = (struct mwifiex_ie_types_auth_type *) pos;
-               auth_tlv->header.type = cpu_to_le16(TLV_TYPE_AUTH_TYPE);
-               auth_tlv->header.len = cpu_to_le16(sizeof(auth_tlv->auth_type));
-               if (priv->sec_info.wep_status == MWIFIEX_802_11_WEP_ENABLED)
-                       auth_tlv->auth_type = cpu_to_le16((u16) priv->sec_info.
-                                                         authentication_mode);
-               else
-                       auth_tlv->auth_type =
-                               cpu_to_le16(MWIFIEX_AUTH_MODE_OPEN);
-               pos += sizeof(auth_tlv->header) +
-                       le16_to_cpu(auth_tlv->header.len);
-       }
+       /* Add the Authentication type to be used for Auth frames */
+       auth_tlv = (struct mwifiex_ie_types_auth_type *) pos;
+       auth_tlv->header.type = cpu_to_le16(TLV_TYPE_AUTH_TYPE);
+       auth_tlv->header.len = cpu_to_le16(sizeof(auth_tlv->auth_type));
+       if (priv->sec_info.wep_status == MWIFIEX_802_11_WEP_ENABLED)
+               auth_tlv->auth_type = cpu_to_le16(
+                               (u16) priv->sec_info.authentication_mode);
+       else
+               auth_tlv->auth_type = cpu_to_le16(MWIFIEX_AUTH_MODE_OPEN);
+
+       pos += sizeof(auth_tlv->header) + le16_to_cpu(auth_tlv->header.len);
 
        if (IS_SUPPORT_MULTI_BANDS(priv->adapter)
            && !(ISSUPP_11NENABLED(priv->adapter->fw_cap_info)

diff --git a/drivers/net/wireless/mwifiex/main.h b/drivers/net/wireless/mwifiex/main.h
index 2b0ad8e3d6e21568627de8c29f384293a89dadfa..f6fe1054a65c51574400601f1d6d53db72e14d41 100644 (file)
--- a/drivers/net/wireless/mwifiex/main.h
+++ b/drivers/net/wireless/mwifiex/main.h
@@ -994,11 +994,6 @@ int mwifiex_get_channel_list(struct mwifiex_private *priv,
 int mwifiex_get_scan_table(struct mwifiex_private *priv,
                           u8 wait_option,
                           struct mwifiex_scan_resp *scanresp);
-int mwifiex_get_auth_mode(struct mwifiex_private *priv,
-                         u8 wait_option, u32 *auth_mode);
-int mwifiex_get_encrypt_mode(struct mwifiex_private *priv,
-                            u8 wait_option,
-                            u32 *encrypt_mode);
 int mwifiex_enable_wep_key(struct mwifiex_private *priv, u8 wait_option);
 int mwifiex_find_best_bss(struct mwifiex_private *priv, u8 wait_option,
                          struct mwifiex_ssid_bssid *ssid_bssid);
@@ -1014,9 +1009,6 @@ int mwifiex_drv_get_mode(struct mwifiex_private *priv, u8 wait_option);
 
 int mwifiex_drv_change_adhoc_chan(struct mwifiex_private *priv, int channel);
 
-int mwifiex_set_auth(struct mwifiex_private *priv, int encrypt_mode,
-                    int auth_mode, int wpa_enabled);
-
 int mwifiex_set_encode(struct mwifiex_private *priv, const u8 *key,
                       int key_len, u8 key_index, int disable);
 

diff --git a/drivers/net/wireless/mwifiex/sta_ioctl.c b/drivers/net/wireless/mwifiex/sta_ioctl.c
index 665a519b14036b10e15434fa5126769c287177e7..362301f417a4462c4d61b3f0a279437aa3d7e212 100644 (file)
--- a/drivers/net/wireless/mwifiex/sta_ioctl.c
+++ b/drivers/net/wireless/mwifiex/sta_ioctl.c
@@ -1690,20 +1690,6 @@ static int mwifiex_sec_ioctl_set_wapi_key(struct mwifiex_adapter *adapter,
        return ret;
 }
 
-/*
- * IOCTL request handler to set/get authentication mode.
- */
-static int mwifiex_set_auth_mode(struct mwifiex_private *priv, u32 auth_mode)
-{
-       int ret = 0;
-
-       priv->sec_info.authentication_mode = auth_mode;
-       if (priv->sec_info.authentication_mode == MWIFIEX_AUTH_MODE_NETWORKEAP)
-               ret = mwifiex_set_wpa_ie_helper(priv, NULL, 0);
-
-       return ret;
-}
-
 /*
  * IOCTL request handler to set WEP network key.
  *
@@ -1998,36 +1984,6 @@ int mwifiex_get_signal_info(struct mwifiex_private *priv, u8 wait_option,
        return status;
 }
 
-/*
- * Sends IOCTL request to set encryption mode.
- *
- * This function allocates the IOCTL request buffer, fills it
- * with requisite parameters and calls the IOCTL handler.
- */
-static int mwifiex_set_encrypt_mode(struct mwifiex_private *priv,
-                                   u8 wait_option, u32 encrypt_mode)
-{
-       priv->sec_info.encryption_mode = encrypt_mode;
-       return 0;
-}
-
-/*
- * This function set the authentication parameters. It sets both encryption
- * mode and authentication mode, and also enables WPA if required.
- */
-int
-mwifiex_set_auth(struct mwifiex_private *priv, int encrypt_mode,
-                int auth_mode, int wpa_enabled)
-{
-       if (mwifiex_set_encrypt_mode(priv, MWIFIEX_IOCTL_WAIT, encrypt_mode))
-               return -EFAULT;
-
-       if (mwifiex_set_auth_mode(priv, auth_mode))
-               return -EFAULT;
-
-       return 0;
-}
-
 /*
  * Sends IOCTL request to set encoding parameters.
  *