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

[Wolfgang Denk]

Dear Prafulla Wadaskar,

In message <73173D32E9439E4ABB5151606C3E19E201CF9E6422 at SC-VEXCH1.marvell.com> you wrote:
> 
> > Just a  question...  Do  you  really  NEED  64  kB  or  even =20
> > 128  kB environement size? In my experience, 16 kB is almost=20
> > always more than sufficient.  Keep  in  mind  that the=20
> > environment size can be smaller than the sector size which=20
> > stores the environment,  and  that  a  big enviroment size=20
> > adds to the boot delay, as the whole environment size needs=20
> > 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 boo=
> t up even though I do saveenv
> Hence I kept it equal to sector size
> This may be a bug...??

This is indeed a bug, then. To give you an example - in
"include/configs/TQM5200.h" we have:

	#define CONFIG_ENV_SIZE         0x4000  /* 16 k - keep small for fast booting */
	...
	#define CONFIG_ENV_SECT_SIZE    0x40000

This works fine - CRC computation covers only the 16 k as set in
CONFIG_ENV_SIZE.

It seems you really have a bug there, which should be fixed.


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
COMPONENT EQUIVALENCY NOTICE:  The  Subatomic  Particles  (Electrons,
Protons,  etc.) Comprising This Product Are Exactly the Same in Every
Measurable Respect as Those Used in the Products of Other  Manufactu-
rers,  and  No Claim to the Contrary May Legitimately Be Expressed or
Implied.