[U-Boot] Timeout in tftp.c

From: Achim Ehrlich <aehrlich@taskit.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] Timeout in tftp.c
Date: Thu, 14 May 2009 17:17:08 +0200	[thread overview]
Message-ID: <4A0C35F4.8080901@taskit.de> (raw)
In-Reply-To: <20090513113906.GG30859@game.jcrosoft.org>

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 