On Fri, Jun 22, 2012 at 01:10:24PM +0200, Sven Eckelmann wrote: > On Friday 22 June 2012 16:22:16 Rajkumar Manoharan wrote: > > > + /* update ampdu factor/density, they may have changed. This may happen > > > + * in HT IBSS when a beacon with HT-info is received after the station > > > + * has already been added. > > > + */ > > > + if (sc->sc_ah->caps.hw_caps & ATH9K_HW_CAP_HT) { > > > + an->maxampdu = 1 << (IEEE80211_HT_MAX_AMPDU_FACTOR + > > > + sta->ht_cap.ampdu_factor); > > > + density = ath9k_parse_mpdudensity(sta->ht_cap.ampdu_density); > > > + an->mpdudensity = density; > > > + } > > > + > > > > No need to declare local variable. ins't it? > > You want more than 80 chars per line or ugly/complex code...? > Never mind. keep the local variable. -Rajkumar -- 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