[Mesa-dev] [PATCH 2/2] radeon/uvd: fix VC-1 simple/main profile decode

Emil Velikov emil.l.velikov at gmail.com
Thu Sep 24 05:04:53 PDT 2015


Hi guys,

A couple of suggestions - feel free to take or leave them.

On 23 September 2015 at 09:11, Christian König <deathsimple at vodafone.de> wrote:
> From: Boyuan Zhang <boyuan.zhang at amd.com>
>
A line or two lines of commit message would be appreciated.

> Signed-off-by: Boyuan Zhang <boyuan.zhang at amd.com>
> Reviewed-by: Christian König <christian.koenig at amd.com>
If you think that one can live with the intermittent glitches Grigori
mentioned, please throw in

Cc: "10.6 11.0" <mesa-stable at lists.freedesktop.org>

> ---
>  src/gallium/drivers/radeon/radeon_uvd.c   | 6 ++++++
>  src/gallium/drivers/radeon/radeon_video.c | 5 +----
>  2 files changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/src/gallium/drivers/radeon/radeon_uvd.c b/src/gallium/drivers/radeon/radeon_uvd.c
> index 81f3f45..9edb511 100644
> --- a/src/gallium/drivers/radeon/radeon_uvd.c
> +++ b/src/gallium/drivers/radeon/radeon_uvd.c
> @@ -940,6 +940,12 @@ static void ruvd_end_frame(struct pipe_video_codec *decoder,
>         dec->msg->body.decode.width_in_samples = dec->base.width;
>         dec->msg->body.decode.height_in_samples = dec->base.height;
>
> +       if ((picture->profile == PIPE_VIDEO_PROFILE_VC1_SIMPLE)
> +        || (picture->profile == PIPE_VIDEO_PROFILE_VC1_MAIN)) {
Too many brackets, place || on previous line (as elsewhere in the file)

Cheers,
Emil


More information about the mesa-dev mailing list