GitHub-Mirror/u-boot
2
0
Fork 0
mirror of https://github.com/Fishwaldo/u-boot.git synced 2025-03-28 10:01:32 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

env: clean env_mmc.c checkpatch and code style

Browse source 
Cleanup the env_mmc.c checkpatch warnings, errors and coding style.
Simplify env_relocate_spec() function implementation.
Also mark internal functions as static.

Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
This commit is contained in:
Igor Grinberg 2011-11-07 01:14:05 +00:00 • committed by Stefano Babic
parent 91494ca6a0
commit e8db8f71c9
1 changed files with 23 additions and 49 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

72
common/env_mmc.c
View file

@ -37,14 +37,9 @@ char *env_name_spec = "MMC";
#ifdef ENV_IS_EMBEDDED #ifdef ENV_IS_EMBEDDED
env_t *env_ptr = &environment; env_t *env_ptr = &environment;
#else /* ! ENV_IS_EMBEDDED */ #else /* ! ENV_IS_EMBEDDED */
env_t *env_ptr = NULL; env_t *env_ptr;
#endif /* ENV_IS_EMBEDDED */ #endif /* ENV_IS_EMBEDDED */
/* local functions */
#if !defined(ENV_IS_EMBEDDED)
static void use_default(void);
#endif
DECLARE_GLOBAL_DATA_PTR; DECLARE_GLOBAL_DATA_PTR;
#if !defined(CONFIG_ENV_OFFSET) #if !defined(CONFIG_ENV_OFFSET)
@ -56,9 +51,8 @@ static int __mmc_get_env_addr(struct mmc *mmc, u32 *env_addr)
*env_addr = CONFIG_ENV_OFFSET; *env_addr = CONFIG_ENV_OFFSET;
return 0; return 0;
} }
__attribute__((weak, alias("__mmc_get_env_addr"))) int mmc_get_env_addr(struct mmc *mmc, u32 *env_addr)
int mmc_get_env_addr(struct mmc *mmc, u32 *env_addr); __attribute__((weak, alias("__mmc_get_env_addr")));
uchar env_get_char_spec(int index) uchar env_get_char_spec(int index)
{ {
@ -68,13 +62,13 @@ uchar env_get_char_spec(int index)
int env_init(void) int env_init(void)
{ {
/* use default */ /* use default */
gd->env_addr = (ulong)&default_environment[0]; gd->env_addr = (ulong)&default_environment[0];
gd->env_valid = 1; gd->env_valid = 1;
return 0; return 0;
} }
int init_mmc_for_env(struct mmc *mmc) static int init_mmc_for_env(struct mmc *mmc)
{ {
if (!mmc) { if (!mmc) {
puts("No MMC card found\n"); puts("No MMC card found\n");
@ -83,21 +77,20 @@ int init_mmc_for_env(struct mmc *mmc)
if (mmc_init(mmc)) { if (mmc_init(mmc)) {
puts("MMC init failed\n"); puts("MMC init failed\n");
return -1; return -1;
} }
return 0; return 0;
} }
#ifdef CONFIG_CMD_SAVEENV #ifdef CONFIG_CMD_SAVEENV
static inline int write_env(struct mmc *mmc, unsigned long size,
inline int write_env(struct mmc *mmc, unsigned long size, unsigned long offset, const void *buffer)
unsigned long offset, const void *buffer)
{ {
uint blk_start, blk_cnt, n; uint blk_start, blk_cnt, n;
blk_start = ALIGN(offset, mmc->write_bl_len) / mmc->write_bl_len; blk_start = ALIGN(offset, mmc->write_bl_len) / mmc->write_bl_len;
blk_cnt = ALIGN(size, mmc->write_bl_len) / mmc->write_bl_len; blk_cnt = ALIGN(size, mmc->write_bl_len) / mmc->write_bl_len;
n = mmc->block_dev.block_write(CONFIG_SYS_MMC_ENV_DEV, blk_start, n = mmc->block_dev.block_write(CONFIG_SYS_MMC_ENV_DEV, blk_start,
blk_cnt, (u_char *)buffer); blk_cnt, (u_char *)buffer);
@ -111,12 +104,9 @@ int saveenv(void)
ssize_t len; ssize_t len;
char *res; char *res;
struct mmc *mmc = find_mmc_device(CONFIG_SYS_MMC_ENV_DEV); struct mmc *mmc = find_mmc_device(CONFIG_SYS_MMC_ENV_DEV);
u32 offset; u32 offset;
if (init_mmc_for_env(mmc)) if (init_mmc_for_env(mmc) || mmc_get_env_addr(mmc, &offset))
return 1;
if(mmc_get_env_addr(mmc, &offset))
return 1; return 1;
res = (char *)&env_new.data; res = (char *)&env_new.data;
@ -125,7 +115,8 @@ int saveenv(void)
error("Cannot export environment: errno = %d\n", errno); error("Cannot export environment: errno = %d\n", errno);
return 1; return 1;
} }
env_new.crc = crc32(0, env_new.data, ENV_SIZE);
env_new.crc = crc32(0, env_new.data, ENV_SIZE);
printf("Writing to MMC(%d)... ", CONFIG_SYS_MMC_ENV_DEV); printf("Writing to MMC(%d)... ", CONFIG_SYS_MMC_ENV_DEV);
if (write_env(mmc, CONFIG_ENV_SIZE, offset, (u_char *)&env_new)) { if (write_env(mmc, CONFIG_ENV_SIZE, offset, (u_char *)&env_new)) {
puts("failed\n"); puts("failed\n");
@ -137,13 +128,13 @@ int saveenv(void)
} }
#endif /* CONFIG_CMD_SAVEENV */ #endif /* CONFIG_CMD_SAVEENV */
inline int read_env(struct mmc *mmc, unsigned long size, static inline int read_env(struct mmc *mmc, unsigned long size,
unsigned long offset, const void *buffer) unsigned long offset, const void *buffer)
{ {
uint blk_start, blk_cnt, n; uint blk_start, blk_cnt, n;
blk_start = ALIGN(offset, mmc->read_bl_len) / mmc->read_bl_len; blk_start = ALIGN(offset, mmc->read_bl_len) / mmc->read_bl_len;
blk_cnt = ALIGN(size, mmc->read_bl_len) / mmc->read_bl_len; blk_cnt = ALIGN(size, mmc->read_bl_len) / mmc->read_bl_len;
n = mmc->block_dev.block_read(CONFIG_SYS_MMC_ENV_DEV, blk_start, n = mmc->block_dev.block_read(CONFIG_SYS_MMC_ENV_DEV, blk_start,
blk_cnt, (uchar *)buffer); blk_cnt, (uchar *)buffer);
@ -155,32 +146,15 @@ void env_relocate_spec(void)
{ {
#if !defined(ENV_IS_EMBEDDED) #if !defined(ENV_IS_EMBEDDED)
char buf[CONFIG_ENV_SIZE]; char buf[CONFIG_ENV_SIZE];
struct mmc *mmc = find_mmc_device(CONFIG_SYS_MMC_ENV_DEV); struct mmc *mmc = find_mmc_device(CONFIG_SYS_MMC_ENV_DEV);
u32 offset; u32 offset;
if (init_mmc_for_env(mmc)) { if (init_mmc_for_env(mmc) || mmc_get_env_addr(mmc, &offset))
use_default(); return set_default_env(NULL);
return;
}
if(mmc_get_env_addr(mmc, &offset)) { if (read_env(mmc, CONFIG_ENV_SIZE, offset, buf))
use_default(); return set_default_env(NULL);
return ;
}
if (read_env(mmc, CONFIG_ENV_SIZE, offset, buf)) {
use_default();
return;
}
env_import(buf, 1); env_import(buf, 1);
#endif #endif
} }
#if !defined(ENV_IS_EMBEDDED)
static void use_default()
{
set_default_env(NULL);
}
#endif