iwlwifi: nvm: remove reading valid antennas from NVM
authorJohannes Berg <johannes.berg@intel.com>
Wed, 12 Feb 2014 21:02:38 +0000 (22:02 +0100)
committerEmmanuel Grumbach <emmanuel.grumbach@intel.com>
Sun, 23 Feb 2014 17:27:26 +0000 (19:27 +0200)
These values aren't used as the firmware values should be used,
so reading them is pointless and hides potential errors when
somebody uses these values. Leave them zero to make it clearer
that they can't be used. We can't remove the struct members as
the DVM driver does read the values from EEPROM/OTP and the
structure is shared between drivers.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
drivers/net/wireless/iwlwifi/iwl-nvm-parse.c
drivers/net/wireless/iwlwifi/mvm/fw.c

index 80c40967cd35602d35383ed34c25f57c04330272..eeb3a838c1bd1be79eb49789e2a9c3a6c452897b 100644 (file)
@@ -425,8 +425,6 @@ static void iwl_set_radio_cfg(const struct iwl_cfg *cfg,
                data->radio_cfg_step = NVM_RF_CFG_STEP_MSK(radio_cfg);
                data->radio_cfg_dash = NVM_RF_CFG_DASH_MSK(radio_cfg);
                data->radio_cfg_pnum = NVM_RF_CFG_PNUM_MSK(radio_cfg);
-               data->valid_tx_ant = NVM_RF_CFG_TX_ANT_MSK(radio_cfg);
-               data->valid_rx_ant = NVM_RF_CFG_RX_ANT_MSK(radio_cfg);
                return;
        }
 
@@ -435,8 +433,6 @@ static void iwl_set_radio_cfg(const struct iwl_cfg *cfg,
        data->radio_cfg_step = NVM_RF_CFG_STEP_MSK_FAMILY_8000(radio_cfg);
        data->radio_cfg_dash = NVM_RF_CFG_DASH_MSK_FAMILY_8000(radio_cfg);
        data->radio_cfg_pnum = NVM_RF_CFG_FLAVOR_MSK_FAMILY_8000(radio_cfg);
-       data->valid_tx_ant = NVM_RF_CFG_TX_ANT_MSK_FAMILY_8000(radio_cfg);
-       data->valid_rx_ant = NVM_RF_CFG_RX_ANT_MSK_FAMILY_8000(radio_cfg);
 }
 
 static void iwl_set_hw_address(const struct iwl_cfg *cfg,
@@ -496,19 +492,6 @@ iwl_parse_nvm_data(struct device *dev, const struct iwl_cfg *cfg,
        if (iwlwifi_mod_params.disable_11n & IWL_DISABLE_HT_ALL)
                data->sku_cap_11n_enable = false;
 
-       /* check overrides (some devices have wrong NVM) */
-       if (cfg->valid_tx_ant)
-               data->valid_tx_ant = cfg->valid_tx_ant;
-       if (cfg->valid_rx_ant)
-               data->valid_rx_ant = cfg->valid_rx_ant;
-
-       if (!data->valid_tx_ant || !data->valid_rx_ant) {
-               IWL_ERR_DEV(dev, "invalid antennas (0x%x, 0x%x)\n",
-                           data->valid_tx_ant, data->valid_rx_ant);
-               kfree(data);
-               return NULL;
-       }
-
        data->n_hw_addrs = iwl_get_n_hw_addrs(cfg, nvm_sw);
 
        if (cfg->device_family != IWL_DEVICE_FAMILY_8000) {
index 979b35bae056ef7ee485c4c5c1d3f9979a5efcd6..93e1b4d64869818740cce91b4bbe4d784e88499d 100644 (file)
@@ -358,8 +358,6 @@ out:
                                        GFP_KERNEL);
                if (!mvm->nvm_data)
                        return -ENOMEM;
-               mvm->nvm_data->valid_rx_ant = 1;
-               mvm->nvm_data->valid_tx_ant = 1;
                mvm->nvm_data->bands[0].channels = mvm->nvm_data->channels;
                mvm->nvm_data->bands[0].n_channels = 1;
                mvm->nvm_data->bands[0].n_bitrates = 1;