Hi Johannes,
And, I'm confused, but isn't the polarity of the scanning check
wrong?
Ah yeah, after you pointed that out I realized 'scanning' is a bit
field. I should be doing:
test_bit(SCAN_HW_SCANNING, &sdata->local->scanning)
I think checking for all the bits is fine (and necessary, just HW scan
is unlikely to be enough, changing the MAC address would also disrupt a
software scan) - just need to invert the polarity?
I concur that scanning should be checked as
if (sdata->local->scanning). So Johannes you're right that the polarity
is reversed. However, __ieee80211_start_scan seems to check for
local->scan_req instead to take deferred scans into account. So I
wonder if that is a better approach.
Regards,
-Denis
