diff options
| author | Arnd Bergmann <[email protected]> | 2021-02-25 15:01:02 +0000 |
|---|---|---|
| committer | Alex Deucher <[email protected]> | 2021-03-02 19:31:45 +0000 |
| commit | 7c7dd77489540d676c2b72304825c6175621e362 (patch) | |
| tree | 65587acfe3d0d3617e78c58a7f9165aed1f530a9 /drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h | |
| parent | drm/amd/display: fix the return of the uninitialized value in ret (diff) | |
| download | kernel-7c7dd77489540d676c2b72304825c6175621e362.tar.gz kernel-7c7dd77489540d676c2b72304825c6175621e362.zip | |
drm/amd/display: Fix an uninitialized index variable
clang points out that the new logic uses an always-uninitialized
array index:
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:9810:38: warning: variable 'i' is uninitialized when used here [-Wuninitialized]
timing = &edid->detailed_timings[i];
^
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:9720:7: note: initialize the variable 'i' to silence this warning
My best guess is that the index should have been returned by the
parse_hdmi_amd_vsdb() function that walks an array here, so do that.
Fixes: f9b4f20c4777 ("drm/amd/display: Add Freesync HDMI support to DM")
Reviewed-by: Nick Desaulniers <[email protected]>
Signed-off-by: Arnd Bergmann <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h')
0 files changed, 0 insertions, 0 deletions
