[FFmpeg-devel] [PATCH] lavfi: Add VAAPI deinterlacer

[Mark Thompson]

On 09/01/17 00:29, Michael Niedermayer wrote:
> On Sun, Jan 08, 2017 at 07:12:47PM +0000, Mark Thompson wrote:
>> (cherry picked from commit ade370a4d7eab1866b6023c91c135d27c77ca465)
>> ---
>> One minor fixup for allocation due to differences in the lavfis, otherwise unchanged.
> [...]
>> diff --git a/libavfilter/vf_deinterlace_vaapi.c b/libavfilter/vf_deinterlace_vaapi.c
>> new file mode 100644
>> index 0000000000..2576cf95a9
>> --- /dev/null
>> +++ b/libavfilter/vf_deinterlace_vaapi.c
> [...]
>> +#define OFFSET(x) offsetof(DeintVAAPIContext, x)
> 
>> +#define FLAGS (AV_OPT_FLAG_VIDEO_PARAM)
> 
> #define FLAGS AV_OPT_FLAG_VIDEO_PARAM|AV_OPT_FLAG_FILTERING_PARAM
> 
On 09/01/17 00:27, Michael Niedermayer wrote:
> On Sun, Jan 08, 2017 at 07:12:47PM +0000, Mark Thompson wrote:
>> (cherry picked from commit ade370a4d7eab1866b6023c91c135d27c77ca465)
>> ---
>> One minor fixup for allocation due to differences in the lavfis, otherwise unchanged.
> [...]
>> diff --git a/libavfilter/vf_deinterlace_vaapi.c b/libavfilter/vf_deinterlace_vaapi.c
>> new file mode 100644
>> index 0000000000..2576cf95a9
>> --- /dev/null
>> +++ b/libavfilter/vf_deinterlace_vaapi.c
> [...]
>> +    params.surface_color_standard =
>> +        vaapi_proc_colour_standard(input_frame->colorspace);
> 
> AVFrame->colorspace is documented as "
>      * It must be accessed using av_frame_get_colorspace() and
>      * av_frame_set_colorspace().
> "
> 
> This is the API until the API is changed ...

Both fixed locally, along with the warning about ff_formats_ref() (unhelpfully it doesn't have a return value in libav).

Thanks,

- Mark