From: Teemu Paasikivi <ext-teemu.3.paasikivi@nokia.com>
To: Luciano Coelho <luciano.coelho@nokia.com>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH v2] wl1271: Fix warning about unsupported RX rate
Date: Fri, 22 Oct 2010 10:49:06 +0300 [thread overview]
Message-ID: <1287733746.2905.19.camel@paavo-desktop> (raw)
In-Reply-To: <1287414729.20107.68.camel@chilepepper>
On Mon, 2010-10-18 at 18:12 +0300, Luciano Coelho wrote:
> On Mon, 2010-10-18 at 16:31 +0200, ext Luciano Coelho wrote:
> > On Thu, 2010-10-14 at 11:00 +0200, Teemu Paasikivi wrote:
> > > While scanning, it is possible that beacon and probe response frames are
> > > received on other band than configured to the driver. In rx status
> > > handling this has caused "Unsupported RX rate from HW" warnings. This
> > > patch changes the wl1271_rate_to_index function to take the band of the
> > > received frame as a parameter instead of using value configuret to
> > > wl->band.
> > >
> > > Signed-off-by: Teemu Paasikivi <ext-teemu.3.paasikivi@nokia.com>
> > > ---
> >
> > Applied, thanks!
>
> Hmmm... I'm now getting loads of these:
>
> [ 465.265594] wl1271: ERROR Unsupported RX rate from HW: 15
>
> ...when scanning with my beagle board. I have also included the 11n
> patches by Shahar and Ido's "optimizations and fixes".
>
> Teemu, I had to merge your patch on top of Ido's and Shahar's patches.
> Can you check your patch in the wl12xx/master tree and confirm that it
> is okay? To me it looks good, but you never know ;)
>
>
Ok, I checked this with a beagle board and it looks like that it's
caused by the firmware marking some packets received during scanning as
being received from 5 GHz band. After some testing I have pinpointed the
problem being caused by the firmware nvs file used. It's just annoying
that those errors are being emitted when scanning. I suppose that the
message could be changed from "ERROR" to "WARNING".
Br,
Teemu
prev parent reply other threads:[~2010-10-22 7:49 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-14 9:00 [PATCH v2] wl1271: Fix warning about unsupported RX rate Teemu Paasikivi
2010-10-15 5:42 ` Juuso Oikarinen
2010-10-18 14:31 ` Luciano Coelho
2010-10-18 15:12 ` Luciano Coelho
2010-10-22 7:49 ` Teemu Paasikivi [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1287733746.2905.19.camel@paavo-desktop \
--to=ext-teemu.3.paasikivi@nokia.com \
--cc=linux-wireless@vger.kernel.org \
--cc=luciano.coelho@nokia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox