Re: [PATCH 2] cfg80211: Extend channel to frequency mapping for 802.11j

[Luciano Coelho <coelho@ti.com>]

On Mon, 2011-01-17 at 05:37 +0100, Bruno Randolf wrote:
> Extend channel to frequency mapping for 802.11j Japan 4.9GHz band, according to
> IEEE802.11 section 17.3.8.3.2 and Annex J. Because there are now overlapping
> channel numbers in the 2GHz and 5GHz band we can't map from channel to
> frequency without knowing the band. This is no problem as in most contexts we
> know the band. In places where we don't know the band (and WEXT compatibility)
> we assume the 2GHz band for channels below 14.
> 
> This patch does not implement all channel to frequency mappings defined in
> 802.11, it's just an extension for 802.11j 20MHz channels. 5MHz and 10MHz
> channels as well as 802.11y channels have been omitted.
> 
> The following drivers have been updated to reflect the API changes:
> iwl-3945, iwl-agn, iwmc3200wifi, libertas, mwl8k, rt2x00, wl1251, wl12xx.
> The drivers have been compile-tested only.
> 
> Signed-off-by: Bruno Randolf <br1@einfach.org>
> 
> ---

For the wl12xx part:

> diff --git a/drivers/net/wireless/wl12xx/rx.c b/drivers/net/wireless/wl12xx/rx.c
> index 682304c..ec8d843 100644
> --- a/drivers/net/wireless/wl12xx/rx.c
> +++ b/drivers/net/wireless/wl12xx/rx.c
> @@ -76,7 +76,7 @@ static void wl1271_rx_status(struct wl1271 *wl,
>          */
>         wl->noise = desc->rssi - (desc->snr >> 1);
> 
> -       status->freq = ieee80211_channel_to_frequency(desc->channel);
> +       status->freq = ieee80211_channel_to_frequency(desc->channel, desc_band);
> 
>         if (desc->flags & WL1271_RX_DESC_ENCRYPT_MASK) {
>                 status->flag |= RX_FLAG_IV_STRIPPED | RX_FLAG_MMIC_STRIPPED;

Acked-by: Luciano Coelho <coelho@ti.com>

-- 
Cheers,
Luca.