On Thu, Jun 4, 2009 at 11:24 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Fri, 2009-06-05 at 01:41 -0400, Luis R. Rodriguez wrote: > >> - u16 fc; >> + __le16 fc; >> >> if (txrc->rts) >> info->control.rates[0].count = >> @@ -291,8 +291,7 @@ rate_control_pid_get_rate(void *priv, struct ieee80211_sta *sta, >> >> /* Send management frames and NO_ACK data using lowest rate. */ >> fc = le16_to_cpu(hdr->frame_control); >> - if (!sta || !spinfo || >> - (fc & IEEE80211_FCTL_FTYPE) != IEEE80211_FTYPE_DATA || >> + if (!sta || !spinfo || !ieee80211_is_data(fc) || > > you might want to run sparse on this entire patchset. I did, PID was not on my kconfig though, so its the only thing I didn't test compile. Luis -- 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
