[U-Boot] [PATCH 1/1] mx51evk: savenv or env save command does not work

[Jason Liu]

2010/11/17 Stefano Babic <sbabic at denx.de>:
> On 11/17/2010 08:29 AM, Jason Liu wrote:
>> Agree. But I think the original commit for 128KiB env size has been
>> reviewed on the mail list and no one against it.
>
> This does not mean that we cannot change this value when we find it is
> wrong....

As Wofgang said, it's not wrong. Just not optimization.

>
>> We really don't know what data that customer will store. So leave much
>> room for them is
>> my first though, But consider the fast boot, what you said make sense.
>
> If someone really needs a so large environment, cand send an e-mail to
> this ML explaining his reason and posting a patch. It will be discussed
> here.
>
>> OK,  do you think we need change all the platform to reflect the 16K
>> env size for MMC case?
>
> Your patch refers to the mx51evk. So change the value according to this
> discussion and post your patch again for inclusion in mainline. We will
> take into account for other boards when new patches will be submitted.

OK, I will set it to 8KiB for mx51evk.

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