On Mon, Mar 18, 2019 at 06:16:55AM +0000, Pankit Garg wrote:

> Signed-off-by: Pankit Garg <pankit.g...@nxp.com>
> ---
> Changes for v2:
>  - Remove CRC calculation as it is already part of env_import
>  - Update env_addr after env_import
>       
>  env/mmc.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/env/mmc.c b/env/mmc.c
> index c3cf35d..d329bea 100644
> --- a/env/mmc.c
> +++ b/env/mmc.c
> @@ -313,6 +313,7 @@ static int env_mmc_load(void)
>       int ret;
>       int dev = mmc_get_env_dev();
>       const char *errmsg;
> +     env_t *ep = NULL;
>  
>       mmc = find_mmc_device(dev);
>  
> @@ -334,6 +335,10 @@ static int env_mmc_load(void)
>       }
>  
>       ret = env_import(buf, 1);
> +     if (!ret) {
> +             ep = (env_t *)buf;
> +             gd->env_addr    = (ulong)&(ep->data);
> +     }
>  
>  fini:
>       fini_mmc_for_env(mmc);

Do we need to update the redundant env location section of code with
this too?

-- 
Tom

Attachment: signature.asc
Description: PGP signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to