From 354e375c9cc937ecfafd7b98a768621c53458528 Mon Sep 17 00:00:00 2001 From: Thong Thai Date: Thu, 15 Oct 2020 15:32:47 -0400 Subject: [PATCH] frontends/va/postproc: Un-break field flag Fixes an issue where deinterlaced videos would play at half the framerate, since only one field was repeated, instead of using both fields. Reverts a change I made previously which broke this. Fixes: 78786a219ea ("frontends/va: Fix deinterlace bottom field first flag") Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/3621 Signed-off-by: Thong Thai Reviewed-by: Leo Liu Part-of: --- src/gallium/frontends/va/postproc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/frontends/va/postproc.c b/src/gallium/frontends/va/postproc.c index bd1228b..1845883 100644 --- a/src/gallium/frontends/va/postproc.c +++ b/src/gallium/frontends/va/postproc.c @@ -321,7 +321,7 @@ vlVaHandleVAProcPipelineParameterBufferType(vlVaDriver *drv, vlVaContext *contex VAProcFilterParameterBufferDeinterlacing *deint = buf->data; switch (deint->algorithm) { case VAProcDeinterlacingBob: - if (deint->flags & VA_DEINTERLACING_BOTTOM_FIELD_FIRST) + if (deint->flags & VA_DEINTERLACING_BOTTOM_FIELD) deinterlace = VL_COMPOSITOR_BOB_BOTTOM; else deinterlace = VL_COMPOSITOR_BOB_TOP; @@ -333,7 +333,7 @@ vlVaHandleVAProcPipelineParameterBufferType(vlVaDriver *drv, vlVaContext *contex case VAProcDeinterlacingMotionAdaptive: src = vlVaApplyDeint(drv, context, param, src, - !!(deint->flags & VA_DEINTERLACING_BOTTOM_FIELD_FIRST)); + !!(deint->flags & VA_DEINTERLACING_BOTTOM_FIELD)); break; default: -- 2.7.4