On Wed, Nov 5, 2008 at 11:59 AM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Wed, 2008-11-05 at 11:55 -0800, Luis R. Rodriguez wrote: > >> >> /* This would make this whole thing pointless */ >> >> - BUG_ON(rd != cfg80211_regdomain); >> >> + if (!last_request->intersect) >> >> + BUG_ON(rd != cfg80211_regdomain); >> >> >> >> /* update all wiphys now with the new established regulatory domain */ >> >> update_all_wiphy_regulatory(last_request->initiator); >> >> >> >> - print_regdomain(rd); >> >> + print_regdomain(cfg80211_regdomain); >> > >> > Maybe the printing here should be removed? It seems confusing now. >> >> Well IMHO its correct to print it here as *this* is after its been set >> and passed through all wiphys. ACK? > > Ah, hmm, rd is intersected before this, right? I think I just got > confused with the code flow, yeah, seems fine. Yeap, OK thanks. Luis -- 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
