From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/i915/color: Add CSC on SDR plane color pipeline Date: Sun, 12 Apr 2026 13:07:23 +1000 Message-ID: In-Reply-To: <20260408051514.608781-3-chaitanya.kumar.borah@intel.com> References: <20260408051514.608781-1-chaitanya.kumar.borah@intel.com> <20260408051514.608781-3-chaitanya.kumar.borah@intel.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review **Overall**: Straightforward pipeline definition for SDR planes. **Observation**: The `is_hdr` check in `_intel_color_pipeline_plane_init` correctly routes to different pipeline definitions. The supported CSC modes (`intel_plane_supported_csc_ff`) include the three YCbCr-to-RGB conversions and RGB709-to-RGB2020, which matches the hardware capabilities indicated by `PLANE_COLOR_CSC_MODE_*` defines. No issues. --- Generated by Claude Code Patch Reviewer