Claude review: drm/rockchip: dw_hdmi_qp: Do not send HPD events for all connectors

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

**Overall: Clean and straightforward.**

- **IRQ registration moved after connector init:** The `devm_request_threaded_irq` is moved to after `drm_bridge_connector_init` and `drm_connector_attach_encoder`, which is correct — HPD interrupts could fire immediately and the handler needs `hdmi->connector` to be valid.

- **`hdmi->connector` set from `drm_bridge_connector_init`:** This is the bridge connector, not a user-allocated connector. Its lifetime is managed by DRM core, so storing it in `hdmi->connector` is fine as long as the driver doesn't outlive the DRM device.

- **Race on `hdmi->connector` in `dw_hdmi_qp_rk3588_hpd_work`:** The work function uses `hdmi->connector` (line 254) which is set during bind. Since the IRQ is now registered after connector init, this is safe. However, during unbind, if the work is still pending, it could access a freed connector. There's no `cancel_delayed_work_sync` in the unbind path for `hpd_work` — but this may be handled by `devm_request_threaded_irq` cleanup preventing new work from being scheduled, and the existing unbind sequence. Worth verifying that the unbind path is safe.

- **Minor: variable rename `irq` → `hpd_irq`:** Clean change to avoid confusion with other IRQs.

---
Generated by Claude Code Patch Reviewer