[U-Boot] [RFC PATCH] env: add default env size for CONFIG_ENV_IS_NOWHERE

Bo Shen voice.shen at gmail.com
Mon Aug 12 15:07:16 CEST 2013


Hi Wolfgang Denk,

On 8/11/2013 10:50 PM, Wolfgang Denk wrote:
> Dear Bo Shen,
>
> In message <1376230503-25331-1-git-send-email-voice.shen at gmail.com> you wrote:
>> when CONFIG_ENV_IS_NOWHERE is enabled, it is still need to define
>> CONFIG_ENV_SIZE. So, add a default size (1024 Bytes) to avoid
>> compile error if not define CONFIG_ENV_SIZE
>
> I don;t understand the rationale for this patch.  In which way is the
> environment seize for the CONFIG_ENV_IS_NOWHERE case different from
> any other cases?  For these, we do not define a default either, so why
> should we handle this case differently?

In my opinion, there is a little different. The CONFIG_ENV_IS_NOWHERE 
case only store environment in volatile memory (e.g. DDR SDRAM) while 
not store in non-volatile device.

At first glance of CONFIG_ENV_IS_NOWHERE, I think no need to define the 
CONFIG_ENV_SIZE, actually, it needs to define the CONFIG_ENV_SIZE. So, I 
think add a default value will be better (Maybe another choice to add a 
description in README file to specify how to use CONFIG_ENV_IS_NOWHERE).

> In any cse, the needed environment seize if a pretty board specific
> thing, and I think it makes sense to let the user define it.

Yes, the environment size is a pretty board specific thing. So, in this 
patch, there is an option for user to define it.

> Best regards,
>
> Wolfgang Denk
>

Best Regards,
Bo Shen


More information about the U-Boot mailing list