freedreno: always set all border colors
authorIlia Mirkin <imirkin@alum.mit.edu>
Fri, 20 Nov 2015 22:04:05 +0000 (17:04 -0500)
committerIlia Mirkin <imirkin@alum.mit.edu>
Fri, 20 Nov 2015 22:44:10 +0000 (17:44 -0500)
Instead of playing the guessing game as to which texture format reads
from which border color encoding type, just write both of them always.

Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
src/gallium/drivers/freedreno/freedreno_texture.c

index f03b65b0ae522db5315e34604bba45bc5cf2499f..f5611abaec8d72c6a70cb90b0c7bd29b9a2ec0f5 100644 (file)
@@ -197,37 +197,15 @@ fd_setup_border_colors(struct fd_texture_stateobj *tex, void *ptr,
                                        continue;
 
                                const struct util_format_channel_description *chan =
-                                               &desc->channel[desc->swizzle[j]];
-                               int size = chan->size;
-
-                               /* The Z16 texture format we use seems to look in the
-                                * 32-bit border color slots
-                                */
-                               if (desc->colorspace == UTIL_FORMAT_COLORSPACE_ZS)
-                                       size = 32;
-
-                               /* Formats like R11G11B10 or RGB9_E5 don't specify
-                                * per-channel sizes properly.
-                                */
-                               if (desc->layout == UTIL_FORMAT_LAYOUT_OTHER)
-                                       size = 16;
-
-                               /* We fake RGTC as if it were RGBA8 */
-                               if (desc->layout == UTIL_FORMAT_LAYOUT_RGTC)
-                                       size = 8;
-
-                               if (chan->pure_integer && size > 16)
-                                       bcolor32[desc->swizzle[j] + 4] =
-                                                       sampler->border_color.i[j];
-                               else if (size > 16)
-                                       bcolor32[desc->swizzle[j]] =
-                                                       fui(sampler->border_color.f[j]);
-                               else if (chan->pure_integer)
-                                       bcolor[desc->swizzle[j] + 8] =
-                                                       sampler->border_color.i[j];
-                               else
+                                       &desc->channel[desc->swizzle[j]];
+                               if (chan->pure_integer) {
+                                       bcolor32[desc->swizzle[j] + 4] = sampler->border_color.i[j];
+                                       bcolor[desc->swizzle[j] + 8] = sampler->border_color.i[j];
+                               } else {
+                                       bcolor32[desc->swizzle[j]] = fui(sampler->border_color.f[j]);
                                        bcolor[desc->swizzle[j]] =
-                                                       util_float_to_half(sampler->border_color.f[j]);
+                                               util_float_to_half(sampler->border_color.f[j]);
+                               }
                        }
                }
        }