On Thu, 28 Jan 2016 12:58:25 +0100
Hendrik Leppkes <h.lepp...@gmail.com> wrote:
> ---
> libavcodec/hevc.c | 13 ++++++++++++-
> 1 file changed, 12 insertions(+), 1 deletion(-)
>
> diff --git a/libavcodec/hevc.c b/libavcodec/hevc.c
> index c245d3b..5677f0f 100644
> --- a/libavcodec/hevc.c
> +++ b/libavcodec/hevc.c
> @@ -346,7 +346,9 @@ static int set_sps(HEVCContext *s, const HEVCSPS *sps,
> enum AVPixelFormat pix_fm
>
> export_stream_params(s->avctx, &s->ps, sps);
>
> - if (sps->pix_fmt == AV_PIX_FMT_YUV420P || sps->pix_fmt ==
> AV_PIX_FMT_YUVJ420P) {
> + switch (sps->pix_fmt) {
> + case AV_PIX_FMT_YUV420P:
> + case AV_PIX_FMT_YUVJ420P:
> #if CONFIG_HEVC_DXVA2_HWACCEL
> *fmt++ = AV_PIX_FMT_DXVA2_VLD;
> #endif
> @@ -359,6 +361,15 @@ static int set_sps(HEVCContext *s, const HEVCSPS *sps,
> enum AVPixelFormat pix_fm
> #if CONFIG_HEVC_VDPAU_HWACCEL
> *fmt++ = AV_PIX_FMT_VDPAU;
> #endif
> + break;
> + case AV_PIX_FMT_YUV420P10:
> +#if CONFIG_HEVC_DXVA2_HWACCEL
> + *fmt++ = AV_PIX_FMT_DXVA2_VLD;
> +#endif
> +#if CONFIG_HEVC_D3D11VA_HWACCEL
> + *fmt++ = AV_PIX_FMT_D3D11VA_VLD;
> +#endif
> + break;
> }
>
> if (pix_fmt == AV_PIX_FMT_NONE) {
Patches 1-3 LGTM. There was a discussion whether there is a
compatibility problem with API users which don't check for the profile.
These would now attempt to decode 10 bit with a 8 bit decoder. On the
other hand, if you don't check for profiles with hw decoding, you're
bound to run into problems anyway.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
