Commit fa8a91b0 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 adb4d6a4
Loading
Loading
Loading
Loading
+2 −2
Original line number Diff line number Diff line
@@ -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)
+2 −2
Original line number Diff line number Diff line
@@ -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)
+2 −2
Original line number Diff line number Diff line
@@ -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)
+2 −2
Original line number Diff line number Diff line
@@ -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;
}

/**