On Wed, 2010-12-08 at 08:50 +0000, Alan Hourihane wrote: > On Tue, 2010-12-07 at 23:20 +0000, Matt Turner wrote: > > On Tue, Dec 7, 2010 at 8:07 PM, Mark Kettenis <[email protected]> > > wrote: > > > No need for byteswapping in YV12 decoding on BE machines > > > > > > The hardware seems to do the proper thing already, so always use the same > > > code on both little-endian and big-endian machines. Fixes xv YV12 on a > > > TechSource Raptor GFX aka Sun PGX32. > > > > > > Signed-off-by: Mark Kettenis <[email protected]> > > > > Nice! Good catch. > > > > I suppose this fixes the problem you discovered in the other thread? > > > > I'll commit this. > > Might want to check the hardware manuals on this, but there might be a > byteswap bit someplace that does it in hardware. The PGX32 may do this > at BIOS init time, whereas a PC version may not. There is a bit for this > on the PM3, so there maybe something on PM2. > > Just thinking if you drop a PC card into a BE machine will it still > work ?
Oh, if I remember correctly this code was added when Amiga VisionPPC board support was added. So it must have been relevant at some point for BE machines. Alan. _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
