[U-Boot-Users] [PATCH V2] Add the Harris QUAD100HD AMCC 405EP-based board.

[Stefan Roese <sr@denx.de>]

On Saturday 26 April 2008, Gary Jennejohn wrote:
> > > +#define CFG_MONITOR_LEN		(256 * 1024)	/* Reserve 256 kB for Monitor	*/
> > > +#define CFG_MALLOC_LEN		(128 * 1024)	/* Reserve 128 kB for malloc()	*/
> > > +#define CFG_MONITOR_BASE	0xFFF80000
> >
> > CFG_MONITOR_BASE at 0xfff80000 makes me think that the U-Boot image has a
> > size of 512k. But CFG_MONITOR_LEN is 256k. Which one is correct? I would
> > think that 256k should be enough for this image.
>
> That's a valid point.  Actually u-boot.bin is 512kB in length, but only
> because there's a reset vector at 0xFFFFFFFC.  u-boot itself is really
> only 256kB in size.
>
> So now I wonder - what's correct?  If CFG_MONITOR_LEN applies to RAM,
> which it seems to since it only really affects TOTAL_MALLOC_LEN, then
> 256kB is correct and no change is required.

I would change the image size to 256k to safe some space in FLASH. For this 
you have to change TEXT_BASE (in config.mk in your board directory) to 
0xfffc0000. And CFG_MONITOR_BASE too. Or even better:

#define CFG_MONITOR_BASE        (TEXT_BASE)

> > Starting here
> >
> > > +#define CFG_FLASH_WORD_SIZE	unsigned short
> > > +#define CFG_FLASH_ADDR0		0x0555
> > > +#define CFG_FLASH_ADDR1		0x02aa
> > > +#define FLASH_BASE0_PRELIM	CFG_FLASH_BASE	/* FLASH bank #0	*/
> >
> > till here. These defines are most likely not needed for the CFI driver.
> > Please remove them.
>
> OK, I'll look into it.

Thanks.

Best regards,
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
=====================================================================