[PATCH] drm/amdgpu: Couple small warning fixes

Deucher, Alexander Alexander.Deucher at amd.com
Tue Mar 28 15:43:06 UTC 2017


> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
> Of Harry Wentland
> Sent: Tuesday, March 28, 2017 11:32 AM
> To: amd-gfx at lists.freedesktop.org
> Cc: Wentland, Harry
> Subject: [PATCH] drm/amdgpu: Couple small warning fixes
> 
> Signed-off-by: Harry Wentland <harry.wentland at amd.com>

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

> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 3 ++-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c | 2 +-
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> index 355e17ac6d27..dde1ecf87288 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> @@ -904,11 +904,12 @@ static int amdgpu_cs_ib_fill(struct amdgpu_device
> *adev,
>  			continue;
> 
>  		if (chunk_ib->ip_type == AMDGPU_HW_IP_GFX &&
> amdgpu_sriov_vf(adev)) {
> -			if (chunk_ib->flags & AMDGPU_IB_FLAG_PREEMPT)
> +			if (chunk_ib->flags & AMDGPU_IB_FLAG_PREEMPT)
> {
>  				if (chunk_ib->flags & AMDGPU_IB_FLAG_CE)
>  					ce_preempt++;
>  				else
>  					de_preempt++;
> +			}
> 
>  			/* each GFX command submit allows 0 or 1 IB
> preemptible for CE & DE */
>  			if (ce_preempt > 1 || de_preempt > 1)
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
> index bcb939220806..e382eb818f7d 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
> @@ -172,7 +172,7 @@ int amdgpu_ib_schedule(struct amdgpu_ring *ring,
> unsigned num_ibs,
>  	if (ring->funcs->init_cond_exec)
>  		patch_offset = amdgpu_ring_init_cond_exec(ring);
> 
> -		if (ring->funcs->emit_hdp_flush
> +	if (ring->funcs->emit_hdp_flush
>  #ifdef CONFIG_X86_64
>  	    && !(adev->flags & AMD_IS_APU)
>  #endif
> --
> 2.11.0
> 
> _______________________________________________
> 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