mesa: get rid of homegrown logbase2 implementation in drivers
authorRoland Scheidegger <sroland@vmware.com>
Wed, 8 Jun 2011 22:47:19 +0000 (00:47 +0200)
committerRoland Scheidegger <sroland@vmware.com>
Wed, 8 Jun 2011 23:14:51 +0000 (01:14 +0200)
Some of the logbase2 functions did just the same as _mesa_logbase2,
though they were taking signed numbers (but it shouldn't matter for them).

src/mesa/drivers/dri/intel/intel_tex_image.c
src/mesa/drivers/dri/unichrome/via_tex.c

index 775fd1008f95d5956aa903f3cd0efb034670d3c9..bc39f4ad83f412539af5630bd9eb758950454769 100644 (file)
  */
 
 
-static int
-logbase2(int n)
-{
-   GLint i = 1;
-   GLint log2 = 0;
-
-   while (n > i) {
-      i *= 2;
-      log2++;
-   }
-
-   return log2;
-}
-
 
 /* Otherwise, store it in memory if (Border != 0) or (any dimension ==
  * 1).
@@ -118,7 +104,7 @@ intel_miptree_create_for_teximage(struct intel_context *intel,
          (intel->gen < 4 || firstLevel == 0)) {
         lastLevel = firstLevel;
       } else {
-        lastLevel = firstLevel + logbase2(MAX2(MAX2(width, height), depth));
+        lastLevel = firstLevel + _mesa_logbase2(MAX2(MAX2(width, height), depth));
       }
    }
 
index 1a0d1eaf0709333998cf46cce31c87290c283ede..fe5480ff25cd008a59a7e98453468b9372688fa0 100644 (file)
@@ -196,18 +196,6 @@ viaChooseTexFormat( struct gl_context *ctx, GLint internalFormat,
    return MESA_FORMAT_NONE; /* never get here */
 }
 
-static int logbase2(int n)
-{
-   GLint i = 1;
-   GLint log2 = 0;
-
-   while (n > i) {
-      i *= 2;
-      log2++;
-   }
-
-   return log2;
-}
 
 static const char *get_memtype_name( GLint memType )
 {
@@ -690,7 +678,7 @@ static void viaTexImage(struct gl_context *ctx,
    }
 
    assert(texImage->RowStride == postConvWidth);
-   viaImage->pitchLog2 = logbase2(postConvWidth * texelBytes);
+   viaImage->pitchLog2 = _mesa_logbase2(postConvWidth * texelBytes);
 
    /* allocate memory */
    if (_mesa_is_format_compressed(texImage->TexFormat))