On 2020-05-22 13:38, Johannes Berg wrote:
On Fri, 2020-05-22 at 00:24 -0700, Pradeep Kumar Chitrapu wrote:
6GHz channels are divided into preferred scanning channels(PSC)
and non-PSC channels. One in every four 20MHz channels is a PSC.
Spec mandates to use only PSC channels as primary channels for
setting up BSS on 6GHz only AP.
The set of 20 MHz channels in the 6 GHz band, with channel center
frequency, ch_a = Channel starting frequency – 55 + 80 × n (MHz)
are referred to as preferred scanning channels (PSCs) where,
n = 1, …, 15 as per IEEE P802.11ax/D6.0.
This flag also will be used when making scanning decision on
6GHz channels.
Is there much value in exposing this as a *flag*?
We have a similar patch, but it just adds the function and everyone who
cares can call it. Do we really need to precalculate that?
johannes
Thanks Johannes for the review..
IMO, accessing flag would be faster instead of computation, as this info
is handy (already in cache) when accessing ieee80211_channel.
However, considering general usage for this info would be only for
control
path, it should be ok to expose function instead of maintaining separate
flag.
Please let me know your suggestion..
Thanks
Pradeep
