Re: [PATCH] drm/amdgpu: remove ip dump reg_count variable

[Christian König <christian.koenig@xxxxxxx>]

Am 02.05.24 um 10:56 schrieb Sunil Khatri:
> reg_count is not used and the register count is
> directly derived from the array size and hence
> removed.
>
> Signed-off-by: Sunil Khatri <sunil.khatri@xxxxxxx>

Acked-by: Christian König <christian.koenig@xxxxxxx>

> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h | 1 -
>   drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c  | 2 --
>   2 files changed, 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
> index 64f197bbc866..9a946f0e015c 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
> @@ -436,7 +436,6 @@ struct amdgpu_gfx {
>   
>   	/* IP reg dump */
>   	uint32_t			*ip_dump;
> -	uint32_t			reg_count;
>   };
>   
>   struct amdgpu_gfx_ras_reg_entry {
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> index 536287ddd2ec..3171ed5e5af3 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> @@ -4592,10 +4592,8 @@ static void gfx_v10_0_alloc_dump_mem(struct amdgpu_device *adev)
>   	if (ptr == NULL) {
>   		DRM_ERROR("Failed to allocate memory for IP Dump\n");
>   		adev->gfx.ip_dump = NULL;
> -		adev->gfx.reg_count = 0;
>   	} else {
>   		adev->gfx.ip_dump = ptr;
> -		adev->gfx.reg_count = reg_count;
>   	}
>   }
>