Hi Michal, On 12 April 2018 at 04:26, Michal Simek <michal.si...@xilinx.com> wrote: > From: Siva Durga Prasad Paladugu <siva.durga.palad...@xilinx.com> > > Relocate env drivers if manual relocation is enabled. This > patch fixes the issue of u-boot hang incase if env is > present in any of the flash devices. > > Signed-off-by: Siva Durga Prasad Paladugu <siva...@xilinx.com> > Signed-off-by: Michal Simek <michal.si...@xilinx.com> > --- > > env/common.c | 1 + > env/env.c | 21 +++++++++++++++++++++ > include/environment.h | 5 +++++ > 3 files changed, 27 insertions(+)
Reviewed-by: Simon Glass <s...@chromium.org> Please see below > > diff --git a/env/common.c b/env/common.c > index f21ff7009695..d282868ee51b 100644 > --- a/env/common.c > +++ b/env/common.c > @@ -227,6 +227,7 @@ void env_relocate(void) > { > #if defined(CONFIG_NEEDS_MANUAL_RELOC) > env_reloc(); > + fix_envdriver(); > env_htab.change_ok += gd->reloc_off; > #endif > if (gd->env_valid == ENV_INVALID) { > diff --git a/env/env.c b/env/env.c > index 3795dbc24e2b..15a0597100f4 100644 > --- a/env/env.c > +++ b/env/env.c > @@ -10,6 +10,27 @@ > > DECLARE_GLOBAL_DATA_PTR; > > +#if defined(CONFIG_NEEDS_MANUAL_RELOC) > +void fix_envdriver(void) How about env_fix_drivers() ? The env_ prefix is nice for something in the environment code. [..] Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot