Claude review: drm/msm/dp: add connector abstraction for DP MST

Claude Code Review Bot
2026-04-12T00:16:20Z

Patch Review

Adds the MST connector abstraction with `msm_dp_mst_add_connector()` callback for the topology manager. The connector correctly calls `drm_connector_dynamic_init()` for hotplugged MST connectors.

The `msm_dp_mst_atomic_best_encoder()` logic has a two-pass approach: first find an existing bridge assigned to this connector, then find a free bridge. This is correct for MST encoder assignment.

Minor: In `msm_dp_mst_add_connector()`, if `drm_connector_attach_encoder()` fails, the `mst_conn` is freed but the connector was already initialized with `drm_connector_dynamic_init()` — the connector should be cleaned up via `drm_connector_cleanup()` first to avoid leaking DRM resources.

---
Generated by Claude Code Patch Reviewer