On 02/08/2013 09:07 PM, Joe Hershberger wrote:
Allow the user to specify two UBI volumes to use for the environment
Signed-off-by: Joe Hershberger joe.hershberger@ni.com
Some minor comments below.
README | 6 +++ common/env_ubi.c | 115 ++++++++++++++++++++++++++++++++++++++++++++++++++ include/environment.h | 3 ++ tools/env/fw_env.c | 3 ++ 4 files changed, 127 insertions(+)
diff --git a/README b/README index c46ca3a..baf828a 100644 --- a/README +++ b/README @@ -3457,6 +3457,12 @@ but it can not erase, write this NOR flash by SRIO or PCIE interface. Define this to the name of the volume that you want to store the environment in.
- CONFIG_ENV_UBI_VOLUME_REDUND:
Define this to the name of another volume to store a second copy ofthe environment in. This will enable redundant environments in UBI.It is assumed that both volumes are in the same MTD partition.
CONFIG_SYS_SPI_INIT_OFFSET
Defines offset to the initial SPI buffer area in DPRAM. The
diff --git a/common/env_ubi.c b/common/env_ubi.c index 9a47fd2..48f0bcb 100644 --- a/common/env_ubi.c +++ b/common/env_ubi.c @@ -47,6 +47,58 @@ int env_init(void) }
#ifdef CONFIG_CMD_SAVEENV +#ifdef CONFIG_SYS_REDUNDAND_ENVIRONMENT +static unsigned char env_flags;
+int saveenv(void) +{
- ALLOC_CACHE_ALIGN_BUFFER(env_t, env_new, 1);
- ssize_t len;
- char *res;
- 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);return 1;- }
- if (ubi_part(CONFIG_ENV_UBI_PART, NULL)) {
printf("\n** Cannot find mtd partition \"%s\"\n",CONFIG_ENV_UBI_PART);return 1;- }
- env_new->crc = crc32(0, env_new->data, ENV_SIZE);
- env_new->flags = ++env_flags; /* increase the serial */
- if (gd->env_valid == 1) {
puts("Writing to redundant UBI... ");if (ubi_volume_write(CONFIG_ENV_UBI_VOLUME_REDUND,(void *)env_new, CONFIG_ENV_SIZE)) {printf("\n** Unable to write env to %s:%s **\n",CONFIG_ENV_UBI_PART,CONFIG_ENV_UBI_VOLUME_REDUND);return 1;}- } else {
puts("Writing to UBI... ");if (ubi_volume_write(CONFIG_ENV_UBI_VOLUME,(void *)env_new, CONFIG_ENV_SIZE)) {printf("\n** Unable to write env to %s:%s **\n",CONFIG_ENV_UBI_PART,CONFIG_ENV_UBI_VOLUME);return 1;}- }
- puts("done\n");
- gd->env_valid = gd->env_valid == 2 ? 1 : 2;
- return 0;
+} +#else /* ! CONFIG_SYS_REDUNDAND_ENVIRONMENT */ int saveenv(void) { ALLOC_CACHE_ALIGN_BUFFER(env_t, env_new, 1); @@ -78,8 +130,70 @@ int saveenv(void) puts("done\n"); return 0; } +#endif /* CONFIG_SYS_REDUNDAND_ENVIRONMENT */ #endif /* CONFIG_CMD_SAVEENV */
+#ifdef CONFIG_SYS_REDUNDAND_ENVIRONMENT +void env_relocate_spec(void) +{
- ALLOC_CACHE_ALIGN_BUFFER(char, env1_buf, CONFIG_ENV_SIZE);
- ALLOC_CACHE_ALIGN_BUFFER(char, env2_buf, CONFIG_ENV_SIZE);
- int crc1_ok = 0, crc2_ok = 0;
- env_t *ep, *tmp_env1, *tmp_env2;
- tmp_env1 = (env_t *)env1_buf;
- tmp_env2 = (env_t *)env2_buf;
- if (ubi_part(CONFIG_ENV_UBI_PART, NULL)) {
printf("\n** Cannot find mtd partition \"%s\"\n",CONFIG_ENV_UBI_PART);set_default_env(NULL);return;- }
- if (ubi_volume_read(CONFIG_ENV_UBI_VOLUME, (void *)tmp_env1,
CONFIG_ENV_SIZE))printf("\n** Unable to read env from %s:%s **\n",CONFIG_ENV_UBI_PART, CONFIG_ENV_UBI_VOLUME);
Parentheses for multi-line statements.
- if (ubi_volume_read(CONFIG_ENV_UBI_VOLUME_REDUND, (void *)tmp_env2,
CONFIG_ENV_SIZE))printf("\n** Unable to read redundant env from %s:%s **\n",CONFIG_ENV_UBI_PART, CONFIG_ENV_UBI_VOLUME_REDUND);
Here too.
- crc1_ok = crc32(0, tmp_env1->data, ENV_SIZE) == tmp_env1->crc;
- crc2_ok = crc32(0, tmp_env2->data, ENV_SIZE) == tmp_env2->crc;
- if (!crc1_ok && !crc2_ok) {
set_default_env("!bad CRC");return;- } else if (crc1_ok && !crc2_ok) {
gd->env_valid = 1;- } else if (!crc1_ok && crc2_ok) {
gd->env_valid = 2;- } else {
/* both ok - check serial */if (tmp_env1->flags == 255 && tmp_env2->flags == 0)gd->env_valid = 2;else if (tmp_env2->flags == 255 && tmp_env1->flags == 0)gd->env_valid = 1;else if (tmp_env1->flags > tmp_env2->flags)gd->env_valid = 1;else if (tmp_env2->flags > tmp_env1->flags)gd->env_valid = 2;else /* flags are equal - almost impossible */gd->env_valid = 1;- }
- if (gd->env_valid == 1)
ep = tmp_env1;- else
ep = tmp_env2;- env_flags = ep->flags;
- env_import((char *)ep, 0);
I might be wrong, but these lines above are most likely copied from other env handling source files, correct? It would be great to extract this code into some common file then. What do you think?
Other that that:
Acked-by: Stefan Roese sr@denx.de
Thanks, Stefan