convert sis and radeon driver to use spantmp2 instead of spantmp template. Remove old (pre-renderbuffer) span code instead of converting that too. Remove this old code from mach64 (the dead code was not fully converted to spantmp2 previously) too.

1 files changed, 15 insertions, 140 deletions

diff --git a/src/mesa/drivers/dri/sis/sis_span.c b/src/mesa/drivers/dri/sis/sis_span.c
index 78ce74b8725..f0cf771d616 100644
--- a/src/mesa/drivers/dri/sis/sis_span.c
+++ b/src/mesa/drivers/dri/sis/sis_span.c
@@ -44,6 +44,7 @@ USE OR OTHER DEALINGS IN THE SOFTWARE.
 #define LOCAL_VARS							\
    sisContextPtr smesa = SIS_CONTEXT(ctx);				\
    __DRIdrawablePrivate *dPriv = smesa->driDrawable;			\
+   GLuint pitch = smesa->drawPitch;					\
    char *buf = (char *)(smesa->FbBase + smesa->drawOffset);		\
    char *read_buf = (char *)(smesa->FbBase + smesa->readOffset);	\
    GLuint p;								\
@@ -61,57 +62,23 @@ USE OR OTHER DEALINGS IN THE SOFTWARE.
 #define HW_UNLOCK() do {} while(0);
 
 /* RGB565 */
-#define INIT_MONO_PIXEL(p, color) \
-  p = SISPACKCOLOR565( 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*smesa->drawPitch) =			\
-					     (((r & 0xf8) << 8) |	\
-					     ((g & 0xfc) << 3) |	\
-					     (b >> 3))
-
-#define WRITE_PIXEL( _x, _y, p )  \
-   *(GLushort *)(buf + _x*2 + _y*smesa->drawPitch) = p
-
-#define READ_RGBA( rgba, _x, _y )			\
-do {							\
-   GLushort p = *(GLushort *)(read_buf + _x*2 + _y*smesa->readPitch);	\
-   rgba[0] = (p & 0xf800) >> 8;				\
-   rgba[1] = (p & 0x07e0) >> 3;			        \
-   rgba[2] = (p & 0x001f) << 3;			        \
-   rgba[3] = 0xff;					\
-} while(0)
-
-#define TAG(x) sis##x##_565
-#include "spantmp.h"
+#define TAG(x)    sis##x##_RGB565
+#define TAG2(x,y) sis##x##_RGB565##y
+#include "spantmp2.h"
 
 
 /* ARGB8888 */
-#undef INIT_MONO_PIXEL
-#define INIT_MONO_PIXEL(p, color) \
-  p = SISPACKCOLOR8888( color[0], color[1], color[2], color[3] )
-
-#define WRITE_RGBA( _x, _y, r, g, b, a )			\
-   *(GLuint *)(buf + _x*4 + _y*smesa->drawPitch) =		\
-					   (((a) << 24) |	\
-					   ((r) << 16) |	\
-					   ((g) << 8) |		\
-					   ((b)))
-
-#define WRITE_PIXEL( _x, _y, p )  \
-   *(GLuint *)(buf + _x*4 + _y*smesa->drawPitch)  = p
+/* FIXME the old code always read back alpha as 0xff, i.e. fully opaque.
+   Was there a reason to do so ? If so that'll won't work with that template... */
+#define SPANTMP_PIXEL_FMT GL_BGRA
+#define SPANTMP_PIXEL_TYPE GL_UNSIGNED_INT_8_8_8_8_REV
 
