[U-Boot-Users] isp116x-hcd and usb_storage

[Markus Klotzbücher]

Hi Timo,

tike64 <tike64 at yahoo.com> writes:

> I think I got it...

Thats good news!

> --- tike64 wrote:
>> --- Markus Klotzbücher wrote:
>> > tike64 writes:
>> > > Now I don't understand how transactions can fail with 6 or 5
>> > > occasionally. Are sticks known to be unreliable or should I
>> > > suspect my hardware? What should be done after that kind of
>> > > failure?
>
> It seems to me now that this is hardware problem in sticks or our HW or
> both. But nevertheless, failed transactions can be simply retried.

Ok. I think thats a good thing to do. We simply live in a world of
misbehaving USB Hardware.

>> > Can you test different sticks to see if that makes any difference?
>
> They indeed are different. Some fail occasionally with wrong PID bits
> and some with CRC errors and so on. I saw one combination of stick and
> hub where I couldn't get any errors at all.



> Retries were working properly but after that program logic error led to
> the data overrun error. I fixed it and now it seems to work reliably.
>
> I have tested it with several memory sticks (Kingston, Toshiba, SanDisk
> and no-name) by repeatedly loading 165K file and comparing it each time
> with an image loaded with tftpboot.
>
> How do I proceed to upload the file (isp116x-hcd.c) for review and
> public consumption?

Please post a patch to the list. See this link for detailed information
on submitting patches:

http://www.denx.de/wiki/UBoot/Patches

Thanks!

Best regards

Markus Klotzbücher

--
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de