iwl3945: Getting rid of priv->antenna
The iwl_priv antenna field is useless as we can simply use the corresponding mod_params antenna field. Signed-off-by: Samuel Ortiz <samuel.ortiz@intel.com> Signed-off-by: Reinette Chatre <reinette.chatre@intel.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
parent
d08853a399
commit
7e4bca5e5b
3 changed files with 39 additions and 41 deletions
|
@ -197,41 +197,6 @@ static int iwl3945_hwrate_to_plcp_idx(u8 plcp)
|
|||
return -1;
|
||||
}
|
||||
|
||||
/**
|
||||
* iwl3945_get_antenna_flags - Get antenna flags for RXON command
|
||||
* @priv: eeprom and antenna fields are used to determine antenna flags
|
||||
*
|
||||
* priv->eeprom39 is used to determine if antenna AUX/MAIN are reversed
|
||||
* priv->antenna specifies the antenna diversity mode:
|
||||
*
|
||||
* IWL_ANTENNA_DIVERSITY - NIC selects best antenna by itself
|
||||
* IWL_ANTENNA_MAIN - Force MAIN antenna
|
||||
* IWL_ANTENNA_AUX - Force AUX antenna
|
||||
*/
|
||||
__le32 iwl3945_get_antenna_flags(const struct iwl_priv *priv)
|
||||
{
|
||||
struct iwl3945_eeprom *eeprom = (struct iwl3945_eeprom *)priv->eeprom;
|
||||
|
||||
switch (priv->antenna) {
|
||||
case IWL_ANTENNA_DIVERSITY:
|
||||
return 0;
|
||||
|
||||
case IWL_ANTENNA_MAIN:
|
||||
if (eeprom->antenna_switch_type)
|
||||
return RXON_FLG_DIS_DIV_MSK | RXON_FLG_ANT_B_MSK;
|
||||
return RXON_FLG_DIS_DIV_MSK | RXON_FLG_ANT_A_MSK;
|
||||
|
||||
case IWL_ANTENNA_AUX:
|
||||
if (eeprom->antenna_switch_type)
|
||||
return RXON_FLG_DIS_DIV_MSK | RXON_FLG_ANT_A_MSK;
|
||||
return RXON_FLG_DIS_DIV_MSK | RXON_FLG_ANT_B_MSK;
|
||||
}
|
||||
|
||||
/* bad antenna selector value */
|
||||
IWL_ERR(priv, "Bad antenna selector value (0x%x)\n", priv->antenna);
|
||||
return 0; /* "diversity" is default if error */
|
||||
}
|
||||
|
||||
#ifdef CONFIG_IWLWIFI_DEBUG
|
||||
#define TX_STATUS_ENTRY(x) case TX_STATUS_FAIL_ ## x: return #x
|
||||
|
||||
|
|
|
@ -978,7 +978,6 @@ struct iwl_priv {
|
|||
u16 rates_mask;
|
||||
|
||||
u32 power_mode;
|
||||
u32 antenna;
|
||||
u8 bssid[ETH_ALEN];
|
||||
u16 rts_threshold;
|
||||
u8 mac_addr[ETH_ALEN];
|
||||
|
|
|
@ -447,6 +447,43 @@ static int iwl3945_send_rxon_assoc(struct iwl_priv *priv)
|
|||
return rc;
|
||||
}
|
||||
|
||||
/**
|
||||
* iwl3945_get_antenna_flags - Get antenna flags for RXON command
|
||||
* @priv: eeprom and antenna fields are used to determine antenna flags
|
||||
*
|
||||
* priv->eeprom39 is used to determine if antenna AUX/MAIN are reversed
|
||||
* iwl3945_mod_params.antenna specifies the antenna diversity mode:
|
||||
*
|
||||
* IWL_ANTENNA_DIVERSITY - NIC selects best antenna by itself
|
||||
* IWL_ANTENNA_MAIN - Force MAIN antenna
|
||||
* IWL_ANTENNA_AUX - Force AUX antenna
|
||||
*/
|
||||
__le32 iwl3945_get_antenna_flags(const struct iwl_priv *priv)
|
||||
{
|
||||
struct iwl3945_eeprom *eeprom = (struct iwl3945_eeprom *)priv->eeprom;
|
||||
|
||||
switch (iwl3945_mod_params.antenna) {
|
||||
case IWL_ANTENNA_DIVERSITY:
|
||||
return 0;
|
||||
|
||||
case IWL_ANTENNA_MAIN:
|
||||
if (eeprom->antenna_switch_type)
|
||||
return RXON_FLG_DIS_DIV_MSK | RXON_FLG_ANT_B_MSK;
|
||||
return RXON_FLG_DIS_DIV_MSK | RXON_FLG_ANT_A_MSK;
|
||||
|
||||
case IWL_ANTENNA_AUX:
|
||||
if (eeprom->antenna_switch_type)
|
||||
return RXON_FLG_DIS_DIV_MSK | RXON_FLG_ANT_A_MSK;
|
||||
return RXON_FLG_DIS_DIV_MSK | RXON_FLG_ANT_B_MSK;
|
||||
}
|
||||
|
||||
/* bad antenna selector value */
|
||||
IWL_ERR(priv, "Bad antenna selector value (0x%x)\n",
|
||||
iwl3945_mod_params.antenna);
|
||||
|
||||
return 0; /* "diversity" is default if error */
|
||||
}
|
||||
|
||||
/**
|
||||
* iwl3945_commit_rxon - commit staging_rxon to hardware
|
||||
*
|
||||
|
@ -5804,7 +5841,7 @@ static ssize_t show_antenna(struct device *d,
|
|||
if (!iwl_is_alive(priv))
|
||||
return -EAGAIN;
|
||||
|
||||
return sprintf(buf, "%d\n", priv->antenna);
|
||||
return sprintf(buf, "%d\n", iwl3945_mod_params.antenna);
|
||||
}
|
||||
|
||||
static ssize_t store_antenna(struct device *d,
|
||||
|
@ -5824,7 +5861,7 @@ static ssize_t store_antenna(struct device *d,
|
|||
|
||||
if ((ant >= 0) && (ant <= 2)) {
|
||||
IWL_DEBUG_INFO("Setting antenna select to %d.\n", ant);
|
||||
priv->antenna = (enum iwl3945_antenna)ant;
|
||||
iwl3945_mod_params.antenna = (enum iwl3945_antenna)ant;
|
||||
} else
|
||||
IWL_DEBUG_INFO("Bad antenna select value %d.\n", ant);
|
||||
|
||||
|
@ -6089,9 +6126,6 @@ static int iwl3945_pci_probe(struct pci_dev *pdev, const struct pci_device_id *e
|
|||
hw->rate_control_algorithm = "iwl-3945-rs";
|
||||
hw->sta_data_size = sizeof(struct iwl3945_sta_priv);
|
||||
|
||||
/* Select antenna (may be helpful if only one antenna is connected) */
|
||||
priv->antenna = (enum iwl3945_antenna)iwl3945_mod_params.antenna;
|
||||
|
||||
/* Tell mac80211 our characteristics */
|
||||
hw->flags = IEEE80211_HW_SIGNAL_DBM |
|
||||
IEEE80211_HW_NOISE_DBM;
|
||||
|
|
Loading…
Reference in a new issue