Re: [PATCH] wl12xx: Fix potential incorrect band in rx-status

[Luciano Coelho <coelho@ti.com>]

On Mon, 2011-03-21 at 15:23 +0200, Luciano Coelho wrote:
> On Thu, 2011-03-10 at 10:01 +0200, juuso.oikarinen@nokia.com wrote:
> > From: Juuso Oikarinen <juuso.oikarinen@nokia.com>
> > 
> > The rx-status passed to mac80211 along with each received frame
> > contains the
> > band on which the frame was received. Under certain circumstances,
> > this band
> > information may be incorrect, causing in worst case a WARNING from
> > mac80211,
> > and causes the received frame to be dropped.
> > 
> > This scenario mainly occurs when performing connected-mode scans, when
> > the
> > received scan results are from the other band than the one currentl
> > associated to.
> > 
> > Signed-off-by: Juuso Oikarinen <juuso.oikarinen@nokia.com> 
> 
> Looks good, thanks!
> 
> Reviewed-by: Luciano Coelho <coelho@ti.com>
> 
> Applied to wl12xx/master with a small s/currentl/currently/ ;)

Ahm, there was a compilation problem with this on my tree.  The
following line had to be changed as well:

-	status->freq = ieee80211_channel_to_frequency(desc->channel, desc_band);
+	status->freq = ieee80211_channel_to_frequency(desc->channel,
+						      status->band);

Fixed the patch and reapplied.

-- 
Cheers,
Luca.