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-02-18T06:43:45.823156

---

This two-patch series adds per-connector hotplug event reporting to the DRM subsystem. Instead of sending a single global hotplug uevent (which forces userspace to re-probe all connectors), it sends individual uevents identifying the specific connector(s) that changed. The motivation is well-described: compositors like Weston can miss rapid on-off-on transitions when they have to guess which connector caused the event.

The approach is straightforward: patch 1 adds a `pending_hp` boolean to `drm_connector` that tracks whether a connector has a status change pending notification, and patch 2 converts the polling and HPD paths to iterate over connectors with `pending_hp` set and send individual events for each.

There is one bug in the series: the `delayed_event` path in `output_poll_execute` sets `changed = true` without any connector having `pending_hp` set, so the new connector-iterating loop will silently drop the hotplug notification. This is a functional regression. There is also a performance concern around `drm_client_dev_hotplug()` being invoked N times instead of once when N connectors change simultaneously.

---
Generated by Claude Code Patch Reviewer