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

[Magnus Damm]

On Fri, Aug 8, 2008 at 2:57 PM, Diego Biurrun <diego at biurrun.de> wrote:
> On Fri, Aug 08, 2008 at 12:02:15PM +0900, Magnus Damm wrote:
>> 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.
>
> Just enable it everywhere, then we might get bug reports for other
> platforms.

Ok, I missed that in V2. Please let me know if you still have issues
with the vidix driver itself and i'll cook up a V3.

Thanks,

/ magnus