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: set min_acc length to 64 for all UBWC 1.0 targets Date: Mon, 25 May 2026 21:42:01 +1000 Message-ID: In-Reply-To: <20260520-ubwc-rework-v5-22-72f2749bc807@oss.qualcomm.com> References: <20260520-ubwc-rework-v5-0-72f2749bc807@oss.qualcomm.com> <20260520-ubwc-rework-v5-22-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 Simplifies `qcom_ubwc_min_acc_length_64b()`: ```c - return cfg->ubwc_enc_version == UBWC_1_0 && - (cfg->ubwc_dec_version == UBWC_2_0 || - cfg->ubwc_dec_version == UBWC_3_0); + return cfg->ubwc_enc_version == UBWC_1_0; ``` Justified by documentation saying MAL should be set for all UBWC 1.0 targets. The old check was a "best guess" inherited from the MDSS driver. Since `ubwc_dec_version` is being removed, this needs to happen. --- Generated by Claude Code Patch Reviewer