On Tue, 2012-10-09 at 13:26 -0700, Marco Porsch wrote: > On 10/09/2012 11:39 AM, Johannes Berg wrote: > >> Does it make any sense to test for NL80211_IFTYPE_AP and > >> NL80211_IFTYPE_AP_VLAN everytime? > > > > Not sure how you mean that? > > > > See, I use the disjunction > + if (sta->sdata->vif.type == NL80211_IFTYPE_AP || > + sta->sdata->vif.type == NL80211_IFTYPE_AP_VLAN) { > at least 5 times. I thought in some situations it may be only necessary > to check for one of them, because of the function context. I don't think so, but honestly, I haven't looked very closely :) 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