[PATCH 1/4] brcm80211: fix compile issue

Brett Rudley brudley at broadcom.com
Mon Feb 14 22:22:59 UTC 2011


> -----Original Message-----
> From: Stanislav Fomichev [mailto:kernel at fomichev.me]
> Sent: Monday, February 14, 2011 2:05 PM
> To: devel at linuxdriverproject.org
> Cc: Arend Van Spriel; Brett Rudley; Roland Vossen
> Subject: [PATCH 1/4] brcm80211: fix compile issue
> 
>     Commit dbd2fd656f2060abfd3a16257f8b51ec60f6d2ed changed the
>     definition of ieee80211_channel_to_frequency; so fix its usage in
> brcmfmac.
> 
> Signed-off-by: Stanislav Fomichev <kernel at fomichev.me>
> ---
>  drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c
> b/drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c
> index fa2316b..cc4cf83 100644
> --- a/drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c
> +++ b/drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c
> @@ -2312,7 +2312,9 @@ static s32 wl_inform_single_bss(struct wl_priv *wl,
> struct wl_bss_info *bi)
>  	notif_bss_info->frame_len =
>  	    offsetof(struct ieee80211_mgmt,
>  		     u.beacon.variable) + wl_get_ielen(wl);
> -	freq = ieee80211_channel_to_frequency(notif_bss_info->channel);
> +	freq = ieee80211_channel_to_frequency(notif_bss_info->channel,
> +					      IEEE80211_BAND_2GHZ);
> +
>  	channel = ieee80211_get_channel(wiphy, freq);
> 
>  	WL_DBG("SSID : \"%s\", rssi %d, channel %d, capability : 0x04%x,
> bssid %pM\n",
> --
> 1.7.1
> 

Hard coding 2.4 Ghz to always be the band for channel_to_freq translations seems very wrong.  

Thanks
brett




More information about the devel mailing list