ath10k.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Youghandhar Chintala <quic_youghand@quicinc.com>
To: <ath10k@lists.infradead.org>, <johannes@sipsolutions.net>
Cc: <linux-wireless@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<quic_mpubbise@quicinc.com>,
	Youghandhar Chintala <quic_youghand@quicinc.com>
Subject: [PATCH 2/2] wifi: ath10k: update the channel list if change in channel flags.
Date: Thu, 22 Dec 2022 18:12:21 +0530	[thread overview]
Message-ID: <20221222124221.30894-3-quic_youghand@quicinc.com> (raw)
In-Reply-To: <20221222124221.30894-1-quic_youghand@quicinc.com>

There are connection failures in hidden SSID case when the device is
with default reg domain WW.
For WW reg domain most of the 5 GHz channels are passive. When device
listens to the beacon on that channel, the driver is updating its
channel flag but firmware is not aware of it and firmware is not
sending probes on that channels.
Due to this, we are seeing connection failures when a device is trying
to connect with hidden SSID AP.
Register beacon hint notifier to the regulatory core so that driver get
notified when there is a change in channel flags. Driver's notifier
callback will send the updated flags to the firmware.

Tested-on: WCN3990 hw1.0 SNOC WLAN.HL.3.2.2.c10-00754-QCAHLSWMTPL-1
Tested-on: QCA6174 hw3.2 SDIO WLAN.RMH.4.4.1-00174

Signed-off-by: Youghandhar Chintala <quic_youghand@quicinc.com>
---
 drivers/net/wireless/ath/ath10k/mac.c | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c
index ec8d5b29bc72..91a957295456 100644
--- a/drivers/net/wireless/ath/ath10k/mac.c
+++ b/drivers/net/wireless/ath/ath10k/mac.c
@@ -3534,6 +3534,15 @@ static void ath10k_mac_update_channel_list(struct ath10k *ar,
 	}
 }
 
+static void ath10k_mac_beacon_notifier(struct wiphy *wiphy)
+{
+	struct ieee80211_hw *hw = wiphy_to_ieee80211_hw(wiphy);
+	struct ath10k *ar = hw->priv;
+
+	if (ath10k_update_channel_list(ar))
+		ath10k_warn(ar, "failed to update channel list\n");
+}
+
 static void ath10k_reg_notifier(struct wiphy *wiphy,
 				struct regulatory_request *request)
 {
@@ -10286,6 +10295,8 @@ int ath10k_mac_register(struct ath10k *ar)
 			goto err_unregister;
 	}
 
+	ar->hw->wiphy->beacon_hint_notifier = ath10k_mac_beacon_notifier;
+
 	return 0;
 
 err_unregister:
-- 
2.38.0


_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

  parent reply	other threads:[~2022-12-22 13:49 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-22 12:42 [PATCH 0/2] Update the channel list if the change in channel flags Youghandhar Chintala
2022-12-22 12:42 ` [PATCH 1/2] wifi: cfg80211: Add beacon hint notifier support Youghandhar Chintala
2023-01-18 16:30   ` Johannes Berg
2023-02-15 11:26     ` Youghandhar Chintala (Temp)
2023-02-15 15:10       ` Johannes Berg
2023-02-22  0:22         ` Wen Gong
2023-02-22  8:00           ` Johannes Berg
2022-12-22 12:42 ` Youghandhar Chintala [this message]
2023-01-19  2:26   ` [PATCH 2/2] wifi: ath10k: update the channel list if change in channel flags Wen Gong
2023-01-19  5:47     ` Youghandhar Chintala (Temp)
2022-12-22 14:00 ` [PATCH 0/2] Update the channel list if the " Kalle Valo
     [not found] <20221222120529.31436-1-quic_youghand@quicinc.com>
     [not found] ` <20221222120529.31436-3-quic_youghand@quicinc.com>
2022-12-22 13:38   ` [PATCH 2/2] wifi: ath10k: update the channel list if " Kalle Valo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20221222124221.30894-3-quic_youghand@quicinc.com \
    --to=quic_youghand@quicinc.com \
    --cc=ath10k@lists.infradead.org \
    --cc=johannes@sipsolutions.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=quic_mpubbise@quicinc.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).