[PATCH 5/7] imx8image: Remove unused cont_img_count variable

[Peng Fan]

On 2/28/2023 6:08 AM, Tom Rini wrote:
> With clang-15, it is now reported that cont_img_count is unused. This is
> true as the code will increment / reset this counter, but never
> functionally use it. Remove it.
> 
> Signed-off-by: Tom Rini <trini at konsulko.com>

Reviewed-by: Peng Fan <peng.fan at nxp.com>

> ---
> Cc: Peng Fan <peng.fan at nxp.com>
> Cc: Mikhail Ilin <ilin.mikhail.ol at gmail.com>
> Cc: Stefano Babic <sbabic at denx.de>
> Cc: Fabio Estevam <festevam at gmail.com>
> Cc: "NXP i.MX U-Boot Team" <uboot-imx at nxp.com>
> ---
>   tools/imx8image.c | 5 -----
>   1 file changed, 5 deletions(-)
> 
> diff --git a/tools/imx8image.c b/tools/imx8image.c
> index 395d5c64bdf0..c25ea84e25c5 100644
> --- a/tools/imx8image.c
> +++ b/tools/imx8image.c
> @@ -829,7 +829,6 @@ static int build_container(soc_type_t soc, uint32_t sector_size,
>   	int ret;
>   
>   	int container = -1;
> -	int cont_img_count = 0; /* indexes to arrange the container */
>   
>   	memset((char *)&imx_header, 0, sizeof(imx_header_v3_t));
>   
> @@ -879,7 +878,6 @@ static int build_container(soc_type_t soc, uint32_t sector_size,
>   			img_sp->src = file_off;
>   
>   			file_off += ALIGN(sbuf.st_size, sector_size);
> -			cont_img_count++;
>   			break;
>   
>   		case SECO:
> @@ -899,7 +897,6 @@ static int build_container(soc_type_t soc, uint32_t sector_size,
>   			img_sp->src = file_off;
>   
>   			file_off += sbuf.st_size;
> -			cont_img_count++;
>   			break;
>   
>   		case NEW_CONTAINER:
> @@ -908,8 +905,6 @@ static int build_container(soc_type_t soc, uint32_t sector_size,
>   				      CONTAINER_ALIGNMENT,
>   				      CONTAINER_FLAGS_DEFAULT,
>   				      fuse_version);
> -			/* reset img count when moving to new container */
> -			cont_img_count = 0;
>   			scfw_flags = 0;
>   			break;
>