[FFmpeg-devel] [PATCH] powerpc --> ppc

[Måns Rullgård]

Diego Biurrun <diego at biurrun.de> writes:

> On Sat, Dec 27, 2008 at 11:55:33AM +0100, Luca Barbato wrote:
>> Diego Biurrun wrote:
>> >>> Another thing I noticed: What is ARCH_POWERPC_405?  It is never set...
>> >> This was added by this _very old_ commit:
>> >>
>> >> commit 45c7805311002362b42e98a664419bfe8ff23931
>> >> Author: bellard <bellard at 9553f0bf-9b14-0410-a0b8-cfaf0461ba5b>
>> >> Date:   Wed Jun 5 14:25:06 2002 +0000
>> >>
>> >>     removed unused code - began to merge alpha specific stuff - added
>> >> mac macros for suitable CPUs
>> >>
>> >>     git-svn-id: file:///var/local/repositories/ffmpeg/trunk at 667
>> >> 9553f0bf-9b14-0410-a0b8-cfaf0461ba5b
>> > 
>> > That's just the one in simple_idct.c, the one in ppc/mathops.h is from
>> > September 2006.
>> > 
>> >> I don't think there's many people who actively use PowerPC 405 besides
>> >> people who hack their DreamBox (http://en.wikipedia.org/wiki/Dreambox)
>> >> and students (since the core is freely available), so I don't care if
>> >> ever you want to remove the core relative to it.
>> > 
>> > Luca, what do you say?
>> 
>> That the simple_idct.c should be cleaned and the code should remain 
>> since there are many ppc4xx, some pretty much recent, and are used on 
>> stb IIRC.
>> 
>> We could rename the symbol to PPC4XX and update the configure 
>> accordingly IF there are people interested (if anybody is using them 
>> please tell)
>
> The important part is that one of either really happens.  We should
> not be keeping cruft around.

I'll see about getting my hardware running, but not before January.

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