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, 09 Mar 2026 08:32:18 +1000 Message-ID: In-Reply-To: <20260306-ubwc-rework-v1-18-9cfdff12f2bb@oss.qualcomm.com> References: <20260306-ubwc-rework-v1-0-9cfdff12f2bb@oss.qualcomm.com> <20260306-ubwc-rework-v1-18-9cfdff12f2bb@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()` to just check `ubwc_enc_version == UBWC_1_0`, removing the decoder version check: ```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; ``` The commit message explains this is correct per documentation. The old check was from DPU/GPU context where the bit wasn't separately controllable before UBWC 2.0 decoders. --- Generated by Claude Code Patch Reviewer