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/sysfs: update outdated comment for renamed drm_sysfs_connector_status_event() Date: Sun, 22 Mar 2026 03:06:17 +1000 Message-ID: In-Reply-To: <20260321105739.6472-1-kexinsun@smail.nju.edu.cn> References: <20260321105739.6472-1-kexinsun@smail.nju.edu.cn> <20260321105739.6472-1-kexinsun@smail.nju.edu.cn> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review **Assessment: Correct and complete.** The patch makes two changes in the comment block above `drm_sysfs_hotplug_e= vent()`: 1. Updates the function reference: ``` - * Any new uapi should be using the drm_sysfs_connector_status_event() + * Any new uapi should be using the drm_sysfs_connector_property_event() ``` 2. Updates the description to match: ``` - * for uevents on connector status change. + * for uevents on connector property change. ``` Both changes are accurate =E2=80=94 the function is indeed named `drm_sysfs= _connector_property_event()` in the current tree (line 490 of `drm_sysfs.c`= ), and the rename commit `0cf8d292ba5e` is correctly cited. **Minor nit:** The `Assisted-by: unnamed:deepseek-v3.2 coccinelle` tag is n= on-standard. The kernel community typically uses tags like `Assisted-by: ` without mixing tool names with semicolons in this way. Ho= wever, since this is a trivial comment fix found via tooling, this is unlik= ely to be a blocker =E2=80=94 a maintainer may just ask for the tag to be c= leaned up or dropped. **No technical concerns.** The patch is minimal, correct, and well-describe= d in its commit message. --- Generated by Claude Code Patch Reviewer