Hi guys,

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

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

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

Cc: "10.6 11.0" <mesa-sta...@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
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to