Hi On 3/19/20 10:59 AM, Patrick Delaunay wrote: > Manage 2 copy at the end of the partition selected by config > "u-boot,mmc-env-partition" to save the U-Boot environment, > with CONFIG_ENV_SIZE and 2*CONFIG_ENV_SIZE offset. > > This patch allows to support redundancy (CONFIG_ENV_OFFSET_REDUND). > > Signed-off-by: Patrick Delaunay <[email protected]> > --- > > env/mmc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/env/mmc.c b/env/mmc.c > index c24b169f3e..677a3d4668 100644 > --- a/env/mmc.c > +++ b/env/mmc.c > @@ -34,7 +34,7 @@ __weak int mmc_get_env_dev(void) > } > > #if CONFIG_IS_ENABLED(OF_CONTROL) > -static inline int mmc_offset_try_partition(const char *str, s64 *val) > +static inline int mmc_offset_try_partition(const char *str, int copy, s64 > *val) > { > disk_partition_t info; > struct blk_desc *desc; > @@ -59,7 +59,7 @@ static inline int mmc_offset_try_partition(const char *str, > s64 *val) > len = DIV_ROUND_UP(CONFIG_ENV_SIZE, info.blksz); > > /* use the top of the partion for the environment */ > - *val = (info.start + info.size - len) * info.blksz; > + *val = (info.start + info.size - (1 + copy) * len) * info.blksz; > > return 0; > } > @@ -84,7 +84,7 @@ static inline s64 mmc_offset(int copy) > str = fdtdec_get_config_string(gd->fdt_blob, dt_prop.partition); > if (str) { > /* try to place the environment at end of the partition */ > - err = mmc_offset_try_partition(str, &val); > + err = mmc_offset_try_partition(str, copy, &val); > if (!err) > return val; > }
Reviewed-by: Patrice Chotard <[email protected]> Thanks

