On Sat, 28 Oct 2023 17:54:27 +0300, Kalle Valo wrote: >Shiji Yang <yangshiji66@xxxxxxxxxxx> writes: > >> Disable the RTS threshold for OFDM and CCK rates by default as the >> initial RTS threshold is 'IEEE80211_MAX_RTS_THRESHOLD'. And RTS >> thresholds for all other rates have already been disabled when init. >> >> Signed-off-by: Shiji Yang <yangshiji66@xxxxxxxxxxx> > >The commit log should always answer to the question "Why?". What problem >does this patch fix? Hi! Thanks for your review. rt2800 has a lot of registers to control the RTS enable/disable status for different rates. And the driver control them via rt2800_set_rts_threshold(). I found that when RTS was disabled in user interface, this function won't be called at all. This means that the RTS is still 'on' for CCK and OFDM rates. So we'd better to disable them by default in case they did some bad things. The RTS for HT20 and HT40 is already default off so we don't need to touch them. If we toggle the RTS status, these register bits will be enabled/disabled again by rt2800_set_rts_threshold(). If this patch is acceptable, I will add more explanations in the v2 patch. Anyway, I don't know if it really solves some existing problems, but I think it should be like this. Regards, Shiji Yang
