[U-Boot] denx/master is not building for OMAP3 platforms except Beagle

[Hiremath, Vaibhav]

> -----Original Message-----
> From: u-boot-bounces at lists.denx.de [mailto:u-boot-bounces at lists.denx.de]
> On Behalf Of Albert ARIBAUD
> Sent: Sunday, October 03, 2010 4:37 PM
> To: u-boot at lists.denx.de
> Subject: Re: [U-Boot] denx/master is not building for OMAP3 platforms
> except Beagle
> 
> Le 03/10/2010 12:50, Hiremath, Vaibhav a écrit :
> > Hi All,
> >
> > I think the current denx/matser is broken and is not building up for
> OMAP3 platforms except Beagle board.
> >
> > The error message on build is -
> >
> > board.c: In function '__dram_init_banksize':
> > board.c:468: error: 'CONFIG_SYS_SDRAM_BASE' undeclared (first use in
> this function)
> > board.c:468: error: (Each undeclared identifier is reported only once
> > board.c:468: error: for each function it appears in.)
> > board.c: In function 'board_init_f':
> > board.c:514: error: 'CONFIG_SYS_INIT_SP_ADDR' undeclared (first use in
> this function)
> > board.c:547: error: 'CONFIG_SYS_SDRAM_BASE' undeclared (first use in
> this function)
> > make[1]: *** [board.o] Error 1
> > make[1]: Leaving directory `/home/devel/git-work/denx-
> uboot/arch/arm/lib'
> > make: *** [arch/arm/lib/libarm.a] Error 2
> >
> >
> > The root-cause is-
> >
> > +#undef CONFIG_SYS_ARM_WITHOUT_RELOC
> > +#define CONFIG_SYS_SDRAM_BASE   PHYS_SDRAM_1
> > +#define CONFIG_SYS_INIT_SP_ADDR (LOW_LEVEL_SRAM_STACK -
> CONFIG_SYS_GBL_DATA_SIZE)
> >
> >
> > The above definition is missing in all (zoom1, zoom2, evm, am3517evm,
> sdp3430, overo, Pandora) config files.
> >
> > Am I missing something here? Just wanted to cross-check before posting
> patch to the list.
> 
> This is due to the introduction of relocation on ARM platforms -- see
> Wolfgang's recent release announcement.
[Hiremath, Vaibhav] Yes that's correct. I am aware of that.

> 
> Note that adding the definition may not be sufficient to get relocation
> working.
>
[Hiremath, Vaibhav] But atleast it will make code to compile cleanly. I feel, at any cost we should not add such patches which will break compilation, the code should build always.

Thanks,
Vaibhav
 
> Amicalement,
> --
> Albert.
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot