On Mon, Mar 07, 2016 at 11:21:00PM +0000, Mark Thompson wrote:
> ---
>  libavcodec/h263dec.c     |  2 +-
>  libavcodec/h264_slice.c  |  2 +-
>  libavcodec/mpeg12dec.c   |  2 +-
>  libavcodec/vaapi_h264.c  |  2 +-
>  libavcodec/vaapi_mpeg2.c |  2 +-
>  libavcodec/vaapi_mpeg4.c |  4 ++--
>  libavcodec/vaapi_vc1.c   |  4 ++--
>  libavcodec/vc1dec.c      |  2 +-
>  libavutil/pixdesc.c      | 14 ++++++++++++++
>  libavutil/pixfmt.h       |  5 +++++
>  libavutil/version.h      |  3 +++
>  11 files changed, 32 insertions(+), 10 deletions(-)

You should update doc/APIchanges.

> --- a/libavutil/pixdesc.c
> +++ b/libavutil/pixdesc.c
> @@ -1093,6 +1093,7 @@ static const AVPixFmtDescriptor 
> av_pix_fmt_descriptors[AV_PIX_FMT_NB] = {
>          },
>          .flags = AV_PIX_FMT_FLAG_RGB,
>      },
> +#if FF_API_VAAPI
>      [AV_PIX_FMT_VAAPI_MOCO] = {
>          .name = "vaapi_moco",
>          .log2_chroma_w = 1,
> @@ -1111,6 +1112,14 @@ static const AVPixFmtDescriptor 
> av_pix_fmt_descriptors[AV_PIX_FMT_NB] = {
>          .log2_chroma_h = 1,
>          .flags = AV_PIX_FMT_FLAG_HWACCEL,
>      },
> +#else
> +    [AV_PIX_FMT_VAAPI] = {
> +        .name = "vaapi",
> +        .log2_chroma_w = 1,
> +        .log2_chroma_h = 1,
> +        .flags = AV_PIX_FMT_FLAG_HWACCEL,
> +    },
> +#endif
>      [AV_PIX_FMT_VDA_VLD] = {
>          .name = "vda_vld",
>          .log2_chroma_w = 1,
> @@ -1655,6 +1664,11 @@ enum AVPixelFormat av_get_pix_fmt(const char *name)
>          snprintf(name2, sizeof(name2), "%s%s", name, X_NE("be", "le"));
>          pix_fmt = get_pix_fmt_internal(name2);
>      }
> +
> +#if FF_API_VAAPI
> +    if (pix_fmt == AV_PIX_FMT_NONE && !strcmp(name, "vaapi"))
> +        pix_fmt = AV_PIX_FMT_VAAPI;
> +#endif
>      return pix_fmt;
>  }

Shouldn't you drop AV_PIX_FMT_VDA_VLD as well?

Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to