From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: soc: qcom: pmic-glink-altmode: pass down HPD_IRQ events
Date: Thu, 23 Apr 2026 09:32:10 +1000
Message-ID: <review-patch5-20260420-hpd-irq-events-v2-5-402ffe27e9e9@oss.qualcomm.com>
In-Reply-To: <20260420-hpd-irq-events-v2-5-402ffe27e9e9@oss.qualcomm.com>
References: <20260420-hpd-irq-events-v2-0-402ffe27e9e9@oss.qualcomm.com> <20260420-hpd-irq-events-v2-5-402ffe27e9e9@oss.qualcomm.com>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0

Patch Review

Straightforward conversion from `drm_aux_hpd_bridge_notify()` to `drm_aux_hpd_bridge_notify_extra()`:

```c
	drm_aux_hpd_bridge_notify_extra(&alt_port->bridge->dev,
					conn_status,
					alt_port->hpd_irq ?
					DRM_CONNECTOR_DP_IRQ_HPD :
					DRM_CONNECTOR_NO_EXTRA_STATUS);
```

**Looks correct.** The `hpd_irq` field already exists in the pmic_glink_altmode driver. No concerns.

---
Generated by Claude Code Patch Reviewer