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.
_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap
