Commit 57f01856 authored by Colin Ian King's avatar Colin Ian King Committed by Alex Deucher

drm/amdgpu: ensure 0 is returned for success in jpeg_v2_5_wait_for_idle

In the cases where adev->jpeg.num_jpeg_inst is zero or the condition
adev->jpeg.harvest_config & (1 << i) is always non-zero the variable
ret is never set to an error condition and the function returns
an uninitialized value in ret.  Since the only exit condition at
the end if the function is a success then explicitly return
0 rather than a potentially uninitialized value in ret.

Addresses-Coverity: ("Uninitialized scalar variable")
Fixes: 14f43e8f ("drm/amdgpu: move JPEG2.5 out from VCN2.5")
Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 6a8987a8
...@@ -462,7 +462,7 @@ static int jpeg_v2_5_wait_for_idle(void *handle) ...@@ -462,7 +462,7 @@ static int jpeg_v2_5_wait_for_idle(void *handle)
return ret; return ret;
} }
return ret; return 0;
} }
static int jpeg_v2_5_set_clockgating_state(void *handle, static int jpeg_v2_5_set_clockgating_state(void *handle,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment