[U-Boot] [PATCH] env_nand: align NAND buffers

Simon Glass sjg at chromium.org
Mon Jul 30 22:36:31 CEST 2012


On Mon, Jul 30, 2012 at 6:38 PM, Stephen Warren <swarren at wwwdotorg.org> wrote:
> From: Stephen Warren <swarren at nvidia.com>
>
> This allows cache flush/invalidate operations to succeed on the buffers.
>
> Signed-off-by: Stephen Warren <swarren at nvidia.com>

Acked-by: Simon Glass <sjg at chromium.org>

> ---
>  common/env_nand.c |   10 +++++-----
>  1 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/common/env_nand.c b/common/env_nand.c
> index e8daec9..e635472 100644
> --- a/common/env_nand.c
> +++ b/common/env_nand.c
> @@ -226,7 +226,7 @@ int saveenv(void)
>  int saveenv(void)
>  {
>         int     ret = 0;
> -       env_t   env_new;
> +       ALLOC_CACHE_ALIGN_BUFFER(env_t, env_new, sizeof(env_t));

I think this should b
>         ssize_t len;
>         char    *res;
>         nand_erase_options_t nand_erase_options;
> @@ -238,20 +238,20 @@ int saveenv(void)
>         if (CONFIG_ENV_RANGE < CONFIG_ENV_SIZE)
>                 return 1;
>
> -       res = (char *)&env_new.data;
> +       res = (char *)&env_new->data;
>         len = hexport_r(&env_htab, '\0', &res, ENV_SIZE, 0, NULL);
>         if (len < 0) {
>                 error("Cannot export environment: errno = %d\n", errno);
>                 return 1;
>         }
> -       env_new.crc = crc32(0, env_new.data, ENV_SIZE);
> +       env_new->crc = crc32(0, env_new->data, ENV_SIZE);
>
>         puts("Erasing Nand...\n");
>         if (nand_erase_opts(&nand_info[0], &nand_erase_options))
>                 return 1;
>
>         puts("Writing to Nand... ");
> -       if (writeenv(CONFIG_ENV_OFFSET, (u_char *)&env_new)) {
> +       if (writeenv(CONFIG_ENV_OFFSET, (u_char *)env_new)) {
>                 puts("FAILED!\n");
>                 return 1;
>         }
> @@ -398,7 +398,7 @@ void env_relocate_spec(void)
>  {
>  #if !defined(ENV_IS_EMBEDDED)
>         int ret;
> -       char buf[CONFIG_ENV_SIZE];
> +       ALLOC_CACHE_ALIGN_BUFFER(char, buf, CONFIG_ENV_SIZE);
>
>  #if defined(CONFIG_ENV_OFFSET_OOB)
>         ret = get_nand_env_oob(&nand_info[0], &nand_env_oob_offset);
> --
> 1.7.0.4
>


More information about the U-Boot mailing list