On Tue, 2008-04-22 at 09:50 +0800, Zhu Yi wrote: > 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? Sure, can give that a shot. Dan -- 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