wifi: ath9k: fix LNA selection in ath_ant_try_scan()
authorDmitry Antipov <dmantipov@yandex.ru>
Sun, 17 Dec 2023 11:29:03 +0000 (13:29 +0200)
committerKalle Valo <quic_kvalo@quicinc.com>
Mon, 18 Dec 2023 18:46:30 +0000 (20:46 +0200)
In 'ath_ant_try_scan()', (most likely) the 2nd LNA's signal
strength should be used in comparison against RSSI when
selecting first LNA as the main one. Compile tested only.

Found by Linux Verification Center (linuxtesting.org) with SVACE.

Signed-off-by: Dmitry Antipov <dmantipov@yandex.ru>
Acked-by: Toke Høiland-Jørgensen <toke@toke.dk>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://msgid.link/20231211172502.25202-1-dmantipov@yandex.ru
drivers/net/wireless/ath/ath9k/antenna.c

index 988222c..acc84e6 100644 (file)
@@ -643,7 +643,7 @@ static void ath_ant_try_scan(struct ath_ant_comb *antcomb,
                                conf->main_lna_conf = ATH_ANT_DIV_COMB_LNA1;
                                conf->alt_lna_conf = ATH_ANT_DIV_COMB_LNA1_PLUS_LNA2;
                        } else if (antcomb->rssi_sub >
-                                  antcomb->rssi_lna1) {
+                                  antcomb->rssi_lna2) {
                                /* set to A-B */
                                conf->main_lna_conf = ATH_ANT_DIV_COMB_LNA1;
                                conf->alt_lna_conf = ATH_ANT_DIV_COMB_LNA1_MINUS_LNA2;