On Tue, 2010-10-26 at 14:22 +0200, Juuso Oikarinen wrote: > On Tue, 2010-10-26 at 14:18 +0200, ext Johannes Berg wrote: > > On Tue, 2010-10-26 at 15:01 +0300, Juuso Oikarinen wrote: > > > On Tue, 2010-10-26 at 13:40 +0200, ext Johannes Berg wrote: > > > > > > > + if (wl->state == WL1271_STATE_OFF) { > > > > > + /* > > > > > + * We cannot return -EBUSY here because mac80211 will expect > > > > > + * a call to ieee80211_scan_completed if we do - in this case > > > > > + * there won't be any call. > > > > > + */ > > > > > > > > ??? > > > > > I'll try explain what this is about. In net/wireless/sme.c you'll find > > > this: > > > > [snip] > > > > so it's really about cfg80211, that explains my confusion :) > > Yeah, it seems it's my confusion :/ Thanks for the patch, Juuso. Thanks for the comment, Johannes. I have reviewed this patch, changed the comment to cfg80211 and applied to the wl12xx tree. Reviewed-by: Luciano Coelho <luciano.coelho@xxxxxxxxx> -- Cheers, Luca. -- 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