[U-Boot] [PATCH] env: don't set to default env twice when use CONFIG_ENV_IS_NOWHERE
Wolfgang Denk
wd at denx.de
Sat Oct 9 20:42:29 CEST 2010
Dear Lei Wen,
In message <1286618288-14552-1-git-send-email-leiwen at marvell.com> you wrote:
> When use the CONFIG_ENV_IS_NOWHERE, I met such issue:
> Using default environment
>
> *** Warning - bad CRC, using default environment
>
> Signed-off-by: Lei Wen <leiwen at marvell.com>
> ---
> common/env_common.c | 1 -
> 1 files changed, 0 insertions(+), 1 deletions(-)
>
> diff --git a/common/env_common.c b/common/env_common.c
> index 88f068c..10131c7 100644
> --- a/common/env_common.c
> +++ b/common/env_common.c
> @@ -238,7 +238,6 @@ void env_relocate (void)
> #else
> show_boot_progress (-60);
> #endif
> - set_default_env("!bad CRC");
> } else {
> env_relocate_spec ();
> }
NAK.
This fix is wrong, as it removes the call to set_default_env() also in
the cases where it's needed. Instead, please move down the "#endif"
after this line.
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
This is now. Later is later.
More information about the U-Boot
mailing list