On Wed, 2012-06-13 at 10:08 +0200, Johannes Berg wrote: > On Wed, 2012-06-13 at 11:00 +0300, Eliad Peller wrote: > > > > @@ -1329,6 +1329,8 @@ static void ieee80211_set_disassoc(struct ieee80211_sub_if_data *sdata, > > > u32 changed = 0; > > > u8 bssid[ETH_ALEN]; > > > > > > + ieee80211_stop_poll(sdata); > > > + > > > ASSERT_MGD_MTX(ifmgd); > > > > it doesn't matter much, but maybe better add the call after the > > function "prologue" (lockdep, WARN_ONs, ...)? > > I think we shouldn't do it after the warnings since those affect the > control flow. OTOH, they should never happen ... I'll change it, makes it look better :-) johannes -- 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
