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/msm/dp: Drop EV_USER_NOTIFICATION Date: Mon, 16 Mar 2026 12:12:56 +1000 Message-ID: In-Reply-To: <20260314-hpd-refactor-v5-5-0c8450737d64@oss.qualcomm.com> References: <20260314-hpd-refactor-v5-0-0c8450737d64@oss.qualcomm.com> <20260314-hpd-refactor-v5-5-0c8450737d64@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 Replaces `drm_helper_hpd_irq_event()` with `drm_bridge_hpd_notify()` and ad= ds `struct drm_bridge *bridge` to `struct msm_dp`. This is a clean transiti= on. **Minor note:** The `msm_dp_display_send_hpd_notification()` function now c= alls `drm_bridge_hpd_notify()` directly instead of through an event queue. = This changes the context in which the notification happens =E2=80=94 it's n= ow called from the same context as the HPD handler rather than from the eve= nt thread. This should be fine since `drm_bridge_hpd_notify()` schedules wo= rk internally. No blocking issues. --- Generated by Claude Code Patch Reviewer