> +static inline int
> +rate_lowest_index(struct ieee80211_local *local, struct ieee80211_hw_mode *mode,
> + struct sta_info *sta)
> +{
> + int i;
> +
> + for (i = 0; i < mode->num_rates; i++) {
> + if (rate_supported(sta, mode, i))
> + return i;
> + }
> +
> + /* warn when we cannot find a rate. */
> + WARN_ON(i == 0);
> +
> + return 0;
> +}
That doesn't look right. Shouldn't it be WARN_ON(1)? Not that it can
ever happen with proper drivers...
Other than that, looks ok to me.
johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
