[U-Boot] [PATCH] saveenv: standardize enablement

Wolfgang Denk wd at denx.de
Wed Jan 28 09:17:06 CET 2009


Dear Mike,

In message <1230623965-16546-1-git-send-email-vapier at gentoo.org> you wrote:
> Rather than special casing each environment type for enabling the saveenv
> command, have them all behave the same.  This avoids bitrot as new env
> sources are added/removed.
> 
> Signed-off-by: Mike Frysinger <vapier at gentoo.org>
> ---
>  common/cmd_nvedit.c |   26 ++++++++------------------
>  1 files changed, 8 insertions(+), 18 deletions(-)

This patch breaks support for the 

This board is a bit unusual as it is ROM based - it has
CONFIG_ENV_IS_IN_FLASH set because the default environment is embedded in
the ROM, but does not enable the flash driver (because there is no
flash on the board). So the test in "common/cmd_nvedit.c":

549 #if defined(CONFIG_CMD_ENV) && !defined(CONFIG_ENV_IS_NOWHERE)

fails - it enables do_saveenv(), but there is no underlying saveenv()
function. Result:

common/libcommon.a(cmd_nvedit.o): In function `do_saveenv':
common/cmd_nvedit.c:557: undefined reference to `saveenv'
make: *** [u-boot] Error 1


Please fix.

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
Never underestimate the bandwidth of a station wagon full of tapes.
                                -- Dr. Warren Jackson, Director, UTCS


More information about the U-Boot mailing list