diff mbox series

[2/2] wifi: ath10k: fix invalid VHT parameters in supported_vht_mcs_rate_nss2

Message ID 20240711020344.98040-3-quic_bqiang@quicinc.com (mailing list archive)
State Accepted
Commit 52db16ec5bae7bd027804265b968259d1a6c3970
Delegated to: Kalle Valo
Headers show
Series wifi: ath10k: fix invalid VHT parameters | expand

Commit Message

Baochen Qiang July 11, 2024, 2:03 a.m. UTC
In supported_vht_mcs_rate_nss2, the rate for MCS9 & VHT20 is defined as
{1560, 1733}, this does not align with firmware's definition and therefore
fails the verification in ath10k_mac_get_rate_flags_vht():

	invalid vht params rate 1730 100kbps nss 2 mcs 9
and:
	invalid vht params rate 1920 100kbps nss 2 mcs 9

Change it to {1730,  1920} to align with firmware, so this issue could be
fixed.

Since ath10k_hw_params::supports_peer_stats_info is enabled only for
QCA6174, this change does not affect other chips.

Tested-on: QCA6174 hw3.2 PCI WLAN.RM.4.4.1-00309-QCARMSWPZ-1

Fixes: 3344b99d69ab ("ath10k: add bitrate parse for peer stats info")
Reported-by: Paul Menzel <pmenzel@molgen.mpg.de>
Closes: https://lore.kernel.org/lkml/fba24cd3-4a1e-4072-8585-8402272788ff@molgen.mpg.de/
Signed-off-by: Baochen Qiang <quic_bqiang@quicinc.com>
---
 drivers/net/wireless/ath/ath10k/mac.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jeff Johnson July 22, 2024, 11:23 p.m. UTC | #1
On 7/10/2024 7:03 PM, Baochen Qiang wrote:
> In supported_vht_mcs_rate_nss2, the rate for MCS9 & VHT20 is defined as
> {1560, 1733}, this does not align with firmware's definition and therefore
> fails the verification in ath10k_mac_get_rate_flags_vht():
> 
> 	invalid vht params rate 1730 100kbps nss 2 mcs 9
> and:
> 	invalid vht params rate 1920 100kbps nss 2 mcs 9
> 
> Change it to {1730,  1920} to align with firmware, so this issue could be
> fixed.
> 
> Since ath10k_hw_params::supports_peer_stats_info is enabled only for
> QCA6174, this change does not affect other chips.
> 
> Tested-on: QCA6174 hw3.2 PCI WLAN.RM.4.4.1-00309-QCARMSWPZ-1
> 
> Fixes: 3344b99d69ab ("ath10k: add bitrate parse for peer stats info")
> Reported-by: Paul Menzel <pmenzel@molgen.mpg.de>
> Closes: https://lore.kernel.org/lkml/fba24cd3-4a1e-4072-8585-8402272788ff@molgen.mpg.de/
> Signed-off-by: Baochen Qiang <quic_bqiang@quicinc.com>
Acked-by: Jeff Johnson <quic_jjohnson@quicinc.com>
Paul Menzel Aug. 19, 2024, 6:13 p.m. UTC | #2
Dear Boachen,


Thank you for your patch, I found by chance. ;-) (It’d be great if you 
could carbon-copy me in the future.)

Am 11.07.24 um 04:03 schrieb Baochen Qiang:
> In supported_vht_mcs_rate_nss2, the rate for MCS9 & VHT20 is defined as
> {1560, 1733}, this does not align with firmware's definition and therefore
> fails the verification in ath10k_mac_get_rate_flags_vht():
> 
> 	invalid vht params rate 1730 100kbps nss 2 mcs 9
> and:

(I’d add blank lines around the *and*.)

> 	invalid vht params rate 1920 100kbps nss 2 mcs 9
> 
> Change it to {1730,  1920} to align with firmware, so this issue could be
> fixed.

… to fix the issue.

> Since ath10k_hw_params::supports_peer_stats_info is enabled only for
> QCA6174, this change does not affect other chips.
> 
> Tested-on: QCA6174 hw3.2 PCI WLAN.RM.4.4.1-00309-QCARMSWPZ-1

