[V9 4/9] wifi: mac80211: Add support for WBRF features

[Quan, Evan]

[AMD Official Use Only - General]

> -----Original Message-----
> From: Johannes Berg <johannes at sipsolutions.net>
> Sent: Monday, August 21, 2023 5:44 PM
> To: Quan, Evan <Evan.Quan at amd.com>; gregkh at linuxfoundation.org;
> rafael at kernel.org; lenb at kernel.org; davem at davemloft.net;
> edumazet at google.com; kuba at kernel.org; pabeni at redhat.com; Deucher,
> Alexander <Alexander.Deucher at amd.com>; andrew at lunn.ch;
> rdunlap at infradead.org; quic_jjohnson at quicinc.com; horms at kernel.org
> Cc: linux-doc at vger.kernel.org; linux-kernel at vger.kernel.org; linux-
> acpi at vger.kernel.org; amd-gfx at lists.freedesktop.org; linux-
> wireless at vger.kernel.org; netdev at vger.kernel.org; Limonciello, Mario
> <Mario.Limonciello at amd.com>
> Subject: Re: [V9 4/9] wifi: mac80211: Add support for WBRF features
>
> On Fri, 2023-08-18 at 11:26 +0800, Evan Quan wrote:
> > To support the WBRF mechanism, Wifi adapters utilized in the system
> > must register the frequencies in use(or unregister those frequencies
> > no longer
> > used) via the dedicated calls. So that, other drivers responding to
> > the frequencies can take proper actions to mitigate possible interference.
> >
> > Co-developed-by: Mario Limonciello <mario.limonciello at amd.com>
> > Signed-off-by: Mario Limonciello <mario.limonciello at amd.com>
> > Co-developed-by: Evan Quan <evan.quan at amd.com>
> > Signed-off-by: Evan Quan <evan.quan at amd.com>
>
> From WiFi POV, this looks _almost_ fine to me.
>
> > +static void wbrf_get_ranges_from_chandef(struct cfg80211_chan_def
> *chandef,
> > +                                    struct wbrf_ranges_in *ranges_in) {
> > +   u64 start_freq1, end_freq1;
> > +   u64 start_freq2, end_freq2;
> > +   int bandwidth;
> > +
> > +   bandwidth = nl80211_chan_width_to_mhz(chandef->width);
> > +
> > +   get_chan_freq_boundary(chandef->center_freq1,
> > +                          bandwidth,
> > +                          &start_freq1,
> > +                          &end_freq1);
> > +
> > +   ranges_in->band_list[0].start = start_freq1;
> > +   ranges_in->band_list[0].end = end_freq1;
> > +
> > +   if (chandef->width == NL80211_CHAN_WIDTH_80P80) {
> > +           get_chan_freq_boundary(chandef->center_freq2,
> > +                                  bandwidth,
> > +                                  &start_freq2,
> > +                                  &end_freq2);
> > +
> > +           ranges_in->band_list[1].start = start_freq2;
> > +           ranges_in->band_list[1].end = end_freq2;
> > +   }
> > +}
>
> This has to setup ranges_in->num_of_ranges, no?
Yes, better to have that. I add this in V10.
> (Also no real good reason for num_of_ranges to be a u64, btw, since it can
> only go up to 11)
Mainly for data structure alignment. Since other members come with u64.
So, to make the data structure naturally aligned, 'num_of_ranges' also comes with u64.

Evan
>
> With that fixed, you can add
>
> Reviewed-by: Johannes Berg <johannes at sipsolutions.net>
>
> johannes