[Mesa-dev] [PATCH 17/20] radeon/uvd: add YUYV format support for target buffer

[Christian König]

Am 18.08.2017 um 18:31 schrieb Leo Liu:
> YUYV is a packed YUV format, and there is no chorma plane

At better description would be something like: "Make chroma plane 
optional for YUYV support."
>
> Signed-off-by: Leo Liu <leo.liu at amd.com>

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

Regards,
Christian.

> ---
>   src/gallium/drivers/radeon/radeon_uvd.c | 6 ++++--
>   src/gallium/drivers/radeonsi/si_uvd.c   | 2 +-
>   2 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/src/gallium/drivers/radeon/radeon_uvd.c b/src/gallium/drivers/radeon/radeon_uvd.c
> index 9ebc31c170..648a493b59 100644
> --- a/src/gallium/drivers/radeon/radeon_uvd.c
> +++ b/src/gallium/drivers/radeon/radeon_uvd.c
> @@ -1566,10 +1566,12 @@ void ruvd_set_dt_surfaces(struct ruvd_msg *msg, struct radeon_surf *luma,
>   		}
>   
>   		msg->body.decode.dt_luma_top_offset = texture_offset(luma, 0, type);
> -		msg->body.decode.dt_chroma_top_offset = texture_offset(chroma, 0, type);
> +		if (chroma)
> +			msg->body.decode.dt_chroma_top_offset = texture_offset(chroma, 0, type);
>   		if (msg->body.decode.dt_field_mode) {
>   			msg->body.decode.dt_luma_bottom_offset = texture_offset(luma, 1, type);
> -			msg->body.decode.dt_chroma_bottom_offset = texture_offset(chroma, 1, type);
> +			if (chroma)
> +				msg->body.decode.dt_chroma_bottom_offset = texture_offset(chroma, 1, type);
>   		} else {
>   			msg->body.decode.dt_luma_bottom_offset = msg->body.decode.dt_luma_top_offset;
>   			msg->body.decode.dt_chroma_bottom_offset = msg->body.decode.dt_chroma_top_offset;
> diff --git a/src/gallium/drivers/radeonsi/si_uvd.c b/src/gallium/drivers/radeonsi/si_uvd.c
> index d17a6656a4..2441ad248c 100644
> --- a/src/gallium/drivers/radeonsi/si_uvd.c
> +++ b/src/gallium/drivers/radeonsi/si_uvd.c
> @@ -131,7 +131,7 @@ static struct pb_buffer* si_uvd_set_dtb(struct ruvd_msg *msg, struct vl_video_bu
>   
>   	msg->body.decode.dt_field_mode = buf->base.interlaced;
>   
> -	ruvd_set_dt_surfaces(msg, &luma->surface, &chroma->surface, type);
> +	ruvd_set_dt_surfaces(msg, &luma->surface, (chroma) ? &chroma->surface : NULL, type);
>   
>   	return luma->resource.buf;
>   }