Mesa (master): freedreno/a6xx: Consolidate computing blit_cntl

[GitLab Mirror]

Module: Mesa
Branch: master
Commit: 416513d915ba55365b94937578f8bb8c207a3684
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=416513d915ba55365b94937578f8bb8c207a3684

Author: Kristian H. Kristensen <hoegsberg at google.com>
Date:   Thu Jul  2 04:03:16 2020 -0700

freedreno/a6xx: Consolidate computing blit_cntl

Compute the blit_cntl value in one place and group it with the
register writes.

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5717>

---

 src/gallium/drivers/freedreno/a6xx/fd6_blitter.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/gallium/drivers/freedreno/a6xx/fd6_blitter.c b/src/gallium/drivers/freedreno/a6xx/fd6_blitter.c
index 93e460acce2..5207a26abfc 100644
--- a/src/gallium/drivers/freedreno/a6xx/fd6_blitter.c
+++ b/src/gallium/drivers/freedreno/a6xx/fd6_blitter.c
@@ -538,11 +538,7 @@ emit_blit_or_clear_texture(struct fd_context *ctx, struct fd_ringbuffer *ring,
 	OUT_RING(ring, A6XX_GRAS_2D_DST_TL_X(dx1) | A6XX_GRAS_2D_DST_TL_Y(dy1));
 	OUT_RING(ring, A6XX_GRAS_2D_DST_BR_X(dx2) | A6XX_GRAS_2D_DST_BR_Y(dy2));
 
-	uint32_t blit_cntl = blit_control(dfmt, util_format_is_srgb(info->dst.format));
-
 	if (color) {
-		blit_cntl |= A6XX_RB_2D_BLIT_CNTL_SOLID_COLOR;
-
 		switch (info->dst.format) {
 		case PIPE_FORMAT_Z24X8_UNORM:
 		case PIPE_FORMAT_Z24_UNORM_S8_UINT:
@@ -595,9 +591,13 @@ emit_blit_or_clear_texture(struct fd_context *ctx, struct fd_ringbuffer *ring,
 				 A6XX_GRAS_RESOLVE_CNTL_1_Y(info->scissor.miny));
 		OUT_RING(ring, A6XX_GRAS_RESOLVE_CNTL_1_X(info->scissor.maxx - 1) |
 				 A6XX_GRAS_RESOLVE_CNTL_1_Y(info->scissor.maxy - 1));
-		blit_cntl |= A6XX_RB_2D_BLIT_CNTL_SCISSOR;
 	}
 
+	uint32_t blit_cntl =
+		blit_control(dfmt, util_format_is_srgb(info->dst.format)) |
+		COND(color, A6XX_RB_2D_BLIT_CNTL_SOLID_COLOR) |
+		COND(info->scissor_enable, A6XX_RB_2D_BLIT_CNTL_SCISSOR);
+
 	OUT_PKT4(ring, REG_A6XX_RB_2D_BLIT_CNTL, 1);
 	OUT_RING(ring, blit_cntl);