diff options
| author | Marek Vasut <marex@denx.de> | 2014-03-05 19:59:50 +0100 |
|---|---|---|
| committer | Tom Rini <trini@ti.com> | 2014-03-21 16:43:59 -0400 |
| commit | 7ce1526ed2bf7a7499a843d38b30095fa2894659 (patch) | |
| tree | 4cfcf6050115137a4ff027e21ab2fcbf61c58359 /common/env_mmc.c | |
| parent | b401b73d02bb4f97197830e565f19a65577fecc6 (diff) | |
| download | blackbird-obmc-uboot-7ce1526ed2bf7a7499a843d38b30095fa2894659.tar.gz blackbird-obmc-uboot-7ce1526ed2bf7a7499a843d38b30095fa2894659.zip | |
env: Add env_export() wrapper
Implement env_export() wrapper, so that all implementers of saveenv() don't
have to call hexport_r(), crc32() etc. sequence . This trims down a bit of
code duplication.
Signed-off-by: Marek Vasut <marex@denx.de>
Diffstat (limited to 'common/env_mmc.c')
| -rw-r--r-- | common/env_mmc.c | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/common/env_mmc.c b/common/env_mmc.c index 045428c6ec..c99fc750fa 100644 --- a/common/env_mmc.c +++ b/common/env_mmc.c @@ -118,8 +118,6 @@ static unsigned char env_flags; int saveenv(void) { ALLOC_CACHE_ALIGN_BUFFER(env_t, env_new, 1); - ssize_t len; - char *res; struct mmc *mmc = find_mmc_device(CONFIG_SYS_MMC_ENV_DEV); u32 offset; int ret, copy = 0; @@ -127,15 +125,9 @@ int saveenv(void) if (init_mmc_for_env(mmc)) return 1; - res = (char *)&env_new->data; - len = hexport_r(&env_htab, '\0', 0, &res, ENV_SIZE, 0, NULL); - if (len < 0) { - error("Cannot export environment: errno = %d\n", errno); - ret = 1; + ret = env_export(env_new); + if (ret) goto fini; - } - - env_new->crc = crc32(0, &env_new->data[0], ENV_SIZE); #ifdef CONFIG_ENV_OFFSET_REDUND env_new->flags = ++env_flags; /* increase the serial */ |
