Hi Tom, On Mon, Feb 20, 2012 at 11:15 PM, Tom Rini <tr...@ti.com> wrote:
> On Fri, Feb 17, 2012 at 05:35:27PM +0530, R Sricharan wrote: > > > Adding the nessecary changes for OMAP5430 ES1.0 silicon. > > > > Signed-off-by: R Sricharan <r.sricha...@ti.com> > [snip] > > +/* OPP SETTING */ > > +#define OPP_NOM > > Where is this used? > Right. This is unused. Will remove this. We were having > OPP_LOW and OPP_HIGH settings in the clocks table, for internal testing. But that being eventually removed, this is not required. > [snip] > > /* Defines for SPL */ > > #define CONFIG_SPL > > -#define CONFIG_SPL_TEXT_BASE 0x40304350 > > -#define CONFIG_SPL_MAX_SIZE 0x1E000 /* 120K */ > > +#define CONFIG_SPL_TEXT_BASE 0x40300350 > > +#define CONFIG_SPL_MAX_SIZE 0x20000 /* 128K */ > > #define CONFIG_SPL_STACK LOW_LEVEL_SRAM_STACK > > This is wrong, you have to account for stack within SPL_MAX_SIZE. > > To understand, So now, LOW_LEVEL_SRAM_STACK points to SRAM base + 128KB address (which is the SRAM size), pointed to by NON_SECURE_SRAM_END Should this be changed ? Also CONFIG_SYS_INIT_SP_ADDR should be adjusted. Will change this > -- > Tom >
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot