From: José Fonseca <jfonseca@vmware.com>
Date: Mon, 10 Aug 2009 07:16:31 +0000 (+0100)
Subject: llvmpipe: Tiles in rgba8 format.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=2101c2941c1ff3ae7f3f587e6e9921ed336ce17c;p=mesa.git

llvmpipe: Tiles in rgba8 format.
---

diff --git a/src/gallium/drivers/llvmpipe/lp_context.h b/src/gallium/drivers/llvmpipe/lp_context.h
index 72387301a6b..7ac83c1e7c1 100644
--- a/src/gallium/drivers/llvmpipe/lp_context.h
+++ b/src/gallium/drivers/llvmpipe/lp_context.h
@@ -61,7 +61,7 @@ struct llvmpipe_context {
    const struct lp_vertex_shader *vs;
 
    /** Other rendering state */
-   float ALIGN16_ATTRIB blend_color[4][QUAD_SIZE];
+   uint8_t ALIGN16_ATTRIB blend_color[4][16];
    struct pipe_clip_state clip;
    struct pipe_constant_buffer constants[PIPE_SHADER_TYPES];
    struct pipe_framebuffer_state framebuffer;
diff --git a/src/gallium/drivers/llvmpipe/lp_quad_blend.c b/src/gallium/drivers/llvmpipe/lp_quad_blend.c
index fbfd51f6003..2b060574ae1 100644
--- a/src/gallium/drivers/llvmpipe/lp_quad_blend.c
+++ b/src/gallium/drivers/llvmpipe/lp_quad_blend.c
@@ -46,30 +46,15 @@
 
 static void
 logicop_quad(struct quad_stage *qs, 
-             float (*quadColor)[4],
-             float (*dest)[4])
+             uint8_t src[][16],
+             uint8_t dst[][16])
 {
    struct llvmpipe_context *llvmpipe = qs->llvmpipe;
-   ubyte src[4][4], dst[4][4], res[4][4];
-   uint *src4 = (uint *) src;
-   uint *dst4 = (uint *) dst;
-   uint *res4 = (uint *) res;
+   uint32_t *src4 = (uint32_t *) src;
+   uint32_t *dst4 = (uint32_t *) dst;
+   uint32_t *res4 = (uint32_t *) src;
    uint j;
 
-
-   /* convert to ubyte */
-   for (j = 0; j < 4; j++) { /* loop over R,G,B,A channels */
-      dst[j][0] = float_to_ubyte(dest[j][0]); /* P0 */
-      dst[j][1] = float_to_ubyte(dest[j][1]); /* P1 */
-      dst[j][2] = float_to_ubyte(dest[j][2]); /* P2 */
-      dst[j][3] = float_to_ubyte(dest[j][3]); /* P3 */
-
-      src[j][0] = float_to_ubyte(quadColor[j][0]); /* P0 */
-      src[j][1] = float_to_ubyte(quadColor[j][1]); /* P1 */
-      src[j][2] = float_to_ubyte(quadColor[j][2]); /* P2 */
-      src[j][3] = float_to_ubyte(quadColor[j][3]); /* P3 */
-   }
-
    switch (llvmpipe->blend->base.logicop_func) {
    case PIPE_LOGICOP_CLEAR:
       for (j = 0; j < 4; j++)
@@ -138,13 +123,6 @@ logicop_quad(struct quad_stage *qs,
    default:
       assert(0);
    }
-
-   for (j = 0; j < 4; j++) {
-      quadColor[j][0] = ubyte_to_float(res[j][0]);
-      quadColor[j][1] = ubyte_to_float(res[j][1]);
-      quadColor[j][2] = ubyte_to_float(res[j][2]);
-      quadColor[j][3] = ubyte_to_float(res[j][3]);
-   }
 }
 
 
@@ -161,53 +139,62 @@ blend_run(struct quad_stage *qs,
    struct llvmpipe_context *llvmpipe = qs->llvmpipe;
    struct lp_blend_state *blend = llvmpipe->blend;
    unsigned cbuf;
-   uint q, i, j;
+   uint q, i, j, k;
 
    for (cbuf = 0; cbuf < llvmpipe->framebuffer.nr_cbufs; cbuf++) 
    {
-      float ALIGN16_ATTRIB dest[4][QUAD_SIZE];
+      uint8_t ALIGN16_ATTRIB src[4][16];
+      uint8_t ALIGN16_ATTRIB dst[4][16];
       struct llvmpipe_cached_tile *tile
          = lp_get_cached_tile(llvmpipe->cbuf_cache[cbuf],
                               quads[0]->input.x0, 
                               quads[0]->input.y0);
 
-      for (q = 0; q < nr; q++) {
-         struct quad_header *quad = quads[q];
-         float (*quadColor)[4] = quad->output.color[cbuf];
-         const int itx = (quad->input.x0 & (TILE_SIZE-1));
-         const int ity = (quad->input.y0 & (TILE_SIZE-1));
+      for (q = 0; q < nr; q += 4) {
+         for (k = 0; k < 4 && q + k < nr; ++k) {
+            struct quad_header *quad = quads[q + k];
+            const int itx = (quad->input.x0 & (TILE_SIZE-1));
+            const int ity = (quad->input.y0 & (TILE_SIZE-1));
 
-         /* get/swizzle dest colors 
-          */
-         for (j = 0; j < QUAD_SIZE; j++) {
-            int x = itx + (j & 1);
-            int y = ity + (j >> 1);
-            for (i = 0; i < 4; i++) {
-               dest[i][j] = tile->data.color[i][y][x];
+            /* get/swizzle src/dest colors
+             */
+            for (j = 0; j < QUAD_SIZE; j++) {
+               int x = itx + (j & 1);
+               int y = ity + (j >> 1);
+               for (i = 0; i < 4; i++) {
+                  src[i][4*k + j] = float_to_ubyte(quad->output.color[cbuf][i][j]);
+                  dst[i][4*k + j] = tile->data.color[i][y][x];
+               }
             }
          }
 
 
          if (blend->base.logicop_enable) {
-            logicop_quad( qs, quadColor, dest );
+            logicop_quad( qs, src, dst );
          }
          else {
             assert(blend->jit_function);
-            assert((((uintptr_t)quadColor) & 0xf) == 0);
-            assert((((uintptr_t)dest) & 0xf) == 0);
+            assert((((uintptr_t)src) & 0xf) == 0);
+            assert((((uintptr_t)dst) & 0xf) == 0);
             assert((((uintptr_t)llvmpipe->blend_color) & 0xf) == 0);
             if(blend->jit_function)
-               blend->jit_function( quadColor, dest, llvmpipe->blend_color, quadColor );
+               blend->jit_function( src, dst, llvmpipe->blend_color, src );
          }
 
          /* Output color values
           */
-         for (j = 0; j < QUAD_SIZE; j++) {
-            if (quad->inout.mask & (1 << j)) {
-               int x = itx + (j & 1);
-               int y = ity + (j >> 1);
-               for (i = 0; i < 4; i++) { /* loop over color chans */
-                  tile->data.color[i][y][x] = quadColor[i][j];
+         for (k = 0; k < 4 && q + k < nr; ++k) {
+            struct quad_header *quad = quads[q + k];
+            const int itx = (quad->input.x0 & (TILE_SIZE-1));
+            const int ity = (quad->input.y0 & (TILE_SIZE-1));
+
+            for (j = 0; j < QUAD_SIZE; j++) {
+               if (quad->inout.mask & (1 << j)) {
+                  int x = itx + (j & 1);
+                  int y = ity + (j >> 1);
+                  for (i = 0; i < 4; i++) { /* loop over color chans */
+                     tile->data.color[i][y][x] = src[i][4*k + j];
+                  }
                }
             }
          }
diff --git a/src/gallium/drivers/llvmpipe/lp_state.h b/src/gallium/drivers/llvmpipe/lp_state.h
index 78c47785804..6b757a6ba71 100644
--- a/src/gallium/drivers/llvmpipe/lp_state.h
+++ b/src/gallium/drivers/llvmpipe/lp_state.h
@@ -93,10 +93,10 @@ struct lp_vertex_shader {
 
 
 typedef void
-(*lp_blend_func)(float src[][4],
-                 float dst[][4],
-                 float con[][4],
-                 float res[][4]);
+(*lp_blend_func)(uint8_t src[][16],
+                 uint8_t dst[][16],
+                 uint8_t con[][16],
+                 uint8_t res[][16]);
 
 
 struct lp_blend_state
diff --git a/src/gallium/drivers/llvmpipe/lp_state_blend.c b/src/gallium/drivers/llvmpipe/lp_state_blend.c
index ed4ea3b3304..c75214430f1 100644
--- a/src/gallium/drivers/llvmpipe/lp_state_blend.c
+++ b/src/gallium/drivers/llvmpipe/lp_state_blend.c
@@ -32,6 +32,7 @@
  */
 
 #include "util/u_memory.h"
+#include "util/u_math.h"
 #include "util/u_debug_dump.h"
 #include "lp_screen.h"
 #include "lp_context.h"
@@ -66,11 +67,11 @@ blend_generate(struct llvmpipe_screen *screen,
    unsigned i;
 
    type.value = 0;
-   type.floating = TRUE;
+   type.floating = FALSE;
    type.sign = FALSE;
    type.norm = TRUE;
-   type.width = 32;
-   type.length = 4;
+   type.width = 8;
+   type.length = 16;
 
    vec_type = lp_build_vec_type(type);
 
@@ -181,8 +182,8 @@ void llvmpipe_set_blend_color( struct pipe_context *pipe,
    unsigned i, j;
 
    for (i = 0; i < 4; ++i)
-      for (j = 0; j < QUAD_SIZE; ++j)
-         llvmpipe->blend_color[i][j] = blend_color->color[i];
+      for (j = 0; j < 16; ++j)
+         llvmpipe->blend_color[i][j] = float_to_ubyte(blend_color->color[i]);
 
    llvmpipe->dirty |= LP_NEW_BLEND;
 }
diff --git a/src/gallium/drivers/llvmpipe/lp_tile_cache.c b/src/gallium/drivers/llvmpipe/lp_tile_cache.c
index 65b62c44dcd..28be3659024 100644
--- a/src/gallium/drivers/llvmpipe/lp_tile_cache.c
+++ b/src/gallium/drivers/llvmpipe/lp_tile_cache.c
@@ -221,7 +221,7 @@ clear_tile_rgba(struct llvmpipe_cached_tile *tile,
       for (i = 0; i < 4; ++i)
          for (y = 0; y < TILE_SIZE; y++)
             for (x = 0; x < TILE_SIZE; x++)
-               tile->data.color[i][y][x] = clear_value[i];
+               tile->data.color[i][y][x] = float_to_ubyte(clear_value[i]);
    }
 }
 
diff --git a/src/gallium/drivers/llvmpipe/lp_tile_cache.h b/src/gallium/drivers/llvmpipe/lp_tile_cache.h
index 2f904d9e530..df63d47e1b5 100644
--- a/src/gallium/drivers/llvmpipe/lp_tile_cache.h
+++ b/src/gallium/drivers/llvmpipe/lp_tile_cache.h
@@ -66,7 +66,7 @@ struct llvmpipe_cached_tile
    union {
 
       /** color in SOA format (rrrr, gggg, bbbb, aaaa) */
-      float color[4][TILE_SIZE][TILE_SIZE];
+      uint8_t color[4][TILE_SIZE][TILE_SIZE];
 
       uint color32[TILE_SIZE][TILE_SIZE];
       uint depth32[TILE_SIZE][TILE_SIZE];
diff --git a/src/gallium/drivers/llvmpipe/lp_tile_soa.c b/src/gallium/drivers/llvmpipe/lp_tile_soa.c
index 9d37cf68053..05b95e10bd3 100644
--- a/src/gallium/drivers/llvmpipe/lp_tile_soa.c
+++ b/src/gallium/drivers/llvmpipe/lp_tile_soa.c
@@ -38,36 +38,30 @@
 #include "util/u_memory.h"
 #include "util/u_rect.h"
 #include "util/u_tile.h"
+#include "lp_tile_cache.h"
 #include "lp_tile_soa.h"
 
 
 #define PIXEL(_p, _x, _y, _c) ((_p)[(_c)*TILE_SIZE*TILE_SIZE + (_y)*TILE_SIZE + (_x)])
 
 
-/** Convert short in [-32768,32767] to GLfloat in [-1.0,1.0] */
-#define SHORT_TO_FLOAT(S)   ((2.0F * (S) + 1.0F) * (1.0F/65535.0F))
-
-#define UNCLAMPED_FLOAT_TO_SHORT(us, f)  \
-   us = ( (short) ( CLAMP((f), -1.0, 1.0) * 32767.0F) )
-
-
 
 /*** PIPE_FORMAT_A8R8G8B8_UNORM ***/
 
 static void
 a8r8g8b8_get_tile_rgba(const unsigned *src,
                        unsigned w, unsigned h,
-                       float *p)
+                       uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          const unsigned pixel = *src++;
-         PIXEL(p, j, i, 0) = ubyte_to_float((pixel >> 16) & 0xff);
-         PIXEL(p, j, i, 1) = ubyte_to_float((pixel >>  8) & 0xff);
-         PIXEL(p, j, i, 2) = ubyte_to_float((pixel >>  0) & 0xff);
-         PIXEL(p, j, i, 3) = ubyte_to_float((pixel >> 24) & 0xff);
+         PIXEL(p, j, i, 0) = (pixel >> 16) & 0xff;
+         PIXEL(p, j, i, 1) = (pixel >>  8) & 0xff;
+         PIXEL(p, j, i, 2) = (pixel >>  0) & 0xff;
+         PIXEL(p, j, i, 3) = (pixel >> 24) & 0xff;
       }
    }
 }
@@ -76,17 +70,17 @@ a8r8g8b8_get_tile_rgba(const unsigned *src,
 static void
 a8r8g8b8_put_tile_rgba(unsigned *dst,
                        unsigned w, unsigned h,
-                       const float *p)
+                       const uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          unsigned r, g, b, a;
-         r = float_to_ubyte(PIXEL(p, j, i, 0));
-         g = float_to_ubyte(PIXEL(p, j, i, 1));
-         b = float_to_ubyte(PIXEL(p, j, i, 2));
-         a = float_to_ubyte(PIXEL(p, j, i, 3));
+         r = PIXEL(p, j, i, 0);
+         g = PIXEL(p, j, i, 1);
+         b = PIXEL(p, j, i, 2);
+         a = PIXEL(p, j, i, 3);
          *dst++ = (a << 24) | (r << 16) | (g << 8) | b;
       }
    }
@@ -98,17 +92,17 @@ a8r8g8b8_put_tile_rgba(unsigned *dst,
 static void
 x8r8g8b8_get_tile_rgba(const unsigned *src,
                        unsigned w, unsigned h,
-                       float *p)
+                       uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          const unsigned pixel = *src++;
-         PIXEL(p, j, i, 0) = ubyte_to_float((pixel >> 16) & 0xff);
-         PIXEL(p, j, i, 1) = ubyte_to_float((pixel >>  8) & 0xff);
-         PIXEL(p, j, i, 2) = ubyte_to_float((pixel >>  0) & 0xff);
-         PIXEL(p, j, i, 3) = ubyte_to_float(0xff);
+         PIXEL(p, j, i, 0) = (pixel >> 16) & 0xff;
+         PIXEL(p, j, i, 1) = (pixel >>  8) & 0xff;
+         PIXEL(p, j, i, 2) = (pixel >>  0) & 0xff;
+         PIXEL(p, j, i, 3) = 0xff;
       }
    }
 }
@@ -117,16 +111,16 @@ x8r8g8b8_get_tile_rgba(const unsigned *src,
 static void
 x8r8g8b8_put_tile_rgba(unsigned *dst,
                        unsigned w, unsigned h,
-                       const float *p)
+                       const uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          unsigned r, g, b;
-         r = float_to_ubyte(PIXEL(p, j, i, 0));
-         g = float_to_ubyte(PIXEL(p, j, i, 1));
-         b = float_to_ubyte(PIXEL(p, j, i, 2));
+         r = PIXEL(p, j, i, 0);
+         g = PIXEL(p, j, i, 1);
+         b = PIXEL(p, j, i, 2);
          *dst++ = (0xff << 24) | (r << 16) | (g << 8) | b;
       }
    }
@@ -138,17 +132,17 @@ x8r8g8b8_put_tile_rgba(unsigned *dst,
 static void
 b8g8r8a8_get_tile_rgba(const unsigned *src,
                        unsigned w, unsigned h,
-                       float *p)
+                       uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          const unsigned pixel = *src++;
-         PIXEL(p, j, i, 0) = ubyte_to_float((pixel >>  8) & 0xff);
-         PIXEL(p, j, i, 1) = ubyte_to_float((pixel >> 16) & 0xff);
-         PIXEL(p, j, i, 2) = ubyte_to_float((pixel >> 24) & 0xff);
-         PIXEL(p, j, i, 3) = ubyte_to_float((pixel >>  0) & 0xff);
+         PIXEL(p, j, i, 0) = (pixel >>  8) & 0xff;
+         PIXEL(p, j, i, 1) = (pixel >> 16) & 0xff;
+         PIXEL(p, j, i, 2) = (pixel >> 24) & 0xff;
+         PIXEL(p, j, i, 3) = (pixel >>  0) & 0xff;
       }
    }
 }
@@ -157,17 +151,17 @@ b8g8r8a8_get_tile_rgba(const unsigned *src,
 static void
 b8g8r8a8_put_tile_rgba(unsigned *dst,
                        unsigned w, unsigned h,
-                       const float *p)
+                       const uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          unsigned r, g, b, a;
-         r = float_to_ubyte(PIXEL(p, j, i, 0));
-         g = float_to_ubyte(PIXEL(p, j, i, 1));
-         b = float_to_ubyte(PIXEL(p, j, i, 2));
-         a = float_to_ubyte(PIXEL(p, j, i, 3));
+         r = PIXEL(p, j, i, 0);
+         g = PIXEL(p, j, i, 1);
+         b = PIXEL(p, j, i, 2);
+         a = PIXEL(p, j, i, 3);
          *dst++ = (b << 24) | (g << 16) | (r << 8) | a;
       }
    }
@@ -179,17 +173,17 @@ b8g8r8a8_put_tile_rgba(unsigned *dst,
 static void
 a1r5g5b5_get_tile_rgba(const ushort *src,
                        unsigned w, unsigned h,
-                       float *p)
+                       uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          const ushort pixel = *src++;
-         PIXEL(p, j, i, 0) = ((pixel >> 10) & 0x1f) * (1.0f / 31.0f);
-         PIXEL(p, j, i, 1) = ((pixel >>  5) & 0x1f) * (1.0f / 31.0f);
-         PIXEL(p, j, i, 2) = ((pixel      ) & 0x1f) * (1.0f / 31.0f);
-         PIXEL(p, j, i, 3) = ((pixel >> 15)       ) * 1.0f;
+         PIXEL(p, j, i, 0) = ((pixel >> 10) & 0x1f) * 255 / 31;
+         PIXEL(p, j, i, 1) = ((pixel >>  5) & 0x1f) * 255 / 31;
+         PIXEL(p, j, i, 2) = ((pixel      ) & 0x1f) * 255 / 31;
+         PIXEL(p, j, i, 3) = ((pixel >> 15)       ) * 255;
       }
    }
 }
