From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/msm/adreno: use new helper to set amsbc Date: Mon, 25 May 2026 21:42:00 +1000 Message-ID: In-Reply-To: <20260520-ubwc-rework-v5-17-72f2749bc807@oss.qualcomm.com> References: <20260520-ubwc-rework-v5-0-72f2749bc807@oss.qualcomm.com> <20260520-ubwc-rework-v5-17-72f2749bc807@oss.qualcomm.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Replaces `amsbc = true` from each switch case with `amsbc = qcom_ubwc_enable_amsbc(cfg)` at initialization: ```c - bool rgb565_predicator = false, amsbc = false; + bool rgb565_predicator = false; + bool amsbc = qcom_ubwc_enable_amsbc(cfg); ``` And removes the per-case `amsbc = true` lines. This depends on patch 15 having already added amsbc for UBWC_6_0, and patch 3 defining the helper as `>= UBWC_3_0`. Correct. --- Generated by Claude Code Patch Reviewer