linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Larry Finger <Larry.Finger@lwfinger.net>
To: Herton Ronaldo Krzesinski <herton@mandriva.com.br>
Cc: linux-wireless@vger.kernel.org,
	"John W. Linville" <linville@tuxdriver.com>,
	Pavel Roskin <proski@gnu.org>
Subject: Re: [PATCH] Add Realtek 8187B support
Date: Wed, 09 Apr 2008 10:46:53 -0500	[thread overview]
Message-ID: <47FCE4ED.9010308@lwfinger.net> (raw)
In-Reply-To: <200804081931.07638.herton@mandriva.com.br>

Herton Ronaldo Krzesinski wrote:
> Hi, this patch (made against wireless-testing repository) adds support for
> 8187B to the rtl8187 module.

Thanks for your efforts. If this driver had been available two months 
ago, it would have made my life much simpler.

Don't get too discouraged about the negative comments - that kind of 
diligence is what makes Linux so robust, and keeps this open-source 
project from chaos.

Once your patches get more stabilized, I will be submitting one to 
handle my hardware, which is a LevelOne WNC-0301USB. For some reason, 
it has an idProduct code of 0x8187, not 0x8189, and one has to look at 
the iProduct value. For early testing, I am just forcing it to be an 
8187B. For my information, what is the iProduct value for your device?

I did only minimal testing. The USB device was only about 1 meter from 
the AP. It connected with a WPA-encrypted network in about the same 
time as my BCM43xx devices do.

I do have one minor comment - I prefer to see signal and noise values 
in dBm, not as xx/65.

Good job,

Larry

  parent reply	other threads:[~2008-04-09 15:47 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-08 22:31 [PATCH] Add Realtek 8187B support Herton Ronaldo Krzesinski
2008-04-09  4:22 ` Pavel Roskin
2008-04-09 17:36   ` Herton Ronaldo Krzesinski
2008-04-09 18:59     ` Pavel Roskin
2008-04-09  6:31 ` Pavel Roskin
2008-04-09 18:07   ` Herton Ronaldo Krzesinski
2008-04-09 23:31     ` Herton Ronaldo Krzesinski
2008-04-09 15:46 ` Larry Finger [this message]
2008-04-09 18:12   ` Herton Ronaldo Krzesinski
2008-04-10  3:39 ` Larry Finger
2008-04-10  4:43   ` Herton Ronaldo Krzesinski
2008-04-11  4:57     ` Larry Finger
2008-04-12  2:36       ` Herton Ronaldo Krzesinski
2008-04-12 17:29         ` Pavel Roskin
2008-04-12 22:49           ` Larry Finger

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=47FCE4ED.9010308@lwfinger.net \
    --to=larry.finger@lwfinger.net \
    --cc=herton@mandriva.com.br \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=proski@gnu.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).