mirror of
https://github.com/torvalds/linux.git
synced 2026-09-23 05:04:02 +02:00
drm/amd/pm: silence uninitialized variable warnings
Smatch complains that:
drivers/gpu/drm/amd/amdgpu/../pm/swsmu/smu15/smu_v15_0_8_ppt.c:1964
smu_v15_0_8_set_performance_level() error: uninitialized symbol 'ret'.
In this line there is an "if (ret)" condition where "ret" is either
zero or uninitialized. Initialize "ret" at the start of the function to
avoid a potential uninitialized variable bug. But also delete the
condition since it is never true.
Fixes: 422b399b09 ("drm/amd/pm: Add od_edit_dpm_table support")
Reviewed-by: Lijo Lazar <lijo.lazar@amd.com>
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
parent
24775b2e8b
commit
b4cb43789b
|
|
@ -1951,7 +1951,7 @@ static int smu_v15_0_8_set_performance_level(struct smu_context *smu,
|
|||
struct smu_dpm_table *gfx_table = &dpm_context->dpm_tables.gfx_table;
|
||||
struct smu_dpm_table *uclk_table = &dpm_context->dpm_tables.uclk_table;
|
||||
struct smu_umd_pstate_table *pstate_table = &smu->pstate_table;
|
||||
int ret;
|
||||
int ret = 0;
|
||||
|
||||
switch (level) {
|
||||
case AMD_DPM_FORCED_LEVEL_PERF_DETERMINISM:
|
||||
|
|
@ -1992,9 +1992,6 @@ static int smu_v15_0_8_set_performance_level(struct smu_context *smu,
|
|||
SMU_DPM_TABLE_MAX(uclk_table);
|
||||
}
|
||||
|
||||
if (ret)
|
||||
goto out;
|
||||
|
||||
smu_cmn_reset_custom_level(smu);
|
||||
|
||||
break;
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user