Commit e126b5d9 authored by Johannes Berg's avatar Johannes Berg

iwlwifi: mvm: remove unneeded argument from iwl_mvm_tx_protection()

The LQ command argument isn't needed, it's always taken from the
station struct that's already passed, remove the argument.
Reviewed-by: default avatarEmmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: default avatarJohannes Berg <johannes@sipsolutions.net>
parent 2be01fa8
...@@ -3193,13 +3193,14 @@ void iwl_mvm_rate_control_unregister(void) ...@@ -3193,13 +3193,14 @@ void iwl_mvm_rate_control_unregister(void)
* iwl_mvm_tx_protection - Gets LQ command, change it to enable/disable * iwl_mvm_tx_protection - Gets LQ command, change it to enable/disable
* Tx protection, according to this rquest and previous requests, * Tx protection, according to this rquest and previous requests,
* and send the LQ command. * and send the LQ command.
* @lq: The LQ command
* @mvmsta: The station * @mvmsta: The station
* @enable: Enable Tx protection? * @enable: Enable Tx protection?
*/ */
int iwl_mvm_tx_protection(struct iwl_mvm *mvm, struct iwl_lq_cmd *lq, int iwl_mvm_tx_protection(struct iwl_mvm *mvm, struct iwl_mvm_sta *mvmsta,
struct iwl_mvm_sta *mvmsta, bool enable) bool enable)
{ {
struct iwl_lq_cmd *lq = &mvmsta->lq_sta.lq;
lockdep_assert_held(&mvm->mutex); lockdep_assert_held(&mvm->mutex);
if (enable) { if (enable) {
......
...@@ -404,7 +404,7 @@ extern void iwl_mvm_rate_control_unregister(void); ...@@ -404,7 +404,7 @@ extern void iwl_mvm_rate_control_unregister(void);
struct iwl_mvm_sta; struct iwl_mvm_sta;
int iwl_mvm_tx_protection(struct iwl_mvm *mvm, struct iwl_lq_cmd *lq, int iwl_mvm_tx_protection(struct iwl_mvm *mvm, struct iwl_mvm_sta *mvmsta,
struct iwl_mvm_sta *mvmsta, bool enable); bool enable);
#endif /* __rs__ */ #endif /* __rs__ */
...@@ -807,8 +807,7 @@ int iwl_mvm_sta_tx_agg_oper(struct iwl_mvm *mvm, struct ieee80211_vif *vif, ...@@ -807,8 +807,7 @@ int iwl_mvm_sta_tx_agg_oper(struct iwl_mvm *mvm, struct ieee80211_vif *vif,
* method for HT traffic * method for HT traffic
* this function also sends the LQ command * this function also sends the LQ command
*/ */
return iwl_mvm_tx_protection(mvm, &mvmsta->lq_sta.lq, return iwl_mvm_tx_protection(mvm, mvmsta, true);
mvmsta, true);
/* /*
* TODO: remove the TLC_RTS flag when we tear down the last * TODO: remove the TLC_RTS flag when we tear down the last
* AGG session (agg_tids_count in DVM) * AGG session (agg_tids_count in DVM)
......
...@@ -391,8 +391,7 @@ static void iwl_mvm_tt_tx_protection(struct iwl_mvm *mvm, bool enable) ...@@ -391,8 +391,7 @@ static void iwl_mvm_tt_tx_protection(struct iwl_mvm *mvm, bool enable)
mvmsta = (void *)sta->drv_priv; mvmsta = (void *)sta->drv_priv;
if (enable == mvmsta->tt_tx_protection) if (enable == mvmsta->tt_tx_protection)
continue; continue;
err = iwl_mvm_tx_protection(mvm, &mvmsta->lq_sta.lq, err = iwl_mvm_tx_protection(mvm, mvmsta, enable);
mvmsta, enable);
if (err) { if (err) {
IWL_ERR(mvm, "Failed to %s Tx protection\n", IWL_ERR(mvm, "Failed to %s Tx protection\n",
enable ? "enable" : "disable"); enable ? "enable" : "disable");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment