Claude review: drm/i915/color: Program plane YUV range correction colorop

Claude Code Review Bot
2026-04-12T03:07:25Z

Patch Review

**Critical**: This patch adds lines 1303-1304 to `glk_plane_color_ctl()`:
```c
plane_color_ctl |= plane_state->hw.yuv_range_correct ? 0 :
PLANE_COLOR_YUV_RANGE_CORRECTION_DISABLE;
```

As noted in patch 3 review, this unconditionally disables YUV range correction on all SDR planes when `yuv_range_correct` is false (the zero-initialized default). This is a **regression for limited-range YUV content** that doesn't use the color pipeline — the legacy path at lines 1292-1293 would correctly leave range correction enabled for limited-range, but this new code overrides that by always setting the DISABLE bit.

The intent is presumably that when using the color pipeline, userspace controls range correction via the colorop bypass state. But the code runs unconditionally for all SDR planes regardless of whether a color pipeline is active.