> From: Petko Manolov <[EMAIL PROTECTED]>
> Date: Mon, 22 Jul 2002 21:51:39 -0700 (PDT)

>  Do you by any chance know the vendor and device id of that Melco device?

The patch was attached to the bug report. This part was sane:

@@ -72,11 +72,15 @@
 
 /* Define these values to match your device */
 #define VENDOR_ID_REALTEK              0x0bda
+#define VENDOR_ID_MELCO                        0x411
+
 #define PRODUCT_ID_RTL8150             0x8150
+#define PRODUCT_ID_LUAKTX              0x12
 
 /* table of devices that work with this driver */
 static struct usb_device_id rtl8150_table [] = {
        { USB_DEVICE(VENDOR_ID_REALTEK, PRODUCT_ID_RTL8150) },
+       { USB_DEVICE(VENDOR_ID_MELCO, PRODUCT_ID_LUAKTX) },
        { }                             
 };
 

This got me thinking:

@@ -671,8 +675,11 @@
                err("usb_set_configuration() failed");
                return NULL;
        }
-       if ((udev->descriptor.idVendor != VENDOR_ID_REALTEK) ||
-           (udev->descriptor.idProduct != PRODUCT_ID_RTL8150)) {
+       if ((udev->descriptor.idVendor == VENDOR_ID_REALTEK) ||
+           (udev->descriptor.idProduct == PRODUCT_ID_RTL8150) ||
+           ((udev->descriptor.idVendor == VENDOR_ID_MELCO) &&
+            (udev->descriptor.idProduct == PRODUCT_ID_LUAKTX))) {
+       } else {
                err("Not the one we are interested about");
                return NULL;
        }

I think you better drop this if statement altogether.

-- Pete


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to