Claude review: Pass down hot-plug CONNECTOR ID to user-space

[Claude Code Review Bot <claude-review@example.com>]

Overall Series Review

Subject: Pass down hot-plug CONNECTOR ID to user-space
Author: Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
Patches: 3
Reviewed: 2026-04-23T07:22:52.089066

---

This is a well-motivated 2-patch series at v9 that improves DRM hotplug event reporting by sending per-connector `CONNECTOR=<id>` uevents instead of generic global hotplug events. The cover letter clearly explains the real-world problem: compositors like Weston can miss rapid on-off-on HPD transitions because they can't identify which connector changed. The approach is sound — split the sysfs uevent (which is safe to call with the mutex held) from `drm_client_dev_hotplug()` (which needs to be called without mode_config locks), thus avoiding the deadlock that plagued earlier versions.

The series is clean and simple at 2 patches / net -1 lines changed. The first patch is a trivial doc fix, and the second implements the actual behavioral change. The code is a simplification over the current baseline which already had partial per-connector support (the `first_changed_connector` logic in `drm_helper_hpd_irq_event`). I have a few observations but nothing blocking.

---
Generated by Claude Code Patch Reviewer