From cd58efd0554d9c97373a056e116934579ede207b Mon Sep 17 00:00:00 2001 From: Lucas Stach Date: Sun, 8 Jun 2025 19:19:07 +0200 Subject: [PATCH] etnaviv: drop superfluous PE alignment check This check and especially the XXX in the comment hasn't been valid for a long time, as our mipmap layout and render shadow code makes sure that all levels are properly aligned for PE usage. Signed-off-by: Lucas Stach Reviewed-by: Christian Gmeiner Part-of: --- src/gallium/drivers/etnaviv/etnaviv_state.c | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/src/gallium/drivers/etnaviv/etnaviv_state.c b/src/gallium/drivers/etnaviv/etnaviv_state.c index 293643925a7..20553b8d16a 100644 --- a/src/gallium/drivers/etnaviv/etnaviv_state.c +++ b/src/gallium/drivers/etnaviv/etnaviv_state.c @@ -208,16 +208,6 @@ etna_set_framebuffer_state(struct pipe_context *pctx, * VIVS_PE_COLOR_FORMAT_OVERWRITE comes from blend_state * but only if we set the bits above. */ /* merged with depth_stencil_alpha */ - if ((cbuf->offset & 63) || - (((cbuf->level->stride * 4) & 63) && cbuf->level->height > 4)) { - /* XXX Must make temporary surface here. - * Need the same mechanism on gc2000 when we want to do mipmap - * generation by - * rendering to levels > 1 due to multitiled / tiled conversion. */ - BUG("Alignment error, trying to render to offset %08x with tile " - "stride %i", - cbuf->offset, cbuf->level->stride * 4); - } cs->PE_COLOR_STRIDE = cbuf->level->stride;