@@ -198,17 +192,17 @@ a1r5g5b5_get_tile_rgba(const ushort *src,
 static void
 a1r5g5b5_put_tile_rgba(ushort *dst,
                        unsigned w, unsigned h,
-                       const float *p)
+                       const uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          unsigned r, g, b, a;
-         r = float_to_ubyte(PIXEL(p, j, i, 0));
-         g = float_to_ubyte(PIXEL(p, j, i, 1));
-         b = float_to_ubyte(PIXEL(p, j, i, 2));
-         a = float_to_ubyte(PIXEL(p, j, i, 3));
+         r = PIXEL(p, j, i, 0);
+         g = PIXEL(p, j, i, 1);
+         b = PIXEL(p, j, i, 2);
+         a = PIXEL(p, j, i, 3);
          r = r >> 3;  /* 5 bits */
          g = g >> 3;  /* 5 bits */
          b = b >> 3;  /* 5 bits */
@@ -224,17 +218,17 @@ a1r5g5b5_put_tile_rgba(ushort *dst,
 static void
 a4r4g4b4_get_tile_rgba(const ushort *src,
                        unsigned w, unsigned h,
-                       float *p)
+                       uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          const ushort pixel = *src++;
-         PIXEL(p, j, i, 0) = ((pixel >>  8) & 0xf) * (1.0f / 15.0f);
-         PIXEL(p, j, i, 1) = ((pixel >>  4) & 0xf) * (1.0f / 15.0f);
-         PIXEL(p, j, i, 2) = ((pixel      ) & 0xf) * (1.0f / 15.0f);
-         PIXEL(p, j, i, 3) = ((pixel >> 12)      ) * (1.0f / 15.0f);
+         PIXEL(p, j, i, 0) = ((pixel >>  8) & 0xf) * 255 / 15;
+         PIXEL(p, j, i, 1) = ((pixel >>  4) & 0xf) * 255 / 15;
+         PIXEL(p, j, i, 2) = ((pixel      ) & 0xf) * 255 / 15;
+         PIXEL(p, j, i, 3) = ((pixel >> 12)      ) * 255 / 15;
       }
    }
 }
