Claude review: drm/atomic: track individual colorop updates

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

Overall Series Review

Subject: drm/atomic: track individual colorop updates
Author: Melissa Wen <mwen@igalia.com>
Patches: 5
Reviewed: 2026-06-04T12:16:43.925332

---

This is a clean, well-structured 4-patch bug-fix series at v8. It addresses two related issues in the DRM color pipeline (colorop) infrastructure:

1. **lut1d/lut3d interpolation properties were incorrectly treated as immutable** — stored on `drm_colorop` instead of `drm_colorop_state`, meaning changes from userspace went to the wrong place and weren't tracked per-atomic-state.

2. **Individual colorop property changes were not tracked** — changing a colorop property (bypass, curve type, LUT data, multiplier, interpolation) didn't set `plane_state->color_mgmt_changed`, so drivers had no way to know color blocks needed updating. This broke gamescope night-mode (shaper/3D-LUT updates).

The series is logically ordered: first fix docs (patch 1), then fix the data model (patch 2), then add change tracking in the core (patch 3), then wire it up in AMD's driver (patch 4). Each patch is independently correct, has appropriate Fixes tags, and carries reviewed-by tags from multiple reviewers (Intel, AMD). The code is straightforward, follows existing patterns, and I found no correctness bugs.

**Recommendation: This series looks ready to merge.**

---

---
Generated by Claude Code Patch Reviewer