From mboxrd@z Thu Jan  1 00:00:00 1970
From: Joakim Tjernlund <Joakim.Tjernlund@transmode.se>
Date: Fri, 5 Sep 2008 21:32:14 +0200
Subject: [U-Boot] [REGRESSION] commit b502611b51... "Change env_get_char
	from a..." breaks imx31_phycore
In-Reply-To: <59b21cf20809051215t41a887c7n6f5b7b564bc1056e@mail.gmail.com>
References: <Pine.LNX.4.64.0809051954400.5482@axis700.grange>	
	<6884773996468454042@unknownmsgid>
	<59b21cf20809051215t41a887c7n6f5b7b564bc1056e@mail.gmail.com>
Message-ID: <00ae01c90f8e$1a01fb80$4e05f280$@Tjernlund@transmode.se>
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

> -----Original Message-----
> From: Magnus Lilja [mailto:lilja.magnus at gmail.com]
> Sent: den 5 september 2008 21:16
> To: Joakim Tjernlund
> Cc: Guennadi Liakhovetski; U-Boot at 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 <Joakim.Tjernlund@transmode.se>:
> >
> >> -----Original Message-----
> >> From: Guennadi Liakhovetski [mailto:lg at denx.de]
> >> Sent: den 5 september 2008 20:01
> >> To: U-Boot at 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 <joakim.tjernlund@transmode.se>
> >> 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 <Joakim.Tjernlund@transmode.se>
> >>
> >> 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.