diff options
author | Colin Ian King <colin.king@canonical.com> | 2021-03-02 14:05:09 +0000 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2021-03-03 23:05:16 -0500 |
commit | b934dd9b44e8ad180b3203ce7d6df3133453ee91 (patch) | |
tree | 0c3f0f7cd301ea525ff94242e68368d21fc7eba9 | |
parent | 25951362db7b3791488ec45bf56c0043f107b94b (diff) | |
download | linux-b934dd9b44e8ad180b3203ce7d6df3133453ee91.tar.gz linux-b934dd9b44e8ad180b3203ce7d6df3133453ee91.tar.bz2 linux-b934dd9b44e8ad180b3203ce7d6df3133453ee91.zip |
drm/amd/display: fix the return of the uninitialized value in ret
Currently if stream->signal is neither SIGNAL_TYPE_DISPLAY_PORT_MST or
SIGNAL_TYPE_DISPLAY_PORT then variable ret is uninitialized and this is
checked for > 0 at the end of the function. Ret should be initialized,
I believe setting it to zero is a correct default.
Addresses-Coverity: ("Uninitialized scalar variable")
Fixes: bd0c064c161c ("drm/amd/display: Add return code instead of boolean for future use")
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
-rw-r--r-- | drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c index 5159399f8239..5750818db8f6 100644 --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c @@ -530,7 +530,7 @@ bool dm_helpers_dp_write_dsc_enable( { uint8_t enable_dsc = enable ? 1 : 0; struct amdgpu_dm_connector *aconnector; - uint8_t ret; + uint8_t ret = 0; if (!stream) return false; |