[PATCH] drm/amdgpu: use the num_rings variable for checking vce rings

Christian König deathsimple at vodafone.de
Thu Jan 12 12:20:02 UTC 2017


Am 11.01.2017 um 22:13 schrieb Alex Deucher:
> Difference families may have different numbers of rings. Use
> the variable rather than a hardcoded number.
>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>

Reviewed-by: Christian König <christian.koenig at amd.com>.

> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> index 29d6d84..3ff2929 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> @@ -75,10 +75,10 @@ int amdgpu_cs_get_ring(struct amdgpu_device *adev, u32 ip_type,
>   		*out_ring = &adev->uvd.ring;
>   		break;
>   	case AMDGPU_HW_IP_VCE:
> -		if (ring < 2){
> +		if (ring < adev->vce.num_rings){
>   			*out_ring = &adev->vce.ring[ring];
>   		} else {
> -			DRM_ERROR("only two VCE rings are supported\n");
> +			DRM_ERROR("only %d VCE rings are supported\n", adev->vce.num_rings);
>   			return -EINVAL;
>   		}
>   		break;




More information about the amd-gfx mailing list