r300: Last of the indent changes. :)
[mesa.git] / src / mesa / drivers / dri / r300 / radeon_span.c
index 0ab6b9b7e0caf9ce0087dfe5e550a3171e138d55..cc779d684fbd173f92a49bfcecb5d6335fa237bc 100644 (file)
@@ -1,10 +1,15 @@
-/*
+/**************************************************************************
+
 Copyright (C) The Weather Channel, Inc.  2002.  All Rights Reserved.
+Copyright 2000, 2001 ATI Technologies Inc., Ontario, Canada, and
+                     VA Linux Systems Inc., Fremont, California.
 
 The Weather Channel (TM) funded Tungsten Graphics to develop the
 initial release of the Radeon 8500 driver under the XFree86 license.
 This notice must be preserved.
 
+All Rights Reserved.
+
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the
 "Software"), to deal in the Software without restriction, including
@@ -29,147 +34,84 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
 /*
  * Authors:
+ *   Kevin E. Martin <martin@valinux.com>
+ *   Gareth Hughes <gareth@valinux.com>
  *   Keith Whitwell <keith@tungstengraphics.com>
+ *
  */
-
+#include <unistd.h>
 #include "glheader.h"
 #include "imports.h"
 #include "swrast/swrast.h"
-#include "colormac.h"
 
-#include "r200_context.h"
+#include "r300_state.h"
 #include "radeon_ioctl.h"
 #include "r300_ioctl.h"
 #include "radeon_span.h"
 
+#include "drirenderbuffer.h"
+
+
 #define DBG 0
 
-#define LOCAL_VARS                                                     \
-   radeonContextPtr radeon = RADEON_CONTEXT(ctx);                      \
-   radeonScreenPtr radeonScreen = radeon->radeonScreen;                        \
-   __DRIscreenPrivate *sPriv = radeon->dri.screen;                     \
-   __DRIdrawablePrivate *dPriv = radeon->dri.drawable;                 \
-   GLuint pitch = radeonScreen->frontPitch * radeonScreen->cpp;                \
-   GLuint height = dPriv->h;                                           \
-   char *buf = (char *)(sPriv->pFB +                                   \
-                       radeon->state.color.drawOffset +                \
-                       (dPriv->x * radeonScreen->cpp) +                \
-                       (dPriv->y * pitch));                            \
-   char *read_buf = (char *)(sPriv->pFB +                              \
-                            radeon->state.pixel.readOffset +           \
-                            (dPriv->x * radeonScreen->cpp) +           \
-                            (dPriv->y * pitch));                       \
-   GLuint p;                                                           \
-   (void) read_buf; (void) buf; (void) p
-
-#define LOCAL_DEPTH_VARS                                               \
-   radeonContextPtr radeon = RADEON_CONTEXT(ctx);                      \
-   radeonScreenPtr radeonScreen = radeon->radeonScreen;                        \
-   __DRIscreenPrivate *sPriv = radeon->dri.screen;                     \
-   __DRIdrawablePrivate *dPriv = radeon->dri.drawable;                 \
-   GLuint height = dPriv->h;                                           \
-   GLuint xo = dPriv->x;                                               \
-   GLuint yo = dPriv->y;                                               \
-   char *buf = (char *)(sPriv->pFB + radeonScreen->depthOffset);       \
-   GLuint pitch = radeonScreen->depthPitch;                            \
-   (void) buf; (void) pitch
-
-#define LOCAL_STENCIL_VARS     LOCAL_DEPTH_VARS
-
-#define CLIPPIXEL( _x, _y )                                            \
-   ((_x >= minx) && (_x < maxx) && (_y >= miny) && (_y < maxy))
-
-#define CLIPSPAN( _x, _y, _n, _x1, _n1, _i )                           \
-   if ( _y < miny || _y >= maxy ) {                                    \
-      _n1 = 0, _x1 = x;                                                        \
-   } else {                                                            \
-      _n1 = _n;                                                                \
-      _x1 = _x;                                                                \
-      if ( _x1 < minx ) _i += (minx-_x1), n1 -= (minx-_x1), _x1 = minx; \
-      if ( _x1 + _n1 >= maxx ) n1 -= (_x1 + n1 - maxx);                        \
-   }
 
