On Sun, Oct 10, 2004 at 06:10:22PM -0700, David Brownell wrote:
> On Sunday 10 October 2004 9:43 am, Russell King wrote:
> 
> > 7. This one is the killer.  Once it's been hacked to compile, plugging
> >    in a USB flash key results in:
> > 
> > usb 1-1: new full speed USB device using address 2
> > usb 1-1: device not accepting address 2, error -110
> > usb 1-1: new full speed USB device using address 3
> > usb 1-1: device not accepting address 3, error -110
> > 
> >    which is the same error as the previous driver gave.
> 
> I suspect that problem isn't associated with Lothar's code at all;
> a lot of folk have been running into problems like that lately.

Actually it is.  After running the SL811HS against the SA1110 UDC driver,
the problem comes out immediately.  The SL811HS driver is _NOT_ following
the protocol for bringing a device out of "default suspend" state.

Sure, it's resetting the bus, but then it's waiting for about 10ms before
enabling SOFs, by which time the device has given up completely and gone
back to suspend mode.

With this kind of bug, this driver could never have worked.

I have it almost working now, mainly by unhooking the reset device
and device detection from the "OHCI emulation".

As I've said previously, I think the whole design of this driver is
flawed by the mere fact that it tries to pretend to be an OHCI device
when it isn't.  That leads to a hell of a lot of extra code and
complexity - which directly leads to these types of bugs.

The code I have here now does a port reset according to the USB spec,
and it's damned obvious from the new code what it's doing to achieve
that.  That's how it should've been in the first place.

-- 
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:  2.6 PCMCIA      - http://pcmcia.arm.linux.org.uk/
                 2.6 Serial core


-------------------------------------------------------
This SF.net email is sponsored by: IT Product Guide on ITManagersJournal
Use IT products in your business? Tell us what you think of them. Give us
Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more
http://productguide.itmanagersjournal.com/guidepromo.tmpl
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to