[Mesa-dev] [PATCH v3 3/8] freedreno: a2xx: Fix fd2_tex_swiz

Wladimir J. van der Laan laanwj at gmail.com
Fri Mar 23 11:21:34 UTC 2018


Compose swizzles using util_format_compose_swizzles instead
of the custom code (which somehow had a bug).

This makes the GL_ALPHA internal format work.

Signed-off-by: Wladimir J. van der Laan <laanwj at gmail.com>
Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>
---
 src/gallium/drivers/freedreno/a2xx/fd2_util.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/gallium/drivers/freedreno/a2xx/fd2_util.c b/src/gallium/drivers/freedreno/a2xx/fd2_util.c
index 0bdcfcd..25f2bf4 100644
--- a/src/gallium/drivers/freedreno/a2xx/fd2_util.c
+++ b/src/gallium/drivers/freedreno/a2xx/fd2_util.c
@@ -309,14 +309,14 @@ fd2_tex_swiz(enum pipe_format format, unsigned swizzle_r, unsigned swizzle_g,
 {
 	const struct util_format_description *desc =
 			util_format_description(format);
-	uint8_t swiz[] = {
-			swizzle_r, swizzle_g, swizzle_b, swizzle_a,
-			PIPE_SWIZZLE_0, PIPE_SWIZZLE_1,
-			PIPE_SWIZZLE_1, PIPE_SWIZZLE_1,
-	};
+	unsigned char swiz[4] = {
+		swizzle_r, swizzle_g, swizzle_b, swizzle_a,
+	}, rswiz[4];
 
-	return A2XX_SQ_TEX_3_SWIZ_X(tex_swiz(swiz[desc->swizzle[0]])) |
-			A2XX_SQ_TEX_3_SWIZ_Y(tex_swiz(swiz[desc->swizzle[1]])) |
-			A2XX_SQ_TEX_3_SWIZ_Z(tex_swiz(swiz[desc->swizzle[2]])) |
-			A2XX_SQ_TEX_3_SWIZ_W(tex_swiz(swiz[desc->swizzle[3]]));
+	util_format_compose_swizzles(desc->swizzle, swiz, rswiz);
+
+	return A2XX_SQ_TEX_3_SWIZ_X(tex_swiz(rswiz[0])) |
+			A2XX_SQ_TEX_3_SWIZ_Y(tex_swiz(rswiz[1])) |
+			A2XX_SQ_TEX_3_SWIZ_Z(tex_swiz(rswiz[2])) |
+			A2XX_SQ_TEX_3_SWIZ_W(tex_swiz(rswiz[3]));
 }
-- 
2.7.4



More information about the mesa-dev mailing list