On 07-06-17 16:53, Kalle Valo wrote: > Arend van Spriel <[email protected]> writes: > >> On 5/22/2017 5:16 PM, Kalle Valo wrote: >>> Arend Van Spriel <[email protected]> wrote: >>>> From: Arend Van Spriel <[email protected]> >>>> >>>> This change adds support for multi-scheduled scan in the driver. It >>>> currently relies on g-scan support in firmware and will set struct >>>> wiphy::max_sched_scan_reqs accordingly. This is limited to 16 concurrent >>>> requests. >>>> >>>> The firmware currently has a limit of 64 channels that can be configured >>>> for all requests in total regardless whether there are duplicates. So if >>>> a request uses 35 channels there are 29 channels left for another request. >>>> When user-space does not specify any channels cfg80211 will add all >>>> channels defined by the wiphy instance to the request, which makes >>>> reaching the limit rather easy for dual-band devices. >>>> >>>> Reviewed-by: Hante Meuleman <[email protected]> >>>> Reviewed-by: Pieter-Paul Giesberts <[email protected]> >>>> Reviewed-by: Franky Lin <[email protected]> >>>> Signed-off-by: Arend van Spriel <[email protected]> >>> >>> I see new warnings: >>> >>> drivers/net/wireless/broadcom/brcm80211/brcmfmac/pno.c: In function >>> ‘brcmf_pno_config_sched_scans’: >>> drivers/net/wireless/broadcom/brcm80211/brcmfmac/pno.c:166:6: warning: >>> ‘mac_mask’ may be used uninitialized in this function >>> [-Wmaybe-uninitialized] >>> u8 *mac_mask; >>> ^ >>> drivers/net/wireless/broadcom/brcm80211/brcmfmac/pno.c:183:2: warning: >>> ‘mac_addr’ may be used uninitialized in this function >>> [-Wmaybe-uninitialized] >>> memcpy(pfn_mac.mac, mac_addr, ETH_ALEN); >>> ^ >>> drivers/net/wireless/broadcom/brcm80211/brcmfmac/pno.c:165:6: note: >>> ‘mac_addr’ was declared here >>> u8 *mac_addr; >>> ^ >>> >>> 3 patches set to Changes Requested. >>> >>> 9736151 [V4,1/3] brcmfmac: add support multi-scheduled scan >>> 9736147 [V4,2/3] brcmfmac: add mutex to protect pno requests >>> 9736149 [V4,3/3] brcmfmac: add scheduled scan support for specified BSSIDs >> >> Not seeing it here using gcc version 4.8.4 (Ubuntu >> 4.8.4-2ubuntu1~14.04.3). > > My version is: > > gcc (Ubuntu 5.4.0-6ubuntu1~16.04.4) 5.4.0 20160609 > > I don't think that's really exocit version of the compiler, or is it?
Nope. >> Also tried using W=1 on the make command >> line. Getting a bunck of other warnings, but not the ones above. > > This was without W=1, just a default compilation with all wireless > drivers enabled. > >> Do you want me to fix it? > > Yes, please. I try to keep the patches warning free, even if they are > false warnings. Will do. And thanks again for the reminder ;-) Regards, Arend
