[U-Boot-Users] bug in at91rm9200 interrupt.c [udelay()] ?

[U-Boot-Users] bug in at91rm9200 interrupt.c [udelay()] ?

[Mirco Fuchs]

hello,

i'm porting u-boot to the csb637 board from cogent.

I think i found a bug in the udelay (); function in file interrupt.c (at91rm9200).

Assume the following C code:

...
start = get_timer(0);

/* some code */

udelay(1500)     // delay of about 1500us
next = get_timer(start);
...

what we expect is that the value of next holds the difference between next and start in milliseconds. This works fine without the udelay() function between the get_timer() function calls. But in udelay() the global variable timestamp is reset to zero and so there is no longer any relationship between next and start. The effect will be that any time measurements will fail.

I allready looked in some other interrupt.c files, but the udelay() function is handled in a similar way.

Maybe i'm wrong with my assumption, but i don't think so. I'm very interested in your answers.


Regards

Mirco

_________________________________________________________________________
Mit der Gruppen-SMS von WEB.DE FreeMail k?nnen Sie eine SMS an alle 
Freunde gleichzeitig schicken: http://freemail.web.de/features/?mc=021179

[U-Boot-Users] bug in at91rm9200 interrupt.c [udelay()] ?

[Anders Larsen]

Mirco Fuchs <mircofuchs@web.de> schreibt:
>i'm porting u-boot to the csb637 board from cogent.

Did you check the list archives?

U-Boot already runs on the csb637 board - the patches were submitted
to the list on the 3.rd of May (but alas, haven't been merged yet).
This was mentioned again on this list less than one week ago.

Cheers
 Anders

[U-Boot-Users] bug in at91rm9200 interrupt.c [udelay()] ?

[Mirco Fuchs]

Thank you for your fast reply,

i didn't check the list archive on csb637 because i downloaded u-boot version 1.1.2 and thought that it was the newest version available - this thought was obvioulsy wrong ...

But i did check the list now and i couldn't find any patch. The only entry i found was that there was a patch added (on 3.rd May) , but i couldn't find the patch itself.

I also browsed the CVS Repository, but it seems that there is no folder csb637 in the /board directory.

So what do i have to do to download the patch???
(Sorry if this question sounds stupid, but i'm new to this mailing list)


best regards

Mirco
______________________________________________________________
Verschicken Sie romantische, coole und witzige Bilder per SMS!
Jetzt bei WEB.DE FreeMail: http://f.web.de/?mc=021193

[U-Boot-Users] bug in at91rm9200 interrupt.c [udelay()] ?

[Anders Larsen]

Mirco Fuchs <mircofuchs@web.de> schreibt:
>i didn't check the list archive on csb637 because i downloaded u-boot
>version 1.1.2 and thought that it was the newest version available - this
>thought was obvioulsy wrong ...

Indeed - you should use top-of-CVS whenever possible.
>
>But i did check the list now and i couldn't find any patch. The only
>entry i found was that there was a patch added (on 3.rd May) , but i
>couldn't find the patch itself.

Bummers - it seems SourceForge does not store attachments.
I'll send you a copy of the patches per private e-mail.
>
>I also browsed the CVS Repository, but it seems that there is no folder
>csb637 in the /board directory.

Like I wrote - the patches haven't been merged yet.
>
Cheers
 Anders

[U-Boot-Users] bug in at91rm9200 interrupt.c [udelay()] ?

[Wolfgang Denk]

In message <fc.004c4e48003562c23b9aca00c0e693d5.3562c7@rea.de> you wrote:
>
> Bummers - it seems SourceForge does not store attachments.

But the gmane archive does.


Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"A complex system that works is invariably found to have evolved from
a simple system that worked."             - John Gall, _Systemantics_