On 13-12-2016 17:19, Johannes Berg wrote: > On Mon, 2016-12-12 at 11:59 +0000, Arend van Spriel wrote: >> This patch adds support for GScan which is a scan offload feature >> used in Android. > > Found a few places with spaces instead of tabs as indentation, and > spurious braces around single-statement things, but other than that it > looks fine from a patch/nl80211 POV. I added a check in wiphy_register() in this patch which actually is more appropriate with the "gscan capabilities" patch. > Haven't really looked into the details of gscan itself now though, > sorry. > > There's a bit of a weird hard-coded restriction to 16 channels too, > that's due to the bucket map? Uhm. Is there? I will check, but if you can give me a pointer where to look it is appreciated. Regards, Arend