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/connector: report IRQ_HPD events to drm_connector_oob_hotplug_event() Date: Sat, 16 May 2026 11:25:50 +1000 Message-ID: In-Reply-To: <20260513-hpd-irq-events-v3-1-086857017f16@oss.qualcomm.com> References: <20260513-hpd-irq-events-v3-0-086857017f16@oss.qualcomm.com> <20260513-hpd-irq-events-v3-1-086857017f16@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 This introduces the `enum drm_connector_status_extra` type and extends `drm= _connector_oob_hotplug_event()` and the USB typec DP altmode callers. The enum design is clean =E2=80=94 using a separate enum rather than adding= values to `drm_connector_status` avoids confusion between connection state= and event type. The DP altmode changes in `displayport.c` correctly pass `DRM_CONNECTOR_DP_= IRQ_HPD` only when both `hpd` and `irq_hpd` are asserted, and `DRM_CONNECTO= R_NO_EXTRA_STATUS` for disconnect / remove paths. Minor nit in the cover letter: "singnalling" typo (line 183 of the mbox). N= ot in the patch itself. **No issues.** --- Generated by Claude Code Patch Reviewer