[MPlayer-dev-eng] [PATCH] partial bigendian support for nvidia_vid

Aurelien Jacobs aurel at gnuage.org
Mon Oct 24 23:12:18 CEST 2005


On Mon, 24 Oct 2005 17:26:37 +0200
Sascha Sommer <saschasommer at freenet.de> wrote:

> Hi,
> 
> On Monday 24 October 2005 14:27, Diego Biurrun wrote:
> > Sascha,
> >
> > this removes to le2me_32 macros from nvidia_vid.c since apparently
> > the values are already in machine endian order.  It's not enough for
> > a complete bigendian port, but it is a necessary step and simplifies
> > the code somewhat.
> >
> > OK to commit?  You wrote this yourself, so I assume it is safe, but
> > I cannot test on x86, so I'd love to hear a quick go ahead for this.
> >
> 
> It should be safe. However I'm not sure if all cards that also work on
> PPC can  swap the bytes automatically and I also do not know if the
> swapping is  required. It might be possible that depending on the (X)
> driver the bytes are  swapped or not swapped. Maybe we can keep the
> current versions of the macros  under #if 0 for now.

I don't know if all nvidia cards are able to swap bytes, but anyway, to
do so, a driver as to enable this feature.
And IIRC, nvfb, rivafb and the nv X driver have a different behavior.
So one will enable automatic byte swap and the other won't. That's only
a vague remembering so I may be wrong, but you should at least test your
patch either with the framebuffer and with X.

Aurel




More information about the MPlayer-dev-eng mailing list