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

[Diego Biurrun]

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.

Diego