On Fri, 2015-01-16 at 11:38 +0100, Janusz Dziedzic wrote: > + /* we both use VHT */ > + struct ieee80211_vht_cap vhtcap_ie; > + struct ieee80211_sta_vht_cap vht_cap = sta->sta.vht_cap; > + > + ieee80211_vht_oper_to_chandef(channel, > + elems->vht_operation, > + &chandef); Ok maybe I'm missing something - but can't this erroneously configure the local HW to 160 MHz when it doesn't even support it, or so? > + memcpy(&vhtcap_ie, elems->vht_cap_elem, sizeof(vhtcap_ie)); > + ieee80211_vht_cap_ie_to_sta_vht_cap(sdata, sband, &vhtcap_ie, sta); > + if (memcmp(&vht_cap, &sta->sta.vht_cap, sizeof(vht_cap))) > + rates_updated |= true; Indentation is wrong here. 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