[Libva] [PATCH 2/2] mpeg2: check frame_pred_frame_dct instead of progressive_frame

[Gwenole Beauchesne]

OK.

Small typo:
s/conformed to/conforming to/.

Thanks,
Gwenole.

2014-05-09 10:53 GMT+02:00 Xiang, Haihao <haihao.xiang at intel.com>:
> From: "Xiang, Haihao" <haihao.xiang at intel.com>
>
> Some MPEG-2 videos set progressive_frame to 1 and set
> frame_pred_frame_dct to 0, which is not conformed to MPEG-2 spec.
> bottom field may be used to form prediction if frame_pred_frame_dct is
> 0. Previously the bottom field is excluded from the frame store list
>
> https://bugs.freedesktop.org/show_bug.cgi?id=73424
>
> Signed-off-by: Xiang, Haihao <haihao.xiang at intel.com>
> ---
>  src/i965_decoder_utils.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/i965_decoder_utils.c b/src/i965_decoder_utils.c
> index d346cd3..ac4ae96 100644
> --- a/src/i965_decoder_utils.c
> +++ b/src/i965_decoder_utils.c
> @@ -139,7 +139,7 @@ mpeg2_set_reference_surfaces(
>          ref_frames[n++].surface_id = ref_frames[0].surface_id;
>      }
>
> -    if (pic_param->picture_coding_extension.bits.progressive_frame)
> +    if (pic_param->picture_coding_extension.bits.frame_pred_frame_dct)
>          return;
>
>      ref_frames[2].surface_id = VA_INVALID_ID;
> --
> 1.8.3.2
>
> _______________________________________________
> Libva mailing list
> Libva at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libva