-#define READ_RGBA( rgba, _x, _y )			\
-do {							\
-   GLuint p = *(GLuint *)(read_buf + _x*4 + _y*smesa->readPitch);	\
-   rgba[0] = (p >> 16) & 0xff;				\
-   rgba[1] = (p >> 8) & 0xff;				\
-   rgba[2] = (p >> 0) & 0xff;				\
-   rgba[3] = 0xff;					\
-} while(0)
-
-#define TAG(x) sis##x##_8888
-#include "spantmp.h"
+#define TAG(x)    sis##x##_ARGB8888
+#define TAG2(x,y) sis##x##_ARGB8888##y
+#include "spantmp2.h"
 
 
 /* 16 bit depthbuffer functions.
@@ -212,89 +179,9 @@ void sisSpanRenderFinish( GLcontext *ctx )
 void
 sisDDInitSpanFuncs( GLcontext *ctx )
 {
-   sisContextPtr smesa = SIS_CONTEXT(ctx);
    struct swrast_device_driver *swdd = _swrast_GetDeviceDriverReference(ctx);
 
    swdd->SetBuffer = sisDDSetBuffer;
-
-   switch (smesa->zFormat)
-   {
-   case SiS_ZFORMAT_Z16:
-#if 0
-      swdd->ReadDepthSpan = sisReadDepthSpan_16;
-      swdd->ReadDepthPixels = sisReadDepthPixels_16;
-      swdd->WriteDepthSpan = sisWriteDepthSpan_16;
-      swdd->WriteDepthPixels = sisWriteDepthPixels_16;
-
-      swdd->ReadStencilSpan = NULL;
-      swdd->ReadStencilPixels = NULL;
-      swdd->WriteStencilSpan = NULL;
-      swdd->WriteStencilPixels = NULL;
-#endif
-      break;
-   case SiS_ZFORMAT_Z32:
-#if 0
-      swdd->ReadDepthSpan = sisReadDepthSpan_32;
-      swdd->ReadDepthPixels = sisReadDepthPixels_32;
-      swdd->WriteDepthSpan = sisWriteDepthSpan_32;
-      swdd->WriteDepthPixels = sisWriteDepthPixels_32;
-
-      swdd->ReadStencilSpan = NULL;
-      swdd->ReadStencilPixels = NULL;
-      swdd->WriteStencilSpan = NULL;
-      swdd->WriteStencilPixels = NULL;
-#endif
-      break;
-   case SiS_ZFORMAT_S8Z24:
-#if 0
-      swdd->ReadDepthSpan = sisReadDepthSpan_24_8;
-      swdd->ReadDepthPixels = sisReadDepthPixels_24_8;
-      swdd->WriteDepthSpan = sisWriteDepthSpan_24_8;
-      swdd->WriteDepthPixels = sisWriteDepthPixels_24_8;
-
-      swdd->ReadStencilSpan = sisReadStencilSpan_24_8;
-      swdd->ReadStencilPixels = sisReadStencilPixels_24_8;
-      swdd->WriteStencilSpan = sisWriteStencilSpan_24_8;
-      swdd->WriteStencilPixels = sisWriteStencilPixels_24_8;
-#endif
-      break;
-   }
-
-#if 0
-   switch ( smesa->bytesPerPixel )
-   {
-   case 2:
-      swdd->WriteRGBASpan = sisWriteRGBASpan_565;
-      swdd->WriteRGBSpan = sisWriteRGBSpan_565;
-      swdd->WriteMonoRGBASpan = sisWriteMonoRGBASpan_565;
-      swdd->WriteRGBAPixels = sisWriteRGBAPixels_565;
-      swdd->WriteMonoRGBAPixels = sisWriteMonoRGBAPixels_565;
-      swdd->ReadRGBASpan = sisReadRGBASpan_565;
-      swdd->ReadRGBAPixels = sisReadRGBAPixels_565;
-      break;
-   case 4:
-      swdd->WriteRGBASpan = sisWriteRGBASpan_8888;
-      swdd->WriteRGBSpan = sisWriteRGBSpan_8888;
-      swdd->WriteMonoRGBASpan = sisWriteMonoRGBASpan_8888;
-      swdd->WriteRGBAPixels = sisWriteRGBAPixels_8888;
-      swdd->WriteMonoRGBAPixels = sisWriteMonoRGBAPixels_8888;
-      swdd->ReadRGBASpan = sisReadRGBASpan_8888;
-      swdd->ReadRGBAPixels = sisReadRGBAPixels_8888;
-      break;
-    default:
-      sis_fatal_error("Bad bytesPerPixel.\n");
-      break;
-   }
-
-   swdd->WriteCI8Span      = NULL;
-   swdd->WriteCI32Span     = NULL;
-   swdd->WriteMonoCISpan   = NULL;
-   swdd->WriteCI32Pixels   = NULL;
-   swdd->WriteMonoCIPixels = NULL;
-   swdd->ReadCI32Span      = NULL;
-   swdd->ReadCI32Pixels    = NULL;
-#endif
-
    swdd->SpanRenderStart   = sisSpanRenderStart;
    swdd->SpanRenderFinish  = sisSpanRenderFinish; 
 }
@@ -309,22 +196,10 @@ sisSetSpanFunctions(driRenderbuffer *drb, const GLvisual *vis)
 {
    if (drb->Base.InternalFormat == GL_RGBA) {
       if (vis->redBits == 5 && vis->greenBits == 6 && vis->blueBits == 5) {
-         drb->Base.GetRow        = sisReadRGBASpan_565;
-         drb->Base.GetValues     = sisReadRGBAPixels_565;
-         drb->Base.PutRow        = sisWriteRGBASpan_565;
-         drb->Base.PutRowRGB     = sisWriteRGBSpan_565;
-         drb->Base.PutMonoRow    = sisWriteMonoRGBASpan_565;
-         drb->Base.PutValues     = sisWriteRGBAPixels_565;
-         drb->Base.PutMonoValues = sisWriteMonoRGBAPixels_565;
+         sisInitPointers_RGB565( &drb->Base );
       }
       else {
-         drb->Base.GetRow        = sisReadRGBASpan_8888;
-         drb->Base.GetValues     = sisReadRGBAPixels_8888;
-         drb->Base.PutRow        = sisWriteRGBASpan_8888;
-         drb->Base.PutRowRGB     = sisWriteRGBSpan_8888;
-         drb->Base.PutMonoRow    = sisWriteMonoRGBASpan_8888;
-         drb->Base.PutValues     = sisWriteRGBAPixels_8888;
-         drb->Base.PutMonoValues = sisWriteMonoRGBAPixels_8888;
+         sisInitPointers_ARGB8888( &drb->Base );
       }
    }
    else if (drb->Base.InternalFormat == GL_DEPTH_COMPONENT16) {