@@ -243,17 +237,17 @@ a4r4g4b4_get_tile_rgba(const ushort *src,
 static void
 a4r4g4b4_put_tile_rgba(ushort *dst,
                        unsigned w, unsigned h,
-                       const float *p)
+                       const uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          unsigned r, g, b, a;
-         r = float_to_ubyte(PIXEL(p, j, i, 0));
-         g = float_to_ubyte(PIXEL(p, j, i, 1));
-         b = float_to_ubyte(PIXEL(p, j, i, 2));
-         a = float_to_ubyte(PIXEL(p, j, i, 3));
+         r = PIXEL(p, j, i, 0);
+         g = PIXEL(p, j, i, 1);
+         b = PIXEL(p, j, i, 2);
+         a = PIXEL(p, j, i, 3);
          r >>= 4;
          g >>= 4;
          b >>= 4;
@@ -269,17 +263,17 @@ a4r4g4b4_put_tile_rgba(ushort *dst,
 static void
 r5g6b5_get_tile_rgba(const ushort *src,
                      unsigned w, unsigned h,
-                     float *p)
+                     uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          const ushort pixel = *src++;
-         PIXEL(p, j, i, 0) = ((pixel >> 11) & 0x1f) * (1.0f / 31.0f);
-         PIXEL(p, j, i, 1) = ((pixel >>  5) & 0x3f) * (1.0f / 63.0f);
-         PIXEL(p, j, i, 2) = ((pixel      ) & 0x1f) * (1.0f / 31.0f);
-         PIXEL(p, j, i, 3) = 1.0f;
+         PIXEL(p, j, i, 0) = ((pixel >> 11) & 0x1f) * 255 / 31;
+         PIXEL(p, j, i, 1) = ((pixel >>  5) & 0x3f) * 255 / 63;
+         PIXEL(p, j, i, 2) = ((pixel      ) & 0x1f) * 255 / 31;
+         PIXEL(p, j, i, 3) = 255;
       }
    }
 }
