Resending this -- patch doesn't seem to be merged, and it may resolve some of the problems Ben H reported.
- Dave
David Brownell wrote:
Paul Mackerras wrote:
Alan Stern writes:
How does this work? It's a smaller change, against BK-current. But maybe David will prefer to change ohci-pci.c instead.
It works fine, and I have been using it for a while now. This, or something similar, needs to go in, since others have been complaining about the problems this fixes.
I didn't see anything obviously wrong here, and it's the best patch we have right now. I'd say merge it.
- Dave
===== hcd.c 1.107 vs edited ===== --- 1.107/drivers/usb/core/hcd.c Thu Jun 12 10:28:01 2003 +++ edited/drivers/usb/core/hcd.c Wed Jun 25 10:02:25 2003 @@ -483,7 +483,7 @@ { struct urb *urb; struct usb_hcd *hcd; - int length; + int length = 0; unsigned long flags;
urb = (struct urb *) ptr; @@ -499,7 +499,9 @@ return; }
- length = hcd->driver->hub_status_data (hcd, urb->transfer_buffer); + if (!HCD_IS_SUSPENDED (hcd->state)) + length = hcd->driver->hub_status_data ( + hcd, urb->transfer_buffer);
/* complete the status urb, or retrigger the timer */ spin_lock (&hcd_data_lock);
Paul.
------------------------------------------------------- This SF.net email is sponsored by: VM Ware With VMware you can run multiple operating systems on a single machine. WITHOUT REBOOTING! Mix Linux / Windows / Novell virtual machines at the same time. Free trial click here:http://www.vmware.com/wl/offer/358/0 _______________________________________________ [EMAIL PROTECTED] To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
