[PATCH] ACS: Add 320MHz support for 80211be

[Nicolas Escande]

On Sat Nov 11, 2023 at 9:55 PM CET, Jouni Malinen wrote:
> On Thu, Nov 09, 2023 at 04:04:08PM +0100, Nicolas Escande wrote:
> > This enables support for ACS in the 80211be/320MHZ. To do so we just:
> >   - add channel definitions for the 3 320MHZ blocks of the 6GHz band
> >   - handle CONF_OPER_CHWIDTH_320MHZ in the different cases
> >   - restrict 320MHZ to 80211be only
>
> > diff --git a/src/ap/acs.c b/src/ap/acs.c
> > +static const struct bw_item bw_320[] = {
> > +	{ 6115, 6415, 63 }, { 6435, 6735, 127 }, { 66755, 7055, 191 },
> > +	{ -1, -1, -1 }
>
> That 66755 should have been 6755..

Indeed I'll fix it in v2

>
> Is there some specific reason for including only the channel center
> frequency index values 63, 127, and 191 here? IEEE P802.11be/D4.0
> defines the operating class 137 with the following set of possible
> channel center frequency index values: 31, 63, 95, 127, 159, 191.

Nope I missed it completely, I'll fix it in v2.

Thanks for the review.