From mboxrd@z Thu Jan 1 00:00:00 1970 From: Achim Ehrlich Date: Thu, 14 May 2009 17:17:08 +0200 Subject: [U-Boot] Timeout in tftp.c In-Reply-To: <20090513113906.GG30859@game.jcrosoft.org> References: <4A0969FC.2060504@taskit.de> <20090512184841.8F54A832E416@gemini.denx.de> <4A0A53BD.6010403@tqs.de> <4A0AB0F7.2000406@taskit.de> <20090513113906.GG30859@game.jcrosoft.org> Message-ID: <4A0C35F4.8080901@taskit.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Jean-Christophe PLAGNIOL-VILLARD schrieb: > On 13:37 Wed 13 May , Achim Ehrlich wrote: >> Jens Gehrlein schrieb: >>> Wolfgang Denk schrieb: >>>> Dear Achim Ehrlich, >>>> >>>> In message <4A0969FC.2060504@taskit.de> you wrote: >>>>> The timeout for lost packages in tftp.c is defined to 5000 msecs. But >>>>> when setting the timeout get_timer(0) is called in NetSetTimeout in >>>>> net.c. >>>>> But on AT91 the timer code returns usecs and not msecs resulting in a >>>>> timeout of 5 ms instead of 5 secs, which is kind of short. I looked >>>>> in the timer code of other ARM-based platforms and they seem to use >>>>> usec-based timers as well. >>>>> Of course I can change the defined timeout in tftp.c to one second or >>>>> so, but that doesn't seem to be the right way to solve this issue. >>>>> >>>>> Any suggestions or ideas or do I miss something here? >>>> Any chance that you are using an old version of U-Boot? >>> Hi Achim, >>> >>> I'm interested in this, too. I observed timeouts on ARM11 (MX31), >>> but couldn't do any investigation due to other projects. >>> >> I'm working on a custom developed board with AT91Sam9G20 and on the >> current at91-tree of u-boot, so all new. I will do further work this >> afternoon on this issue. > which value do you put in the CONFIG_SYS_HZ? > > if you can send patch we could take a look on this > > Best Regards, > J. My value was at 1000000. Switching back to 1000 didn't solve my problem, but instead causes erase and write operations on nand flash to timeout as well. My u-boot was built on commit 03bab0091948196b9558248684c04f60943ca4b5 of the at-91 tree. Rebasing it to the current version with the changed timer code caused u-boot to not boot at all on my hardware. I just made a simple board port and didn't change much, but I add my diff against u-boot-at91 anyway. It should work on the Atmel Board as well, but I don't own any. Thank you for your help Achim -- product manager email:aehrlich at taskit.de Tel.: ++49 30 611295-25 Fax: ++49 30 611295-11 -- taskit GmbH Seelenbinderstr. 33 | D-12555 Berlin web:http://www.taskit.de Amtsgericht Charlottenburg: 93HRB39014 Managing director: Thorsten Raulfs -- -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: stamp9g20.diff Url: http://lists.denx.de/pipermail/u-boot/attachments/20090514/c1285c00/attachment.txt