RE: [PATCH rtw-next 3/5] wifi: rtl8xxxu: Fix the 40 MHz subchannel for RTL8192EU, RTL8723BU

[Ping-Ke Shih <pkshih@realtek.com>]

Bitterblue Smith <rtl8821cerfe2@gmail.com> wrote:
> rtl8xxxu_gen2_config_channel() was missing the subchannel setting.
> This function is used by RTL8192EU and RTL8723BU.
> 
> This change seems to make no difference in my testing on channel 13
> with either chip.
> 
> Signed-off-by: Bitterblue Smith <rtl8821cerfe2@gmail.com>

Reviewed-by: Ping-Ke Shih <pkshih@realtek.com>


> ---
>  drivers/net/wireless/realtek/rtl8xxxu/core.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/net/wireless/realtek/rtl8xxxu/core.c
> b/drivers/net/wireless/realtek/rtl8xxxu/core.c
> index 7700e4074dc3..739456640ca8 100644
> --- a/drivers/net/wireless/realtek/rtl8xxxu/core.c
> +++ b/drivers/net/wireless/realtek/rtl8xxxu/core.c
> @@ -1373,9 +1373,11 @@ void rtl8xxxu_gen2_config_channel(struct ieee80211_hw *hw)
>                     hw->conf.chandef.chan->center_freq) {
>                         sec_ch_above = 1;
>                         channel += 2;
> +                       subchannel = 2;

(For record)
By vendor driver, 'subchannel = 2' means primary channel at lower 20MHz.

>                 } else {
>                         sec_ch_above = 0;
>                         channel -= 2;
> +                       subchannel = 1;
>                 }
> 
>                 val32 = rtl8xxxu_read32(priv, REG_FPGA0_RF_MODE);
> --
> 2.51.1