On Mon, 2009-03-23 at 14:52 -0500, Bob Copeland wrote: > On Mon, 23 Mar 2009 20:36:28 +0100, Johannes Berg wrote > > I think I'm in favour of "downgrading" that to a WARN_ON, like this: > > if (WARN_ON(...)) > > info->control.rates[0].idx = rate_lowest() > > > > johannes > > How about the similar WARN_ON() in ieee80211_get_tx_rate()? Does > it make sense to try and tx a frame but return null for the rate > structure? Not really, I think. But it should only return NULL if the above condition was true. johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