@@ -288,15 +282,15 @@ r5g6b5_get_tile_rgba(const ushort *src,
 static void
 r5g6b5_put_tile_rgba(ushort *dst,
                      unsigned w, unsigned h,
-                     const float *p)
+                     const uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
-         uint r = (uint) (CLAMP(PIXEL(p, j, i, 0), 0.0, 1.0) * 31.0);
-         uint g = (uint) (CLAMP(PIXEL(p, j, i, 1), 0.0, 1.0) * 63.0);
-         uint b = (uint) (CLAMP(PIXEL(p, j, i, 2), 0.0, 1.0) * 31.0);
+         uint r = (uint) PIXEL(p, j, i, 0) * 31 / 255;
+         uint g = (uint) PIXEL(p, j, i, 1) * 63 / 255;
+         uint b = (uint) PIXEL(p, j, i, 2) * 31 / 255;
          *dst++ = (r << 11) | (g << 5) | (b);
       }
    }
@@ -312,7 +306,7 @@ r5g6b5_put_tile_rgba(ushort *dst,
 static void
 z16_get_tile_rgba(const ushort *src,
                   unsigned w, unsigned h,
-                  float *p)
+                  uint8_t *p)
 {
    const float scale = 1.0f / 65535.0f;
    unsigned i, j;
@@ -335,7 +329,7 @@ z16_get_tile_rgba(const ushort *src,
 static void
 l8_get_tile_rgba(const ubyte *src,
                  unsigned w, unsigned h,
-                 float *p)
+                 uint8_t *p)
 {
    unsigned i, j;
 
@@ -343,8 +337,8 @@ l8_get_tile_rgba(const ubyte *src,
       for (j = 0; j < w; j++, src++) {
          PIXEL(p, j, i, 0) =
          PIXEL(p, j, i, 1) =
-         PIXEL(p, j, i, 2) = ubyte_to_float(*src);
-         PIXEL(p, j, i, 3) = 1.0;
+         PIXEL(p, j, i, 2) = *src;
+         PIXEL(p, j, i, 3) = 255;
       }
    }
 }
@@ -353,14 +347,14 @@ l8_get_tile_rgba(const ubyte *src,
 static void
 l8_put_tile_rgba(ubyte *dst,
                  unsigned w, unsigned h,
-                 const float *p)
+                 const uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          unsigned r;
-         r = float_to_ubyte(PIXEL(p, j, i, 0));
+         r = PIXEL(p, j, i, 0);
          *dst++ = (ubyte) r;
       }
    }
