Message-ID: <0d4b99ab0a5bedc82f35bc1e548a611564b010e5.camel@sipsolutions.net>
Subject: Re: [PATCH 1/2] wifi: cfg80211: Add beacon hint notifier support
From:   Johannes Berg <johannes@sipsolutions.net>
To:     "Youghandhar Chintala (Temp)" <quic_youghand@quicinc.com>,
        ath10k@lists.infradead.org
Cc:     linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org,
        quic_mpubbise@quicinc.com
Date:   Wed, 15 Feb 2023 16:10:11 +0100
In-Reply-To: <96cab5d9-9090-4cb2-ff17-eadc9da12f50@quicinc.com>
References: <20221222124221.30894-1-quic_youghand@quicinc.com>
         <20221222124221.30894-2-quic_youghand@quicinc.com>
         <1df59863e78e8ddbe7eb3a74e6dd4c8f0bd7f098.camel@sipsolutions.net>
         <96cab5d9-9090-4cb2-ff17-eadc9da12f50@quicinc.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.46.3 (3.46.3-1.fc37) 
MIME-Version: 1.0
Precedence: bulk

> >=20
> > Why is this even needed? You should always get reg_notifier after this
> > anyway?
>=20
> Currently when channel flag changed through the beacon hints are not=20
> informed to driver.
>=20
> reg_notifier will be triggered for regdomain changes but not for channel =
flag changes due to beacon hints.
>=20

So maybe triggering reg notifier once would be sufficient, a la Wen's
patch that I recently merged?

johannes