ath6kl: Remove redundant key_index check.

Less-than-zero comparison of an unsigned value is never true.

kvalo: remove WMI_MIN_KEY_INDEX altogether, it's useless

Signed-off-by: Vivek Natarajan <nataraja@qca.qualcomm.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>

3 files changed, 6 insertions, 8 deletions

diff --git a/drivers/net/wireless/ath/ath6kl/cfg80211.c b/drivers/net/wireless/ath/ath6kl/cfg80211.c
index c756425abf73..2a166cc7cc3c 100644
--- a/drivers/net/wireless/ath/ath6kl/cfg80211.c
+++ b/drivers/net/wireless/ath/ath6kl/cfg80211.c
@@ -524,8 +524,7 @@ static int ath6kl_cfg80211_connect(struct wiphy *wiphy, struct net_device *dev,
             (vif->prwise_crypto == WEP_CRYPT)) {
                 struct ath6kl_key *key = NULL;
 
-                if (sme->key_idx < WMI_MIN_KEY_INDEX ||
-                    sme->key_idx > WMI_MAX_KEY_INDEX) {
+                if (sme->key_idx > WMI_MAX_KEY_INDEX) {
                         ath6kl_err("key index %d out of bounds\n",
                                    sme->key_idx);
                         up(&ar->sem);
@@ -997,7 +996,7 @@ static int ath6kl_cfg80211_add_key(struct wiphy *wiphy, struct net_device *ndev,
                                               params->key);
         }
 
-        if (key_index < WMI_MIN_KEY_INDEX || key_index > WMI_MAX_KEY_INDEX) {
+        if (key_index > WMI_MAX_KEY_INDEX) {
                 ath6kl_dbg(ATH6KL_DBG_WLAN_CFG,
                            "%s: key index %d out of bounds\n", __func__,
                            key_index);
@@ -1115,7 +1114,7 @@ static int ath6kl_cfg80211_del_key(struct wiphy *wiphy, struct net_device *ndev,
         if (!ath6kl_cfg80211_ready(vif))
                 return -EIO;
 
-        if (key_index < WMI_MIN_KEY_INDEX || key_index > WMI_MAX_KEY_INDEX) {
+        if (key_index > WMI_MAX_KEY_INDEX) {
                 ath6kl_dbg(ATH6KL_DBG_WLAN_CFG,
                            "%s: key index %d out of bounds\n", __func__,
                            key_index);
@@ -1148,7 +1147,7 @@ static int ath6kl_cfg80211_get_key(struct wiphy *wiphy, struct net_device *ndev,
         if (!ath6kl_cfg80211_ready(vif))
                 return -EIO;
 
-        if (key_index < WMI_MIN_KEY_INDEX || key_index > WMI_MAX_KEY_INDEX) {
+        if (key_index > WMI_MAX_KEY_INDEX) {
                 ath6kl_dbg(ATH6KL_DBG_WLAN_CFG,
                            "%s: key index %d out of bounds\n", __func__,
                            key_index);
@@ -1184,7 +1183,7 @@ static int ath6kl_cfg80211_set_default_key(struct wiphy *wiphy,
         if (!ath6kl_cfg80211_ready(vif))
                 return -EIO;
 
-        if (key_index < WMI_MIN_KEY_INDEX || key_index > WMI_MAX_KEY_INDEX) {
+        if (key_index > WMI_MAX_KEY_INDEX) {
                 ath6kl_dbg(ATH6KL_DBG_WLAN_CFG,
                            "%s: key index %d out of bounds\n",
                            __func__, key_index);
diff --git a/drivers/net/wireless/ath/ath6kl/main.c b/drivers/net/wireless/ath/ath6kl/main.c
index f3d0184a5cea..f74986d62098 100644
--- a/drivers/net/wireless/ath/ath6kl/main.c
+++ b/drivers/net/wireless/ath/ath6kl/main.c
@@ -363,7 +363,7 @@ static void ath6kl_install_static_wep_keys(struct ath6kl_vif *vif)
         u8 index;
         u8 keyusage;
 
-        for (index = WMI_MIN_KEY_INDEX; index <= WMI_MAX_KEY_INDEX; index++) {
+        for (index = 0; index <= WMI_MAX_KEY_INDEX; index++) {
                 if (vif->wep_key_list[index].key_len) {
                         keyusage = GROUP_USAGE;
                         if (index == vif->def_txkey_index)
diff --git a/drivers/net/wireless/ath/ath6kl/wmi.h b/drivers/net/wireless/ath/ath6kl/wmi.h
index 9f8425b97db7..2b435994b013 100644
--- a/drivers/net/wireless/ath/ath6kl/wmi.h
+++ b/drivers/net/wireless/ath/ath6kl/wmi.h
@@ -649,7 +649,6 @@ enum auth_mode {
         WPA2_AUTH_CCKM = 0x40,
 };
 
-#define WMI_MIN_KEY_INDEX   0
 #define WMI_MAX_KEY_INDEX   3
 
 #define WMI_MAX_KEY_LEN     32