diff options
author | Alex Deucher <alexander.deucher@amd.com> | 2016-02-19 15:18:45 -0500 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2016-02-22 11:30:50 -0500 |
commit | 10f950f640e689336af0a0d587a4b3c26a73cd44 (patch) | |
tree | ce7f19a59aad309c1c31d0bb080046cad76074b2 /drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c | |
parent | 04ab3b7622a618ea081d17427c15a292710b40cb (diff) |
drm/amdgpu: fix locking in force performance level
Looks like a copy paste typo when we added powerplay
support.
Reviewed-by: Christian König <christian.koenig@amd.com>
Acked-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c index 7d8d84eaea4a..5ed5598997e9 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c | |||
@@ -157,6 +157,7 @@ static ssize_t amdgpu_set_dpm_forced_performance_level(struct device *dev, | |||
157 | mutex_lock(&adev->pm.mutex); | 157 | mutex_lock(&adev->pm.mutex); |
158 | if (adev->pm.dpm.thermal_active) { | 158 | if (adev->pm.dpm.thermal_active) { |
159 | count = -EINVAL; | 159 | count = -EINVAL; |
160 | mutex_unlock(&adev->pm.mutex); | ||
160 | goto fail; | 161 | goto fail; |
161 | } | 162 | } |
162 | ret = amdgpu_dpm_force_performance_level(adev, level); | 163 | ret = amdgpu_dpm_force_performance_level(adev, level); |
@@ -167,8 +168,6 @@ static ssize_t amdgpu_set_dpm_forced_performance_level(struct device *dev, | |||
167 | mutex_unlock(&adev->pm.mutex); | 168 | mutex_unlock(&adev->pm.mutex); |
168 | } | 169 | } |
169 | fail: | 170 | fail: |
170 | mutex_unlock(&adev->pm.mutex); | ||
171 | |||
172 | return count; | 171 | return count; |
173 | } | 172 | } |
174 | 173 | ||