@@ -373,7 +367,7 @@ l8_put_tile_rgba(ubyte *dst,
 static void
 a8_get_tile_rgba(const ubyte *src,
                  unsigned w, unsigned h,
-                 float *p)
+                 uint8_t *p)
 {
    unsigned i, j;
 
@@ -381,8 +375,8 @@ a8_get_tile_rgba(const ubyte *src,
       for (j = 0; j < w; j++, src++) {
          PIXEL(p, j, i, 0) =
          PIXEL(p, j, i, 1) =
-         PIXEL(p, j, i, 2) = 0.0;
-         PIXEL(p, j, i, 3) = ubyte_to_float(*src);
+         PIXEL(p, j, i, 2) = 0;
+         PIXEL(p, j, i, 3) = *src;
       }
    }
 }
@@ -391,14 +385,14 @@ a8_get_tile_rgba(const ubyte *src,
 static void
 a8_put_tile_rgba(ubyte *dst,
                  unsigned w, unsigned h,
-                 const float *p)
+                 const uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          unsigned a;
-         a = float_to_ubyte(PIXEL(p, j, i, 3));
+         a = PIXEL(p, j, i, 3);
          *dst++ = (ubyte) a;
       }
    }
@@ -411,16 +405,16 @@ a8_put_tile_rgba(ubyte *dst,
 static void
 r16_get_tile_rgba(const short *src,
                   unsigned w, unsigned h,
-                  float *p)
+                  uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++, src++) {
-         PIXEL(p, j, i, 0) = SHORT_TO_FLOAT(src[0]);
+         PIXEL(p, j, i, 0) = MAX2(src[0] >> 7, 0);
          PIXEL(p, j, i, 1) =
-         PIXEL(p, j, i, 2) = 0.0;
-         PIXEL(p, j, i, 3) = 1.0;
+         PIXEL(p, j, i, 2) = 0;
+         PIXEL(p, j, i, 3) = 255;
       }
    }
 }
