On Mon, Oct 04, 2010 at 03:09:06PM +0200, Johannes Berg wrote: > On Fri, 2010-10-01 at 14:05 +0200, Stanislaw Gruszka wrote: > > > out_complete: > > + finish = false; > > + if (rc) { > > + hw_scan = test_bit(SCAN_HW_SCANNING, &local->scanning); > > + finish = __ieee80211_scan_completed(&local->hw, aborted, hw_scan); > > + } > > + > > mutex_unlock(&local->mtx); > > - if (rc) > > - __ieee80211_scan_completed(&local->hw, true); > > + > > + if (finish) > > + __ieee80211_scan_completed_finish(&local->hw, hw_scan); > > } > > And this makes the logic here even more confusing :-) I'll try to rewrite that more clean. Stanislaw -- 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