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

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

Patch Review

The conversion from `intel_crtc_alloc()` + `drm_crtc_init_with_planes()` to `drmm_crtc_alloc_with_planes()` is structurally correct. The `drmm_add_action_or_reset()` for `cpu_latency_qos` cleanup is appropriate.

Concern: the commit message is very thin ("It is not an issue yet, but in order to comply with the latest DRM requirement"). This is a substantial reordering of initialization (planes before CRTC) and removal of error paths. The commit message should explain *why* the reordering is safe and document which cleanup paths are now handled by devres vs. callbacks.

---
Generated by Claude Code Patch Reviewer