Nice cleanups. I also tested these changes on "nouveau" driver with one
video per format (except HEVC) and works fine with mpv and gstreamer.
Patches "[PATCH 2/8] st/va: move post processing function into own file" to
"[PATCH 8/8] st/va: fix indentation"
are:
Reviewed-by: Julien Isorce <j.iso...@samsung.com>
On 23 November 2015 at 14:37, Christian König <deathsim...@vodafone.de>
wrote:
> From: Christian König <christian.koe...@amd.com>
>
> Signed-off-by: Christian König <christian.koe...@amd.com>
> ---
> src/gallium/state_trackers/va/picture.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/src/gallium/state_trackers/va/picture.c
> b/src/gallium/state_trackers/va/picture.c
> index ce75e2d..25d2940 100644
> --- a/src/gallium/state_trackers/va/picture.c
> +++ b/src/gallium/state_trackers/va/picture.c
> @@ -192,19 +192,19 @@ handleVASliceDataBufferType(vlVaContext *context,
> vlVaBuffer *buf)
> format = u_reduce_video_profile(context->decoder->profile);
> switch (format) {
> case PIPE_VIDEO_FORMAT_MPEG4_AVC:
> - if (bufHasStartcode(buf, 0x000001, 24))
> - break;
> + if (bufHasStartcode(buf, 0x000001, 24))
> + break;
>
> - buffers[num_buffers] = (void *const)&start_code_h264;
> - sizes[num_buffers++] = sizeof(start_code_h264);
> + buffers[num_buffers] = (void *const)&start_code_h264;
> + sizes[num_buffers++] = sizeof(start_code_h264);
> break;
> case PIPE_VIDEO_FORMAT_HEVC:
> if (bufHasStartcode(buf, 0x000001, 24))
> break;
>
> - buffers[num_buffers] = (void *const)&start_code_h265;
> - sizes[num_buffers++] = sizeof(start_code_h265);
> - break;
> + buffers[num_buffers] = (void *const)&start_code_h265;
> + sizes[num_buffers++] = sizeof(start_code_h265);
> + break;
> case PIPE_VIDEO_FORMAT_VC1:
> if (bufHasStartcode(buf, 0x0000010d, 32) ||
> bufHasStartcode(buf, 0x0000010c, 32) ||
> --
> 2.5.0
>
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
