[U-Boot] [PATCH] Revert "arm: Switch 32-bit ARM to using generic global_data setup"

[Albert ARIBAUD <albert.u.boot@aribaud.net>]

Hello Simon,

On Tue, 10 Nov 2015 06:41:25 -0800, Simon Glass <sjg@chromium.org>
wrote:
> Hi Fabio,
> 
> On 10 November 2015 at 04:40, Fabio Estevam <fabio.estevam@freescale.com> wrote:
> > This reverts commit 5ba534d247d418e09c5b4fe5fb7fa780aac08e49.
> >
> > This commit causes cgtqmx6eval to not boot anymore:
> >
> > U-Boot SPL 2015.10-00527-g8800bee (Nov 09 2015 - 21:23:54)
> > mxc_spi: SPI Slave not allocated !
> >
> > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> > ---
> >  arch/arm/lib/crt0.S | 28 ++++++++++++++++++++++++----
> >  1 file changed, 24 insertions(+), 4 deletions(-)
> 
> We're at the very start the release process, so I wonder if we can try
> to figure out what is wrong here?
> 
> Is it because malloc() is not working, perhaps?
> 
> The C code should be roughly equivalent to the assembly code.

"Roughly". :)

However:

> Albert
> found a problem with the code on toolchain 5.2.1 to do with setting
> 'gd', so may have some thoughts on this. But this might be a different
> problem.

I've looked into cgtqmx6eval, and if I'm not mistaken it builds ARM,
not Thumb, code, whereas the bug I found is on Thumb code (thumb-1 at
least).

So yes, this seems like a different problem than the gcc-5.2.1-induced
one.

> Regards,
> Simon

Amicalement,
-- 
Albert.