From: Jason Andryuk <jandryuk@gmail.com>
To: Kalle Valo <kalle.valo@iki.fi>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH] at76c50x-usb: support cfg80211 scanning
Date: Wed, 18 Feb 2009 22:32:10 -0500 [thread overview]
Message-ID: <1235014330.6344.18.camel@rainbow> (raw)
In-Reply-To: <8763j7qyfx.fsf@litku.valot.fi>
On Wed, 2009-02-18 at 22:31 +0200, Kalle Valo wrote:
> Jason Andryuk <jandryuk@gmail.com> writes:
>
> > With the latest mac80211 stack, the driver needs to be updated for
> > cfg80211 scanning. I based the changes off of modifications for
> > at76_usb found here:
> > http://johannes.sipsolutions.net/patches/old/all/2008-09-19-13:35/020-cfg80211-scan.patch
> >
> > The trick was that max_signal also needs to be set to avoid a divide
> > by zero Oops. I just guessed and used the value 100 for now.
>
> Thank you for doing this. Unfortunately the patch was corrupted, so I
> had to manually take your changes. I also included the
> ieee80211_scan_compileted() fix to this patch.
>
> > Additionally, I added some ieee80211_{wake,stop}_queues calls that
> > were contained in the older at76_usb. There is ambiguity as to
> > whether at76_join should be called before or after waking the queues.
>
> I would like to have a separate patch for this. Also I would like to
> know what kind of bug this fixes.
>
> Sorry to be so picky, but I'm going to push the driver into mainline
> soon and I try avoid regressions right now.
I must admit, I do not have the best understanding of the driver. As
such, I was making modifications to at76c50x-usb to be as similar to
at76_usb+mac80211 as possible to prevent the driver from oops-ing.
I don't know the proper usage of ieee80211_{wake,stop}_queues, but my
understanding is they should be called at different times to indicate
readiness for transmitting frames to the mac80211 stack.
I am having issues with my Linksys WUSB11 adapter. It has the intersil
3861 and the scan command is reporting Command Status of 0x03 - Invalid
parameter. For right now, scanning does not work for me.
Sorry about the corrupt patches. The gmail web interface was causing
problems.
Take care.
Jason
next prev parent reply other threads:[~2009-02-19 3:32 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-17 0:43 [PATCH] at76c50x-usb: support cfg80211 scanning Jason Andryuk
2009-02-18 20:31 ` Kalle Valo
2009-02-19 3:32 ` Jason Andryuk [this message]
2009-02-19 6:33 ` Kalle Valo
2009-02-20 4:07 ` Jason Andryuk
2009-02-21 7:57 ` Kalle Valo
2009-02-21 21:44 ` Jason Andryuk
2009-02-21 22:00 ` Jason Andryuk
2009-02-22 11:49 ` Kalle Valo
2009-02-22 23:54 ` Jason Andryuk
2009-02-23 3:29 ` Jason Andryuk
2009-02-22 11:46 ` Kalle Valo
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=1235014330.6344.18.camel@rainbow \
--to=jandryuk@gmail.com \
--cc=kalle.valo@iki.fi \
--cc=linux-wireless@vger.kernel.org \
/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;
as well as URLs for NNTP newsgroup(s).