[U-Boot-Users] AT91RM9200 Errata (Was: CSB637 support - big bug..)

[Steven Scholz <steven.scholz@imc-berlin.de>]

Patrick,

 .. wrote:
>>--- Urspr?ngliche Nachricht ---
>>Von: Steven Scholz <steven.scholz@imc-berlin.de>
>>An: Anders Larsen <alarsen@rea.de>
>>Kopie: "Patrick .." <oc3an@gmx.net>, u-boot-users at lists.sourceforge.net
>>Betreff: Re: [U-Boot-Users] AT91RM9200 Errata (Was: CSB637 support - big
>>bug..)
>>Datum: Wed, 24 Aug 2005 11:01:30 +0200

No need to sent the above text!

[snip]

>>-------------------------------------------------------
>>SF.Net email is Sponsored by the Better Software Conference & EXPO
>>September 19-22, 2005 * San Francisco, CA * Development Lifecycle
>>Practices
>>Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA
>>Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf
>>_______________________________________________
>>U-Boot-Users mailing list
>>U-Boot-Users at lists.sourceforge.net
>>https://lists.sourceforge.net/lists/listinfo/u-boot-users
>>

No need to sent the above text!

[snip]

> No need to wait between the writes.

Due to all the rubbish text above I forget the question... ;-)

> This fix is quite simple to implement.  in lowlevel_init.S edit the values
> for MCKR (as shown below).  You also need to modify the length of the
> assembly loop some lines above which iterates through this list (also shown
> below).  This length should really be calculated (but it works).

Thanks a million.

> Then define MCKR_VAL0 as the first value to be written and MCKR_VAL1 as the
> second in the includes/common/csb637.h file.
> 
> Hope this helps,

Sure. Maybe you could send unified diffs next time as they're much
easier to read.

Thanks anyway!

> 	/* memory control configuration */
> 	/* this isn't very elegant, but	 what the heck */
> 	ldr	r0, =SMRDATA
> 	ldr	r1, _MTEXT_BASE
> 	sub	r0, r0, r1
> 	add	r2, r0, #88  // Edited by Patrick

Aah. I forgot that. Cause I thought the original autjort would never do
a bad thing like hardcoding such a loop count ... ;-)

--
Steven