[U-Boot] [PATCH 3/3] Replace CONFIG_SYS_GBL_DATA_SIZE by auto-generated value

Stefan Roese sr at denx.de
Tue Oct 26 17:22:58 CEST 2010


Hi Wolfgang,

On Tuesday 26 October 2010 16:41:07 Wolfgang Denk wrote:
> You are right, especially as the resulting start address of the global
> data is usually also used as the top of the internal stack. Guess we
> should align it on 16 bytes, to be sure.

Yes, we already had this 8-byte stack align problem on ARM, IIRC.
 
> BTW - this uncovers a number of other problems.
> 
> commit 800eb096 "POST cleanup." inserts a single data word between
> global data and initial stack, thus also breaking stack alignment.
> 
> I think this approach is wrong, the POST word should be made part of
> the global_data instead.

Good idea.
 
Cheers,
Stefan

--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de


More information about the U-Boot mailing list