> -----Original Message----- > From: Johannes Berg <johannes@xxxxxxxxxxxxxxxx> > Sent: Monday, March 21, 2022 2:28 PM > To: Rameshkumar Sundaram (QUIC) <quic_ramess@xxxxxxxxxxx> > Cc: linux-wireless@xxxxxxxxxxxxxxx; ath11k@xxxxxxxxxxxxxxxxxxx > Subject: Re: [PATCH 1/2] mac80211: add support to configure 6GHz non-ht > duplicate transmission > > On Mon, 2022-03-21 at 05:12 +0000, Rameshkumar Sundaram (QUIC) wrote: > > > > > > > > I am wondering though if this should even be detected from the HE > > > operation element itself, rather than from the beacon rate settings > > > that are separate in nl80211? > > This is a BW dependent bit in HE Oper IE and user space can switch BW > > (CSA/chan switch scenarios). > > Which can call assign_beacon directly, Hence adding the logic here to > > cover all Beacon change cases. > > > > Yeah but still ... why do we take it from the content rather than adding a > control for it? So, Suggestion here is to add a new NL attribute to carry the information from user space for START_AP and SET_BEACON NL commands and use it here, is that understanding correct ? > > johannes