On Mon October 4 2010 20:21:40 Johannes Berg wrote: > On Mon, 2010-10-04 at 11:17 +0900, Bruno Randolf wrote: > > Enable WME QoS in IBSS mode by adding a WME information element to > > beacons and probe respones and by checking for it and marking stations > > as WME capable if it is present. > > > > Signed-off-by: Bruno Randolf <br1@xxxxxxxxxxx> > > > > --- > > v3: use GFP_ATOMIC and re-organize nesting of "if" statements. allthough > > in practice we would probaby always have the rates element i think it's > > more correct to code it like this. > > You're probably right, but shouldn't that be a separate patch? It > doesn't relate to WME at all, afaict. it does. before we only looked at elems->supp_rates, so it was o.k. to combine that check into the first if (adhoc mode && bssid matches). now we also check elems->wmm_info so we should make separate cases for each. as i noted, it probably does not matter much in practice, since elems->supp_rates should be present in all proper beacons and probe responses. bruno -- 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
