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/bridge: dw-hdmi-qp: Provide dw_hdmi_qp_hpd_notify() helper Date: Thu, 04 Jun 2026 13:33:50 +1000 Message-ID: In-Reply-To: <20260602-dw-hdmi-qp-scramb-v7-15-445eb54ee1ed@collabora.com> References: <20260602-dw-hdmi-qp-scramb-v7-0-445eb54ee1ed@collabora.com> <20260602-dw-hdmi-qp-scramb-v7-15-445eb54ee1ed@collabora.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review The logic for choosing between `drm_connector_helper_hpd_irq_event()` and `drm_bridge_hpd_notify()` is subtle but well-commented. The key insight is that `drm_bridge_hpd_notify()` defers to a delayed path when the pipeline is active, which doesn't invoke `detect_ctx` synchronously. ### PATCHES 16-18: Rockchip cleanups Trivial and correct. Missing newlines, `dev` variable consistency, stale include removal. --- Generated by Claude Code Patch Reviewer