From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jerry Van Baren <gvb.uboot@gmail.com>
Date: Fri, 25 Sep 2009 06:30:36 -0400
Subject: [U-Boot] Broken board: mpc5121ads
Message-ID: <4ABC9BCC.9060801@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Martha, list,

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,
         };



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>

If someone fixes it, that would be great, otherwise I'll look at it tonight.

Thanks,
gvb