From mboxrd@z Thu Jan  1 00:00:00 1970
From: Markus =?iso-8859-1?Q?Klotzb=FCcher?= <mk@denx.de>
Date: Tue, 18 Sep 2007 16:56:26 +0200
Subject: [U-Boot-Users] isp116x-hcd and usb_storage
In-Reply-To: <89014.45054.qm@web59211.mail.re1.yahoo.com> (tike64@yahoo.com's
	message of "Tue, 18 Sep 2007 04:54:46 -0700 (PDT)")
References: <89014.45054.qm@web59211.mail.re1.yahoo.com>
Message-ID: <87hclsggyt.fsf@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de


tike64 <tike64@yahoo.com> writes:
> --- tike64 wrote:
>> --- Markus Klotzb?cher wrote:
>> > U-Boots USB implementation is known to have some timing issues
>
> OK, it really seems to be a timing issue. I have now a correct 'fatls'
> output with all debugging disabled. I added one delay into the
> usb_stor_BBB_transport function like this:
>
> 	if (srb->datalen == 0)
> 		goto st;
> 	USB_STOR_PRINTF("DATA phase\n");
> -->	wait_ms(200);
> 	if (dir_in)
>
> 100 ms was too short.
>
> Now the question is, what is the correct way to fix this?

Have you got an idea why this delay is needed? The correct way would be
to figure this out and send a patch and explanation to the list.

Thanks in advance.

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