Hi Wolfgang,
I really don't like the ordering of those registers the way you fixed it last
night. My comment before the array was "in order of the 4 mddrc registers" I
recommented to "in memory order" If you change it to your way then you should
recomment at least.
I redid it to save this ordering AND not have confusing array indexes. Please
reconsider.
-Martha
PS - the last patch is the good one ... got flustered and sent the wrong one
first and then forgot to fix the reordered ELPIDA settings so I hgad to redo
again.
-----Original Message-----
From: Wolfgang Denk w...@denx.de
Sent 9/25/2009 8:22:09 AM
To: Jerry Van Baren gvb.ub...@gmail.com
Cc: U-Boot u-boot@lists.denx.de, mm...@silicontkx.com
Subject: Re: [U-Boot] Broken board: mpc5121ads
Dear Jerry Van Baren,
In message 4abc9bcc.9060...@gmail.com you wrote:
A couple of lines got scrambled (exchanged!?) on the mpc5121. I did not
have the time to do a "git bisect" or "blame" to figure out where it
happened.
The error is:
$ cat LOG/mpc5121ads.ERR
mpc5121ads.c: In function 'initdram':
mpc5121ads.c:176: error: expected '}' before numeric constant
make[1]: *** [mpc5121ads.o] Error 1
make: *** [board/freescale/mpc5121ads/libmpc5121ads.a] Error 2
The problem is the list is re-ordered in
./board/freescale/mpc5121ads/mpc5121ads.c
u32 elpida_mddrc_config[4] = {
CONFIG_SYS_MDDRC_TIME_CFG0,
CONFIG_SYS_MDDRC_TIME_CFG1_ELPIDA,
CONFIG_SYS_MDDRC_TIME_CFG2_ELPIDA -- should be last?
CONFIG_SYS_MDDRC_SYS_CFG_ELPIDA,
D*mn. I did fix this. I though. And I did run MAKEALL after that.
I thought.
Doing a quick search in gmain, the code in question was added by Martha
M Stan (CC:ed):
http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/68478
No, this was not Martha's fault, it was mine. I reordered the array,
and messed it up. Sorry.
If someone fixes it, that would be great, otherwise I'll look at it tonight.
Done. Thanks for pointing out.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
The biggest difference between time and space is that you can't reuse
time. - Merrick Furst
����������
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
