Commit 0ca57515 authored by Lijo Lazar's avatar Lijo Lazar Committed by Alex Deucher
Browse files

drm/amdgpu: Remove redundant logic in GC v9.4.3 



GFXOFF check is not needed for GC v9.4.3. Also, save/restore list is
available by default.

Signed-off-by: default avatarLijo Lazar <lijo.lazar@amd.com>
Reviewed-by: default avatarHawking Zhang <Hawking.Zhang@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 793ee232

drivers/gpu/drm/amd/amdgpu/gfx_v9_4_3.c

+1 −16
Original line number Diff line number Diff line
@@ -563,17 +563,6 @@ static int gfx_v9_4_3_init_rlc_microcode(struct amdgpu_device *adev, 
	return err;

}



static bool gfx_v9_4_3_should_disable_gfxoff(struct pci_dev *pdev)

{

	return true;

}



static void gfx_v9_4_3_check_if_need_gfxoff(struct amdgpu_device *adev)

{

	if (gfx_v9_4_3_should_disable_gfxoff(adev->pdev))

		adev->pm.pp_feature &= ~PP_GFXOFF_MASK;

}



static int gfx_v9_4_3_init_cp_compute_microcode(struct amdgpu_device *adev,

					  const char *chip_name)

{
@@ -600,8 +589,6 @@ static int gfx_v9_4_3_init_cp_compute_microcode(struct amdgpu_device *adev, 
	adev->gfx.mec2_fw_version = adev->gfx.mec_fw_version;

	adev->gfx.mec2_feature_version = adev->gfx.mec_feature_version;



	gfx_v9_4_3_check_if_need_gfxoff(adev);



out:

	if (err)

		amdgpu_ucode_release(&adev->gfx.mec_fw);
@@ -1362,9 +1349,7 @@ static void gfx_v9_4_3_xcc_init_pg(struct amdgpu_device *adev, int xcc_id) 
{

	/*

	 * Rlc save restore list is workable since v2_1.

	 * And it's needed by gfxoff feature.

	 */

	if (adev->gfx.rlc.is_rlc_v2_1)

	gfx_v9_4_3_xcc_enable_save_restore_machine(adev, xcc_id);

}