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/panthor: Pass vm_bind_op to vm_prepare_map_op_ctx Date: Mon, 25 May 2026 18:13:05 +1000 Message-ID: In-Reply-To: <20260522185206.2798288-3-adrian.larumbe@collabora.com> References: <20260522185206.2798288-1-adrian.larumbe@collabora.com> <20260522185206.2798288-3-adrian.larumbe@collabora.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 Mechanical refactor =E2=80=94 the function now takes `const struct drm_pant= hor_vm_bind_op *op` instead of four separate parameters. The internal-only = caller `panthor_vm_map_bo_range` constructs a local `struct drm_panthor_vm_= bind_op` on the stack, which is clean. The `const` qualifier on the paramet= er is appropriate. No issues. --- Generated by Claude Code Patch Reviewer