-#define Y_FLIP( _y )           (height - _y - 1)
+/*
+ * Note that all information needed to access pixels in a renderbuffer
+ * should be obtained through the gl_renderbuffer parameter, not per-context
+ * information.
+ */
+#define LOCAL_VARS                                             \
+   driRenderbuffer *drb = (driRenderbuffer *) rb;              \
+   const __DRIdrawablePrivate *dPriv = drb->dPriv;             \
+   const GLuint bottom = dPriv->h - 1;                         \
+   GLubyte *buf = (GLubyte *) drb->flippedData                 \
+      + (dPriv->y * drb->flippedPitch + dPriv->x) * drb->cpp;  \
+   GLuint p;                                                   \
+   (void) p;
+
+#define LOCAL_DEPTH_VARS                               \
+   driRenderbuffer *drb = (driRenderbuffer *) rb;      \
+   const __DRIdrawablePrivate *dPriv = drb->dPriv;     \
+   const GLuint bottom = dPriv->h - 1;                 \
+   GLuint xo = dPriv->x;                               \
+   GLuint yo = dPriv->y;                               \
+   GLubyte *buf = (GLubyte *) drb->Base.Data;
+
+#define LOCAL_STENCIL_VARS LOCAL_DEPTH_VARS
+
+#define Y_FLIP(Y) (bottom - (Y))
 
 #define HW_LOCK()
 
-#define HW_CLIPLOOP()                                                  \
-   do {                                                                        \
-      __DRIdrawablePrivate *dPriv = radeon->dri.drawable;              \
-      int _nc = dPriv->numClipRects;                                   \
-                                                                       \
-      while ( _nc-- ) {                                                        \
-        int minx = dPriv->pClipRects[_nc].x1 - dPriv->x;               \
-        int miny = dPriv->pClipRects[_nc].y1 - dPriv->y;               \
-        int maxx = dPriv->pClipRects[_nc].x2 - dPriv->x;               \
-        int maxy = dPriv->pClipRects[_nc].y2 - dPriv->y;
-
-#define HW_ENDCLIPLOOP()                                               \
-      }                                                                        \
-   } while (0)
-
 #define HW_UNLOCK()
 
+
+
 /* ================================================================
  * Color buffer
  */
 
 /* 16 bit, RGB565 color spanline and pixel functions
  */
-#define INIT_MONO_PIXEL(p, color) \
-  p = PACK_COLOR_565( color[0], color[1], color[2] )
+#define SPANTMP_PIXEL_FMT GL_RGB
+#define SPANTMP_PIXEL_TYPE GL_UNSIGNED_SHORT_5_6_5
 
-#define WRITE_RGBA( _x, _y, r, g, b, a )                               \
-   *(GLushort *)(buf + _x*2 + _y*pitch) = ((((int)r & 0xf8) << 8) |    \
-                                          (((int)g & 0xfc) << 3) |     \
-                                          (((int)b & 0xf8) >> 3))
+#define TAG(x)    radeon##x##_RGB565
+#define TAG2(x,y) radeon##x##_RGB565##y
+#define GET_PTR(X,Y) (buf + ((Y) * drb->flippedPitch + (X)) * 2)
+#include "spantmp2.h"
 
-#define WRITE_PIXEL( _x, _y, p )                                       \
-   *(GLushort *)(buf + _x*2 + _y*pitch) = p
-
-#define READ_RGBA( rgba, _x, _y )                                      \
-   do {                                                                        \
-      GLushort p = *(GLushort *)(read_buf + _x*2 + _y*pitch);          \
-      rgba[0] = ((p >> 8) & 0xf8) * 255 / 0xf8;                                \
-      rgba[1] = ((p >> 3) & 0xfc) * 255 / 0xfc;                                \
-      rgba[2] = ((p << 3) & 0xf8) * 255 / 0xf8;                                \
-      rgba[3] = 0xff;                                                  \
-   } while (0)
-
-#define TAG(x) radeon##x##_RGB565
-#include "spantmp.h"
 
 /* 32 bit, ARGB8888 color spanline and pixel functions
  */
-#undef INIT_MONO_PIXEL
-#define INIT_MONO_PIXEL(p, color) \
-  p = PACK_COLOR_8888( color[3], color[0], color[1], color[2] )
-
-#define WRITE_RGBA( _x, _y, r, g, b, a )                       \
-do {                                                           \
-   *(GLuint *)(buf + _x*4 + _y*pitch) = ((b <<  0) |           \
-                                        (g <<  8) |            \
-                                        (r << 16) |            \
-                                        (a << 24) );           \
-} while (0)
+#define SPANTMP_PIXEL_FMT GL_BGRA
+#define SPANTMP_PIXEL_TYPE GL_UNSIGNED_INT_8_8_8_8_REV
 
