Hi Igor,

Thank you for the patch.

On mar., févr. 06, 2024 at 23:31, Igor Opaniuk <igor.opan...@foundries.io> 
wrote:

> From: Igor Opaniuk <igor.opan...@gmail.com>
>
> mmc_switch_part() is used for switching between hw partitions
> on eMMC (boot0, boot1, user, rpmb).
> There is no need to do that for SD card.
>
> Signed-off-by: Igor Opaniuk <igor.opan...@gmail.com>

Reviewed-by: Mattijs Korpershoek <mkorpersh...@baylibre.com>

> ---
>
>  common/avb_verify.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/common/avb_verify.c b/common/avb_verify.c
> index 48ba8db51e5..59f2c25e0de 100644
> --- a/common/avb_verify.c
> +++ b/common/avb_verify.c
> @@ -358,9 +358,11 @@ static struct mmc_part *get_partition(AvbOps *ops, const 
> char *partition)
>               goto err;
>       }
>  
> -     ret = mmc_switch_part(part->mmc, part_num);
> -     if (ret)
> -             goto err;
> +     if (IS_MMC(part->mmc)) {
> +             ret = mmc_switch_part(part->mmc, part_num);
> +             if (ret)
> +                     goto err;
> +     }
>  
>       mmc_blk = mmc_get_blk_desc(part->mmc);
>       if (!mmc_blk) {
> -- 
> 2.34.1

Reply via email to