Remove extra multiplication.

CIK GPUs such as Hawaii appear to use PP_TABLE_V0 in which case
the shutdown temperature is hardcoded in smu7_init_dpm_defaults
and is already multiplied by 1000. The value was mistakenly
multiplied another time by smu7_get_thermal_temperature_range.

Fixes: 4ba082572a42 ("drm/amd/powerplay: export the thermal ranges of VI asics 
(V2)")
Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/1676
Signed-off-by: Timur Kristóf <[email protected]>
---
 drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c 
b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c
index 8da882c51856..9b28c0728269 100644
--- a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c
+++ b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c
@@ -5444,8 +5444,7 @@ static int smu7_get_thermal_temperature_range(struct 
pp_hwmgr *hwmgr,
                thermal_data->max = 
table_info->cac_dtp_table->usSoftwareShutdownTemp *
                        PP_TEMPERATURE_UNITS_PER_CENTIGRADES;
        else if (hwmgr->pp_table_version == PP_TABLE_V0)
-               thermal_data->max = 
data->thermal_temp_setting.temperature_shutdown *
-                       PP_TEMPERATURE_UNITS_PER_CENTIGRADES;
+               thermal_data->max = 
data->thermal_temp_setting.temperature_shutdown;
 
        thermal_data->sw_ctf_threshold = thermal_data->max;
 
-- 
2.51.0

Reply via email to