[PATCH 1/6] drm/amdgpu: add VCN JPEG support amdgpu_ctx_num_entities

Christian K├Ânig ckoenig.leichtzumerken at gmail.com
Wed Nov 28 08:12:10 UTC 2018


Maybe drop patch #2 when you refactor that code in patch #3 and #4 anyway.

On patch #3 I would put the new helper into amdgpu_ctx.c instead because 
that is rather how rings map to userspace queues.

Apart from that it looks good to me.

Christian.

Am 27.11.18 um 22:10 schrieb Alex Deucher:
> Looks like it was missed when setting support was added.
>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c
> index f9b54236102d..95f4c4139fc6 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c
> @@ -39,6 +39,7 @@ const unsigned int amdgpu_ctx_num_entities[AMDGPU_HW_IP_NUM] = {
>   	[AMDGPU_HW_IP_UVD_ENC]	=	1,
>   	[AMDGPU_HW_IP_VCN_DEC]	=	1,
>   	[AMDGPU_HW_IP_VCN_ENC]	=	1,
> +	[AMDGPU_HW_IP_VCN_JPEG]	=	1,
>   };
>   
>   static int amdgput_ctx_total_num_entities(void)



More information about the amd-gfx mailing list