diff options
author | Alex Deucher <alexander.deucher@amd.com> | 2016-09-28 16:37:15 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2016-10-04 11:15:59 -0400 |
commit | c64474e4f72f21aec4ae27eab39b563cfd25561d (patch) | |
tree | b9f7efbd831ba0d71345f45be402b799f2be6e11 /drivers/gpu/drm/amd/amdgpu/amdgpu_powerplay.c | |
parent | 363926dc9ef65a3813fa778e85f85dd756c8652f (diff) |
drm/amdgpu: remove DRM_AMD_POWERPLAY
Powerplay is no longer optional after the recently cleanups
Reviewed-by: Edward O'Callaghan <funfunctor@folklore1984.net>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_powerplay.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_powerplay.c | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_powerplay.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_powerplay.c index 68ad24101a36..7532ff822aa7 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_powerplay.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_powerplay.c | |||
@@ -42,7 +42,6 @@ static int amdgpu_powerplay_init(struct amdgpu_device *adev) | |||
42 | amd_pp = &(adev->powerplay); | 42 | amd_pp = &(adev->powerplay); |
43 | 43 | ||
44 | if (adev->pp_enabled) { | 44 | if (adev->pp_enabled) { |
45 | #ifdef CONFIG_DRM_AMD_POWERPLAY | ||
46 | struct amd_pp_init *pp_init; | 45 | struct amd_pp_init *pp_init; |
47 | 46 | ||
48 | pp_init = kzalloc(sizeof(struct amd_pp_init), GFP_KERNEL); | 47 | pp_init = kzalloc(sizeof(struct amd_pp_init), GFP_KERNEL); |
@@ -55,7 +54,6 @@ static int amdgpu_powerplay_init(struct amdgpu_device *adev) | |||
55 | pp_init->device = amdgpu_cgs_create_device(adev); | 54 | pp_init->device = amdgpu_cgs_create_device(adev); |
56 | ret = amd_powerplay_init(pp_init, amd_pp); | 55 | ret = amd_powerplay_init(pp_init, amd_pp); |
57 | kfree(pp_init); | 56 | kfree(pp_init); |
58 | #endif | ||
59 | } else { | 57 | } else { |
60 | amd_pp->pp_handle = (void *)adev; | 58 | amd_pp->pp_handle = (void *)adev; |
61 | 59 | ||
@@ -97,7 +95,6 @@ static int amdgpu_pp_early_init(void *handle) | |||
97 | struct amdgpu_device *adev = (struct amdgpu_device *)handle; | 95 | struct amdgpu_device *adev = (struct amdgpu_device *)handle; |
98 | int ret = 0; | 96 | int ret = 0; |
99 | 97 | ||
100 | #ifdef CONFIG_DRM_AMD_POWERPLAY | ||
101 | switch (adev->asic_type) { | 98 | switch (adev->asic_type) { |
102 | case CHIP_POLARIS11: | 99 | case CHIP_POLARIS11: |
103 | case CHIP_POLARIS10: | 100 | case CHIP_POLARIS10: |
@@ -120,9 +117,6 @@ static int amdgpu_pp_early_init(void *handle) | |||
120 | adev->pp_enabled = false; | 117 | adev->pp_enabled = false; |
121 | break; | 118 | break; |
122 | } | 119 | } |
123 | #else | ||
124 | adev->pp_enabled = false; | ||
125 | #endif | ||
126 | 120 | ||
127 | ret = amdgpu_powerplay_init(adev); | 121 | ret = amdgpu_powerplay_init(adev); |
128 | if (ret) | 122 | if (ret) |
@@ -144,12 +138,11 @@ static int amdgpu_pp_late_init(void *handle) | |||
144 | ret = adev->powerplay.ip_funcs->late_init( | 138 | ret = adev->powerplay.ip_funcs->late_init( |
145 | adev->powerplay.pp_handle); | 139 | adev->powerplay.pp_handle); |
146 | 140 | ||
147 | #ifdef CONFIG_DRM_AMD_POWERPLAY | ||
148 | if (adev->pp_enabled && adev->pm.dpm_enabled) { | 141 | if (adev->pp_enabled && adev->pm.dpm_enabled) { |
149 | amdgpu_pm_sysfs_init(adev); | 142 | amdgpu_pm_sysfs_init(adev); |
150 | amdgpu_dpm_dispatch_task(adev, AMD_PP_EVENT_COMPLETE_INIT, NULL, NULL); | 143 | amdgpu_dpm_dispatch_task(adev, AMD_PP_EVENT_COMPLETE_INIT, NULL, NULL); |
151 | } | 144 | } |
152 | #endif | 145 | |
153 | return ret; | 146 | return ret; |
154 | } | 147 | } |
155 | 148 | ||
@@ -162,10 +155,8 @@ static int amdgpu_pp_sw_init(void *handle) | |||
162 | ret = adev->powerplay.ip_funcs->sw_init( | 155 | ret = adev->powerplay.ip_funcs->sw_init( |
163 | adev->powerplay.pp_handle); | 156 | adev->powerplay.pp_handle); |
164 | 157 | ||
165 | #ifdef CONFIG_DRM_AMD_POWERPLAY | ||
166 | if (adev->pp_enabled) | 158 | if (adev->pp_enabled) |
167 | adev->pm.dpm_enabled = true; | 159 | adev->pm.dpm_enabled = true; |
168 | #endif | ||
169 | 160 | ||
170 | return ret; | 161 | return ret; |
171 | } | 162 | } |
@@ -216,7 +207,6 @@ static int amdgpu_pp_hw_fini(void *handle) | |||
216 | 207 | ||
217 | static void amdgpu_pp_late_fini(void *handle) | 208 | static void amdgpu_pp_late_fini(void *handle) |
218 | { | 209 | { |
219 | #ifdef CONFIG_DRM_AMD_POWERPLAY | ||
220 | struct amdgpu_device *adev = (struct amdgpu_device *)handle; | 210 | struct amdgpu_device *adev = (struct amdgpu_device *)handle; |
221 | 211 | ||
222 | if (adev->pp_enabled) { | 212 | if (adev->pp_enabled) { |
@@ -227,7 +217,6 @@ static void amdgpu_pp_late_fini(void *handle) | |||
227 | if (adev->powerplay.ip_funcs->late_fini) | 217 | if (adev->powerplay.ip_funcs->late_fini) |
228 | adev->powerplay.ip_funcs->late_fini( | 218 | adev->powerplay.ip_funcs->late_fini( |
229 | adev->powerplay.pp_handle); | 219 | adev->powerplay.pp_handle); |
230 | #endif | ||
231 | } | 220 | } |
232 | 221 | ||
233 | static int amdgpu_pp_suspend(void *handle) | 222 | static int amdgpu_pp_suspend(void *handle) |