drm/amdgpu: enable sw smu driver for navi10 by default
authorHawking Zhang <Hawking.Zhang@amd.com>
Mon, 1 Apr 2019 07:32:48 +0000 (15:32 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 21 Jun 2019 23:59:25 +0000 (18:59 -0500)
Navi10 will use sw smu driver for dynamic power managment,
while vega20 could also use sw smu driver when amdgpu_dpm is
set to 2

Signed-off-by: Hawking Zhang <Hawking.Zhang@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
drivers/gpu/drm/amd/powerplay/amdgpu_smu.c

index 6920e237263273292a6660cf30f649263ea1d962..cb081f2e6829c48b826705df5309211daac4d70b 100644 (file)
@@ -251,7 +251,9 @@ module_param_string(lockup_timeout, amdgpu_lockup_timeout, sizeof(amdgpu_lockup_
 
 /**
  * DOC: dpm (int)
- * Override for dynamic power management setting (1 = enable, 0 = disable). The default is -1 (auto).
+ * Override for dynamic power management setting
+ * (0 = disable, 1 = enable, 2 = enable sw smu driver for vega20)
+ * The default is -1 (auto).
  */
 MODULE_PARM_DESC(dpm, "DPM support (1 = enable, 0 = disable, -1 = auto)");
 module_param_named(dpm, amdgpu_dpm, int, 0444);
index 63df59e7335d53bd8c10d2011e0d6175ea692fd1..785321da11410b846fbbe2656b6274da110f4de3 100644 (file)
@@ -168,13 +168,12 @@ int smu_update_table_with_arg(struct smu_context *smu, uint16_t table_id, uint16
 
 bool is_support_sw_smu(struct amdgpu_device *adev)
 {
-       if (amdgpu_dpm != 1)
-               return false;
-
-       if (adev->asic_type >= CHIP_VEGA20 && adev->asic_type != CHIP_RAVEN)
+       if (adev->asic_type == CHIP_VEGA20)
+               return (amdgpu_dpm == 2) ? true : false;
+       else if (adev->asic_type >= CHIP_NAVI10)
                return true;
-
-       return false;
+       else
+               return false;
 }
 
 int smu_sys_get_pp_table(struct smu_context *smu, void **table)