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

[Baptiste Coudurier]

On 5/25/10 3:31 PM, Michael Niedermayer wrote:
> On Mon, May 24, 2010 at 07:16:57PM -0700, Baptiste Coudurier wrote:
>> On 05/24/2010 04:18 AM, M?ns Rullg?rd wrote:
>>> Michael Niedermayer<michaelni at gmx.at>   writes:
>>>
>>>> On Sun, May 23, 2010 at 06:03:50PM -0700, Baptiste Coudurier wrote:
>>>>> On 5/23/10 4:22 AM, Michael Niedermayer wrote:
>>>>>> On Sun, May 23, 2010 at 04:10:57AM -0700, Baptiste Coudurier wrote:
>>>> [...]
>>>>>>> In short what's required before svn inclusion ?
>>>>>>
>>>>>> cpuid + matching md5 to mplayer
>>>>>> (id also like to see double frame output to work if possible but if
>>>>>> thats
>>>>>>     a problem we could look into it later)
>>>>>
>>>>> I'd go for later :)
>>>>>
>>>>>> and if fast_memcpy isnt ported we need a todo/fixme somewhere so we
>>>>>> dont
>>>>>> forget it
>>>>>
>>>>> md5 matches, cpuid is a bit complicated to port since MM_FLAGS are
>>>>> defined
>>>>> in avcodec.h
>>>>
>>>> where is the problem with moving them to some header in avutil ?
>>>
>>> Make that a new header only for cpuid stuff.
>>>
>>
>> All right, first attempt attached.
>>
>> --
>> Baptiste COUDURIER
>> Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
>> FFmpeg maintainer                                  http://www.ffmpeg.org
>
>>   libavcodec/avcodec.h    |    4 +++-
>>   libavcodec/dsputil.h    |    6 +-----
>>   libavcodec/x86/Makefile |    3 +--
>>   libavutil/Makefile      |    2 ++
>>   libavutil/cpuid.h       |   44 ++++++++++++++++++++++++++++++++++++++++++++
>>   libavutil/x86/cpuid.c   |    2 +-
>>   6 files changed, 52 insertions(+), 9 deletions(-)
>> c75422b037d027c7bdb9c2bbe859bdac4c438304  avutil_mm_support.patch
>> Index: libavcodec/avcodec.h
>> ===================================================================
>> --- libavcodec/avcodec.h	(revision 23288)
>> +++ libavcodec/avcodec.h	(working copy)
>> @@ -30,7 +30,7 @@
>>   #include "libavutil/avutil.h"
>>
>>   #define LIBAVCODEC_VERSION_MAJOR 52
>> -#define LIBAVCODEC_VERSION_MINOR 68
>> +#define LIBAVCODEC_VERSION_MINOR 69
>>   #define LIBAVCODEC_VERSION_MICRO  3
>
> micro should be zerod if minor is bumped
>
> remaining code ok with me

All right, updated.

yadif updated as well.
Issues are named args in asm
yadif should be compiled only when gpl is enabled.

-- 
Baptiste COUDURIER
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: avutil_mm_support.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100606/8abe340d/attachment.asc>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: yadif.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100606/8abe340d/attachment.txt>