[MPlayer-dev-eng] [PATCHES] vidix: SuperH fixes, NV12 support, new vidix driver

Magnus Damm magnus.damm at gmail.com
Fri Aug 8 05:02:15 CEST 2008


On Thu, Aug 7, 2008 at 9:01 PM, Diego Biurrun <diego at biurrun.de> wrote:
> On Thu, Aug 07, 2008 at 08:21:10PM +0900, Magnus Damm wrote:
>>
>> Patch 1 - SuperH configure changes and vidix build fixes.
>> Vidix doesn't build on SuperH and the configure script doesn't support
>> sh4 and sh4a at the moment.
>>
>> --- 0001/vidix/AsmMacros.h
>> +++ work/vidix/AsmMacros.h    2008-08-07 19:29:08.000000000 +0900
>> @@ -77,6 +77,8 @@
>>  #include "sysdep/AsmMacros_arm32.h"
>>  #elif defined(__powerpc__)
>>  #include "sysdep/AsmMacros_powerpc.h"
>> +#elif defined(__sh__) && defined(linux)
>
> __linux__, not linux

Correct, good catch.

> But why does it have to be Linux-only?

I guess adding Linux support is good as a first step. If someone else
runs a different operating system then maybe they can adjust and test.
So for this, I'm basically doing the same thing as ia64 but without
dragging in the need for an extra header file.

Thanks!

/ magnus



More information about the MPlayer-dev-eng mailing list