diff options
author | Wey-Yi Guy <wey-yi.w.guy@intel.com> | 2011-07-07 11:33:01 -0400 |
---|---|---|
committer | Wey-Yi Guy <wey-yi.w.guy@intel.com> | 2011-07-16 10:37:28 -0400 |
commit | 90c300cbd89e76789dbff101a1cb1ec226af277f (patch) | |
tree | 2b25a09768aab208765fc28795bdd7c1874a9199 /drivers/net/wireless/iwlwifi/iwl-agn.c | |
parent | e505c433d35900d98870a2e266759166a03030dc (diff) |
iwlagn: remove dual-indirect call to simply the code
After driver split, no need to make the code so complex
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
Diffstat (limited to 'drivers/net/wireless/iwlwifi/iwl-agn.c')
-rw-r--r-- | drivers/net/wireless/iwlwifi/iwl-agn.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/drivers/net/wireless/iwlwifi/iwl-agn.c b/drivers/net/wireless/iwlwifi/iwl-agn.c index dceb45063669..41c3e15104ad 100644 --- a/drivers/net/wireless/iwlwifi/iwl-agn.c +++ b/drivers/net/wireless/iwlwifi/iwl-agn.c | |||
@@ -375,7 +375,7 @@ static void iwl_continuous_event_trace(struct iwl_priv *priv) | |||
375 | u32 next_entry; /* index of next entry to be written by uCode */ | 375 | u32 next_entry; /* index of next entry to be written by uCode */ |
376 | 376 | ||
377 | base = priv->device_pointers.error_event_table; | 377 | base = priv->device_pointers.error_event_table; |
378 | if (priv->cfg->ops->lib->is_valid_rtc_data_addr(base)) { | 378 | if (priv->cfg->lib->is_valid_rtc_data_addr(base)) { |
379 | capacity = iwl_read_targ_mem(priv, base); | 379 | capacity = iwl_read_targ_mem(priv, base); |
380 | num_wraps = iwl_read_targ_mem(priv, base + (2 * sizeof(u32))); | 380 | num_wraps = iwl_read_targ_mem(priv, base + (2 * sizeof(u32))); |
381 | mode = iwl_read_targ_mem(priv, base + (1 * sizeof(u32))); | 381 | mode = iwl_read_targ_mem(priv, base + (1 * sizeof(u32))); |
@@ -1614,7 +1614,7 @@ void iwl_dump_nic_error_log(struct iwl_priv *priv) | |||
1614 | base = priv->_agn.inst_errlog_ptr; | 1614 | base = priv->_agn.inst_errlog_ptr; |
1615 | } | 1615 | } |
1616 | 1616 | ||
1617 | if (!priv->cfg->ops->lib->is_valid_rtc_data_addr(base)) { | 1617 | if (!priv->cfg->lib->is_valid_rtc_data_addr(base)) { |
1618 | IWL_ERR(priv, | 1618 | IWL_ERR(priv, |
1619 | "Not valid error log pointer 0x%08X for %s uCode\n", | 1619 | "Not valid error log pointer 0x%08X for %s uCode\n", |
1620 | base, | 1620 | base, |
@@ -1805,7 +1805,7 @@ int iwl_dump_nic_event_log(struct iwl_priv *priv, bool full_log, | |||
1805 | base = priv->_agn.inst_evtlog_ptr; | 1805 | base = priv->_agn.inst_evtlog_ptr; |
1806 | } | 1806 | } |
1807 | 1807 | ||
1808 | if (!priv->cfg->ops->lib->is_valid_rtc_data_addr(base)) { | 1808 | if (!priv->cfg->lib->is_valid_rtc_data_addr(base)) { |
1809 | IWL_ERR(priv, | 1809 | IWL_ERR(priv, |
1810 | "Invalid event log pointer 0x%08X for %s uCode\n", | 1810 | "Invalid event log pointer 0x%08X for %s uCode\n", |
1811 | base, | 1811 | base, |
@@ -2939,7 +2939,7 @@ static void iwlagn_mac_channel_switch(struct ieee80211_hw *hw, | |||
2939 | if (!iwl_is_associated_ctx(ctx)) | 2939 | if (!iwl_is_associated_ctx(ctx)) |
2940 | goto out; | 2940 | goto out; |
2941 | 2941 | ||
2942 | if (!priv->cfg->ops->lib->set_channel_switch) | 2942 | if (!priv->cfg->lib->set_channel_switch) |
2943 | goto out; | 2943 | goto out; |
2944 | 2944 | ||
2945 | ch = channel->hw_value; | 2945 | ch = channel->hw_value; |
@@ -2991,7 +2991,7 @@ static void iwlagn_mac_channel_switch(struct ieee80211_hw *hw, | |||
2991 | */ | 2991 | */ |
2992 | set_bit(STATUS_CHANNEL_SWITCH_PENDING, &priv->status); | 2992 | set_bit(STATUS_CHANNEL_SWITCH_PENDING, &priv->status); |
2993 | priv->switch_channel = cpu_to_le16(ch); | 2993 | priv->switch_channel = cpu_to_le16(ch); |
2994 | if (priv->cfg->ops->lib->set_channel_switch(priv, ch_switch)) { | 2994 | if (priv->cfg->lib->set_channel_switch(priv, ch_switch)) { |
2995 | clear_bit(STATUS_CHANNEL_SWITCH_PENDING, &priv->status); | 2995 | clear_bit(STATUS_CHANNEL_SWITCH_PENDING, &priv->status); |
2996 | priv->switch_channel = 0; | 2996 | priv->switch_channel = 0; |
2997 | ieee80211_chswitch_done(ctx->vif, false); | 2997 | ieee80211_chswitch_done(ctx->vif, false); |
@@ -3198,8 +3198,8 @@ static void iwl_setup_deferred_work(struct iwl_priv *priv) | |||
3198 | 3198 | ||
3199 | iwl_setup_scan_deferred_work(priv); | 3199 | iwl_setup_scan_deferred_work(priv); |
3200 | 3200 | ||
3201 | if (priv->cfg->ops->lib->setup_deferred_work) | 3201 | if (priv->cfg->lib->setup_deferred_work) |
3202 | priv->cfg->ops->lib->setup_deferred_work(priv); | 3202 | priv->cfg->lib->setup_deferred_work(priv); |
3203 | 3203 | ||
3204 | init_timer(&priv->statistics_periodic); | 3204 | init_timer(&priv->statistics_periodic); |
3205 | priv->statistics_periodic.data = (unsigned long)priv; | 3205 | priv->statistics_periodic.data = (unsigned long)priv; |
@@ -3216,8 +3216,8 @@ static void iwl_setup_deferred_work(struct iwl_priv *priv) | |||
3216 | 3216 | ||
3217 | static void iwl_cancel_deferred_work(struct iwl_priv *priv) | 3217 | static void iwl_cancel_deferred_work(struct iwl_priv *priv) |
3218 | { | 3218 | { |
3219 | if (priv->cfg->ops->lib->cancel_deferred_work) | 3219 | if (priv->cfg->lib->cancel_deferred_work) |
3220 | priv->cfg->ops->lib->cancel_deferred_work(priv); | 3220 | priv->cfg->lib->cancel_deferred_work(priv); |
3221 | 3221 | ||
3222 | cancel_work_sync(&priv->run_time_calib_work); | 3222 | cancel_work_sync(&priv->run_time_calib_work); |
3223 | cancel_work_sync(&priv->beacon_update); | 3223 | cancel_work_sync(&priv->beacon_update); |
@@ -3397,7 +3397,7 @@ static int iwl_set_hw_params(struct iwl_priv *priv) | |||
3397 | priv->cfg->sku &= ~EEPROM_SKU_CAP_11N_ENABLE; | 3397 | priv->cfg->sku &= ~EEPROM_SKU_CAP_11N_ENABLE; |
3398 | 3398 | ||
3399 | /* Device-specific setup */ | 3399 | /* Device-specific setup */ |
3400 | return priv->cfg->ops->lib->set_hw_params(priv); | 3400 | return priv->cfg->lib->set_hw_params(priv); |
3401 | } | 3401 | } |
3402 | 3402 | ||
3403 | static const u8 iwlagn_bss_ac_to_fifo[] = { | 3403 | static const u8 iwlagn_bss_ac_to_fifo[] = { |