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: 78786a219e ("frontends/va: Fix deinterlace bottom field first flag")
Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/3621
Signed-off-by: Thong Thai <thong.thai@amd.com>
Reviewed-by: Leo Liu <leo.liu@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7194>
(cherry picked from commit 354e375c9c)
This commit is contained in:
Thong Thai 2020-10-15 15:32:47 -04:00 committed by Dylan Baker
parent a1f8f5e241
commit b9693aa851
2 changed files with 3 additions and 3 deletions

View file

@ -274,7 +274,7 @@
"description": "frontends/va/postproc: Un-break field flag",
"nominated": true,
"nomination_type": 1,
"resolution": 0,
"resolution": 1,
"master_sha": null,
"because_sha": "78786a219ea2322af09576472dcc2d6d01cb9060"
},

View file

@ -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: