shinys000114/linux
1
0
Fork 0
mirror of https://github.com/hardkernel/linux.git synced 2026-06-05 10:31:46 +09:00
Code Issues Packages Projects Releases Wiki Activity

drm/amd/pm: fix uninitialized variable warning for smu8_hwmgr

Browse Source 
[ Upstream commit 86df36b934640866eb249a4488abb148b985a0d9 ]

Clear warnings that using uninitialized value level when fails
to get the value from SMU.

Signed-off-by: Tim Huang <Tim.Huang@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Tim Huang
2024-04-26 12:52:45 +08:00
committed by Greg Kroah-Hartman
parent 92cb4425f5
commit 0f97f31acc
1 changed files with 12 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu8_hwmgr.c
View File

@@ -584,6 +584,7 @@ static int smu8_init_uvd_limit(struct pp_hwmgr *hwmgr)
hwmgr->dyn_state.uvd_clock_voltage_dependency_table; hwmgr->dyn_state.uvd_clock_voltage_dependency_table;
unsigned long clock = 0; unsigned long clock = 0;
uint32_t level; uint32_t level;
int ret;
if (NULL == table || table->count <= 0) if (NULL == table || table->count <= 0)
return -EINVAL; return -EINVAL;
@@ -591,7 +592,9 @@ static int smu8_init_uvd_limit(struct pp_hwmgr *hwmgr)
data->uvd_dpm.soft_min_clk = 0; data->uvd_dpm.soft_min_clk = 0;
data->uvd_dpm.hard_min_clk = 0; data->uvd_dpm.hard_min_clk = 0;
smum_send_msg_to_smc(hwmgr, PPSMC_MSG_GetMaxUvdLevel, &level); ret = smum_send_msg_to_smc(hwmgr, PPSMC_MSG_GetMaxUvdLevel, &level);
if (ret)
return ret;
if (level < table->count) if (level < table->count)
clock = table->entries[level].vclk; clock = table->entries[level].vclk;
@@ -611,6 +614,7 @@ static int smu8_init_vce_limit(struct pp_hwmgr *hwmgr)
hwmgr->dyn_state.vce_clock_voltage_dependency_table; hwmgr->dyn_state.vce_clock_voltage_dependency_table;
unsigned long clock = 0; unsigned long clock = 0;
uint32_t level; uint32_t level;
int ret;
if (NULL == table || table->count <= 0) if (NULL == table || table->count <= 0)
return -EINVAL; return -EINVAL;
@@ -618,7 +622,9 @@ static int smu8_init_vce_limit(struct pp_hwmgr *hwmgr)
data->vce_dpm.soft_min_clk = 0; data->vce_dpm.soft_min_clk = 0;
data->vce_dpm.hard_min_clk = 0; data->vce_dpm.hard_min_clk = 0;
smum_send_msg_to_smc(hwmgr, PPSMC_MSG_GetMaxEclkLevel, &level); ret = smum_send_msg_to_smc(hwmgr, PPSMC_MSG_GetMaxEclkLevel, &level);
if (ret)
return ret;
if (level < table->count) if (level < table->count)
clock = table->entries[level].ecclk; clock = table->entries[level].ecclk;
@@ -638,6 +644,7 @@ static int smu8_init_acp_limit(struct pp_hwmgr *hwmgr)
hwmgr->dyn_state.acp_clock_voltage_dependency_table; hwmgr->dyn_state.acp_clock_voltage_dependency_table;
unsigned long clock = 0; unsigned long clock = 0;
uint32_t level; uint32_t level;
int ret;
if (NULL == table || table->count <= 0) if (NULL == table || table->count <= 0)
return -EINVAL; return -EINVAL;
@@ -645,7 +652,9 @@ static int smu8_init_acp_limit(struct pp_hwmgr *hwmgr)
data->acp_dpm.soft_min_clk = 0; data->acp_dpm.soft_min_clk = 0;
data->acp_dpm.hard_min_clk = 0; data->acp_dpm.hard_min_clk = 0;
smum_send_msg_to_smc(hwmgr, PPSMC_MSG_GetMaxAclkLevel, &level); ret = smum_send_msg_to_smc(hwmgr, PPSMC_MSG_GetMaxAclkLevel, &level);
if (ret)
return ret;
if (level < table->count) if (level < table->count)
clock = table->entries[level].acpclk; clock = table->entries[level].acpclk;