From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: media: iris: don't specify bank_spreading in the source code Date: Thu, 07 May 2026 12:40:17 +1000 Message-ID: In-Reply-To: <20260507-iris-ubwc-v5-5-e9a3aee53c49@oss.qualcomm.com> References: <20260507-iris-ubwc-v5-0-e9a3aee53c49@oss.qualcomm.com> <20260507-iris-ubwc-v5-5-e9a3aee53c49@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 Replaces `core->iris_platform_data->ubwc_config->bank_spreading` with: ```c payload =3D qcom_ubwc_bank_spread(ubwc); ``` The helper returns `cfg->ubwc_bank_spread` which is `bool`. The old field w= as `u32` set to `1`. The `bool =E2=86=92 u32` implicit conversion produces = 1 for true. Correct. No issues. --- Generated by Claude Code Patch Reviewer