diff options
author | Ernst Sjöstrand <ernstp@gmail.com> | 2017-11-19 12:52:42 -0500 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2017-12-06 12:47:59 -0500 |
commit | 19fef71cd1f4928b958913bfee38662471f04faf (patch) | |
tree | 19d5927e57eef9d4e45fd670d1c5bcbb8cf1125c /drivers/gpu/drm/amd | |
parent | 235943189db8ce05b888f48a04ded4448eebc408 (diff) |
drm/amd/powerplay: Minor fixes in processpptables.c (v2)
Reported by smatch:
init_overdrive_limits() error: uninitialized symbol 'result'.
get_clock_voltage_dependency_table() warn: inconsistent indenting
v2: set result to 0 (Alex)
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Ernst Sjöstrand <ernstp@gmail.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd')
-rw-r--r-- | drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.c b/drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.c index afae32ee2b0d..c3e7e34535e8 100644 --- a/drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.c +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.c | |||
@@ -394,8 +394,8 @@ static int get_clock_voltage_dependency_table(struct pp_hwmgr *hwmgr, | |||
394 | dep_table->entries[i].clk = | 394 | dep_table->entries[i].clk = |
395 | ((unsigned long)table->entries[i].ucClockHigh << 16) | | 395 | ((unsigned long)table->entries[i].ucClockHigh << 16) | |
396 | le16_to_cpu(table->entries[i].usClockLow); | 396 | le16_to_cpu(table->entries[i].usClockLow); |
397 | dep_table->entries[i].v = | 397 | dep_table->entries[i].v = |
398 | (unsigned long)le16_to_cpu(table->entries[i].usVoltage); | 398 | (unsigned long)le16_to_cpu(table->entries[i].usVoltage); |
399 | } | 399 | } |
400 | 400 | ||
401 | *ptable = dep_table; | 401 | *ptable = dep_table; |
@@ -1042,7 +1042,7 @@ static int init_overdrive_limits_V2_1(struct pp_hwmgr *hwmgr, | |||
1042 | static int init_overdrive_limits(struct pp_hwmgr *hwmgr, | 1042 | static int init_overdrive_limits(struct pp_hwmgr *hwmgr, |
1043 | const ATOM_PPLIB_POWERPLAYTABLE *powerplay_table) | 1043 | const ATOM_PPLIB_POWERPLAYTABLE *powerplay_table) |
1044 | { | 1044 | { |
1045 | int result; | 1045 | int result = 0; |
1046 | uint8_t frev, crev; | 1046 | uint8_t frev, crev; |
1047 | uint16_t size; | 1047 | uint16_t size; |
1048 | 1048 | ||