On 24-1-2017 10:40, Johannes Berg wrote: >> + /* leave request id zero for legacy request >> + * or if driver does not support multi-scheduled scan >> + */ >> + if (want_multi && rdev->wiphy.max_sched_scan_reqs > 1) { > > Why do the >1 check here? It probably doesn't really make a difference > since only one can be running at a time, but it might be nicer - at > least for debug in userspace - to have a real value for all multi > scans? I added the check, because drivers not supporting multi-scheduled scan might not bother using the request id in event reporting. Regards, Arend