Re: [PATCH v3] drm/amdgpu: Fix missing error code in 'gmc_v6/7/8/9_0_hw_init()'

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

Am 30.01.24 um 11:19 schrieb Srinivasan Shanmugam:
> Return 0 for success scenairos in 'gmc_v6/7/8/9_0_hw_init()'
>
> Fixes the below:
> drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c:920 gmc_v6_0_hw_init() warn: missing error code? 'r'
> drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1104 gmc_v7_0_hw_init() warn: missing error code? 'r'
> drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1224 gmc_v8_0_hw_init() warn: missing error code? 'r'
> drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c:2347 gmc_v9_0_hw_init() warn: missing error code? 'r'
>
> Fixes: 8301de8fcadc ("drm/amdgpu: Fix with right return code '-EIO' in 'amdgpu_gmc_vram_checking()'")
> Cc: Christian König <christian.koenig@xxxxxxx>
> Cc: Alex Deucher <alexander.deucher@xxxxxxx>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@xxxxxxx>

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

> ---
> v3:
>    - Changed from 'return r;' to 'return 0' (Christian)
>
>   drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c | 4 ++--
>   drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c | 4 ++--
>   drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c | 4 ++--
>   drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 4 ++--
>   4 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c
> index 229263e407e0..23b478639921 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c
> @@ -916,8 +916,8 @@ static int gmc_v6_0_hw_init(void *handle)
>   
>   	if (amdgpu_emu_mode == 1)
>   		return amdgpu_gmc_vram_checking(adev);
> -	else
> -		return r;
> +
> +	return 0;
>   }
>   
>   static int gmc_v6_0_hw_fini(void *handle)
> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
> index d95f719eec55..3da7b6a2b00d 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
> @@ -1100,8 +1100,8 @@ static int gmc_v7_0_hw_init(void *handle)
>   
>   	if (amdgpu_emu_mode == 1)
>   		return amdgpu_gmc_vram_checking(adev);
> -	else
> -		return r;
> +
> +	return 0;
>   }
>   
>   static int gmc_v7_0_hw_fini(void *handle)
> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
> index 4eb0cccdb413..969a9e867170 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
> @@ -1220,8 +1220,8 @@ static int gmc_v8_0_hw_init(void *handle)
>   
>   	if (amdgpu_emu_mode == 1)
>   		return amdgpu_gmc_vram_checking(adev);
> -	else
> -		return r;
> +
> +	return 0;
>   }
>   
>   static int gmc_v8_0_hw_fini(void *handle)
> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> index a3a11538207b..4a50537252ac 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> @@ -2343,8 +2343,8 @@ static int gmc_v9_0_hw_init(void *handle)
>   
>   	if (amdgpu_emu_mode == 1)
>   		return amdgpu_gmc_vram_checking(adev);
> -	else
> -		return r;
> +
> +	return 0;
>   }
>   
>   /**