-#define WRITE_PIXEL( _x, _y, p )                       \
-do {                                                   \
-   *(GLuint *)(buf + _x*4 + _y*pitch) = p;             \
-} while (0)
+#define TAG(x)    radeon##x##_ARGB8888
+#define TAG2(x,y) radeon##x##_ARGB8888##y
+#define GET_PTR(X,Y) (buf + ((Y) * drb->flippedPitch + (X)) * 4)
+#include "spantmp2.h"
 
-#define READ_RGBA( rgba, _x, _y )                              \
-do {                                                           \
-   volatile GLuint *ptr = (volatile GLuint *)(read_buf + _x*4 + _y*pitch); \
-   GLuint p = *ptr;                                    \
-   rgba[0] = (p >> 16) & 0xff;                                 \
-   rgba[1] = (p >>  8) & 0xff;                                 \
-   rgba[2] = (p >>  0) & 0xff;                                 \
-   rgba[3] = (p >> 24) & 0xff;                                 \
-} while (0)
-
-#define TAG(x) radeon##x##_ARGB8888
-#include "spantmp.h"
 
 /* ================================================================
  * Depth buffer
@@ -180,82 +122,76 @@ do {                                                              \
  * manner as the engine.  In each case, the linear block address (ba)
  * is calculated, and then wired with x and y to produce the final
  * memory address.
+ * The chip will do address translation on its own if the surface registers
+ * are set up correctly. It is not quite enough to get it working with hyperz
+ * too...
  */
 
-#define BIT(x,b) ((x & (1<<b))>>b)
-static GLuint radeon_mba_z32(radeonContextPtr radeon, GLint x, GLint y)
+static GLuint
+radeon_mba_z32( const driRenderbuffer *drb, GLint x, GLint y )
 {
-       GLuint pitch = radeon->radeonScreen->depthPitch;
-       GLuint b =
-           ((y & 0x3FF) >> 4) * ((pitch & 0xFFF) >> 5) + ((x & 0x3FF) >> 5);
-       GLuint a =
-           (BIT(x, 0) << 2) | (BIT(y, 0) << 3) | (BIT(x, 1) << 4) | (BIT(y, 1)
-                                                                     << 5) |
-           (BIT(x, 3) << 6) | (BIT(x, 4) << 7) | (BIT(x, 2) << 8) | (BIT(y, 2)
-                                                                     << 9) |
-           (BIT(y, 3) << 10) |
-           (((pitch & 0x20) ? (b & 0x01) : ((b & 0x01) ^ (BIT(y, 4)))) << 11) |
-           ((b >> 1) << 12);
-       return a;
-}
+   GLuint pitch = drb->pitch;
+   if (1 /*|| drb->depthHasSurface */) {
+      return 4 * (x + y * pitch);
+   }
+   else {
+      GLuint ba, address = 0;                  /* a[0..1] = 0           */
 
-static GLuint radeon_mba_z16(radeonContextPtr radeon, GLint x, GLint y)
-{
-       GLuint pitch = radeon->radeonScreen->depthPitch;
-       GLuint b =
-           ((y & 0x3FF) >> 4) * ((pitch & 0xFFF) >> 6) + ((x & 0x3FF) >> 6);
-       GLuint a =
-           (BIT(x, 0) << 1) | (BIT(y, 0) << 2) | (BIT(x, 1) << 3) | (BIT(y, 1)
-                                                                     << 4) |
-           (BIT(x, 2) << 5) | (BIT(x, 4) << 6) | (BIT(x, 5) << 7) | (BIT(x, 3)
-                                                                     << 8) |
-           (BIT(y, 2) << 9) | (BIT(y, 3) << 10) |
-           (((pitch & 0x40) ? (b & 0x01) : ((b & 0x01) ^ (BIT(y, 4)))) << 11) |
-           ((b >> 1) << 12);
-       return a;
-}
+      ba = (y / 8) * (pitch / 8) + (x / 8);
 
+      address |= (x & 0x7) << 2;               /* a[2..4] = x[0..2]     */
+      address |= (y & 0x3) << 5;               /* a[5..6] = y[0..1]     */
+      address |=
+         (((x & 0x10) >> 2) ^ (y & 0x4)) << 5; /* a[7]    = x[4] ^ y[2] */
+      address |= (ba & 0x3) << 8;              /* a[8..9] = ba[0..1]    */
 
-/* 16-bit depth buffer functions
- */
-#define WRITE_DEPTH( _x, _y, d )                                       \
-   *(GLushort *)(buf + radeon_mba_z16( radeon, _x + xo, _y + yo )) = d;
+      address |= (y & 0x8) << 7;               /* a[10]   = y[3]        */
+      address |=
+         (((x & 0x8) << 1) ^ (y & 0x10)) << 7; /* a[11]   = x[3] ^ y[4] */
+      address |= (ba & ~0x3) << 10;            /* a[12..] = ba[2..]     */
 
-#define READ_DEPTH( d, _x, _y )                                                \
-   d = *(GLushort *)(buf + radeon_mba_z16( radeon, _x + xo, _y + yo ));
+      return address;
+   }
+}
 
