taken from windows inf file
Signed-off-by: Xose Vazquez Perez <[email protected]>
---
drivers/net/wireless/rtl818x/rtl8180_dev.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/drivers/net/wireless/rtl818x/rtl8180_dev.c b/drivers/net/wireless/rtl818x/rtl8180_dev.c
index 4801589..6d6140d 100644
--- a/drivers/net/wireless/rtl818x/rtl8180_dev.c
+++ b/drivers/net/wireless/rtl818x/rtl8180_dev.c
@@ -36,6 +36,7 @@ MODULE_LICENSE("GPL");
static DEFINE_PCI_DEVICE_TABLE(rtl8180_table) = {
/* rtl8185 */
+ { PCI_DEVICE(PCI_VENDOR_ID_REALTEK, 0x8184) },
{ PCI_DEVICE(PCI_VENDOR_ID_REALTEK, 0x8185) },
{ PCI_DEVICE(PCI_VENDOR_ID_BELKIN, 0x700f) },
{ PCI_DEVICE(PCI_VENDOR_ID_BELKIN, 0x701f) },
@@ -45,6 +46,8 @@ static DEFINE_PCI_DEVICE_TABLE(rtl8180_table) = {
{ PCI_DEVICE(0x1799, 0x6001) },
{ PCI_DEVICE(0x1799, 0x6020) },
{ PCI_DEVICE(PCI_VENDOR_ID_DLINK, 0x3300) },
+ { PCI_DEVICE(PCI_VENDOR_ID_DLINK, 0x3301) },
+ { PCI_DEVICE(0x1432, 0x7106) },
{ }
};
--
1.6.6.1
--- On Sat, 15/5/10, Xose Vazquez Perez <[email protected]> wrote:
> Sorry, but no realtek devices here.
I am sorry, if you do not own any of the devices for which you just copied a bunch of IDs from the windows driver, that means your patch is untested. What is your purpose of copying IDs from the windows driver?
There is no code sharing between the windows driver and the linux driver, and there are at least 5 variants of the device - so what the windows driver supports really bears no relationship with what the linux driver supports.
I don't think it is a good idea to blindly copy IDs from the windows driver without testing.
On 05/15/2010 01:49 AM, Hin-Tak Leung wrote:
> --- On Sat, 15/5/10, Xose Vazquez Perez <[email protected]> wrote:
>> taken from windows inf file
>>
>> Signed-off-by: Xose Vazquez Perez <[email protected]>
>
> Acked-by: Hin-Tak Leung <[email protected]>
>
> You added 3 device ids - do you own one of them by any chance? We like to know who has what hardware sometimes - just in case they are slightly different.
Sorry, but no realtek devices here.
--
If less is more than more, most is more than less.
--- On Sat, 15/5/10, Xose Vazquez Perez <[email protected]> wrote:
> taken from windows inf file
>
> Signed-off-by: Xose Vazquez Perez <[email protected]>
Acked-by: Hin-Tak Leung <[email protected]>
You added 3 device ids - do you own one of them by any chance? We like to know who has what hardware sometimes - just in case they are slightly different.
Hin-Tak
taken from windows inf file
Signed-off-by: Xose Vazquez Perez <[email protected]>
---
drivers/net/wireless/rtl818x/rtl8187_dev.c | 5 +++++
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/drivers/net/wireless/rtl818x/rtl8187_dev.c b/drivers/net/wireless/rtl818x/rtl8187_dev.c
index b332c1e..f641bde 100644
--- a/drivers/net/wireless/rtl818x/rtl8187_dev.c
+++ b/drivers/net/wireless/rtl818x/rtl8187_dev.c
@@ -53,6 +53,7 @@ static struct usb_device_id rtl8187_table[] __devinitdata = {
{USB_DEVICE(0x0bda, 0x8189), .driver_info = DEVICE_RTL8187B},
{USB_DEVICE(0x0bda, 0x8197), .driver_info = DEVICE_RTL8187B},
{USB_DEVICE(0x0bda, 0x8198), .driver_info = DEVICE_RTL8187B},
+ {USB_DEVICE(0x0bda, 0x8199), .driver_info = DEVICE_RTL8187B},
/* Surecom */
{USB_DEVICE(0x0769, 0x11F2), .driver_info = DEVICE_RTL8187},
/* Logitech */
@@ -79,6 +80,10 @@ static struct usb_device_id rtl8187_table[] __devinitdata = {
{USB_DEVICE(0x1b75, 0x8187), .driver_info = DEVICE_RTL8187},
/* Linksys */
{USB_DEVICE(0x1737, 0x0073), .driver_info = DEVICE_RTL8187B},
+ /* Trust */
+ {USB_DEVICE(0x145f, 0x0161), .driver_info = DEVICE_RTL8187},
+ /* Edimax */
+ {USB_DEVICE(0x7392, 0x7612), .driver_info = DEVICE_RTL8187},
{}
};
--
1.6.6.1