@@ -429,13 +423,13 @@ r16_get_tile_rgba(const short *src,
 static void
 r16_put_tile_rgba(short *dst,
                   unsigned w, unsigned h,
-                  const float *p)
+                  const uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++, dst++) {
-         UNCLAMPED_FLOAT_TO_SHORT(dst[0], PIXEL(p, j, i, 0));
+         dst[0] = PIXEL(p, j, i, 0) << 7;
       }
    }
 }
@@ -446,16 +440,16 @@ r16_put_tile_rgba(short *dst,
 static void
 r16g16b16a16_get_tile_rgba(const short *src,
                            unsigned w, unsigned h,
-                           float *p)
+                           uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++, src += 4) {
-         PIXEL(p, j, i, 0) = SHORT_TO_FLOAT(src[0]);
-         PIXEL(p, j, i, 1) = SHORT_TO_FLOAT(src[1]);
-         PIXEL(p, j, i, 2) = SHORT_TO_FLOAT(src[2]);
-         PIXEL(p, j, i, 3) = SHORT_TO_FLOAT(src[3]);
+         PIXEL(p, j, i, 0) = src[0] >> 8;
+         PIXEL(p, j, i, 1) = src[1] >> 8;
+         PIXEL(p, j, i, 2) = src[2] >> 8;
+         PIXEL(p, j, i, 3) = src[3] >> 8;
       }
    }
 }
@@ -464,16 +458,16 @@ r16g16b16a16_get_tile_rgba(const short *src,
 static void
 r16g16b16a16_put_tile_rgba(short *dst,
                            unsigned w, unsigned h,
-                           const float *p)
+                           const uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++, dst += 4) {
-         UNCLAMPED_FLOAT_TO_SHORT(dst[0], PIXEL(p, j, i, 0));
-         UNCLAMPED_FLOAT_TO_SHORT(dst[1], PIXEL(p, j, i, 1));
-         UNCLAMPED_FLOAT_TO_SHORT(dst[2], PIXEL(p, j, i, 2));
-         UNCLAMPED_FLOAT_TO_SHORT(dst[3], PIXEL(p, j, i, 3));
+         dst[0] = PIXEL(p, j, i, 0) << 8;
+         dst[1] = PIXEL(p, j, i, 1) << 8;
+         dst[2] = PIXEL(p, j, i, 2) << 8;
+         dst[3] = PIXEL(p, j, i, 3) << 8;
       }
    }
 }
