> -----Original Message----- > From: Magnus Lilja [mailto:[EMAIL PROTECTED] > Sent: den 5 september 2008 21:16 > To: Joakim Tjernlund > Cc: Guennadi Liakhovetski; U-Boot@lists.denx.de > Subject: Re: [U-Boot] [REGRESSION] commit b502611b51... "Change env_get_char > from a..." breaks > imx31_phycore > > Hi, > > 2008/9/5 Joakim Tjernlund <[EMAIL PROTECTED]>: > > > >> -----Original Message----- > >> From: Guennadi Liakhovetski [mailto:[EMAIL PROTECTED] > >> Sent: den 5 september 2008 20:01 > >> To: U-Boot@lists.denx.de > >> Cc: Joakim Tjernlund > >> Subject: [REGRESSION] commit b502611b51... "Change env_get_char from a..." > >> breaks imx31_phycore > >> > >> Hi, > >> > >> The aforementioned commit > >> > >> commit b502611b51f02718c2d1117d4981dabceb5af6de > >> Author: Joakim Tjernlund <[EMAIL PROTECTED]> > >> Date: Sun Jul 6 12:30:09 2008 +0200 > >> > >> Change env_get_char from a global function ptr to a function > >> > >> This avoids an early global data reference. > >> > >> Signed-off-by: Joakim Tjernlund <[EMAIL PROTECTED]> > >> > >> found by bisection and causes at least the imx31_phycore board to break. > >> The boot process becomes slow, printenv is very slow too, booting does not > >> always come to the bootdelay countdown, tftp wtops working too. Reverting > >> this commit from the current HEAD fixes the problem. > > > > Your board probably don't flip the GD_FLG_RELOC flag after relocation. A few > > ARM boards had a problem with this too. > > Unless I've missed something all ARM boards have this problem. I can't > find any ARM boards flipping the RELOC flag (I 'grep'-ed for > GD_FLG_RELOC). For most other architectures the flip is done in > lib_<arch>/ but for ARM that's only done if the > CONFIG_SKIP_RELOCATE_UBOOT option is set. > > Wouldn't it be possible to set GD_FLG_RELOC near the beginning of > start_armboot()? After the memset() of the gd area.
I can't say, I am not an ARM guy. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
