From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: accel/qda: Add DRM accel device registration for QDA driver Date: Tue, 24 Feb 2026 18:20:12 +1000 Message-ID: In-Reply-To: <20260224-qda-firstpost-v1-7-fe46a9c1a046@oss.qualcomm.com> References: <20260224-qda-firstpost-v1-0-fe46a9c1a046@oss.qualcomm.com> <20260224-qda-firstpost-v1-7-fe46a9c1a046@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 Finally allocates the DRM device.=20 - Dmitry said: "devm_drm_dev_alloc() please. Move this patch to the front o= f the series, making everything else depend on the allocated data structure= ." This is correct =E2=80=94 the DRM device should be the foundation, not a= dded as patch 7. - The shared data structure `qda_shared_data` was questioned by Dmitry: "Sh= ared between what and what? Why do you need a separate structure instead of= using qda_dev?" --- Generated by Claude Code Patch Reviewer