On Mon, 2008-04-21 at 13:49 -0400, Dan Williams wrote: > When another scan is in progress, a direct scan gets dropped on the > floor. However, that direct scan is usually the scan that's really > needed by userspace, and gets stomped on by all the broadcast scans > the > ipw2200 driver issues internally. Make sure the direct scan happens > eventually, and as a bonus ensure that the passive scan worker is > cleaned up when appropriate. > > The change of request_passive_scan form a struct work to struct > delayed_work is only to make the set_wx_scan() code a bit simpler, > it's > still only used with a delay of 0 to match previous behavior. > > Signed-off-by: Dan Williams <dcbw@xxxxxxxxxx> I agree with you the idea to split the current active request_scan into active direct scan and active broadcast scan. But can you merge your ipw_request_direct_scan_helper into ipw_request_scan_helper so that we don't duplicate the code? Thanks, -yi -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html