From 904015ae855a6a3ae4d25a38e593410be60d1aa5 Mon Sep 17 00:00:00 2001 From: Vinson Lee Date: Sun, 12 Nov 2023 10:33:43 -0800 Subject: [PATCH] ac/surface/tests: Remove duplicate variable block_size_bits MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fix defect reported by Coverity Scan. Evaluation order violation (EVALUATION_ORDER) write_write_typo: In block_size_bits = block_size_bits = ((surf.u.gfx9.swizzle_mode >= ADDR_SW_256KB_Z_X) ? 18 : 16), block_size_bits is written twice with the same value. Fixes: 44eaf50a34b ("ac/surface/tests: cosmetic changes") Signed-off-by: Vinson Lee Reviewed-by: Marek Olšák Part-of: --- src/amd/common/ac_surface_modifier_test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/amd/common/ac_surface_modifier_test.c b/src/amd/common/ac_surface_modifier_test.c index 49854a8ff8f..82d2d4d6f50 100644 --- a/src/amd/common/ac_surface_modifier_test.c +++ b/src/amd/common/ac_surface_modifier_test.c @@ -261,7 +261,7 @@ static void test_modifier(const struct radeon_info *info, unsigned aligned_pitch, aligned_height; if (modifier != DRM_FORMAT_MOD_LINEAR) { unsigned block_size_bits = - block_size_bits = surf.u.gfx9.swizzle_mode >= ADDR_SW_256KB_Z_X ? 18 : 16; + surf.u.gfx9.swizzle_mode >= ADDR_SW_256KB_Z_X ? 18 : 16; surf_size = block_count(dims[i][0], dims[i][1], elem_bits, block_size_bits, &aligned_pitch,