Petko:

The following check seems redundant:

static void * rtl8150_probe(struct usb_device *udev, unsigned int ifnum,
                            const struct usb_device_id *id)
{
......
        if ((udev->descriptor.idVendor != VENDOR_ID_REALTEK) ||
            (udev->descriptor.idProduct != PRODUCT_ID_RTL8150)) {
                err("Not the one we are interested about");
                return NULL;
        }

A correct ID table is supplied, so why do you want to
do it manually?

The question came up after reviewing a bug (RFE actually)
 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=68565

-- 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