From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: drm/i915/dp: Add YCBCR444 handling for sink formats
Date: Mon, 25 May 2026 18:48:56 +1000
Message-ID: <review-patch12-20260522-color-format-v15-12-21fb136c9df2@collabora.com>
In-Reply-To: <20260522-color-format-v15-12-21fb136c9df2@collabora.com>
References: <20260522-color-format-v15-0-21fb136c9df2@collabora.com> <20260522-color-format-v15-12-21fb136c9df2@collabora.com>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0

Patch Review

The `intel_dp_can_ycbcr444` function properly checks both direct source output and DFP conversion paths:

```c
if (source_can_output(intel_dp, INTEL_OUTPUT_FORMAT_YCBCR444) &&
    !drm_dp_is_branch(intel_dp->dpcd))
    return true;
if (source_can_output(intel_dp, INTEL_OUTPUT_FORMAT_RGB) &&
    dfp_can_convert_from_rgb(intel_dp, INTEL_OUTPUT_FORMAT_YCBCR444))
    return true;
```

This is more thorough than the HDMI counterpart. No issues.

---
Generated by Claude Code Patch Reviewer