[4/9] drm/amdgpu: add uvd enc rings

[Christian König]

Am 02.10.2017 um 16:13 schrieb James Zhu:
> UVD 6.3 has two UVD encode rings.  Add the ring structures and initialize the hw ring buffers.
> Currently only ASIC Polaris10/11/12 uses UVD6.3 encode engine on HEVC encoding.
>
> Signed-off-by: James Zhu <James.Zhu at amd.com>
> ---
>   drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c | 44 +++++++++++++++++++++++++++++++++--
>   1 file changed, 42 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c b/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
> index d642915..f4f56c9 100644
> --- a/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
> @@ -47,6 +47,8 @@ static int uvd_v6_0_set_clockgating_state(void *handle,
>   static void uvd_v6_0_enable_mgcg(struct amdgpu_device *adev,
>   				 bool enable);
>   
> +#define uvd_v6_0_enc_support(x) ((x->asic_type >= CHIP_POLARIS10) && (x->asic_type <= CHIP_POLARIS12))

Either make this a function are use a capital name for the macro. I 
prefer to define a local function for this.

Regards,
Christian.

> +
>   /**
>    * uvd_v6_0_ring_get_rptr - get read pointer
>    *
> @@ -146,6 +148,11 @@ static int uvd_v6_0_early_init(void *handle)
>   	struct amdgpu_device *adev = (struct amdgpu_device *)handle;
>   
>   	uvd_v6_0_set_ring_funcs(adev);
> +
> +	if (uvd_v6_0_enc_support(adev)) {
> +		adev->uvd.num_enc_rings = 2;
> +	}
> +
>   	uvd_v6_0_set_irq_funcs(adev);
>   
>   	return 0;
> @@ -154,7 +161,7 @@ static int uvd_v6_0_early_init(void *handle)
>   static int uvd_v6_0_sw_init(void *handle)
>   {
>   	struct amdgpu_ring *ring;
> -	int r;
> +	int i, r;
>   	struct amdgpu_device *adev = (struct amdgpu_device *)handle;
>   
>   	/* UVD TRAP */
> @@ -173,19 +180,36 @@ static int uvd_v6_0_sw_init(void *handle)
>   	ring = &adev->uvd.ring;
>   	sprintf(ring->name, "uvd");
>   	r = amdgpu_ring_init(adev, ring, 512, &adev->uvd.irq, 0);
> +	if (r)
> +		return r;
> +
> +	if (uvd_v6_0_enc_support(adev)) {
> +		for (i = 0; i < adev->uvd.num_enc_rings; ++i) {
> +			ring = &adev->uvd.ring_enc[i];
> +			sprintf(ring->name, "uvd_enc%d", i);
> +			r = amdgpu_ring_init(adev, ring, 512, &adev->uvd.irq, 0);
> +			if (r)
> +				return r;
> +		}
> +	}
>   
>   	return r;
>   }
>   
>   static int uvd_v6_0_sw_fini(void *handle)
>   {
> -	int r;
> +	int i, r;
>   	struct amdgpu_device *adev = (struct amdgpu_device *)handle;
>   
>   	r = amdgpu_uvd_suspend(adev);
>   	if (r)
>   		return r;
>   
> +	if (uvd_v6_0_enc_support(adev)) {
> +		for (i = 0; i < adev->uvd.num_enc_rings; ++i)
> +			amdgpu_ring_fini(&adev->uvd.ring_enc[i]);
> +	}
> +
>   	return amdgpu_uvd_sw_fini(adev);
>   }
>   
> @@ -566,6 +590,22 @@ static int uvd_v6_0_start(struct amdgpu_device *adev)
>   
>   	WREG32_FIELD(UVD_RBC_RB_CNTL, RB_NO_FETCH, 0);
>   
> +	if (uvd_v6_0_enc_support(adev)) {
> +		ring = &adev->uvd.ring_enc[0];
> +		WREG32(mmUVD_RB_RPTR, lower_32_bits(ring->wptr));
> +		WREG32(mmUVD_RB_WPTR, lower_32_bits(ring->wptr));
> +		WREG32(mmUVD_RB_BASE_LO, ring->gpu_addr);
> +		WREG32(mmUVD_RB_BASE_HI, upper_32_bits(ring->gpu_addr));
> +		WREG32(mmUVD_RB_SIZE, ring->ring_size / 4);
> +
> +		ring = &adev->uvd.ring_enc[1];
> +		WREG32(mmUVD_RB_RPTR2, lower_32_bits(ring->wptr));
> +		WREG32(mmUVD_RB_WPTR2, lower_32_bits(ring->wptr));
> +		WREG32(mmUVD_RB_BASE_LO2, ring->gpu_addr);
> +		WREG32(mmUVD_RB_BASE_HI2, upper_32_bits(ring->gpu_addr));
> +		WREG32(mmUVD_RB_SIZE2, ring->ring_size / 4);
> +	}
> +
>   	return 0;
>   }
>