From mboxrd@z Thu Jan 1 00:00:00 1970 From: Markus =?iso-8859-1?Q?Klotzb=FCcher?= Date: Mon, 24 Sep 2007 08:44:52 +0200 Subject: [U-Boot-Users] isp116x-hcd and usb_storage References: <334763.49344.qm@web59204.mail.re1.yahoo.com> Message-ID: <87hclkd0kb.fsf@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Timo, tike64 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