Claude review: drm/bridge: analogix_dp: Move the color format check to .atomic_check() for Rockchip platforms

Claude Code Review Bot
2026-04-12T01:37:40Z

Patch Review

Moves the YUV-to-RGB color format fixup from `rockchip_dp_get_modes()` into `analogix_dp_atomic_check()`, gated by `is_rockchip()`. This makes the check run at atomic_check time rather than get_modes time, which is architecturally more appropriate.

One minor concern: modifying `display_info` (which lives on the connector, not in state) from `atomic_check` is a bit unusual. The `display_info` is typically populated during EDID parsing and is not per-state. However, this pattern was already present in the original code (just in `get_modes()` instead), so this isn't a regression. The bridge `atomic_check` in patch 11 continues this approach.

No blocking issues.

---
Generated by Claude Code Patch Reviewer