Nice, I've added this to my tree. I also made the following patch, to
fix up the proper return value for the probe() call, and fix a memory
leak.
Let me know if this patch causes any problems for you.
thanks,
greg k-h
diff -Nru a/drivers/usb/serial/usb-serial.c b/drivers/usb/serial/usb-serial.c
--- a/drivers/usb/serial/usb-serial.c Sun Oct 13 13:35:48 2002
+++ b/drivers/usb/serial/usb-serial.c Sun Oct 13 13:35:48 2002
@@ -1249,7 +1249,6 @@
}
#if defined(CONFIG_USB_SERIAL_PL2303) || defined(CONFIG_USB_SERIAL_PL2303_MODULE)
-#if 1
/* BEGIN HORRIBLE HACK FOR PL2303 */
/* this is needed due to the looney way its endpoints are set up */
if (((dev->descriptor.idVendor == PL2303_VENDOR_ID) &&
@@ -1280,11 +1279,11 @@
*/
if (num_bulk_in == 0 || num_bulk_out == 0) {
info("PL-2303 hack: descriptors matched but endpoints did not");
- return NULL;
+ kfree (serial);
+ return -ENODEV;
}
}
/* END HORRIBLE HACK FOR PL2303 */
-#endif
#endif
/* found all that we need */
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/