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: Fri, 13 Mar 2026 14:12:50 +1000 Message-ID: In-Reply-To: <20260312-ubwc-rework-v3-16-b7e8f800176a@oss.qualcomm.com> References: <20260312-ubwc-rework-v3-0-b7e8f800176a@oss.qualcomm.com> <20260312-ubwc-rework-v3-16-b7e8f800176a@oss.qualcomm.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review The a8xx change is more complex than a simple substitution =E2=80=94 the th= ree per-case `amsbc =3D true` assignments are replaced with a guarded call = before the switch: ```c if (ubwc_version < UBWC_6_0) amsbc =3D qcom_ubwc_enable_amsbc(cfg); ``` Correctly excludes UBWC 6.0. Also adds `case UBWC_3_1:` to the switch. --- Generated by Claude Code Patch Reviewer