[PATCH] drm/amdgpu: Cleanup 2 compiler warnings

Alex Deucher alexdeucher at gmail.com
Thu Jan 3 14:14:32 UTC 2019


On Thu, Jan 3, 2019 at 8:37 AM Russell, Kent <Kent.Russell at amd.com> wrote:
>
> These 2 variables are unused now, so remove their references.
>
> Fixes: e4ae0fc drm/amdgpu: implement gfx8 post_soft_reset
> Fixes: 5e01c09 drm/amdgpu/gfx_v8_0: Reorder the gfx, kiq and kcq rings
> test sequence
>
> Change-Id: Ic3f39c48b8deef2ff81589c31ce1cc24573b47eb
> Signed-off-by: Kent Russell <kent.russell at amd.com>

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> index 164ffc9..57cb3a5 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> @@ -4233,7 +4233,6 @@ static int gfx_v8_0_cp_gfx_resume(struct amdgpu_device *adev)
>         u32 tmp;
>         u32 rb_bufsz;
>         u64 rb_addr, rptr_addr, wptr_gpu_addr;
> -       int r;
>
>         /* Set the write pointer delay */
>         WREG32(mmCP_RB_WPTR_DELAY, 0);
> @@ -5075,7 +5074,6 @@ static int gfx_v8_0_post_soft_reset(void *handle)
>  {
>         struct amdgpu_device *adev = (struct amdgpu_device *)handle;
>         u32 grbm_soft_reset = 0;
> -       struct amdgpu_ring *ring;
>
>         if ((!adev->gfx.grbm_soft_reset) &&
>             (!adev->gfx.srbm_soft_reset))
> --
> 2.7.4
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


More information about the amd-gfx mailing list