Re: [PATCH 06/10] stm32mp: stm32prog: correctly handle DM_PMIC

From: Patrice CHOTARD <patrice.chotard@foss.st.com>
To: Patrick Delaunay <patrick.delaunay@foss.st.com>, <u-boot@lists.denx.de>
Cc: Patrick Delaunay <patrick.delaunay@st.com>,
	Simon Glass <sjg@chromium.org>,
	U-Boot STM32 <uboot-stm32@st-md-mailman.stormreply.com>
Subject: Re: [PATCH 06/10] stm32mp: stm32prog: correctly handle DM_PMIC
Date: Fri, 28 May 2021 14:54:08 +0200	[thread overview]
Message-ID: <f3155f93-f2ba-de1f-da12-5cdb8926ff0c@foss.st.com> (raw)
In-Reply-To: <20210518151206.6.Id5ec1c084e307f296256a9764772f23492ee4766@changeid>

Hi Patrick

On 5/18/21 3:12 PM, Patrick Delaunay wrote:
> Correctly handle number of alternate when DM_PMIC is not activated.
> This patch remove the last UNKNOWN partition in this case.
> 
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> ---
> 
>  arch/arm/mach-stm32mp/cmd_stm32prog/stm32prog.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/mach-stm32mp/cmd_stm32prog/stm32prog.c b/arch/arm/mach-stm32mp/cmd_stm32prog/stm32prog.c
> index ea69d5dd16..ab687c272d 100644
> --- a/arch/arm/mach-stm32mp/cmd_stm32prog/stm32prog.c
> +++ b/arch/arm/mach-stm32mp/cmd_stm32prog/stm32prog.c
> @@ -1150,7 +1150,10 @@ static int dfu_init_entities(struct stm32prog_data *data)
>  	struct dfu_entity *dfu;
>  	int alt_nb;
>  
> -	alt_nb = 3; /* number of virtual = CMD, OTP, PMIC*/
> +	alt_nb = 2; /* number of virtual = CMD, OTP*/
> +	if (CONFIG_IS_ENABLED(DM_PMIC))
> +		alt_nb++; /* PMIC NVMEM*/
> +
>  	if (data->part_nb == 0)
>  		alt_nb++;  /* +1 for FlashLayout */
>  	else
> 

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

Thanks
Patrice