Claude review: drm/i915/display: Switch to managed for connector

Claude Code Review Bot
2026-04-12T00:51:25Z

Patch Review

The overall approach is correct. The MST connector exception is well-motivated -- MST connectors have dynamic lifetimes managed by the topology layer.

Concern about the `drmm_connector_init()` + `drmm_add_action_or_reset(intel_connector_destroy)` boilerplate repeated ~10 times. A helper like `drmm_intel_connector_init()` wrapping both calls would reduce duplication and the risk of one call being added without the other.

The dual-use of `intel_connector_destroy()` as both a `drmm_action` callback `(struct drm_device *, void *)` and called directly from `mst_connector_destroy()` is functional but awkward. Consider a wrapper for the direct call path.

---
Generated by Claude Code Patch Reviewer