Hi Heinrich

On 1/11/22 3:58 PM, Heinrich Schuchardt wrote:
> MAX_SEARCH_PARTITIONS is the highest possible partition number.
> Do not skip the last partition in board_get_alt_info_mmc().
> 
> Signed-off-by: Heinrich Schuchardt <heinrich.schucha...@canonical.com>
> ---
>  board/st/common/stm32mp_dfu.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/board/st/common/stm32mp_dfu.c b/board/st/common/stm32mp_dfu.c
> index a3f0da5b5b..fa48b2a35e 100644
> --- a/board/st/common/stm32mp_dfu.c
> +++ b/board/st/common/stm32mp_dfu.c
> @@ -57,7 +57,7 @@ static void board_get_alt_info_mmc(struct udevice *dev, 
> char *buf)
>               first = false;
>       }
>  
> -     for (p = 1; p < MAX_SEARCH_PARTITIONS; p++) {
> +     for (p = 1; p <= MAX_SEARCH_PARTITIONS; p++) {
>               if (part_get_info(desc, p, &info))
>                       continue;
>               if (!first)
> 

Reviewed-by: Patrice Chotard <patrice.chot...@foss.st.com>
Thanks
Patrice

Reply via email to