lkml.org 
[lkml]   [2023]   [Oct]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 2/5] [brcmfmac] Add support for 6G bands
From
On 10/20/2023 6:35 PM, Daniel Berlin wrote:
>>> @@ -6965,6 +7066,10 @@ static int brcmf_construct_chaninfo(struct brcmf_cfg80211_info *cfg,
>>> for (i = 0; i < band->n_channels; i++)
>>> band->channels[i].flags = IEEE80211_CHAN_DISABLED;
>>> band = wiphy->bands[NL80211_BAND_5GHZ];
>>> + if (band)
>> Eh. Why is this conditional? We are creating all bands in the wiphy
>> instance so why the null check here?
>
> I just matched what was there, I can remove all of them if we want.
>
> (I'll take care of the rest of the comments between here)

I stumbled upon the bandlist usage later in this patch (I think) so we
do add bands only when the device supports them. Hence you can ignore my
comment here.

Regards,
Arend
[unhandled content-type:application/pkcs7-signature]
\
 
 \ /
  Last update: 2023-10-20 20:39    [W:0.654 / U:0.156 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site