diff options
| author | Colin Ian King <[email protected]> | 2018-06-06 13:18:31 +0100 |
|---|---|---|
| committer | Alex Deucher <[email protected]> | 2018-06-13 13:45:22 -0500 |
| commit | c4ff91dd40e2253ab6dd028011469c2c694e1e19 (patch) | |
| tree | 096607cd20a37096fa2c42cdfaec6427f1b0aeec /drivers/gpu/drm/amd/amdgpu/amdgpu_vf_error.c | |
| parent | c3dade5ef72d9141e28fb86ebd46a9b3f3f4e030 (diff) | |
drm/amd/pp: initialize result to before or'ing in data
The current use of result is or'ing in values and checking for
a non-zero result, however, result is not initialized to zero
so it potentially contains garbage to start with. Fix this by
initializing it to the first return from the call to
vega10_program_didt_config_registers.
Detected by cppcheck:
"(error) Uninitialized variable: result"
Fixes: 9b7b8154cdb8 ("drm/amd/powerplay: added didt support for vega10")
Signed-off-by: Colin Ian King <[email protected]>
Acked-by: Huang Rui <[email protected]>
[Fix the subject as Colin's comment]
Signed-off-by: Huang Rui <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
Cc: [email protected]
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_vf_error.c')
0 files changed, 0 insertions, 0 deletions