iwlwifi: use #define instead of hard coded value
Instead of hard coded value, use the define in iwl-commands.h for better code maintenance Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com> Signed-off-by: Reinette Chatre <reinette.chatre@intel.com> Signed-off-by: Tomas Winkler <tomas.winkler@intel.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
parent
43121432e2
commit
fff7a4346c
|
@ -960,10 +960,10 @@ void iwl_set_rxon_chain(struct iwl_priv *priv)
|
||||||
if (iwl_is_monitor_mode(priv) &&
|
if (iwl_is_monitor_mode(priv) &&
|
||||||
!(priv->staging_rxon.flags & RXON_FLG_BAND_24G_MSK) &&
|
!(priv->staging_rxon.flags & RXON_FLG_BAND_24G_MSK) &&
|
||||||
((priv->hw_rev & CSR_HW_REV_TYPE_MSK) == CSR_HW_REV_TYPE_4965)) {
|
((priv->hw_rev & CSR_HW_REV_TYPE_MSK) == CSR_HW_REV_TYPE_4965)) {
|
||||||
rx_chain = 0x07 << RXON_RX_CHAIN_VALID_POS;
|
rx_chain = ANT_ABC << RXON_RX_CHAIN_VALID_POS;
|
||||||
rx_chain |= 0x06 << RXON_RX_CHAIN_FORCE_SEL_POS;
|
rx_chain |= ANT_BC << RXON_RX_CHAIN_FORCE_SEL_POS;
|
||||||
rx_chain |= 0x07 << RXON_RX_CHAIN_FORCE_MIMO_SEL_POS;
|
rx_chain |= ANT_ABC << RXON_RX_CHAIN_FORCE_MIMO_SEL_POS;
|
||||||
rx_chain |= 0x01 << RXON_RX_CHAIN_DRIVER_FORCE_POS;
|
rx_chain |= 0x1 << RXON_RX_CHAIN_DRIVER_FORCE_POS;
|
||||||
}
|
}
|
||||||
|
|
||||||
priv->staging_rxon.rx_chain = cpu_to_le16(rx_chain);
|
priv->staging_rxon.rx_chain = cpu_to_le16(rx_chain);
|
||||||
|
|
|
@ -580,9 +580,10 @@ static void iwl_bg_request_scan(struct work_struct *data)
|
||||||
int ret = 0;
|
int ret = 0;
|
||||||
u32 rate_flags = 0;
|
u32 rate_flags = 0;
|
||||||
u16 cmd_len;
|
u16 cmd_len;
|
||||||
|
u16 rx_chain = 0;
|
||||||
enum ieee80211_band band;
|
enum ieee80211_band band;
|
||||||
u8 n_probes = 0;
|
u8 n_probes = 0;
|
||||||
u8 rx_chain = priv->hw_params.valid_rx_ant;
|
u8 rx_ant = priv->hw_params.valid_rx_ant;
|
||||||
u8 rate;
|
u8 rate;
|
||||||
bool is_active = false;
|
bool is_active = false;
|
||||||
|
|
||||||
|
@ -723,7 +724,7 @@ static void iwl_bg_request_scan(struct work_struct *data)
|
||||||
* Avoid A (0x1) because of its off-channel reception on A-band.
|
* Avoid A (0x1) because of its off-channel reception on A-band.
|
||||||
*/
|
*/
|
||||||
if ((priv->hw_rev & CSR_HW_REV_TYPE_MSK) == CSR_HW_REV_TYPE_4965)
|
if ((priv->hw_rev & CSR_HW_REV_TYPE_MSK) == CSR_HW_REV_TYPE_4965)
|
||||||
rx_chain = 0x6;
|
rx_ant = ANT_BC;
|
||||||
} else {
|
} else {
|
||||||
IWL_WARN(priv, "Invalid scan band count\n");
|
IWL_WARN(priv, "Invalid scan band count\n");
|
||||||
goto done;
|
goto done;
|
||||||
|
@ -735,10 +736,11 @@ static void iwl_bg_request_scan(struct work_struct *data)
|
||||||
scan->tx_cmd.rate_n_flags = iwl_hw_set_rate_n_flags(rate, rate_flags);
|
scan->tx_cmd.rate_n_flags = iwl_hw_set_rate_n_flags(rate, rate_flags);
|
||||||
|
|
||||||
/* MIMO is not used here, but value is required */
|
/* MIMO is not used here, but value is required */
|
||||||
scan->rx_chain = RXON_RX_CHAIN_DRIVER_FORCE_MSK |
|
rx_chain |= ANT_ABC << RXON_RX_CHAIN_VALID_POS;
|
||||||
cpu_to_le16((0x7 << RXON_RX_CHAIN_VALID_POS) |
|
rx_chain |= ANT_ABC << RXON_RX_CHAIN_FORCE_MIMO_SEL_POS;
|
||||||
(rx_chain << RXON_RX_CHAIN_FORCE_SEL_POS) |
|
rx_chain |= rx_ant << RXON_RX_CHAIN_FORCE_SEL_POS;
|
||||||
(0x7 << RXON_RX_CHAIN_FORCE_MIMO_SEL_POS));
|
rx_chain |= 0x1 << RXON_RX_CHAIN_DRIVER_FORCE_POS;
|
||||||
|
scan->rx_chain = cpu_to_le16(rx_chain);
|
||||||
cmd_len = iwl_fill_probe_req(priv,
|
cmd_len = iwl_fill_probe_req(priv,
|
||||||
(struct ieee80211_mgmt *)scan->data,
|
(struct ieee80211_mgmt *)scan->data,
|
||||||
priv->scan_request->ie,
|
priv->scan_request->ie,
|
||||||
|
|
Loading…
Reference in New Issue