drm/amdgpu/smu: use unified variable smu->is_apu to check apu asic platform
authorKevin Wang <kevin1.wang@amd.com>
Mon, 23 Dec 2019 10:17:36 +0000 (18:17 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 7 Jan 2020 16:57:22 +0000 (11:57 -0500)
use unified variable smu->is_apu to check apu asic in smu driver.

related patch:
drm/amd/powerplay: bypass dpm_context null pointer check guard for some
smu series

Signed-off-by: Kevin Wang <kevin1.wang@amd.com>
Reviewed-by: Huang Rui <ray.huang@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/powerplay/amdgpu_smu.c
drivers/gpu/drm/amd/powerplay/smu_v12_0.c

index d07c4f2ccee741bbd5ff478493564698be87ff72..d04c0ae3cfaab815b87a642987a432f442d383c8 100644 (file)
@@ -643,12 +643,11 @@ int smu_feature_init_dpm(struct smu_context *smu)
 
 int smu_feature_is_enabled(struct smu_context *smu, enum smu_feature_mask mask)
 {
-       struct amdgpu_device *adev = smu->adev;
        struct smu_feature *feature = &smu->smu_feature;
        int feature_id;
        int ret = 0;
 
-       if (adev->flags & AMD_IS_APU)
+       if (smu->is_apu)
                return 1;
 
        feature_id = smu_feature_get_index(smu, mask);
@@ -1242,7 +1241,7 @@ static int smu_hw_init(void *handle)
                return ret;
        }
 
-       if (adev->flags & AMD_IS_APU) {
+       if (smu->is_apu) {
                smu_powergate_sdma(&adev->smu, false);
                smu_powergate_vcn(&adev->smu, false);
                smu_powergate_jpeg(&adev->smu, false);
@@ -1301,7 +1300,7 @@ static int smu_hw_fini(void *handle)
        struct smu_table_context *table_context = &smu->smu_table;
        int ret = 0;
 
-       if (adev->flags & AMD_IS_APU) {
+       if (smu->is_apu) {
                smu_powergate_sdma(&adev->smu, true);
                smu_powergate_vcn(&adev->smu, true);
                smu_powergate_jpeg(&adev->smu, true);
@@ -1377,7 +1376,7 @@ static int smu_suspend(void *handle)
        struct smu_context *smu = &adev->smu;
        bool baco_feature_is_enabled = false;
 
-       if(!(adev->flags & AMD_IS_APU))
+       if(!smu->is_apu)
                baco_feature_is_enabled = smu_feature_is_enabled(smu, SMU_FEATURE_BACO_BIT);
 
        ret = smu_system_features_control(smu, false);
index 2ac7f2f231b6ac1d9dc63e8caf4f4dfb1a8a9397..9e27462d0f4eb195b4fc12eae32210e75d1b7999 100644 (file)
@@ -159,7 +159,7 @@ int smu_v12_0_check_fw_version(struct smu_context *smu)
 
 int smu_v12_0_powergate_sdma(struct smu_context *smu, bool gate)
 {
-       if (!(smu->adev->flags & AMD_IS_APU))
+       if (!smu->is_apu)
                return 0;
 
        if (gate)
@@ -170,7 +170,7 @@ int smu_v12_0_powergate_sdma(struct smu_context *smu, bool gate)
 
 int smu_v12_0_powergate_vcn(struct smu_context *smu, bool gate)
 {
-       if (!(smu->adev->flags & AMD_IS_APU))
+       if (!smu->is_apu)
                return 0;
 
        if (gate)
@@ -181,7 +181,7 @@ int smu_v12_0_powergate_vcn(struct smu_context *smu, bool gate)
 
 int smu_v12_0_powergate_jpeg(struct smu_context *smu, bool gate)
 {
-       if (!(smu->adev->flags & AMD_IS_APU))
+       if (!smu->is_apu)
                return 0;
 
        if (gate)