mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-22 17:01:14 +00:00
wifi: rtlwifi: Ignore IEEE80211_CONF_CHANGE_RETRY_LIMITS
commit819bda58e7
upstream. Since commit0a44dfc070
("wifi: mac80211: simplify non-chanctx drivers") ieee80211_hw_config() is no longer called with changed = ~0. rtlwifi relied on ~0 in order to ignore the default retry limits of 4/7, preferring 48/48 in station mode and 7/7 in AP/IBSS. RTL8192DU has a lot of packet loss with the default limits from mac80211. Fix it by ignoring IEEE80211_CONF_CHANGE_RETRY_LIMITS completely, because it's the simplest solution. Link: https://lore.kernel.org/linux-wireless/cedd13d7691f4692b2a2fa5a24d44a22@realtek.com/ Cc: stable@vger.kernel.org # 6.9.x Signed-off-by: Bitterblue Smith <rtl8821cerfe2@gmail.com> Acked-by: Ping-Ke Shih <pkshih@realtek.com> Signed-off-by: Kalle Valo <kvalo@kernel.org> Link: https://msgid.link/1fabb8e4-adf3-47ae-8462-8aea963bc2a5@gmail.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
f15e3e13e1
commit
6df6f10002
1 changed files with 0 additions and 15 deletions
|
@ -633,21 +633,6 @@ static int rtl_op_config(struct ieee80211_hw *hw, u32 changed)
|
|||
}
|
||||
}
|
||||
|
||||
if (changed & IEEE80211_CONF_CHANGE_RETRY_LIMITS) {
|
||||
rtl_dbg(rtlpriv, COMP_MAC80211, DBG_LOUD,
|
||||
"IEEE80211_CONF_CHANGE_RETRY_LIMITS %x\n",
|
||||
hw->conf.long_frame_max_tx_count);
|
||||
/* brought up everything changes (changed == ~0) indicates first
|
||||
* open, so use our default value instead of that of wiphy.
|
||||
*/
|
||||
if (changed != ~0) {
|
||||
mac->retry_long = hw->conf.long_frame_max_tx_count;
|
||||
mac->retry_short = hw->conf.long_frame_max_tx_count;
|
||||
rtlpriv->cfg->ops->set_hw_reg(hw, HW_VAR_RETRY_LIMIT,
|
||||
(u8 *)(&hw->conf.long_frame_max_tx_count));
|
||||
}
|
||||
}
|
||||
|
||||
if (changed & IEEE80211_CONF_CHANGE_CHANNEL &&
|
||||
!rtlpriv->proximity.proxim_on) {
|
||||
struct ieee80211_channel *channel = hw->conf.chandef.chan;
|
||||
|
|
Loading…
Reference in a new issue