On Fri, May 19, 2023 at 1:06 AM Srinivasan Shanmugam
<srinivasan.shanmugam@xxxxxxx> wrote:
>
> drivers/gpu/drm/amd/amdgpu/jpeg_v4_0_3.c:752:4: error: variable 'ret' is uninitialized when used here [-Werror,-Wuninitialized]
> ret &= ((RREG32_SOC15_OFFSET(
> ^~~
> drivers/gpu/drm/amd/amdgpu/jpeg_v4_0_3.c:745:10: note: initialize the variable 'ret' to silence this warning
> bool ret;
> ^
> = 0
> drivers/gpu/drm/amd/amdgpu/jpeg_v4_0_3.c:774:4: error: variable 'ret' is uninitialized when used here [-Werror,-Wuninitialized]
> ret &= SOC15_WAIT_ON_RREG_OFFSET(
> ^~~
> drivers/gpu/drm/amd/amdgpu/jpeg_v4_0_3.c:767:9: note: initialize the variable 'ret' to silence this warning
> int ret;
> ^
> = 0
> 2 errors generated.
>
> Cc: Luben Tuikov <luben.tuikov@xxxxxxx>
> Cc: Alex Deucher <alexander.deucher@xxxxxxx>
> Cc: Christian König <christian.koenig@xxxxxxx>
> Cc: James Zhu <James.Zhu@xxxxxxx>
> Cc: Leo Liu <leo.liu@xxxxxxx>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@xxxxxxx>
Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>
> ---
> drivers/gpu/drm/amd/amdgpu/jpeg_v4_0_3.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/jpeg_v4_0_3.c b/drivers/gpu/drm/amd/amdgpu/jpeg_v4_0_3.c
> index ede15a3a4701..ce2b22f7e4e4 100644
> --- a/drivers/gpu/drm/amd/amdgpu/jpeg_v4_0_3.c
> +++ b/drivers/gpu/drm/amd/amdgpu/jpeg_v4_0_3.c
> @@ -742,7 +742,7 @@ static void jpeg_v4_0_3_dec_ring_nop(struct amdgpu_ring *ring, uint32_t count)
> static bool jpeg_v4_0_3_is_idle(void *handle)
> {
> struct amdgpu_device *adev = (struct amdgpu_device *)handle;
> - bool ret;
> + bool ret = false;
> int i, j;
>
> for (i = 0; i < adev->jpeg.num_jpeg_inst; ++i) {
> @@ -764,7 +764,7 @@ static bool jpeg_v4_0_3_is_idle(void *handle)
> static int jpeg_v4_0_3_wait_for_idle(void *handle)
> {
> struct amdgpu_device *adev = (struct amdgpu_device *)handle;
> - int ret;
> + int ret = 0;
> int i, j;
>
> for (i = 0; i < adev->jpeg.num_jpeg_inst; ++i) {
> --
> 2.25.1
>
