Oliver may be on to something. Try using this patch instead of your changes:
--- a/drivers/usb/class/cdc-acm.c Fri Feb 11 22:17:32 2005 +++ b/drivers/usb/class/cdc-acm.c Fri Feb 11 22:16:41 2005 @@ -767,8 +767,15 @@
static void acm_disconnect(struct usb_interface *intf) { - struct acm *acm = usb_get_intfdata (intf); - struct usb_device *usb_dev = interface_to_usbdev(intf); + struct acm *acm; + struct usb_device *usb_dev; + + dbg("%s 1: intf = %p\n", __FUNCTION__, intf); + acm = usb_get_intfdata (intf); + dbg(" 2: acm = %p\n", acm); + dbg(" 3: acm->dev = %p\n", acm->dev); + usb_dev = interface_to_usbdev(intf); + dbg(" 4: usb_dev = %p\n", usb_dev);
if (!acm || !acm->dev) { dbg("disconnect on nonexisting interface");
Alan Stern
Eh ! Forgive me my stupidity ;-) I should look at oops before blindly adding dbg()s. Well, yes it is called twice with the same acm ... See usboops.6 devik
------------------------------------------------------- SF email is sponsored by - The IT Product Guide Read honest & candid reviews on hundreds of IT Products from real users. Discover which products truly live up to the hype. Start reading now. http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click _______________________________________________ linux-usb-devel@lists.sourceforge.net To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
