From patchwork Thu May 9 15:41:33 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Connor Abbott X-Patchwork-Id: 13659833 Received: from mail-lj1-f171.google.com (mail-lj1-f171.google.com [209.85.208.171]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4591216F26E for ; Thu, 9 May 2024 15:41:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.171 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715269302; cv=none; b=Zf7yFsaRyaI1kEoPYSajQssEInKZOrNovlZgAe8BPsZZtP/gW9BApunbpUbGxqHn3mPpcdloa4jPtZq0CUws6ng8TjH8MiL2CL05tfVQYUDRVUawp5hflAA4u4g4yPT2g6IJre0AtimNue7PGEhv6BFjY8NOxLb8kxUlPvTR28k= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715269302; c=relaxed/simple; bh=tA+6pkx+Ynhfs2J+gOdH/Qp/VvHds6NwR6jSCdCdBX8=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=gZjVfg//RM10q7A4an9FJTVg0sMupqZRnkKU/YTF6tZwfp8XqK1tfg1ApOJ3Edm3pqYL69cLNyggj0JELV03klFtbE/EYG9DEqjy3NzKmPPGSraMgXdOjInOcfUgWZc+5bfErzn75ETnaLKaqMEIyb4OKut7RKBUSqdX9WFdsx0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=MEmyaNdI; arc=none smtp.client-ip=209.85.208.171 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="MEmyaNdI" Received: by mail-lj1-f171.google.com with SMTP id 38308e7fff4ca-2e1fa824504so14029791fa.0 for ; Thu, 09 May 2024 08:41:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1715269298; x=1715874098; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=60C0NrLWh3mQZJWX6R2x91+C9pDjFcxDbObVrF8FXCk=; b=MEmyaNdI9USSmBWZ5yVxh5OSX7RBOWKFQOQtoHhHPTCujXsL5EbqGJ6OFkOaL03MjV iZekxIwlRMYLR1IpqPlRIEgLDqqjC+NR4ajQJQfK+UlRxGj+0WrmgsGowQrd88MYoVYe X+nE0eo+P1xU16Uz+Tpcv31U8XDh4wK4fzGJwsxmMLON+vO4F2p4qeVp83aJhbIkRbDf xCE7yoeZC/SeHoA/TSlmB3W2Ek90mto7r6RxHmBKSSbQsNWt5SsrSXZRDBWZOAbzPBlu v5/SYwbA47A34VzwNciIuxvwit27VP9xvX3sdbPjh2h6r+NyK1gdQlt0Kp3e7+wUwXL/ DXEg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1715269298; x=1715874098; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=60C0NrLWh3mQZJWX6R2x91+C9pDjFcxDbObVrF8FXCk=; b=AgnY90XxZ+HZlFaCkjjkuYTSYovJP7+VCzhR1dcqDCB5WGa5Kyq1R3ny5IxYgIUIWY sKyxwBbDmX+wBWyN8njcKocWPlpyhCSwyCrAYUfvjPDIS8V1TtiE8xWL0KOzrli4MT09 pZuxhw7ACxOLtBHqtFTG0tz3nMiCyhrtTKnleAgG3dVJXBzTPGHWf+MrIN31RogIW7Dk oMDEf6hD7+kjUQVrkYcfpiTzDMm3bR/9PPtuNfELzk5b37oW97oBZFCQ7QNcSY4jdY7r NYdP0TPuQAVe9wXIZ8M2MSJK6R9CGPn3xk5gOgCBAb4pBjg9I8ONaZXWS2vwf8oVvLJJ KAzA== X-Gm-Message-State: AOJu0YzCkM5o10elQ+XmojcAtq6fTKPu9ktc5p8a0rvRxfPN0TY/MbUR 4YAHkIvutlZcyEpJLRZTpbHDm7182zsFydcC8ttTlFgBwhAPE7L9 X-Google-Smtp-Source: AGHT+IEJWRpyAa5AFHE96im8sOUhoqIb+ceKmf5JbUPShXGbd4D9iFgw3yrji/pxcIwRkAy8GQrAOQ== X-Received: by 2002:a2e:8317:0:b0:2e1:18d:5b4f with SMTP id 38308e7fff4ca-2e4476ad07fmr35697581fa.42.1715269298361; Thu, 09 May 2024 08:41:38 -0700 (PDT) Received: from [192.168.0.20] (cpc115152-dals23-2-0-cust532.20-2.cable.virginm.net. [86.12.82.21]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-41fccce1b17sm29577555e9.12.2024.05.09.08.41.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 09 May 2024 08:41:37 -0700 (PDT) From: Connor Abbott Date: Thu, 09 May 2024 16:41:33 +0100 Subject: [PATCH 1/2] drm/msm: Use a7xx family directly in gpu_state Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20240509-a750-devcoredump-fixes-v1-1-60cc99e2147d@gmail.com> References: <20240509-a750-devcoredump-fixes-v1-0-60cc99e2147d@gmail.com> In-Reply-To: <20240509-a750-devcoredump-fixes-v1-0-60cc99e2147d@gmail.com> To: Rob Clark , Abhinav Kumar , Dmitry Baryshkov , Sean Paul , Marijn Suijten Cc: linux-arm-msm@vger.kernel.org, freedreno@lists.freedesktop.org, Connor Abbott X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=ed25519-sha256; t=1715269296; l=6824; i=cwabbott0@gmail.com; s=20240426; h=from:subject:message-id; bh=tA+6pkx+Ynhfs2J+gOdH/Qp/VvHds6NwR6jSCdCdBX8=; b=rSJYkjhFPTn9uTxy1sR6khmRbTfhjnMWYKFSowgBI5Qoj9Xvh00chIAOV6tgsXeeVEWdlrdYq wE7LP3cWftAAyzw7FRikECk04KF4zI6dUHBpRc7kSxN01Ltbp0l0fTT X-Developer-Key: i=cwabbott0@gmail.com; a=ed25519; pk=dkpOeRSXLzVgqhy0Idr3nsBr4ranyERLMnoAgR4cHmY= With a7xx, we need to import a new header for each new generation and switch to a different list of registers, instead of making backwards-compatible changes. Using the helpers inadvertently made a750 use the a740 list of registers, instead use the family directly to fix this. Fixes: f3f8207d8aed ("drm/msm: Add devcoredump support for a750") Signed-off-by: Connor Abbott --- drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c | 41 ++++++++++++++--------------- 1 file changed, 20 insertions(+), 21 deletions(-) diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c b/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c index 77146d30bcaa..c641ee7dec78 100644 --- a/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c +++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c @@ -390,18 +390,18 @@ static void a7xx_get_debugbus_blocks(struct msm_gpu *gpu, const u32 *debugbus_blocks, *gbif_debugbus_blocks; int i; - if (adreno_is_a730(adreno_gpu)) { + if (adreno_gpu->info->family == ADRENO_7XX_GEN1) { debugbus_blocks = gen7_0_0_debugbus_blocks; debugbus_blocks_count = ARRAY_SIZE(gen7_0_0_debugbus_blocks); gbif_debugbus_blocks = a7xx_gbif_debugbus_blocks; gbif_debugbus_blocks_count = ARRAY_SIZE(a7xx_gbif_debugbus_blocks); - } else if (adreno_is_a740_family(adreno_gpu)) { + } else if (adreno_gpu->info->family == ADRENO_7XX_GEN2) { debugbus_blocks = gen7_2_0_debugbus_blocks; debugbus_blocks_count = ARRAY_SIZE(gen7_2_0_debugbus_blocks); gbif_debugbus_blocks = a7xx_gbif_debugbus_blocks; gbif_debugbus_blocks_count = ARRAY_SIZE(a7xx_gbif_debugbus_blocks); } else { - BUG_ON(!adreno_is_a750(adreno_gpu)); + BUG_ON(adreno_gpu->info->family != ADRENO_7XX_GEN3); debugbus_blocks = gen7_9_0_debugbus_blocks; debugbus_blocks_count = ARRAY_SIZE(gen7_9_0_debugbus_blocks); gbif_debugbus_blocks = gen7_9_0_gbif_debugbus_blocks; @@ -511,7 +511,7 @@ static void a6xx_get_debugbus(struct msm_gpu *gpu, const struct a6xx_debugbus_block *cx_debugbus_blocks; if (adreno_is_a7xx(adreno_gpu)) { - BUG_ON(!(adreno_is_a730(adreno_gpu) || adreno_is_a740_family(adreno_gpu))); + BUG_ON(adreno_gpu->info->family > ADRENO_7XX_GEN3); cx_debugbus_blocks = a7xx_cx_debugbus_blocks; nr_cx_debugbus_blocks = ARRAY_SIZE(a7xx_cx_debugbus_blocks); } else { @@ -662,11 +662,11 @@ static void a7xx_get_dbgahb_clusters(struct msm_gpu *gpu, const struct gen7_sptp_cluster_registers *dbgahb_clusters; unsigned dbgahb_clusters_size; - if (adreno_is_a730(adreno_gpu)) { + if (adreno_gpu->info->family == ADRENO_7XX_GEN1) { dbgahb_clusters = gen7_0_0_sptp_clusters; dbgahb_clusters_size = ARRAY_SIZE(gen7_0_0_sptp_clusters); } else { - BUG_ON(!adreno_is_a740_family(adreno_gpu)); + BUG_ON(adreno_gpu->info->family > ADRENO_7XX_GEN3); dbgahb_clusters = gen7_2_0_sptp_clusters; dbgahb_clusters_size = ARRAY_SIZE(gen7_2_0_sptp_clusters); } @@ -820,14 +820,14 @@ static void a7xx_get_clusters(struct msm_gpu *gpu, const struct gen7_cluster_registers *clusters; unsigned clusters_size; - if (adreno_is_a730(adreno_gpu)) { + if (adreno_gpu->info->family == ADRENO_7XX_GEN1) { clusters = gen7_0_0_clusters; clusters_size = ARRAY_SIZE(gen7_0_0_clusters); - } else if (adreno_is_a740_family(adreno_gpu)) { + } else if (adreno_gpu->info->family == ADRENO_7XX_GEN2) { clusters = gen7_2_0_clusters; clusters_size = ARRAY_SIZE(gen7_2_0_clusters); } else { - BUG_ON(!adreno_is_a750(adreno_gpu)); + BUG_ON(adreno_gpu->info->family != ADRENO_7XX_GEN3); clusters = gen7_9_0_clusters; clusters_size = ARRAY_SIZE(gen7_9_0_clusters); } @@ -895,7 +895,7 @@ static void a7xx_get_shader_block(struct msm_gpu *gpu, if (WARN_ON(datasize > A6XX_CD_DATA_SIZE)) return; - if (adreno_is_a730(adreno_gpu)) { + if (adreno_gpu->info->family == ADRENO_7XX_GEN1) { gpu_rmw(gpu, REG_A7XX_SP_DBG_CNTL, GENMASK(1, 0), 3); } @@ -925,7 +925,7 @@ static void a7xx_get_shader_block(struct msm_gpu *gpu, datasize); out: - if (adreno_is_a730(adreno_gpu)) { + if (adreno_gpu->info->family == ADRENO_7XX_GEN1) { gpu_rmw(gpu, REG_A7XX_SP_DBG_CNTL, GENMASK(1, 0), 0); } } @@ -958,14 +958,14 @@ static void a7xx_get_shaders(struct msm_gpu *gpu, unsigned num_shader_blocks; int i; - if (adreno_is_a730(adreno_gpu)) { + if (adreno_gpu->info->family == ADRENO_7XX_GEN1) { shader_blocks = gen7_0_0_shader_blocks; num_shader_blocks = ARRAY_SIZE(gen7_0_0_shader_blocks); - } else if (adreno_is_a740_family(adreno_gpu)) { + } else if (adreno_gpu->info->family == ADRENO_7XX_GEN2) { shader_blocks = gen7_2_0_shader_blocks; num_shader_blocks = ARRAY_SIZE(gen7_2_0_shader_blocks); } else { - BUG_ON(!adreno_is_a750(adreno_gpu)); + BUG_ON(adreno_gpu->info->family != ADRENO_7XX_GEN3); shader_blocks = gen7_9_0_shader_blocks; num_shader_blocks = ARRAY_SIZE(gen7_9_0_shader_blocks); } @@ -1350,14 +1350,14 @@ static void a7xx_get_registers(struct msm_gpu *gpu, const u32 *pre_crashdumper_regs; const struct gen7_reg_list *reglist; - if (adreno_is_a730(adreno_gpu)) { + if (adreno_gpu->info->family == ADRENO_7XX_GEN1) { reglist = gen7_0_0_reg_list; pre_crashdumper_regs = gen7_0_0_pre_crashdumper_gpu_registers; - } else if (adreno_is_a740_family(adreno_gpu)) { + } else if (adreno_gpu->info->family == ADRENO_7XX_GEN2) { reglist = gen7_2_0_reg_list; pre_crashdumper_regs = gen7_0_0_pre_crashdumper_gpu_registers; } else { - BUG_ON(!adreno_is_a750(adreno_gpu)); + BUG_ON(adreno_gpu->info->family != ADRENO_7XX_GEN3); reglist = gen7_9_0_reg_list; pre_crashdumper_regs = gen7_9_0_pre_crashdumper_gpu_registers; } @@ -1407,8 +1407,7 @@ static void a7xx_get_post_crashdumper_registers(struct msm_gpu *gpu, struct adreno_gpu *adreno_gpu = to_adreno_gpu(gpu); const u32 *regs; - BUG_ON(!(adreno_is_a730(adreno_gpu) || adreno_is_a740_family(adreno_gpu) || - adreno_is_a750(adreno_gpu))); + BUG_ON(adreno_gpu->info->family > ADRENO_7XX_GEN3); regs = gen7_0_0_post_crashdumper_registers; a7xx_get_ahb_gpu_registers(gpu, @@ -1514,11 +1513,11 @@ static void a7xx_get_indexed_registers(struct msm_gpu *gpu, const struct a6xx_indexed_registers *indexed_regs; int i, indexed_count, mempool_count; - if (adreno_is_a730(adreno_gpu) || adreno_is_a740_family(adreno_gpu)) { + if (adreno_gpu->info->family <= ADRENO_7XX_GEN2) { indexed_regs = a7xx_indexed_reglist; indexed_count = ARRAY_SIZE(a7xx_indexed_reglist); } else { - BUG_ON(!adreno_is_a750(adreno_gpu)); + BUG_ON(adreno_gpu->info->family != ADRENO_7XX_GEN3); indexed_regs = gen7_9_0_cp_indexed_reg_list; indexed_count = ARRAY_SIZE(gen7_9_0_cp_indexed_reg_list); } From patchwork Thu May 9 15:41:34 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Connor Abbott X-Patchwork-Id: 13659834 Received: from mail-lj1-f178.google.com (mail-lj1-f178.google.com [209.85.208.178]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6004516F271 for ; Thu, 9 May 2024 15:41:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.178 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715269303; cv=none; b=WziS/GkJVCEWwylNWHvG9/TrvpY+QyHqQwjTwEo677rU6hSzBRsHrwrQNy1FcWViDU3OOrpLDCII3QP9BQoOg16C5EulbNF2RthRDlmySJTxRTTwsRF8aIsp2Lllu/h2z6jS7d6nYUc26u3e88TzsHS1Ua5kZTtaTE4ohEdceDw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715269303; c=relaxed/simple; bh=YDTMlYTd8Jl+1LbuEOwzVy2qVGX9nsvbpTUbbqpunaM=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=bP1fQQf6+S1tMseWQx8neLf83J4I+3PLa1VsW+YMw0LKXwEx7Y46qSch82YJizIlyWVcR7cGg6EAzQUOFZ2f+6gfygf2VHsIWEx+ZJdb5M+vAtwarKNvaRqFVOGHPQP+Aa+8nGIQko6rf7/7Uqb7dSFCGQP7jY2lMYol9gI82yA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=TOLUgBfi; arc=none smtp.client-ip=209.85.208.178 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="TOLUgBfi" Received: by mail-lj1-f178.google.com with SMTP id 38308e7fff4ca-2e242b1dfd6so13109531fa.0 for ; Thu, 09 May 2024 08:41:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1715269299; x=1715874099; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=MHCgfOjCAWuaCf+wlNyjS/SdCnZB5VMR+4B9ySlLbTk=; b=TOLUgBfi++4EV1OsrHY5u6zEi7RLi/t4CeZmc8u80ggI6hlU8IAVMT0tu82ZRbVqER 6AkZnWRv9GkLcXCWIXJ6uq4YXDAaMQS769rzlBGEqDXV/KhJzs32l5tfXkcSo4Pq0Ddu OfayXYVzJPuIlpRIiwQ9/hoGQBwa05nXpZczVpN1xxkpZMo+7wFxAvhHG6lvUL3COGRw fdkUBh7rk9Wa2oXb72Q2vY5gK2EuJlIIKBOctS8m+HMdLHefsKl2vytq7Qpb1ppV21vF FjOxRIPDYAGgth2ZXZ4bbEvgLhyp45g0wqLmxiaq6F3/+j8du5qlN1i8+c07JlWkQQHS 43KQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1715269299; x=1715874099; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=MHCgfOjCAWuaCf+wlNyjS/SdCnZB5VMR+4B9ySlLbTk=; b=mz5eH6zRHgHiRUTwFo1rQVgb914bqvK6yXcfq5+RCmxxTLXSWoty8fIdPYQTS/AXg3 gbf9hwh6PunA24LfKV6L2koamnlMN1c/LMPL1vKZRQc79Ddblhs8L/xh0ahoCxd557Jd wbuG8BAhcJ9TNDsCVxOznN+zn0BMlFtAGTccZDvsB/Mh1XGxgE+4Ab5gIM03WWh0lsQq +lkLjrE2QdKtF/aH7+mUFFlQGrKPD8NWHBZvcKsNh3IvDqUkxQuJ2XriGIiRCRbedFMK O3Ncz+pnlodXi/qJswNzPnNoaB2DQ22flotPi58ml60PDsXJaD45YgVNBm7M4vM4gYiZ yx7Q== X-Gm-Message-State: AOJu0YwqNTEYM3V91WbUmPQt2qUa63vPmkP+EBbLe35QSmrAIvJg1/Mg lj5aI0aENcaJBeltLm5GIgSFoslyYkKFlAqqOUcGpf9fa72Y5LeJ X-Google-Smtp-Source: AGHT+IGW47YrANCiM7MAOv1YCC4wAKiyKqY/iX4tT8I3a2vjfykMHd5JV7q+HsYEiB/NPT6K68bqDQ== X-Received: by 2002:a05:651c:11d3:b0:2e2:f2d:b8ad with SMTP id 38308e7fff4ca-2e447084a8amr35217221fa.26.1715269299347; Thu, 09 May 2024 08:41:39 -0700 (PDT) Received: from [192.168.0.20] (cpc115152-dals23-2-0-cust532.20-2.cable.virginm.net. [86.12.82.21]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-41fccce1b17sm29577555e9.12.2024.05.09.08.41.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 09 May 2024 08:41:38 -0700 (PDT) From: Connor Abbott Date: Thu, 09 May 2024 16:41:34 +0100 Subject: [PATCH 2/2] drm/msm: Dump correct dbgahb clusters on a750 Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20240509-a750-devcoredump-fixes-v1-2-60cc99e2147d@gmail.com> References: <20240509-a750-devcoredump-fixes-v1-0-60cc99e2147d@gmail.com> In-Reply-To: <20240509-a750-devcoredump-fixes-v1-0-60cc99e2147d@gmail.com> To: Rob Clark , Abhinav Kumar , Dmitry Baryshkov , Sean Paul , Marijn Suijten Cc: linux-arm-msm@vger.kernel.org, freedreno@lists.freedesktop.org, Connor Abbott X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=ed25519-sha256; t=1715269296; l=1299; i=cwabbott0@gmail.com; s=20240426; h=from:subject:message-id; bh=YDTMlYTd8Jl+1LbuEOwzVy2qVGX9nsvbpTUbbqpunaM=; b=BKxi8bwquihpob2UmeuDEeaUfWIOAKCHgPIrTzz/DiJ4w779kGeQf/VyOcZzZ0xd9qyIt1H7j Y3w+RuiAezpBAIKcybt38M7n75/2t5DOEoVNJ1g+mM0TfEV2EuG1MUZ X-Developer-Key: i=cwabbott0@gmail.com; a=ed25519; pk=dkpOeRSXLzVgqhy0Idr3nsBr4ranyERLMnoAgR4cHmY= This was missed thanks to the family mixup fixed in the previous commit. Fixes: f3f8207d8aed ("drm/msm: Add devcoredump support for a750") Signed-off-by: Connor Abbott --- drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c b/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c index c641ee7dec78..69f3b942ce9d 100644 --- a/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c +++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c @@ -665,10 +665,13 @@ static void a7xx_get_dbgahb_clusters(struct msm_gpu *gpu, if (adreno_gpu->info->family == ADRENO_7XX_GEN1) { dbgahb_clusters = gen7_0_0_sptp_clusters; dbgahb_clusters_size = ARRAY_SIZE(gen7_0_0_sptp_clusters); - } else { - BUG_ON(adreno_gpu->info->family > ADRENO_7XX_GEN3); + } else if (adreno_gpu->info->family == ADRENO_7XX_GEN2) { dbgahb_clusters = gen7_2_0_sptp_clusters; dbgahb_clusters_size = ARRAY_SIZE(gen7_2_0_sptp_clusters); + } else { + BUG_ON(adreno_gpu->info->family != ADRENO_7XX_GEN3); + dbgahb_clusters = gen7_9_0_sptp_clusters; + dbgahb_clusters_size = ARRAY_SIZE(gen7_9_0_sptp_clusters); } a6xx_state->dbgahb_clusters = state_kcalloc(a6xx_state,