[FFmpeg-devel] [PATCH] yadif port to libavfitler

[Måns Rullgård]

Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:

> On date Saturday 2010-09-04 12:45:08 +0200, Stefano Sabatini encoded:
> [...]
>> Updated against the av_get_cpu_flags() patch.
>> 
>> Still missing documentation (Baptiste feel free to continue to work on
>> this, I don't even know how did you test it).
>
> Updated again. Documentation for the second param should be extended,
> (Mplayer docs is outdated as well), also maybe ASM code should be
> placed in /ARCH dirs.
>
> I'll let someone else work on this though, so don't expect other
> updated patches from me.
>
> Regards.
> -- 
> FFmpeg = Fast and Forgiving Muttering Pure Embarassing Game
>
> From 119e161c3269ce6001b605556a7680248dfe0541 Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> Date: Sat, 4 Sep 2010 12:36:52 +0200
> Subject: [PATCH] Add yadif filter.
>
> Based on the Baptiste port from libmpcodecs.
> ---
>  configure                |    3 +
>  doc/filters.texi         |   29 +++
>  libavfilter/Makefile     |    1 +
>  libavfilter/allfilters.c |    1 +
>  libavfilter/vf_yadif.c   |  496 ++++++++++++++++++++++++++++++++++++++++++++++
>  5 files changed, 530 insertions(+), 0 deletions(-)
>  create mode 100644 libavfilter/vf_yadif.c
>
> diff --git a/configure b/configure
> index ae3d738..a03439e 100755
> --- a/configure
> +++ b/configure
> @@ -1390,6 +1390,9 @@ rtp_protocol_select="udp_protocol"
>  tcp_protocol_deps="network"
>  udp_protocol_deps="network"
>  
> +# filters
> +yadif_filter_deps="gpl"
> +
>  # libraries
>  avdevice_deps="avcodec avformat"
>  avformat_deps="avcodec"

configure part OK.

> +#if HAVE_MMX
> +
> +#define LOAD4(mem,dst) \
> +            "movd      "mem", "#dst" \n\t"\
> +            "punpcklbw %%mm7, "#dst" \n\t"
> +
> +#define PABS(tmp,dst) \
> +            "pxor     "#tmp", "#tmp" \n\t"\
> +            "psubw    "#dst", "#tmp" \n\t"\
> +            "pmaxsw   "#tmp", "#dst" \n\t"

Please, please, please move all asm into the x86/ subdir.

-- 
M?ns Rullg?rd
mans at mansr.com