[U-Boot] [PATCH v3] env_mmc: correct fini partition to match init partition

Pantelis Antoniou panto at antoniou-consulting.com
Thu Oct 2 13:09:56 CEST 2014


Hi Peter,

On Sep 3, 2014, at 8:22 PM, Peter A. Bigot <pab at pabigot.com> wrote:

> The code to set the MMC partition uses an weak function to obtain the
> correct partition number.  Use that instead of the compile-time default
> when deciding whether it needs to switch back.
> 
> Fixes: 6e7b7df4df43574 ("env_mmc: support env partition setup in runtime")
> Signed-off-by: Peter A. Bigot <pab at pabigot.com>
> ---
> V3:
> * Add Fixes line as requested
> 
> V2:
> * Preserve desired behavior of avoiding diagnostic when no HW partition supported
> * Supersedes https://patchwork.ozlabs.org/patch/385355/
> 
> common/env_mmc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/common/env_mmc.c b/common/env_mmc.c
> index a7621a8..14648e3 100644
> --- a/common/env_mmc.c
> +++ b/common/env_mmc.c
> @@ -113,7 +113,7 @@ static void fini_mmc_for_env(struct mmc *mmc)
> #ifdef CONFIG_SPL_BUILD
> 	dev = 0;
> #endif
> -	if (CONFIG_SYS_MMC_ENV_PART != mmc->part_num)
> +	if (mmc_get_env_part(mmc) != mmc->part_num)
> 		mmc_switch_part(dev, mmc->part_num);
> #endif
> }
> -- 
> 1.8.5.5
> 

Applied, thanks.

Acked-by: Pantelis Antoniou <panto at antoniou-consulting.com>



More information about the U-Boot mailing list