Skip to content
Snippets Groups Projects
Commit 82b54b97 authored by Igor Grinberg's avatar Igor Grinberg Committed by Stefano Babic
Browse files

env: clean env_flash.c checkpatch and code style


Cleanup the env_flash.c checkpatch warnings, errors and coding style.

Signed-off-by: default avatarIgor Grinberg <grinberg@compulab.co.il>
parent eb58a7fc
Branches reform2
No related tags found
No related merge requests found
...@@ -39,10 +39,11 @@ DECLARE_GLOBAL_DATA_PTR; ...@@ -39,10 +39,11 @@ DECLARE_GLOBAL_DATA_PTR;
#if defined(CONFIG_CMD_SAVEENV) && defined(CONFIG_CMD_FLASH) #if defined(CONFIG_CMD_SAVEENV) && defined(CONFIG_CMD_FLASH)
#define CMD_SAVEENV #define CMD_SAVEENV
#elif defined(CONFIG_ENV_ADDR_REDUND) #elif defined(CONFIG_ENV_ADDR_REDUND)
#error Cannot use CONFIG_ENV_ADDR_REDUND without CONFIG_CMD_SAVEENV & CONFIG_CMD_FLASH #error CONFIG_ENV_ADDR_REDUND must have CONFIG_CMD_SAVEENV & CONFIG_CMD_FLASH
#endif #endif
#if defined(CONFIG_ENV_SIZE_REDUND) && (CONFIG_ENV_SIZE_REDUND < CONFIG_ENV_SIZE) #if defined(CONFIG_ENV_SIZE_REDUND) && \
(CONFIG_ENV_SIZE_REDUND < CONFIG_ENV_SIZE)
#error CONFIG_ENV_SIZE_REDUND should not be less then CONFIG_ENV_SIZE #error CONFIG_ENV_SIZE_REDUND should not be less then CONFIG_ENV_SIZE
#endif #endif
...@@ -57,7 +58,6 @@ static env_t *flash_addr = (env_t *)CONFIG_ENV_ADDR; ...@@ -57,7 +58,6 @@ static env_t *flash_addr = (env_t *)CONFIG_ENV_ADDR;
env_t *env_ptr = (env_t *)CONFIG_ENV_ADDR; env_t *env_ptr = (env_t *)CONFIG_ENV_ADDR;
static env_t *flash_addr = (env_t *)CONFIG_ENV_ADDR; static env_t *flash_addr = (env_t *)CONFIG_ENV_ADDR;
#endif /* ENV_IS_EMBEDDED */ #endif /* ENV_IS_EMBEDDED */
#if defined(CMD_SAVEENV) || defined(CONFIG_ENV_ADDR_REDUND) #if defined(CMD_SAVEENV) || defined(CONFIG_ENV_ADDR_REDUND)
...@@ -75,11 +75,10 @@ static ulong end_addr_new = CONFIG_ENV_ADDR_REDUND + CONFIG_ENV_SECT_SIZE - 1; ...@@ -75,11 +75,10 @@ static ulong end_addr_new = CONFIG_ENV_ADDR_REDUND + CONFIG_ENV_SECT_SIZE - 1;
uchar env_get_char_spec(int index) uchar env_get_char_spec(int index)
{ {
return (*((uchar *)(gd->env_addr + index))); return *((uchar *)(gd->env_addr + index));
} }
#ifdef CONFIG_ENV_ADDR_REDUND #ifdef CONFIG_ENV_ADDR_REDUND
int env_init(void) int env_init(void)
{ {
int crc1_ok = 0, crc2_ok = 0; int crc1_ok = 0, crc2_ok = 0;
...@@ -91,8 +90,9 @@ int env_init(void) ...@@ -91,8 +90,9 @@ int env_init(void)
ulong addr1 = (ulong)&(flash_addr->data); ulong addr1 = (ulong)&(flash_addr->data);
ulong addr2 = (ulong)&(flash_addr_new->data); ulong addr2 = (ulong)&(flash_addr_new->data);
crc1_ok = (crc32(0, flash_addr->data, ENV_SIZE) == flash_addr->crc); crc1_ok = crc32(0, flash_addr->data, ENV_SIZE) == flash_addr->crc;
crc2_ok = (crc32(0, flash_addr_new->data, ENV_SIZE) == flash_addr_new->crc); crc2_ok =
crc32(0, flash_addr_new->data, ENV_SIZE) == flash_addr_new->crc;
if (crc1_ok && !crc2_ok) { if (crc1_ok && !crc2_ok) {
gd->env_addr = addr1; gd->env_addr = addr1;
...@@ -128,27 +128,23 @@ int saveenv(void) ...@@ -128,27 +128,23 @@ int saveenv(void)
{ {
env_t env_new; env_t env_new;
ssize_t len; ssize_t len;
char *saved_data = NULL; char *res, *saved_data = NULL;
char *res;
int rc = 1;
char flag = OBSOLETE_FLAG, new_flag = ACTIVE_FLAG; char flag = OBSOLETE_FLAG, new_flag = ACTIVE_FLAG;
int rc = 1;
#if CONFIG_ENV_SECT_SIZE > CONFIG_ENV_SIZE #if CONFIG_ENV_SECT_SIZE > CONFIG_ENV_SIZE
ulong up_data = 0; ulong up_data = 0;
#endif #endif
debug("Protect off %08lX ... %08lX\n", debug("Protect off %08lX ... %08lX\n", (ulong)flash_addr, end_addr);
(ulong)flash_addr, end_addr);
if (flash_sect_protect(0, (ulong)flash_addr, end_addr)) { if (flash_sect_protect(0, (ulong)flash_addr, end_addr))
goto done; goto done;
}
debug("Protect off %08lX ... %08lX\n", debug("Protect off %08lX ... %08lX\n",
(ulong)flash_addr_new, end_addr_new); (ulong)flash_addr_new, end_addr_new);
if (flash_sect_protect(0, (ulong)flash_addr_new, end_addr_new)) { if (flash_sect_protect(0, (ulong)flash_addr_new, end_addr_new))
goto done; goto done;
}
res = (char *)&env_new.data; res = (char *)&env_new.data;
len = hexport_r(&env_htab, '\0', &res, ENV_SIZE, 0, NULL); len = hexport_r(&env_htab, '\0', &res, ENV_SIZE, 0, NULL);
...@@ -160,42 +156,42 @@ int saveenv(void) ...@@ -160,42 +156,42 @@ int saveenv(void)
env_new.flags = new_flag; env_new.flags = new_flag;
#if CONFIG_ENV_SECT_SIZE > CONFIG_ENV_SIZE #if CONFIG_ENV_SECT_SIZE > CONFIG_ENV_SIZE
up_data = (end_addr_new + 1 - ((long)flash_addr_new + CONFIG_ENV_SIZE)); up_data = end_addr_new + 1 - ((long)flash_addr_new + CONFIG_ENV_SIZE);
debug("Data to save 0x%lX\n", up_data); debug("Data to save 0x%lX\n", up_data);
if (up_data) { if (up_data) {
if ((saved_data = malloc(up_data)) == NULL) { saved_data = malloc(up_data);
if (saved_data == NULL) {
printf("Unable to save the rest of sector (%ld)\n", printf("Unable to save the rest of sector (%ld)\n",
up_data); up_data);
goto done; goto done;
} }
memcpy(saved_data, memcpy(saved_data,
(void *)((long)flash_addr_new + CONFIG_ENV_SIZE), up_data); (void *)((long)flash_addr_new + CONFIG_ENV_SIZE),
up_data);
debug("Data (start 0x%lX, len 0x%lX) saved at 0x%p\n", debug("Data (start 0x%lX, len 0x%lX) saved at 0x%p\n",
(long)flash_addr_new + CONFIG_ENV_SIZE, (long)flash_addr_new + CONFIG_ENV_SIZE,
up_data, saved_data); up_data, saved_data);
} }
#endif #endif
puts("Erasing Flash..."); puts("Erasing Flash...");
debug(" %08lX ... %08lX ...", debug(" %08lX ... %08lX ...", (ulong)flash_addr_new, end_addr_new);
(ulong)flash_addr_new, end_addr_new);
if (flash_sect_erase((ulong)flash_addr_new, end_addr_new)) { if (flash_sect_erase((ulong)flash_addr_new, end_addr_new))
goto done; goto done;
}
puts("Writing to Flash... "); puts("Writing to Flash... ");
debug(" %08lX ... %08lX ...", debug(" %08lX ... %08lX ...",
(ulong)&(flash_addr_new->data), (ulong)&(flash_addr_new->data),
sizeof(env_ptr->data) + (ulong)&(flash_addr_new->data)); sizeof(env_ptr->data) + (ulong)&(flash_addr_new->data));
if ((rc = flash_write((char *)&env_new, rc = flash_write((char *)&env_new, (ulong)flash_addr_new,
(ulong)flash_addr_new, sizeof(env_new));
sizeof(env_new))) || if (rc)
(rc = flash_write(&flag, goto perror;
(ulong)&(flash_addr->flags),
sizeof(flash_addr->flags))) ) { rc = flash_write(&flag, (ulong)&(flash_addr->flags),
flash_perror(rc); sizeof(flash_addr->flags));
goto done; if (rc)
} goto perror;
#if CONFIG_ENV_SECT_SIZE > CONFIG_ENV_SIZE #if CONFIG_ENV_SECT_SIZE > CONFIG_ENV_SIZE
if (up_data) { /* restore the rest of sector */ if (up_data) { /* restore the rest of sector */
...@@ -203,10 +199,8 @@ int saveenv(void) ...@@ -203,10 +199,8 @@ int saveenv(void)
(long)flash_addr_new + CONFIG_ENV_SIZE, up_data); (long)flash_addr_new + CONFIG_ENV_SIZE, up_data);
if (flash_write(saved_data, if (flash_write(saved_data,
(long)flash_addr_new + CONFIG_ENV_SIZE, (long)flash_addr_new + CONFIG_ENV_SIZE,
up_data)) { up_data))
flash_perror(rc); goto perror;
goto done;
}
} }
#endif #endif
puts("done\n"); puts("done\n");
...@@ -223,12 +217,15 @@ int saveenv(void) ...@@ -223,12 +217,15 @@ int saveenv(void)
} }
rc = 0; rc = 0;
goto done;
perror:
flash_perror(rc);
done: done:
if (saved_data) if (saved_data)
free(saved_data); free(saved_data);
/* try to re-protect */ /* try to re-protect */
(void) flash_sect_protect(1, (ulong)flash_addr, end_addr); flash_sect_protect(1, (ulong)flash_addr, end_addr);
(void) flash_sect_protect(1, (ulong)flash_addr_new, end_addr_new); flash_sect_protect(1, (ulong)flash_addr_new, end_addr_new);
return rc; return rc;
} }
...@@ -241,7 +238,7 @@ int env_init(void) ...@@ -241,7 +238,7 @@ int env_init(void)
if (crc32(0, env_ptr->data, ENV_SIZE) == env_ptr->crc) { if (crc32(0, env_ptr->data, ENV_SIZE) == env_ptr->crc) {
gd->env_addr = (ulong)&(env_ptr->data); gd->env_addr = (ulong)&(env_ptr->data);
gd->env_valid = 1; gd->env_valid = 1;
return(0); return 0;
} }
gd->env_addr = (ulong)&default_environment[0]; gd->env_addr = (ulong)&default_environment[0];
...@@ -250,21 +247,20 @@ int env_init(void) ...@@ -250,21 +247,20 @@ int env_init(void)
} }
#ifdef CMD_SAVEENV #ifdef CMD_SAVEENV
int saveenv(void) int saveenv(void)
{ {
env_t env_new; env_t env_new;
ssize_t len; ssize_t len;
int rc = 1; int rc = 1;
char *res; char *res, *saved_data = NULL;
char *saved_data = NULL;
#if CONFIG_ENV_SECT_SIZE > CONFIG_ENV_SIZE #if CONFIG_ENV_SECT_SIZE > CONFIG_ENV_SIZE
ulong up_data = 0; ulong up_data = 0;
up_data = (end_addr + 1 - ((long)flash_addr + CONFIG_ENV_SIZE)); up_data = end_addr + 1 - ((long)flash_addr + CONFIG_ENV_SIZE);
debug("Data to save 0x%lx\n", up_data); debug("Data to save 0x%lx\n", up_data);
if (up_data) { if (up_data) {
if ((saved_data = malloc(up_data)) == NULL) { saved_data = malloc(up_data);
if (saved_data == NULL) {
printf("Unable to save the rest of sector (%ld)\n", printf("Unable to save the rest of sector (%ld)\n",
up_data); up_data);
goto done; goto done;
...@@ -278,8 +274,7 @@ int saveenv(void) ...@@ -278,8 +274,7 @@ int saveenv(void)
} }
#endif /* CONFIG_ENV_SECT_SIZE */ #endif /* CONFIG_ENV_SECT_SIZE */
debug("Protect off %08lX ... %08lX\n", debug("Protect off %08lX ... %08lX\n", (ulong)flash_addr, end_addr);
(ulong)flash_addr, end_addr);
if (flash_sect_protect(0, (long)flash_addr, end_addr)) if (flash_sect_protect(0, (long)flash_addr, end_addr))
goto done; goto done;
...@@ -298,32 +293,31 @@ int saveenv(void) ...@@ -298,32 +293,31 @@ int saveenv(void)
puts("Writing to Flash... "); puts("Writing to Flash... ");
rc = flash_write((char *)&env_new, (long)flash_addr, CONFIG_ENV_SIZE); rc = flash_write((char *)&env_new, (long)flash_addr, CONFIG_ENV_SIZE);
if (rc != 0) { if (rc != 0)
flash_perror(rc); goto perror;
goto done;
}
#if CONFIG_ENV_SECT_SIZE > CONFIG_ENV_SIZE #if CONFIG_ENV_SECT_SIZE > CONFIG_ENV_SIZE
if (up_data) { /* restore the rest of sector */ if (up_data) { /* restore the rest of sector */
debug("Restoring the rest of data to 0x%lx len 0x%lx\n", debug("Restoring the rest of data to 0x%lx len 0x%lx\n",
(ulong)flash_addr + CONFIG_ENV_SIZE, up_data); (ulong)flash_addr + CONFIG_ENV_SIZE, up_data);
if (flash_write(saved_data, if (flash_write(saved_data,
(long)flash_addr + CONFIG_ENV_SIZE, (long)flash_addr + CONFIG_ENV_SIZE,
up_data)) { up_data))
flash_perror(rc); goto perror;
goto done;
}
} }
#endif #endif
puts("done\n"); puts("done\n");
rc = 0; rc = 0;
goto done;
perror:
flash_perror(rc);
done: done:
if (saved_data) if (saved_data)
free(saved_data); free(saved_data);
/* try to re-protect */ /* try to re-protect */
(void) flash_sect_protect(1, (long)flash_addr, end_addr); flash_sect_protect(1, (long)flash_addr, end_addr);
return rc; return rc;
} }
#endif /* CMD_SAVEENV */ #endif /* CMD_SAVEENV */
#endif /* CONFIG_ENV_ADDR_REDUND */ #endif /* CONFIG_ENV_ADDR_REDUND */
...@@ -343,8 +337,7 @@ void env_relocate_spec(void) ...@@ -343,8 +337,7 @@ void env_relocate_spec(void)
} }
if (flash_addr_new->flags != OBSOLETE_FLAG && if (flash_addr_new->flags != OBSOLETE_FLAG &&
crc32(0, flash_addr_new->data, ENV_SIZE) == crc32(0, flash_addr_new->data, ENV_SIZE) == flash_addr_new->crc) {
flash_addr_new->crc) {
char flag = OBSOLETE_FLAG; char flag = OBSOLETE_FLAG;
gd->env_valid = 2; gd->env_valid = 2;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment