drm/amdgpu: remove needless usage of #ifdef
authorShirish S <shirish.s@amd.com>
Thu, 12 Sep 2019 06:33:55 +0000 (12:03 +0530)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 16 Sep 2019 15:05:20 +0000 (10:05 -0500)
define sched_policy in case CONFIG_HSA_AMD is not
enabled, with this there is no need to check for CONFIG_HSA_AMD
else where in driver code.

Suggested-by: Felix Kuehling <Felix.Kuehling@amd.com>
Signed-off-by: Shirish S <shirish.s@amd.com>
Reviewed-by: Felix Kuehling <Felix.Kuehling@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu.h
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c

index d1bd0c5d81a16f3648e64e0173f9f13d714bbffc..20a37ce56950a637fa7683c643bc324aad06ed70 100644 (file)
@@ -171,6 +171,8 @@ extern int amdgpu_noretry;
 extern int amdgpu_force_asic_type;
 #ifdef CONFIG_HSA_AMD
 extern int sched_policy;
+#else
+static const int sched_policy = KFD_SCHED_POLICY_HWS;
 #endif
 
 #ifdef CONFIG_DRM_AMDGPU_SI
index b893ec935b841c290ba73a6a82ed5b4123cb92bc..1affaa4ee1eecc78a5351777e2e737fa60bf4942 100644 (file)
@@ -1624,11 +1624,7 @@ static int amdgpu_device_ip_early_init(struct amdgpu_device *adev)
        }
 
        adev->pm.pp_feature = amdgpu_pp_feature_mask;
-       if (amdgpu_sriov_vf(adev)
-           #ifdef CONFIG_HSA_AMD
-           || sched_policy == KFD_SCHED_POLICY_NO_HWS
-           #endif
-           )
+       if (amdgpu_sriov_vf(adev) || sched_policy == KFD_SCHED_POLICY_NO_HWS)
                adev->pm.pp_feature &= ~PP_GFXOFF_MASK;
 
        for (i = 0; i < adev->num_ip_blocks; i++) {