@@ -485,7 +479,7 @@ r16g16b16a16_put_tile_rgba(short *dst,
 static void
 i8_get_tile_rgba(const ubyte *src,
                  unsigned w, unsigned h,
-                 float *p)
+                 uint8_t *p)
 {
    unsigned i, j;
 
@@ -494,7 +488,7 @@ i8_get_tile_rgba(const ubyte *src,
          PIXEL(p, j, i, 0) =
          PIXEL(p, j, i, 1) =
          PIXEL(p, j, i, 2) =
-         PIXEL(p, j, i, 3) = ubyte_to_float(*src);
+         PIXEL(p, j, i, 3) = *src;
       }
    }
 }
@@ -503,14 +497,14 @@ i8_get_tile_rgba(const ubyte *src,
 static void
 i8_put_tile_rgba(ubyte *dst,
                  unsigned w, unsigned h,
-                 const float *p)
+                 const uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          unsigned r;
-         r = float_to_ubyte(PIXEL(p, j, i, 0));
+         r = PIXEL(p, j, i, 0);
          *dst++ = (ubyte) r;
       }
    }
@@ -522,7 +516,7 @@ i8_put_tile_rgba(ubyte *dst,
 static void
 a8l8_get_tile_rgba(const ushort *src,
                    unsigned w, unsigned h,
-                   float *p)
+                   uint8_t *p)
 {
    unsigned i, j;
 
@@ -531,8 +525,8 @@ a8l8_get_tile_rgba(const ushort *src,
          ushort ra = *src++;
          PIXEL(p, j, i, 0) =
          PIXEL(p, j, i, 1) =
-         PIXEL(p, j, i, 2) = ubyte_to_float(ra & 0xff);
-         PIXEL(p, j, i, 3) = ubyte_to_float(ra >> 8);
+         PIXEL(p, j, i, 2) = ra & 0xff;
+         PIXEL(p, j, i, 3) = ra >> 8;
       }
    }
 }
@@ -541,15 +535,15 @@ a8l8_get_tile_rgba(const ushort *src,
 static void
 a8l8_put_tile_rgba(ushort *dst,
                    unsigned w, unsigned h,
-                   const float *p)
+                   const uint8_t *p)
 {
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
       for (j = 0; j < w; j++) {
          unsigned r, a;
-         r = float_to_ubyte(PIXEL(p, j, i, 0));
-         a = float_to_ubyte(PIXEL(p, j, i, 3));
+         r = PIXEL(p, j, i, 0);
+         a = PIXEL(p, j, i, 3);
          *dst++ = (a << 8) | r;
       }
    }
