On Thu, 2015-06-11 at 13:06 +0200, Matthias May wrote:

>  const struct ieee80211_reg_rule *freq_reg_info(struct wiphy *wiphy,
> -                                            u32 center_freq)
> +                                            u32 center_freq, u32 min_bw)
>  {
>       const struct ieee80211_regdomain *regd;
> +     const struct ieee80211_reg_rule *rr_tmp = NULL;
> +     const struct ieee80211_reg_rule *reg_rule = NULL;
> +     u32 bw;
>  
>       regd = reg_get_regdomain(wiphy);
>  
> -     return freq_reg_info_regd(wiphy, center_freq, regd);
> +     for(bw=min_bw; bw <= MHZ_TO_KHZ(20); bw=bw*2) {
> +             rr_tmp = freq_reg_info_regd(wiphy, center_freq, bw, regd);
> +             if(!reg_rule || !IS_ERR(rr_tmp))
> +                     reg_rule = rr_tmp;
> +     }
> +
> +     return reg_rule;

This makes no sense to me.

You have the same code below, but why would you care? You can just look
at the reg_rule's max_bandwidth, no?

johannes

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to