[U-Boot] [PATCH v9] Marvell MV88F6281GTW_GE Board support

[Prafulla Wadaskar]

> > > + *  Environment variables configurations  */ #ifdef
> > CONFIG_SPI_FLASH
> > > +#define CONFIG_ENV_IS_IN_SPI_FLASH	1
> > > +#define CONFIG_ENV_SIZE			0x10000	/* spi 
> > flash block (64k) */
> > > +#define CONFIG_ENV_SECT_SIZE		0x10000	/* _64K */
> > > +#else
> > > +#define CONFIG_ENV_IS_NOWHERE		1	/* if 
> > env in SDRAM */
> > > +#define CONFIG_ENV_SIZE			0x20000	/* 
> > default 128k */
> > 
> > Just a  question...  Do  you  really  NEED  64  kB  or  even
> > 128  kB environement size? In my experience, 16 kB is almost always 
> > more than sufficient.  Keep  in  mind  that the environment 
> size can 
> > be smaller than the sector size which stores the environment,  and  
> > that  a  big enviroment size adds to the boot delay, as the whole 
> > environment size needs to be CRC32 checked.
> I agree, even 4kb is sufficient for me
> but if I keep it less than a sector size it gives me bad CRC 
> warning at boot up even though I do saveenv Hence I kept it 
> equal to sector size This may be a bug...??
Hi Wolfgang
This was really a bug in my spi driver, I fixed it,
now it's working for any size (even 4k)
Thanks for pointing this issue, I am using CONFIG_ENV_SIZE = 4k now

Regards..
Prafulla . .