Re: [PATCH 1/1] stm32mp: fix board_get_alt_info_mmc()

Hi,

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.schuchardt@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: Patrick Delaunay patrick.delaunay@foss.st.com

Thanks Patrick