@@ -566,7 +560,7 @@ a8l8_put_tile_rgba(ushort *dst,
 static void
 z32_get_tile_rgba(const unsigned *src,
                   unsigned w, unsigned h,
-                  float *p)
+                  uint8_t *p)
 {
    const double scale = 1.0 / (double) 0xffffffff;
    unsigned i, j;
@@ -590,7 +584,7 @@ z32_get_tile_rgba(const unsigned *src,
 static void
 s8z24_get_tile_rgba(const unsigned *src,
                     unsigned w, unsigned h,
-                    float *p)
+                    uint8_t *p)
 {
    const double scale = 1.0 / ((1 << 24) - 1);
    unsigned i, j;
@@ -614,7 +608,7 @@ s8z24_get_tile_rgba(const unsigned *src,
 static void
 z24s8_get_tile_rgba(const unsigned *src,
                     unsigned w, unsigned h,
-                    float *p)
+                    uint8_t *p)
 {
    const double scale = 1.0 / ((1 << 24) - 1);
    unsigned i, j;
@@ -638,7 +632,7 @@ z24s8_get_tile_rgba(const unsigned *src,
 static void
 z32f_get_tile_rgba(const float *src,
                    unsigned w, unsigned h,
-                   float *p)
+                   uint8_t *p)
 {
    unsigned i, j;
 
@@ -661,10 +655,9 @@ z32f_get_tile_rgba(const float *src,
 static void
 ycbcr_get_tile_rgba(const ushort *src,
                     unsigned w, unsigned h,
-                    float *p,
+                    uint8_t *p,
                     boolean rev)
 {
-   const float scale = 1.0f / 255.0f;
    unsigned i, j;
 
    for (i = 0; i < h; i++) {
@@ -690,19 +683,19 @@ ycbcr_get_tile_rgba(const ushort *src,
          r = 1.164f * (y0-16) + 1.596f * (cr-128);
          g = 1.164f * (y0-16) - 0.813f * (cr-128) - 0.391f * (cb-128);
          b = 1.164f * (y0-16) + 2.018f * (cb-128);
-         PIXEL(p, j, i, 0) = r * scale;
-         PIXEL(p, j, i, 1) = g * scale;
-         PIXEL(p, j, i, 2) = b * scale;
-         PIXEL(p, j, i, 3) = 1.0f;
+         PIXEL(p, j, i, 0) = r;
+         PIXEL(p, j, i, 1) = g;
+         PIXEL(p, j, i, 2) = b;
+         PIXEL(p, j, i, 3) = 255;
 
          /* odd pixel: use y1,cr,cb */
          r = 1.164f * (y1-16) + 1.596f * (cr-128);
          g = 1.164f * (y1-16) - 0.813f * (cr-128) - 0.391f * (cb-128);
          b = 1.164f * (y1-16) + 2.018f * (cb-128);
-         PIXEL(p, j + 1, i, 0) = r * scale;
-         PIXEL(p, j + 1, i, 1) = g * scale;
-         PIXEL(p, j + 1, i, 2) = b * scale;
-         PIXEL(p, j + 1, i, 3) = 1.0f;
+         PIXEL(p, j + 1, i, 0) = r;
+         PIXEL(p, j + 1, i, 1) = g;
+         PIXEL(p, j + 1, i, 2) = b;
+         PIXEL(p, j + 1, i, 3) = 255;
       }
       /* do the last texel */
       if (w & 1) {
@@ -725,10 +718,10 @@ ycbcr_get_tile_rgba(const ushort *src,
          r = 1.164f * (y0-16) + 1.596f * (cr-128);
          g = 1.164f * (y0-16) - 0.813f * (cr-128) - 0.391f * (cb-128);
          b = 1.164f * (y0-16) + 2.018f * (cb-128);
-         PIXEL(p, j, i, 0) = r * scale;
-         PIXEL(p, j, i, 1) = g * scale;
-         PIXEL(p, j, i, 2) = b * scale;
-         PIXEL(p, j, i, 3) = 1.0f;
+         PIXEL(p, j, i, 0) = r;
+         PIXEL(p, j, i, 1) = g;
+         PIXEL(p, j, i, 2) = b;
+         PIXEL(p, j, i, 3) = 255;
       }
    }
 }
@@ -737,7 +730,7 @@ ycbcr_get_tile_rgba(const ushort *src,
 static void
 fake_get_tile_rgba(const ushort *src,
                    unsigned w, unsigned h,
-                   float *p)
+                   uint8_t *p)
 {
    unsigned i, j;
 
@@ -746,7 +739,7 @@ fake_get_tile_rgba(const ushort *src,
          PIXEL(p, j, i, 0) =
          PIXEL(p, j, i, 1) =
          PIXEL(p, j, i, 2) =
-         PIXEL(p, j, i, 3) = (i ^ j) & 1 ? 1.0f : 0.0f;
+         PIXEL(p, j, i, 3) = (i ^ j) & 1 ? 255 : 0;
       }
    }
 }
@@ -756,7 +749,7 @@ static void
 lp_tile_raw_to_rgba_soa(enum pipe_format format,
                         void *src,
                         uint w, uint h,
-                        float *p)
+                        uint8_t *p)
 {
    switch (format) {
    case PIPE_FORMAT_A8R8G8B8_UNORM:
@@ -828,7 +821,7 @@ lp_tile_raw_to_rgba_soa(enum pipe_format format,
 void
 lp_get_tile_rgba_soa(struct pipe_transfer *pt,
                      uint x, uint y,
-                     float *p)
+                     uint8_t *p)
 {
    uint w = TILE_SIZE, h = TILE_SIZE;
    void *packed;
@@ -855,7 +848,7 @@ lp_get_tile_rgba_soa(struct pipe_transfer *pt,
 void
 lp_put_tile_rgba_soa(struct pipe_transfer *pt,
                      uint x, uint y,
-                     const float *p)
+                     const uint8_t *p)
 {
    uint w = TILE_SIZE, h = TILE_SIZE;
    void *packed;
diff --git a/src/gallium/drivers/llvmpipe/lp_tile_soa.h b/src/gallium/drivers/llvmpipe/lp_tile_soa.h
index 23b27b77eb8..96fe87245cc 100644
--- a/src/gallium/drivers/llvmpipe/lp_tile_soa.h
+++ b/src/gallium/drivers/llvmpipe/lp_tile_soa.h
@@ -30,13 +30,10 @@
 
 #include "pipe/p_compiler.h"
 
-#include "lp_tile_cache.h"
-
 
 struct pipe_transfer;
 
 
-
 #ifdef __cplusplus
 extern "C" {
 #endif
@@ -45,12 +42,12 @@ extern "C" {
 void
 lp_get_tile_rgba_soa(struct pipe_transfer *pt,
                      uint x, uint y,
-                     float *p);
+                     uint8_t *p);
 
 void
 lp_put_tile_rgba_soa(struct pipe_transfer *pt,
                      uint x, uint y,
-                     const float *p);
+                     const uint8_t *p);
 
 
 #ifdef __cplusplus