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/todo: clarify drm_syncobj error TODOs Date: Thu, 04 Jun 2026 14:27:35 +1000 Message-ID: In-Reply-To: <20260601085132.683087-1-christian.koenig@amd.com> References: <20260601085132.683087-1-christian.koenig@amd.com> <20260601085132.683087-1-christian.koenig@amd.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 **Purpose:** Prevent repeated "is anybody working on this?" emails by clari= fying that the kernel side is done and only userspace work remains. Also ad= ds Michel D=C3=A4nzer as a second contact. **Review:** The changes are sensible and clearly motivated by the commit message. Two m= inor observations: 1. **Trailing space before closing parenthesis:** ``` +(patches are on the dri-devel mailing list ) but adding support for tha= t in the ``` There's a stray space before the `)` =E2=80=94 `mailing list )` should b= e `mailing list)`. This is a cosmetic nit but worth fixing since it's a doc= umentation file that humans will read. 2. **Pre-existing typo `complection`** (line 894 in current tree / line 110= context in the patch): ``` complection of submission. ``` This should be `completion`. It's not introduced by this patch =E2=80=94= it's pre-existing =E2=80=94 but since the patch is already touching this s= ection, it would be a nice drive-by fix to correct it. Not a blocker. 3. **Vagueness about kernel patches:** The text says "patches are on the dr= i-devel mailing list" without a link or series identifier. It might be help= ful to readers to include a Message-Id or link-to-lore reference so they ca= n find the actual kernel patches, but this is optional. 4. **Level unchanged at "Starter":** Since the kernel work is done and only= userspace remains, keeping this as "Starter" seems appropriate =E2=80=94 u= serspace integration work is a reasonable entry point for new contributors. **Overall: The patch is good to go with the trivial whitespace fix (`mailin= g list )` =E2=86=92 `mailing list)`).** --- Generated by Claude Code Patch Reviewer