Claude review: drm/bridge: aux-hpd: let drivers pass IRQ_HPD events

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

Patch Review

Renames `drm_bridge_hpd_notify` → `drm_bridge_hpd_notify_extra` (exported symbol) and `drm_aux_hpd_bridge_notify` → `drm_aux_hpd_bridge_notify_extra`, while keeping the old names as inline wrappers that pass `DRM_CONNECTOR_NO_EXTRA_STATUS`.

The commit message says: "The drm_aux_hpd_bridge_notify() is keps to ease merging" — typo: "keps" → "kept".

The approach of keeping compatibility wrappers as `static inline` in headers is clean and avoids cross-tree merge headaches. Good.

**Observation:** `drm_aux_hpd_bridge_notify()` inline wrapper is placed *after* the `#endif` of the `CONFIG_DRM_AUX_HPD_BRIDGE` block:

```c
 #endif
 
+static inline void drm_aux_hpd_bridge_notify(struct device *dev, enum drm_connector_status status)
+{
+	drm_aux_hpd_bridge_notify_extra(dev, status, DRM_CONNECTOR_NO_EXTRA_STATUS);
+}
+
 #endif
```

This means it's always defined regardless of `CONFIG_DRM_AUX_HPD_BRIDGE`. When the config is disabled, it will call the stub `drm_aux_hpd_bridge_notify_extra()` which is an empty inline — so this is correct. But it's slightly unusual placement. Consider moving it inside the `#ifdef` block and adding a corresponding stub in the `#else`.

---
Generated by Claude Code Patch Reviewer