From b97aab014c6ba6133df1e26bd20c1ad7f50a5bff Mon Sep 17 00:00:00 2001 From: Alex Deucher Date: Tue, 9 Jun 2015 17:46:30 -0400 Subject: [PATCH 0265/1050] drm/amdgpu/iceland: don't call smu_init on resume smu_init allocates buffers and initializes them. It does not touch the hw. There is no need to do it again on resume. It should really be part of sw_init (and smu_fini should be part of sw_fini), but we need the firmware sizes from the other IPs for firmware loading so we have to wait until sw init is done for all other IPs. Reviewed-by: Sonny Jiang Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/iceland_dpm.c | 25 +++++++++++++++++++------ 1 file changed, 19 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/iceland_dpm.c b/drivers/gpu/drm/amd/amdgpu/iceland_dpm.c index 4b773f2..208d55f4 100644 --- a/drivers/gpu/drm/amd/amdgpu/iceland_dpm.c +++ b/drivers/gpu/drm/amd/amdgpu/iceland_dpm.c @@ -82,6 +82,10 @@ static int iceland_dpm_hw_init(void *handle) mutex_lock(&adev->pm.mutex); + /* smu init only needs to be called at startup, not resume. + * It should be in sw_init, but requires the fw info gathered + * in sw_init from other IP modules. + */ ret = iceland_smu_init(adev); if (ret) { DRM_ERROR("SMU initialization failed\n"); @@ -108,6 +112,10 @@ static int iceland_dpm_hw_fini(void *handle) struct amdgpu_device *adev = (struct amdgpu_device *)handle; mutex_lock(&adev->pm.mutex); + /* smu fini only needs to be called at teardown, not suspend. + * It should be in sw_fini, but we put it here for symmetry + * with smu init. + */ iceland_smu_fini(adev); mutex_unlock(&adev->pm.mutex); return 0; @@ -115,20 +123,25 @@ static int iceland_dpm_hw_fini(void *handle) static int iceland_dpm_suspend(void *handle) { - struct amdgpu_device *adev = (struct amdgpu_device *)handle; - - iceland_dpm_hw_fini(adev); - return 0; } static int iceland_dpm_resume(void *handle) { + int ret; struct amdgpu_device *adev = (struct amdgpu_device *)handle; - iceland_dpm_hw_init(adev); + mutex_lock(&adev->pm.mutex); - return 0; + ret = iceland_smu_start(adev); + if (ret) { + DRM_ERROR("SMU start failed\n"); + goto fail; + } + +fail: + mutex_unlock(&adev->pm.mutex); + return ret; } static int iceland_dpm_set_clockgating_state(void *handle, -- 1.9.1