[FFmpeg-devel] [PATCH] powerpc --> ppc
Sat Dec 27 14:15:44 CET 2008
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.
mans at mansr.com
More information about the ffmpeg-devel