Commit 16da3990 authored by Srinivasan Shanmugam's avatar Srinivasan Shanmugam Committed by Alex Deucher
Browse files

drm/amdgpu: Fix missing error code in 'gmc_v6/7/8/9_0_hw_init()'



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: fac4ebd7 ("drm/amdgpu: Fix with right return code '-EIO' in 'amdgpu_gmc_vram_checking()'")
Cc: Christian König <christian.koenig@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: default avatarSrinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 97cba232
Loading
Loading
Loading
Loading
+2 −2
Original line number Diff line number Diff line
@@ -915,8 +915,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)
+2 −2
Original line number Diff line number Diff line
@@ -1099,8 +1099,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)
+2 −2
Original line number Diff line number Diff line
@@ -1219,8 +1219,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)
+2 −2
Original line number Diff line number Diff line
@@ -2341,8 +2341,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;
}

/**