[U-Boot] [PATCH] TI DaVinci: Fix DM6467 EVM Compilation Warning

Paulraj, Sandeep s-paulraj at ti.com
Wed Oct 14 04:23:24 CEST 2009



> -----Original Message-----
> From: Tom Rix [mailto:tom at bumblecow.com]
> Sent: Tuesday, October 13, 2009 9:49 PM
> To: Paulraj, Sandeep
> Cc: u-boot at lists.denx.de
> Subject: Re: [U-Boot] [PATCH] TI DaVinci: Fix DM6467 EVM Compilation
> Warning
> 
> s-paulraj at ti.com wrote:
> > From: Sandeep Paulraj <s-paulraj at ti.com>
> >
> > Due to new TI boards being added to U-Boot, the hardware.h
> > is getting very messy. The warning being fixed is due to
> > the EMIF addresses being redefined.
> >
> > The long term solution(after 2009.11) to this is to
> > have SOC specific header files.
> >
> > Signed-off-by: Sandeep Paulraj <s-paulraj at ti.com>
> > ---
> >  include/asm-arm/arch-davinci/hardware.h |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> >
> > diff --git a/include/asm-arm/arch-davinci/hardware.h b/include/asm-
> arm/arch-davinci/hardware.h
> > index ac32510..acf12ea 100644
> > --- a/include/asm-arm/arch-davinci/hardware.h
> > +++ b/include/asm-arm/arch-davinci/hardware.h
> > @@ -71,10 +71,12 @@ typedef volatile unsigned int *	dv_reg_p;
> >  #define DAVINCI_SPI_BASE			(0x01c66800)
> >  #define DAVINCI_GPIO_BASE			(0x01c67000)
> >  #define DAVINCI_VPSS_REGS_BASE			(0x01c70000)
> > +#if !defined(CONFIG_SOC_DM646X)
> >  #define DAVINCI_ASYNC_EMIF_DATA_CE0_BASE	(0x02000000)
> >  #define DAVINCI_ASYNC_EMIF_DATA_CE1_BASE	(0x04000000)
> >  #define DAVINCI_ASYNC_EMIF_DATA_CE2_BASE	(0x06000000)
> >  #define DAVINCI_ASYNC_EMIF_DATA_CE3_BASE	(0x08000000)
> > +#endif
> >  #define DAVINCI_DDR_BASE			(0x80000000)
> >
> >  #ifdef CONFIG_SOC_DM644X
> 
> You have already pushed this so mostly this is for the future..
> This can be cleaned up so the default #defines are set
> only if they are not already defined by the board.
> Or handled with the board files are you are planning.
I will clean it up soon. But this will be considered a feature and thus cannot make it to 2009.11; hence this quick fix solution.

In fact I might work on the clean up this weekend and even send for review but even if accepted, I will have to keep it in my next


> Tom

Sandeep


More information about the U-Boot mailing list