Re: [PATCH] wireless: fix supported cards for rtl8187

[Marcin Slusarz <marcin.slusarz@gmail.com>]

Hin-Tak Leung wrote:
> 2009/7/9 Marcin Slusarz <marcin.slusarz@gmail.com>:
>> Hin-Tak Leung pisze:
>>> 2009/7/9 Larry Finger <Larry.Finger@lwfinger.net>:
>>>> Hin-Tak Leung wrote:
>>>>> if they are not labelled v1/v2/v3 as such, it may be more
>>>>> useful/informative to change to
>>>>>
>>>>>     [some variant of ] ... [(other variants are supported by the
>>>>> rt2070/rt3070 modules)]
>>>>>
>>>>> instead of just adding [v2] to the end.
>>>> I don't know about the box as I bought mine used on Ebay, but the
>>>> label on the device clearly states that it is a WG111v2.
>>>>
>>>> Larry
>>>>
>>>>
>>> But that's  a Netgear WG111v2 ? We are talking about Linksys
>>> WUSB54GC-EU and its v1/v2/v3 . I am just thinking that if it is for
>>> informative purposes (for those who actually selectively build their
>>> kernel modules), adding a bit more info, such as suggesting to use
>>> rt73usb/rt2070/rt3070 modules, won't hurt.
>>>
>>> Incidentally I like to know a bit more about this patch - presumably
>>> Marcin has such a device? I am also a little curious, as this commit
>>> should be accompanied by a corresponding update in another Kconfig
>>> which supports the device? I am not trying to be picky, but for an
>>> informative (non-code-changing) patch, it can be more informative.
>> I have a v3 device and this driver does not support it.
>> Below is the commit which added the string my patch corrects.
>> I've sent yesterday another patch which adds support for WUSB54GC v3
>> to the rt3070 driver.
> 
> Here is some info from Gabor (shouldn't this be reply-all rather than just me):
> ------------------
> The v1 and v2 look identical (though the label on the device indicates
> the revision). The v3 looks completely different. They all have
> different USB IDs (v1 is stock RT73, v2 has a custom ID, while v3 is
> AFAIK stock RT2070).
> ------------------
> 
> I guess to be informative, may I suggest adding '(for v1  see rt73usb,
> and v3 see rt3070)' or something similiar to the 'v2' change?
> It is fair customary in Kconfig to mention even alternative/competing
> drivers for the same device. And when you add v3 support to the rt3070
> driver, a reverse mention may be appropriate. (I guess I am spending
> far too long on this discussion - but I reckon, the purpose of a
> non-code-changing change is meant to be informative, so one might want
> to be as informative as appropriate without being too wordy...)

Look, support for Ralink chipsets is a mess. They are supported by too sets
of drivers - Ralink's ugly as hell but working staging drivers (rt2860,
rt2870, rt3070, soon rt3090) and properly developed but lagging rt2x00
drivers (RT2400PCI, RT2500PCI, RT61PCI, RT2500USB, RT73USB, RT2800USB).

Theoretically my WUSB54GCv3 is supported by RT2800USB (which was added during
2.6.31 merge window), but it doesn't work. With tiny patch added to rt3070 
(just usb id) my card works. Some day rt2x00 drivers will be better than Ralink's,
but right now they are not.

So I wouldn't like to encourage any of them. I just wanted to fix clearly
wrong help text which stated that rtl8187 (Realtek's chipset!) supports
all WUSB54GC cards.

Marcin