[FFmpeg-devel] [PATCH 3/3] ffmpeg_dxva2: add a profile check for hevc
Hendrik Leppkes
h.leppkes at gmail.com
Sat Jan 30 17:03:39 CET 2016
On Thu, Jan 28, 2016 at 3:52 PM, Michael Niedermayer
<michael at niedermayer.cc> wrote:
> On Thu, Jan 28, 2016 at 12:58:27PM +0100, Hendrik Leppkes wrote:
>> ---
>> ffmpeg_dxva2.c | 6 ++++++
>> 1 file changed, 6 insertions(+)
>>
>> diff --git a/ffmpeg_dxva2.c b/ffmpeg_dxva2.c
>> index 1b62bb9..35bedec 100644
>> --- a/ffmpeg_dxva2.c
>> +++ b/ffmpeg_dxva2.c
>> @@ -637,6 +637,12 @@ int dxva2_init(AVCodecContext *s)
>> return AVERROR(EINVAL);
>> }
>>
>> + if (s->codec_id == AV_CODEC_ID_HEVC &&
>> + s->profile != FF_PROFILE_HEVC_MAIN && s->profile != FF_PROFILE_HEVC_MAIN_10) {
>> + av_log(NULL, loglevel, "Unsupported HEVC profile for DXVA2 HWAccel: %d\n", s->profile);
> ^^^^
> this should be some context if possible
>
> otherwise LGTM
>
The entire file doesn't use contexts for some reason, I would rather
keep it like this in this patch and then send a follow up to make all
the log commands in that file use contexts.
- Hendrik
More information about the ffmpeg-devel
mailing list