linux-kernel - Re: [PATCH] wireless: fix supported cards for rtl8187

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 09 Jul 2009 13:14:43 +0200
From:	Marcin Slusarz <marcin.slusarz@...il.com>
To:	Hin-Tak Leung <hintak.leung@...il.com>
CC:	Larry Finger <Larry.Finger@...inger.net>,
	"John W. Linville" <linville@...driver.com>,
	Linux wireless <linux-wireless@...r.kernel.org>,
	Przemyslaw Kulczycki <azrael@...ocom.pl>,
	LKML <linux-kernel@...r.kernel.org>,
	Gábor Stefanik <netrolller.3d@...il.com>
Subject: Re: [PATCH] wireless: fix supported cards for rtl8187

Hin-Tak Leung pisze:
> 2009/7/9 Larry Finger <Larry.Finger@...inger.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.

commit aeeab4ff06b8e29cfe2fe730ba626f7e2487ba03
Author: John W. Linville <linville@...driver.com>
Date:   Wed May 27 09:21:57 2009 +0200

    rtl8187: add USB ID for Linksys WUSB54GC-EU v2 USB wifi dongle

    http://bugzilla.kernel.org/show_bug.cgi?id=13383

    Reported-by: Przemyslaw Kulczycki <azrael@...ocom.pl>
    Signed-off-by: John W. Linville <linville@...driver.com>

diff --git a/drivers/net/wireless/Kconfig b/drivers/net/wireless/Kconfig
index 8a08235..3d94e7d 100644
--- a/drivers/net/wireless/Kconfig
+++ b/drivers/net/wireless/Kconfig
@@ -430,6 +430,7 @@ config RTL8187
          ASUS P5B Deluxe
          Toshiba Satellite Pro series of laptops
          Asus Wireless Link
+         Linksys WUSB54GC-EU

          Thanks to Realtek for their support!

diff --git a/drivers/net/wireless/rtl818x/rtl8187_dev.c b/drivers/net/wireless/rtl818x/rtl8187_dev.c
index bac6cfb..d51ba0a 100644
--- a/drivers/net/wireless/rtl818x/rtl8187_dev.c
+++ b/drivers/net/wireless/rtl818x/rtl8187_dev.c
@@ -71,6 +71,8 @@ static struct usb_device_id rtl8187_table[] __devinitdata = {
        {USB_DEVICE(0x18E8, 0x6232), .driver_info = DEVICE_RTL8187},
        /* AirLive */
        {USB_DEVICE(0x1b75, 0x8187), .driver_info = DEVICE_RTL8187},
+       /* Linksys */
+       {USB_DEVICE(0x1737, 0x0073), .driver_info = DEVICE_RTL8187B},
        {}
 };



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/