On Fri, 2012-07-27 at 18:00 +0200, Thomas Huehn wrote: > @@ -283,6 +284,9 @@ bool rate_control_send_low(struct ieee80211_sta *sta, > txrc->bss_conf->basic_rates, > sband); > } > + for (i = 0; i < IEEE80211_TX_MAX_RATES; i++) { > + info->control.rates[i].tpc = txrc->hw->conf.power_level; > + } No need for braces, and a blank line above it would be good. Remember code is meant to be read by people and compilers (in that order) Anyway I don't think I'm going to apply any of this until we have some code that uses it. 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