summaryrefslogtreecommitdiff
path: root/src/mesa/drivers/dri/unichrome/via_texstate.c
diff options
context:
space:
mode:
Diffstat (limited to 'src/mesa/drivers/dri/unichrome/via_texstate.c')
-rw-r--r--src/mesa/drivers/dri/unichrome/via_texstate.c17
1 files changed, 9 insertions, 8 deletions
diff --git a/src/mesa/drivers/dri/unichrome/via_texstate.c b/src/mesa/drivers/dri/unichrome/via_texstate.c
index daf7212e7e..51a27ef118 100644
--- a/src/mesa/drivers/dri/unichrome/via_texstate.c
+++ b/src/mesa/drivers/dri/unichrome/via_texstate.c
@@ -173,7 +173,12 @@ static void viaSetTexImages(viaContextPtr vmesa,
if (VIA_DEBUG) fprintf(stderr, "%s - in\n", __FUNCTION__);
switch (baseImage->TexFormat->MesaFormat) {
case MESA_FORMAT_ARGB8888:
- if (t->image[tObj->BaseLevel].internalFormat == GL_RGB)
+ /* KW: I'm not sure what the thinking behind this test was, but
+ * it definitely didn't work, see failures in demos/texenv.c
+ * and glean texEnv test, both of which are corrected by
+ * disabling the 0888 option:
+ */
+ if (t->image[tObj->BaseLevel].internalFormat == GL_RGB && 0)
texFormat = HC_HTXnFM_ARGB0888;
else
texFormat = HC_HTXnFM_ARGB8888;
@@ -408,13 +413,9 @@ static GLboolean viaUpdateTexUnit(GLcontext *ctx, GLuint unit)
}
}
-void viaUpdateTextureState(GLcontext *ctx)
+GLboolean viaUpdateTextureState(GLcontext *ctx)
{
- viaContextPtr vmesa = VIA_CONTEXT(ctx);
-
- GLuint ok = (viaUpdateTexUnit(ctx, 0) &&
- viaUpdateTexUnit(ctx, 1));
-
- FALLBACK(vmesa, VIA_FALLBACK_TEXTURE, !ok);
+ return (viaUpdateTexUnit(ctx, 0) &&
+ viaUpdateTexUnit(ctx, 1));
}