Claude review: YUV support for VOP2 background color

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

Overall Series Review

Subject: YUV support for VOP2 background color
Author: Cristian Ciocaltea <cristian.ciocaltea@collabora.com>
Patches: 7
Reviewed: 2026-06-04T13:48:14.989595

---

This is a clean, well-structured 2-patch series that adds YUV format support for the VOP2 background color register. The motivation is clear: when the VOP2 overlay operates in YUV mode, the background color must also be programmed in YUV, but previously only RGB was supported.

**Patch 1** is a trivial naming consistency fix. **Patch 2** is the substantive change, adding RGB-to-YCbCr conversion matrices derived from the well-established `v4l2-tpg-core.c` reference implementation. The conversion math is correct (verified against the BT.601/709/2020 standards), and the 16-bit precision arithmetic properly accounts for both limited and full range encoding.

The refactoring of `vop2_post_config` to thread explicit old/new state pointers (rather than relying on `crtc->state`) is a good cleanup. The optimization to skip background color reprogramming when neither the color nor the colorspace changed is sensible.

**No blocking issues found.** A couple of minor observations below.

---

---
Generated by Claude Code Patch Reviewer