-#define TAG(x) radeon##x##_16_TILE
-#include "depthtmp.h"
 
-/* 24 bit depth, 8 bit stencil depthbuffer functions
- */
-#define WRITE_DEPTH( _x, _y, d )                                       \
-do {                                                                   \
-   GLuint offset = radeon_mba_z32( radeon, _x + xo, _y + yo );         \
-   GLuint tmp = *(GLuint *)(buf + offset);                             \
-   tmp &= 0xff000000;                                                  \
-   tmp |= ((d) & 0x00ffffff);                                          \
-   *(GLuint *)(buf + offset) = tmp;                                    \
-} while (0)
+static INLINE GLuint
+radeon_mba_z16( const driRenderbuffer *drb, GLint x, GLint y )
+{
+   GLuint pitch = drb->pitch;
+   if (1 /*|| drb->depthHasSurface */) {
+      return 2 * (x + y * pitch);
+   }
+   else {
+      GLuint ba, address = 0;                  /* a[0]    = 0           */
 
-#define READ_DEPTH( d, _x, _y )                                                \
-   d = *(GLuint *)(buf + radeon_mba_z32( radeon, _x + xo,              \
-                                        _y + yo )) & 0x00ffffff;
+      ba = (y / 16) * (pitch / 32) + (x / 32);
+
+      address |= (x & 0x7) << 1;               /* a[1..3] = x[0..2]     */
+      address |= (y & 0x7) << 4;               /* a[4..6] = y[0..2]     */
+      address |= (x & 0x8) << 4;               /* a[7]    = x[3]        */
+      address |= (ba & 0x3) << 8;              /* a[8..9] = ba[0..1]    */
+      address |= (y & 0x8) << 7;               /* a[10]   = y[3]        */
+      address |= ((x & 0x10) ^ (y & 0x10)) << 7;/* a[11]   = x[4] ^ y[4] */
+      address |= (ba & ~0x3) << 10;            /* a[12..] = ba[2..]     */
+
+      return address;
+   }
+}
 
-#define TAG(x) radeon##x##_24_8_TILE
-#include "depthtmp.h"
 
 /* 16-bit depth buffer functions
  */
 #define WRITE_DEPTH( _x, _y, d )                                       \
-   *(GLushort *)(buf + (_x + xo + (_y + yo)*pitch)*2 ) = d;
+   *(GLushort *)(buf + radeon_mba_z16( drb, _x + xo, _y + yo )) = d;
 
 #define READ_DEPTH( d, _x, _y )                                                \
-   d = *(GLushort *)(buf + (_x + xo + (_y + yo)*pitch)*2 );
+   d = *(GLushort *)(buf + radeon_mba_z16( drb, _x + xo, _y + yo ));
 
-#define TAG(x) radeon##x##_16_LINEAR
+#define TAG(x) radeon##x##_z16
 #include "depthtmp.h"
 
