On Mit, 2002-03-20 at 21:34, Guido Guenther wrote:

> +     {
> +
> +#if X_BYTE_ORDER == X_BIG_ENDIAN
> +       unsigned char *RADEONMMIO = info->MMIO;
> +     CARD32 surface_cntl;
> +
> +     surface_cntl = INREG(RADEON_SURFACE_CNTL);
> +     OUTREG(RADEON_SURFACE_CNTL, (surface_cntl | 
> +             RADEON_NONSURF_AP0_SWP_32BPP) & ~RADEON_NONSURF_AP0_SWP_16BPP);
> +#endif
>       RADEONCopyMungedData(buf + (top * srcPitch) + left, buf + s2offset,
>                          buf + s3offset, dst_start, srcPitch, srcPitch2,
>                          dstPitch, nlines, npixels);
> +#if X_BYTE_ORDER == X_BIG_ENDIAN
> +     /* restore byte swapping */
> +     OUTREG(RADEON_SURFACE_CNTL, surface_cntl);
> +#endif
> +     }
>       break;
>      case FOURCC_UYVY:
>      case FOURCC_YUY2:

Mr. Joshi doesn't like this even in r128, and I agree it would probably
be cleaner to set up a surface with byte swapping disabled for the
overlay. Benjamin Herrenschmidt may alreay have code for that.


-- 
Earthling Michel D�nzer (MrCooper)/ Debian GNU/Linux (powerpc) developer
XFree86 and DRI project member   /  CS student, Free Software enthusiast
_______________________________________________
Xpert mailing list
[EMAIL PROTECTED]
http://XFree86.Org/mailman/listinfo/xpert

Reply via email to