From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: soc: qcom: ubwc: drop ubwc_bank_spread Date: Mon, 25 May 2026 21:42:01 +1000 Message-ID: In-Reply-To: <20260520-ubwc-rework-v5-24-72f2749bc807@oss.qualcomm.com> References: <20260520-ubwc-rework-v5-0-72f2749bc807@oss.qualcomm.com> <20260520-ubwc-rework-v5-24-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 Removes `ubwc_bank_spread` from the struct and makes `qcom_ubwc_bank_spread()` unconditionally return `true`: ```c - return cfg->ubwc_bank_spread; + return true; ``` Justified by documentation saying bank spread should be enabled for all targets. All existing entries had `ubwc_bank_spread = true`, so this is a no-op in practice. No issues. --- Generated by Claude Code Patch Reviewer