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/mdss: use new helper to set UBWC bank spreading Date: Fri, 13 Mar 2026 14:12:50 +1000 Message-ID: In-Reply-To: <20260312-ubwc-rework-v3-12-b7e8f800176a@oss.qualcomm.com> References: <20260312-ubwc-rework-v3-0-b7e8f800176a@oss.qualcomm.com> <20260312-ubwc-rework-v3-12-b7e8f800176a@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 `data->ubwc_bank_spread` with `qcom_ubwc_bank_spread(data)` (which always returns `true`). This makes the `if` guard dead code, but it's cleaned up in patch 23. ### PATCHES 13-15: use new helper to set ubwc_swizzle (adreno + dpu + mdss) Mechanical. The DPU change nicely replaces magic `0x1` with `UBWC_SWIZZLE_ENABLE_LVL1`. --- Generated by Claude Code Patch Reviewer