Linux wireless drivers development
 help / color / mirror / Atom feed
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



      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