diff options
author | Arnd Bergmann <arnd@arndb.de> | 2021-02-25 16:01:02 +0100 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2021-03-02 14:31:45 -0500 |
commit | 7c7dd77489540d676c2b72304825c6175621e362 (patch) | |
tree | 65587acfe3d0d3617e78c58a7f9165aed1f530a9 /drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h | |
parent | 6302aead150e8ffb9abe82683e8e30d519e69024 (diff) | |
download | linux-7c7dd77489540d676c2b72304825c6175621e362.tar.gz linux-7c7dd77489540d676c2b72304825c6175621e362.tar.bz2 linux-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 <ndesaulniers@google.com>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h')
0 files changed, 0 insertions, 0 deletions