All of lore.kernel.org
 help / color / mirror / Atom feed
From: Larry Finger <Larry.Finger@lwfinger.net>
To: Hin-Tak Leung <hintak_leung@yahoo.co.uk>
Cc: Pavel Roskin <proski@gnu.org>, Matthew Garrett <mjg@redhat.com>,
	"John W. Linville" <linville@tuxdriver.com>,
	herton@mandriva.com.br, linux-wireless@vger.kernel.org,
	flamingice@sourmilk.net, andreamrl@tiscali.it,
	linville@redhat.com
Subject: Re: [RFC][PATCH] Realtek 8187B wireless support with product id	0x8197/0x8189
Date: Thu, 26 Jun 2008 15:44:43 -0500	[thread overview]
Message-ID: <4863FFBB.9020504@lwfinger.net> (raw)
In-Reply-To: <4863F1DE.3080503@yahoo.co.uk>

Hin-Tak Leung wrote:
> Pavel Roskin wrote:
>> On Thu, 2008-06-26 at 13:33 -0500, Larry Finger wrote:
>>
>>> Do I understand correctly that it might be better to check for the 
>>> existence of endpoint 02 for an 8187 device? Or would it be better to 
>>> look for endpoint 12 to set it as an 8187B?
>>
>> It's probably OK either way, but I would look at some number that takes
>> more than just 2 values (true and false).  Say, the number of endpoints
>> would be such number.  This way, it there is some elusive 8187A or
>> 8187C, it may have a different number and won't be mistaken for known
>> chipsets.
>>
>> Of course, we cannot guarantee anything.  It's just a matter of general
>> sanity that may or may not help.  But sometimes it helps.
> 
> Larry's 8187 indeed looks like a 8187B i.e. the OEM being naughty and needs
> some serious lashing there... where does it come from - what brand and what
> is it bundled with?

It is a Level One WNC-0301USB dongle. I bought it as an interim 
replacement when the PCIe interface in my laptop expired and I could 
not get it fixed at that time. I was quite put out when Linux couldn't 
drive it as I needed Linux for my E-mail and it had to be wireless. I 
tried ndiswrapper with it, but that didn't work either. I finally 
settled on running Windows XP with Linux as a virtual machine running 
under VirtualBox. It wasn't pretty, but it worked.

> The 8187/8187b code actually *uses* the
> endpoints to send management frames for association - that's a 
> functional difference and it is not possible to drive a 8187 as if it is 
> a 8187b or
> vice versa. In the absence of
> more authoritative answers like reading from a register or something, 
> counting
> the endpoints - I would actually go a bit further and demand the used 
> endpoints
> being there and in the right direction and propertes (int/bulk) - would 
> seem
> to be a good idea. As for what to do "if things don't add up", at least 
> a warning through dmesg "give the OEM some lashing for calling a 8187b 
> 0x8187".


Good idea. It probably won't change anything other than make us feel good.


> I think Larry's question could be re-phrased as: if there is a preculiar 
> device
> with *both* end point 2 and endpoint 12, what to treat it as?
> 
> I would be inclined to treat it as the newer chip - but some big 
> warning, or
> even refusing to carry on, is probably in order.


I will prepare a patch with these warnings.


> Herton's code also seem to distinguish revC/revD/revE of different 
> 8187B's -
> I don't seem to have seen those distinctions in the original vendor 
> code, but
> maybe I haven't look hard enough :-).

I have not found them either. FWIW, my device is reported as 
RTL8187BvE V0 + rtl8225z2. The decision comes from 
rtl818x_ioread8(priv, (u8 *)0xFFE1), but I don't know what is at that 
address.

Larry

  parent reply	other threads:[~2008-06-26 20:43 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-15  0:59 [RFC][PATCH] Realtek 8187B wireless support with product id 0x8197/0x8189 Hin-Tak Leung
2008-06-18 22:31 ` Uwe Hermann
2008-06-19  0:17   ` Hin-Tak Leung
2008-06-25 14:08 ` John W. Linville
2008-06-25 14:59   ` Larry Finger
2008-06-25 15:44     ` Hin-Tak Leung
2008-06-26  0:26     ` Matthew Garrett
2008-06-26  0:44       ` Hin-Tak Leung
2008-06-26  3:41         ` Larry Finger
2008-06-26 17:22           ` Hin-Tak Leung
2008-06-26 18:33             ` Larry Finger
2008-06-26 18:44               ` Pavel Roskin
2008-06-26 19:45                 ` Hin-Tak Leung
2008-06-26 20:40                   ` Herton Ronaldo Krzesinski
2008-06-26 21:24                     ` Herton Ronaldo Krzesinski
2008-06-26 21:53                       ` Larry Finger
2008-06-26 20:44                   ` Larry Finger [this message]
2008-06-26 21:15                     ` Herton Ronaldo Krzesinski

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=4863FFBB.9020504@lwfinger.net \
    --to=larry.finger@lwfinger.net \
    --cc=andreamrl@tiscali.it \
    --cc=flamingice@sourmilk.net \
    --cc=herton@mandriva.com.br \
    --cc=hintak_leung@yahoo.co.uk \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@redhat.com \
    --cc=linville@tuxdriver.com \
    --cc=mjg@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.