From mboxrd@z Thu Jan  1 00:00:00 1970
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
Date: Tue, 10 Nov 2015 16:08:23 +0100
Subject: [U-Boot] [PATCH] Revert "arm: Switch 32-bit ARM to using
 generic global_data setup"
In-Reply-To: <CAPnjgZ1v_UtG_BSyKKTXzq2TMgtJP=hNLrL59s=2+ubo2RQ8hg@mail.gmail.com>
References: <1447159252-10828-1-git-send-email-fabio.estevam@freescale.com>
 <CAPnjgZ1v_UtG_BSyKKTXzq2TMgtJP=hNLrL59s=2+ubo2RQ8hg@mail.gmail.com>
Message-ID: <20151110160823.280e4ebf@lilith>
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

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.