+
 /* 24 bit depth, 8 bit stencil depthbuffer functions
  *
  * Careful: It looks like the R300 uses ZZZS byte order while the R200
@@ -263,7 +199,7 @@ do {                                                                        \
  */
 #define WRITE_DEPTH( _x, _y, d )                                       \
 do {                                                                   \
-   GLuint offset = (_x + xo + (_y + yo)*pitch)*4;                      \
+   GLuint offset = radeon_mba_z32( drb, _x + xo, _y + yo );            \
    GLuint tmp = *(GLuint *)(buf + offset);                             \
    tmp &= 0x000000ff;                                                  \
    tmp |= ((d << 8) & 0xffffff00);                                     \
@@ -271,11 +207,15 @@ do {                                                                      \
 } while (0)
 
 #define READ_DEPTH( d, _x, _y )                                                \
-   d = (*(GLuint *)(buf + (_x + xo + (_y + yo)*pitch)*4) & 0xffffff00) >> 8;
+  do { \
+    d = (*(GLuint *)(buf + radeon_mba_z32( drb, _x + xo,               \
+                                        _y + yo )) & 0xffffff00) >> 8; \
+  }while(0)
 
-#define TAG(x) radeon##x##_24_8_LINEAR
+#define TAG(x) radeon##x##_z24_s8
 #include "depthtmp.h"
 
+
 /* ================================================================
  * Stencil buffer
  */
@@ -284,28 +224,7 @@ do {                                                                       \
  */
 #define WRITE_STENCIL( _x, _y, d )                                     \
 do {                                                                   \
-   GLuint offset = radeon_mba_z32( radeon, _x + xo, _y + yo );         \
-   GLuint tmp = *(GLuint *)(buf + offset);                             \
-   tmp &= 0xffffff00;                                                  \
-   tmp |= (d) & 0xff;                                                  \
-   *(GLuint *)(buf + offset) = tmp;                                    \
-} while (0)
-
-#define READ_STENCIL( d, _x, _y )                                      \
-do {                                                                   \
-   GLuint offset = radeon_mba_z32( radeon, _x + xo, _y + yo );         \
-   GLuint tmp = *(GLuint *)(buf + offset);                             \
-   d = tmp & 0x000000ff;                                               \
-} while (0)
-
-#define TAG(x) radeon##x##_24_8_TILE
-#include "stenciltmp.h"
-
-/* 24 bit depth, 8 bit stencil depthbuffer functions
- */
-#define WRITE_STENCIL( _x, _y, d )                                     \
-do {                                                                   \
-   GLuint offset = (_x + xo + (_y + yo)*pitch)*4;                      \
+   GLuint offset = radeon_mba_z32( drb, _x + xo, _y + yo );            \
    GLuint tmp = *(GLuint *)(buf + offset);                             \
    tmp &= 0xffffff00;                                                  \
    tmp |= (d) & 0xff;                                                  \
@@ -314,266 +233,92 @@ do {                                                                     \
 
 #define READ_STENCIL( d, _x, _y )                                      \
 do {                                                                   \
-   GLuint offset = (_x + xo + (_y + yo)*pitch)*4;                      \
+   GLuint offset = radeon_mba_z32( drb, _x + xo, _y + yo );            \
    GLuint tmp = *(GLuint *)(buf + offset);                             \
    d = tmp & 0x000000ff;                                               \
 } while (0)
 
-#define TAG(x) radeon##x##_24_8_LINEAR
+#define TAG(x) radeon##x##_z24_s8
 #include "stenciltmp.h"
 
-/*
- * This function is called to specify which buffer to read and write
- * for software rasterization (swrast) fallbacks.  This doesn't necessarily
- * correspond to glDrawBuffer() or glReadBuffer() calls.
- */
-static void radeonSetBuffer(GLcontext * ctx,
-                         GLframebuffer * colorBuffer, GLuint bufferBit)
-{
-       radeonContextPtr radeon = RADEON_CONTEXT(ctx);
-       int buffer;
 
-       switch (bufferBit) {
-       case BUFFER_BIT_FRONT_LEFT:
-               buffer = 0;
-               break;
-
-       case BUFFER_BIT_BACK_LEFT:
-               buffer = 1;
-               break;
-
-       default:
-               _mesa_problem(ctx, "Bad bufferBit in %s", __FUNCTION__);
-               return;
-       }
-
-       if (radeon->doPageFlip && radeon->sarea->pfCurrentPage == 1)
-               buffer ^= 1;
-
-#if 0
-       fprintf(stderr, "%s: using %s buffer\n", __FUNCTION__,
-               buffer ? "back" : "front");
-#endif
-
-       if (buffer) {
-               radeon->state.pixel.readOffset =
-                       radeon->radeonScreen->backOffset;
-               radeon->state.pixel.readPitch =
-                       radeon->radeonScreen->backPitch;
-               radeon->state.color.drawOffset =
-                       radeon->radeonScreen->backOffset;
-               radeon->state.color.drawPitch =
-                       radeon->radeonScreen->backPitch;
-       } else {
-               radeon->state.pixel.readOffset =
-                       radeon->radeonScreen->frontOffset;
-               radeon->state.pixel.readPitch =
-                       radeon->radeonScreen->frontPitch;
-               radeon->state.color.drawOffset =
-                       radeon->radeonScreen->frontOffset;
-               radeon->state.color.drawPitch =
-                       radeon->radeonScreen->frontPitch;
-       }
-}
 
 /* Move locking out to get reasonable span performance (10x better
  * than doing this in HW_LOCK above).  WaitForIdle() is the main
  * culprit.
  */
 
-static void radeonSpanRenderStart(GLcontext * ctx)
+static void radeonSpanRenderStart( GLcontext *ctx )
 {
-       radeonContextPtr radeon = RADEON_CONTEXT(ctx);
-
-       if (IS_FAMILY_R200(radeon))
-               R200_FIREVERTICES((r200ContextPtr)radeon);
-       else
+   radeonContextPtr rmesa = RADEON_CONTEXT( ctx );
+   {
+       static int first = 1;
+       r300ContextPtr r300 = (r300ContextPtr)rmesa;
+       
+       if (first) {
+               r300->span_dlocking = getenv("R300_SPAN_DISABLE_LOCKING") ? 1 : 0;
+               if (r300->span_dlocking == 0) {
+                       fprintf(stderr, "Try R300_SPAN_DISABLE_LOCKING env var if this hangs.\n");
+                       fflush(stderr);
+                       sleep(1);
+               }
+               first = 0;
+       }
+       
+       if (r300->span_dlocking) {
                r300Flush(ctx);
-
-       LOCK_HARDWARE(radeon);
-       radeonWaitForIdleLocked(radeon);
-
-       /* Read & rewrite the first pixel in the frame buffer.  This should
-        * be a noop, right?  In fact without this conform fails as reading
-        * from the framebuffer sometimes produces old results -- the
-        * on-card read cache gets mixed up and doesn't notice that the
-        * framebuffer has been updated.
-        *
-        * In the worst case this is buggy too as p might get the wrong
-        * value first time, so really need a hidden pixel somewhere for this.
-        */
-       {
-               int p;
-               volatile int *read_buf =
-                   (volatile int *)(radeon->dri.screen->pFB +
-                                    radeon->state.pixel.readOffset);
-               p = *read_buf;
-               *read_buf = p;
+               LOCK_HARDWARE( rmesa );
+               radeonWaitForIdleLocked( rmesa );
+               UNLOCK_HARDWARE( rmesa );
+               
+               return;
        }
+   }
+   //   R300_FIREVERTICES( rmesa );
+   // old code has flush
+   r300Flush(ctx);
+   LOCK_HARDWARE( rmesa );
+   radeonWaitForIdleLocked( rmesa );
 }
 
-static void radeonSpanRenderFinish(GLcontext * ctx)
+static void radeonSpanRenderFinish( GLcontext *ctx )
 {
-       radeonContextPtr radeon = RADEON_CONTEXT(ctx);
-
-       _swrast_flush(ctx);
-       UNLOCK_HARDWARE(radeon);
+   radeonContextPtr rmesa = RADEON_CONTEXT( ctx );
+   r300ContextPtr r300 = (r300ContextPtr)rmesa;
+   _swrast_flush( ctx );
+   if (r300->span_dlocking == 0)
+       UNLOCK_HARDWARE( rmesa );
 }
 
-void radeonInitSpanFuncs(GLcontext * ctx)
+void radeonInitSpanFuncs( GLcontext *ctx )
 {
-       radeonContextPtr radeon = RADEON_CONTEXT(ctx);
-       struct swrast_device_driver *swdd =
-           _swrast_GetDeviceDriverReference(ctx);
-
-       swdd->SetBuffer = radeonSetBuffer;
-
-       switch (radeon->radeonScreen->cpp) {
-       case 2:
-#if 0
-               swdd->WriteRGBASpan = radeonWriteRGBASpan_RGB565;
-               swdd->WriteRGBSpan = radeonWriteRGBSpan_RGB565;
-               swdd->WriteMonoRGBASpan = radeonWriteMonoRGBASpan_RGB565;
-               swdd->WriteRGBAPixels = radeonWriteRGBAPixels_RGB565;
-               swdd->WriteMonoRGBAPixels = radeonWriteMonoRGBAPixels_RGB565;
-               swdd->ReadRGBASpan = radeonReadRGBASpan_RGB565;
-               swdd->ReadRGBAPixels = radeonReadRGBAPixels_RGB565;
-#endif
-               break;
-
-       case 4:
-#if 0
-               swdd->WriteRGBASpan = radeonWriteRGBASpan_ARGB8888;
-               swdd->WriteRGBSpan = radeonWriteRGBSpan_ARGB8888;
-               swdd->WriteMonoRGBASpan = radeonWriteMonoRGBASpan_ARGB8888;
-               swdd->WriteRGBAPixels = radeonWriteRGBAPixels_ARGB8888;
-               swdd->WriteMonoRGBAPixels = radeonWriteMonoRGBAPixels_ARGB8888;
-               swdd->ReadRGBASpan = radeonReadRGBASpan_ARGB8888;
-               swdd->ReadRGBAPixels = radeonReadRGBAPixels_ARGB8888;
-#endif
-               break;
-
-       default:
-               break;
-       }
-
-       if (IS_FAMILY_R300(radeon))
-       {
-               switch (radeon->glCtx->Visual.depthBits) {
-               case 16:
-#if 0
-                       swdd->ReadDepthSpan = radeonReadDepthSpan_16_LINEAR;
-                       swdd->WriteDepthSpan = radeonWriteDepthSpan_16_LINEAR;
-                       swdd->WriteMonoDepthSpan = radeonWriteMonoDepthSpan_16_LINEAR;
-                       swdd->ReadDepthPixels = radeonReadDepthPixels_16_LINEAR;
-                       swdd->WriteDepthPixels = radeonWriteDepthPixels_16_LINEAR;
-#endif
-                       break;
-
-               case 24:
-#if 0
-                       swdd->ReadDepthSpan = radeonReadDepthSpan_24_8_LINEAR;
-                       swdd->WriteDepthSpan = radeonWriteDepthSpan_24_8_LINEAR;
-                       swdd->WriteMonoDepthSpan = radeonWriteMonoDepthSpan_24_8_LINEAR;
-                       swdd->ReadDepthPixels = radeonReadDepthPixels_24_8_LINEAR;
-                       swdd->WriteDepthPixels = radeonWriteDepthPixels_24_8_LINEAR;
-
-                       swdd->ReadStencilSpan = radeonReadStencilSpan_24_8_LINEAR;
-                       swdd->WriteStencilSpan = radeonWriteStencilSpan_24_8_LINEAR;
-                       swdd->ReadStencilPixels = radeonReadStencilPixels_24_8_LINEAR;
-                       swdd->WriteStencilPixels = radeonWriteStencilPixels_24_8_LINEAR;
-#endif
-                       break;
-
-               default:
-                       break;
-               }
-       }
-       else
-       {
-               switch (radeon->glCtx->Visual.depthBits) {
-               case 16:
-#if 0
-                       swdd->ReadDepthSpan = radeonReadDepthSpan_16_TILE;
-                       swdd->WriteDepthSpan = radeonWriteDepthSpan_16_TILE;
-                       swdd->WriteMonoDepthSpan = radeonWriteMonoDepthSpan_16_TILE;
-                       swdd->ReadDepthPixels = radeonReadDepthPixels_16_TILE;
-                       swdd->WriteDepthPixels = radeonWriteDepthPixels_16_TILE;
-#endif
-                       break;
-
-               case 24:
-#if 0
-                       swdd->ReadDepthSpan = radeonReadDepthSpan_24_8_TILE;
-                       swdd->WriteDepthSpan = radeonWriteDepthSpan_24_8_TILE;
-                       swdd->WriteMonoDepthSpan = radeonWriteMonoDepthSpan_24_8_TILE;
-                       swdd->ReadDepthPixels = radeonReadDepthPixels_24_8_TILE;
-                       swdd->WriteDepthPixels = radeonWriteDepthPixels_24_8_TILE;
-
-                       swdd->ReadStencilSpan = radeonReadStencilSpan_24_8_TILE;
-                       swdd->WriteStencilSpan = radeonWriteStencilSpan_24_8_TILE;
-                       swdd->ReadStencilPixels = radeonReadStencilPixels_24_8_TILE;
-                       swdd->WriteStencilPixels = radeonWriteStencilPixels_24_8_TILE;
-#endif
-                       break;
-
-               default:
-                       break;
-               }
-       }
-
-       swdd->SpanRenderStart = radeonSpanRenderStart;
-       swdd->SpanRenderFinish = radeonSpanRenderFinish;
+   struct swrast_device_driver *swdd = _swrast_GetDeviceDriverReference(ctx);
+   swdd->SpanRenderStart          = radeonSpanRenderStart;
+   swdd->SpanRenderFinish         = radeonSpanRenderFinish; 
 }
 
+
 /**
-         * Plug in the Get/Put routines for the given driRenderbuffer.
-         */
-void radeonSetSpanFunctions(driRenderbuffer *drb, const GLvisual *vis)
+ * Plug in the Get/Put routines for the given driRenderbuffer.
+ */
+void
+radeonSetSpanFunctions(driRenderbuffer *drb, const GLvisual *vis)
 {
-       if (drb->Base.InternalFormat == GL_RGBA) {
-               if (vis->redBits == 5 && vis->greenBits == 6 && vis->blueBits == 5) {
-                       drb->Base.GetRow        = radeonReadRGBASpan_RGB565;
-                       drb->Base.GetValues     = radeonReadRGBAPixels_RGB565;
-                       drb->Base.PutRow        = radeonWriteRGBASpan_RGB565;
-                       drb->Base.PutRowRGB     = radeonWriteRGBSpan_RGB565;
-                       drb->Base.PutMonoRow    = radeonWriteMonoRGBASpan_RGB565;
-                       drb->Base.PutValues     = radeonWriteRGBAPixels_RGB565;
-                       drb->Base.PutMonoValues = radeonWriteMonoRGBAPixels_RGB565;
-               }
-               else {
-                       drb->Base.GetRow        = radeonReadRGBASpan_ARGB8888;
-                       drb->Base.GetValues     = radeonReadRGBAPixels_ARGB8888;
-                       drb->Base.PutRow        = radeonWriteRGBASpan_ARGB8888;
-                       drb->Base.PutRowRGB     = radeonWriteRGBSpan_ARGB8888;
-                       drb->Base.PutMonoRow    = radeonWriteMonoRGBASpan_ARGB8888;
-                       drb->Base.PutValues     = radeonWriteRGBAPixels_ARGB8888;
-                       drb->Base.PutMonoValues = radeonWriteMonoRGBAPixels_ARGB8888;
-               }
-       }
-       else if (drb->Base.InternalFormat == GL_DEPTH_COMPONENT16) {
-               drb->Base.GetRow        = radeonReadDepthSpan_16_LINEAR;
-               drb->Base.GetValues     = radeonReadDepthPixels_16_LINEAR;
-               drb->Base.PutRow        = radeonWriteDepthSpan_16_LINEAR;
-               drb->Base.PutMonoRow    = radeonWriteMonoDepthSpan_16_LINEAR;
-               drb->Base.PutValues     = radeonWriteDepthPixels_16_LINEAR;
-               drb->Base.PutMonoValues = NULL;
-       }
-       else if (drb->Base.InternalFormat == GL_DEPTH_COMPONENT24) {
-               drb->Base.GetRow        = radeonReadDepthSpan_24_8_LINEAR;
-               drb->Base.GetValues     = radeonReadDepthPixels_24_8_LINEAR;
-               drb->Base.PutRow        = radeonWriteDepthSpan_24_8_LINEAR;
-               drb->Base.PutMonoRow    = radeonWriteMonoDepthSpan_24_8_LINEAR;
-               drb->Base.PutValues     = radeonWriteDepthPixels_24_8_LINEAR;
-               drb->Base.PutMonoValues = NULL;
-       }
-       else if (drb->Base.InternalFormat == GL_STENCIL_INDEX8_EXT) {
-               drb->Base.GetRow        = radeonReadStencilSpan_24_8_LINEAR;
-               drb->Base.GetValues     = radeonReadStencilPixels_24_8_LINEAR;
-               drb->Base.PutRow        = radeonWriteStencilSpan_24_8_LINEAR;
-               drb->Base.PutMonoRow    = radeonWriteMonoStencilSpan_24_8_LINEAR;
-               drb->Base.PutValues     = radeonWriteStencilPixels_24_8_LINEAR;
-               drb->Base.PutMonoValues = NULL;
-       }
+   if (drb->Base.InternalFormat == GL_RGBA) {
+      if (vis->redBits == 5 && vis->greenBits == 6 && vis->blueBits == 5) {
+         radeonInitPointers_RGB565(&drb->Base);
+      }
+      else {
+         radeonInitPointers_ARGB8888(&drb->Base);
+      }
+   }
+   else if (drb->Base.InternalFormat == GL_DEPTH_COMPONENT16) {
+      radeonInitDepthPointers_z16(&drb->Base);
+   }
+   else if (drb->Base.InternalFormat == GL_DEPTH_COMPONENT24) {
+      radeonInitDepthPointers_z24_s8(&drb->Base);
+   }
+   else if (drb->Base.InternalFormat == GL_STENCIL_INDEX8_EXT) {
+      radeonInitStencilPointers_z24_s8(&drb->Base);
+   }
 }
-