How did you test it? Were you able to reproduce the errors?

Anyway, with Linux v6.11-rc3-306-gdf6cbc62cc9b (Merge tag 'scsi-fixes' 
of git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi), I was able 
to reproduce the issue addressed in this patch at the airport BER (I 
guess they use Cisco), and applying your two patches, I wasn’t able to 
see the messages anymore.

> Fixes: 3344b99d69ab ("ath10k: add bitrate parse for peer stats info")
> Reported-by: Paul Menzel <pmenzel@molgen.mpg.de>
> Closes: https://lore.kernel.org/lkml/fba24cd3-4a1e-4072-8585-8402272788ff@molgen.mpg.de/
> Signed-off-by: Baochen Qiang <quic_bqiang@quicinc.com>
> ---
>   drivers/net/wireless/ath/ath10k/mac.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c
> index 3c34f3d8e555..e62b251405fc 100644
> --- a/drivers/net/wireless/ath/ath10k/mac.c
> +++ b/drivers/net/wireless/ath/ath10k/mac.c
> @@ -9136,7 +9136,7 @@ static const struct ath10k_index_vht_data_rate_type supported_vht_mcs_rate_nss2[
>   	{6,  {5265, 5850}, {2430, 2700}, {1170, 1300} },
>   	{7,  {5850, 6500}, {2700, 3000}, {1300, 1444} },
>   	{8,  {7020, 7800}, {3240, 3600}, {1560, 1733} },
> -	{9,  {7800, 8667}, {3600, 4000}, {1560, 1733} }
> +	{9,  {7800, 8667}, {3600, 4000}, {1730, 1920} }
>   };
>   
>   static void ath10k_mac_get_rate_flags_ht(struct ath10k *ar, u32 rate, u8 nss, u8 mcs,

Tested-by: Paul Menzel <pmenzel@molgen.mpg.de> # Dell XPS 13 9360
(qca6174 hw3.2 target 0x05030000 chip_id 0x00340aff sub 1a56:1535, 
firmware ver WLAN.RM.4.4.1-00309- api 6 features wowlan,ignore-otp,mfp 
crc32 0793bcf2)


Kind regards,

Paul


PS: For the record the log messages without your patches:

```
$ journalctl -b -2 _TRANSPORT=kernel -o short-monotonic -g ath10k | cat
[    9.235159] abreu kernel: ath10k_pci 0000:3a:00.0: enabling device 
(0000 -> 0002)
[    9.235737] abreu kernel: ath10k_pci 0000:3a:00.0: pci irq msi 
oper_irq_mode 2 irq_mode 0 reset_mode 0
[    9.536027] abreu kernel: ath10k_pci 0000:3a:00.0: qca6174 hw3.2 
target 0x05030000 chip_id 0x00340aff sub 1a56:1535
[    9.536283] abreu kernel: ath10k_pci 0000:3a:00.0: kconfig debug 0 
debugfs 0 tracing 0 dfs 0 testmode 0
[    9.536448] abreu kernel: ath10k_pci 0000:3a:00.0: firmware ver 
WLAN.RM.4.4.1-00309- api 6 features wowlan,ignore-otp,mfp crc32 0793bcf2
[    9.664028] abreu kernel: ath10k_pci 0000:3a:00.0: board_file api 2 
bmi_id N/A crc32 d2863f91
[    9.804022] abreu kernel: ath10k_pci 0000:3a:00.0: htt-ver 3.87 
wmi-op 4 htt-op 3 cal otp max-sta 32 raw 0 hwcrypto 1
[    9.888048] abreu kernel: ath10k_pci 0000:3a:00.0 wlp58s0: renamed 
from wlan0
[  513.580087] abreu kernel: ath10k_pci 0000:3a:00.0: failed to flush 
transmit queue (skip 0 ar-state 1): 0
[ 2567.504260] abreu kernel: ath10k_pci 0000:3a:00.0: pci irq msi 
oper_irq_mode 2 irq_mode 0 reset_mode 0
[ 2567.780099] abreu kernel: ath10k_pci 0000:3a:00.0: qca6174 hw3.2 
target 0x05030000 chip_id 0x00340aff sub 1a56:1535
[ 2567.780840] abreu kernel: ath10k_pci 0000:3a:00.0: kconfig debug 0 
debugfs 0 tracing 0 dfs 0 testmode 0
[ 2567.781291] abreu kernel: ath10k_pci 0000:3a:00.0: firmware ver 
WLAN.RM.4.4.1-00309- api 6 features wowlan,ignore-otp,mfp crc32 0793bcf2
[ 2567.856082] abreu kernel: ath10k_pci 0000:3a:00.0: board_file api 2 
bmi_id N/A crc32 d2863f91
[ 2567.948225] abreu kernel: ath10k_pci 0000:3a:00.0: htt-ver 3.87 
wmi-op 4 htt-op 3 cal otp max-sta 32 raw 0 hwcrypto 1
[ 2568.055481] abreu kernel: ath10k_pci 0000:3a:00.0 wlp58s0: renamed 
from wlan0
[ 2594.004036] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2606.716035] abreu kernel: ath10k_pci 0000:3a:00.0: pci irq msi 
oper_irq_mode 2 irq_mode 0 reset_mode 0
[ 2606.996065] abreu kernel: ath10k_pci 0000:3a:00.0: qca6174 hw3.2 
target 0x05030000 chip_id 0x00340aff sub 1a56:1535
[ 2606.997132] abreu kernel: ath10k_pci 0000:3a:00.0: kconfig debug 0 
debugfs 0 tracing 0 dfs 0 testmode 0
[ 2606.997787] abreu kernel: ath10k_pci 0000:3a:00.0: firmware ver 
WLAN.RM.4.4.1-00309- api 6 features wowlan,ignore-otp,mfp crc32 0793bcf2
[ 2607.076065] abreu kernel: ath10k_pci 0000:3a:00.0: board_file api 2 
bmi_id N/A crc32 d2863f91
[ 2607.172110] abreu kernel: ath10k_pci 0000:3a:00.0: htt-ver 3.87 
wmi-op 4 htt-op 3 cal otp max-sta 32 raw 0 hwcrypto 1
[ 2607.288083] abreu kernel: ath10k_pci 0000:3a:00.0 wlp58s0: renamed 
from wlan0
[ 2619.004042] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2634.004063] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2634.004555] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2646.008072] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2658.004636] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2670.004124] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2676.004124] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2682.008029] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2694.008097] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2700.008035] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2706.008036] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2712.008036] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2718.013839] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2724.008034] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2730.008047] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2736.008160] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2742.012285] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2754.004105] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1730 100kbps nss 2 mcs 9
[ 2760.008098] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2766.008097] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2772.008111] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2784.008045] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2790.008116] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2796.004150] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2802.008109] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2808.008074] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2820.008060] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2838.008137] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2844.008135] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2856.008179] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2862.008038] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2874.004098] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2892.008086] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2898.008175] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2904.008100] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2910.008031] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2940.008087] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2946.008062] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2964.008098] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2970.008089] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2976.008094] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2982.004167] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2988.012097] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 2994.008078] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3000.008129] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3018.008091] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1730 100kbps nss 2 mcs 9
[ 3036.008060] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3042.004087] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3048.008104] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3054.012046] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3060.008090] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3066.008157] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3072.008299] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3078.004268] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3084.012112] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3102.004068] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3108.012337] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3114.008081] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3120.008092] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3126.004109] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3132.008110] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3162.004063] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3168.008092] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3192.012049] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1730 100kbps nss 2 mcs 9
[ 3198.008053] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3204.012100] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3210.012098] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3216.008087] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3222.008052] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3228.012313] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3246.008047] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1730 100kbps nss 2 mcs 9
[ 3252.008035] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3258.004121] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3264.008099] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3270.012084] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3276.008129] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3282.004088] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3294.004099] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3318.004049] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3324.008203] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3330.008069] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3336.012074] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3342.008087] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3348.004111] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3366.008075] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3372.008026] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3378.008111] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3384.008285] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3390.008103] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3396.008092] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3402.008038] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3408.008092] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3414.008273] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3438.008111] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1730 100kbps nss 2 mcs 9
[ 3444.004040] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3450.008046] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3456.008056] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3462.008052] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
[ 3467.020210] abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params 
rate 1920 100kbps nss 2 mcs 9
```

With both of your patches:

```
$ journalctl -b -1 _TRANSPORT=kernel -o short-monotonic -g ath10k | cat
[   15.136140] abreu kernel: ath10k_pci 0000:3a:00.0: enabling device 
(0000 -> 0002)
[   15.148602] abreu kernel: ath10k_pci 0000:3a:00.0: pci irq msi 
oper_irq_mode 2 irq_mode 0 reset_mode 0
[   15.468121] abreu kernel: ath10k_pci 0000:3a:00.0: qca6174 hw3.2 
target 0x05030000 chip_id 0x00340aff sub 1a56:1535
[   15.468401] abreu kernel: ath10k_pci 0000:3a:00.0: kconfig debug 0 
debugfs 0 tracing 0 dfs 0 testmode 0
[   15.468598] abreu kernel: ath10k_pci 0000:3a:00.0: firmware ver 
WLAN.RM.4.4.1-00309- api 6 features wowlan,ignore-otp,mfp crc32 0793bcf2
[   15.568036] abreu kernel: ath10k_pci 0000:3a:00.0: board_file api 2 
bmi_id N/A crc32 d2863f91
[   15.672086] abreu kernel: ath10k_pci 0000:3a:00.0: htt-ver 3.87 
wmi-op 4 htt-op 3 cal otp max-sta 32 raw 0 hwcrypto 1
[   15.788147] abreu kernel: ath10k_pci 0000:3a:00.0 wlp58s0: renamed 
from wlan0
[   75.232061] abreu kernel: ath10k_pci: unknown parameter 'debug_mask' 
ignored
[   75.240101] abreu kernel: ath10k_pci 0000:3a:00.0: pci irq msi 
oper_irq_mode 2 irq_mode 0 reset_mode 0
[   75.516113] abreu kernel: ath10k_pci 0000:3a:00.0: qca6174 hw3.2 
target 0x05030000 chip_id 0x00340aff sub 1a56:1535
[   75.516835] abreu kernel: ath10k_pci 0000:3a:00.0: kconfig debug 0 
debugfs 0 tracing 0 dfs 0 testmode 0
[   75.517283] abreu kernel: ath10k_pci 0000:3a:00.0: firmware ver 
WLAN.RM.4.4.1-00309- api 6 features wowlan,ignore-otp,mfp crc32 0793bcf2
[   75.592080] abreu kernel: ath10k_pci 0000:3a:00.0: board_file api 2 
bmi_id N/A crc32 d2863f91
[   75.684191] abreu kernel: ath10k_pci 0000:3a:00.0: htt-ver 3.87 
wmi-op 4 htt-op 3 cal otp max-sta 32 raw 0 hwcrypto 1
[   75.787369] abreu kernel: ath10k_pci 0000:3a:00.0 wlp58s0: renamed 
from wlan0
```
diff mbox series

Patch

diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c
index 3c34f3d8e555..e62b251405fc 100644
--- a/drivers/net/wireless/ath/ath10k/mac.c
+++ b/drivers/net/wireless/ath/ath10k/mac.c
@@ -9136,7 +9136,7 @@  static const struct ath10k_index_vht_data_rate_type supported_vht_mcs_rate_nss2[
 	{6,  {5265, 5850}, {2430, 2700}, {1170, 1300} },
 	{7,  {5850, 6500}, {2700, 3000}, {1300, 1444} },
 	{8,  {7020, 7800}, {3240, 3600}, {1560, 1733} },
-	{9,  {7800, 8667}, {3600, 4000}, {1560, 1733} }
+	{9,  {7800, 8667}, {3600, 4000}, {1730, 1920} }
 };
 
 static void ath10k_mac_get_rate_flags_ht(struct ath10k *ar, u32 rate, u8 nss, u8 mcs,