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 support for 1D LUT in SDR planes Date: Sun, 12 Apr 2026 13:07:23 +1000 Message-ID: In-Reply-To: <20260408051514.608781-5-chaitanya.kumar.borah@intel.com> References: <20260408051514.608781-1-chaitanya.kumar.borah@intel.com> <20260408051514.608781-5-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**: Extends the SDR pipeline with pre/post-CSC LUT blocks using a smaller LUT size (32 vs 128 for HDR). **Observation**: The `is_hdr` check now selects the LUT size at init time: ```c is_hdr ? PLANE_DEGAMMA_SIZE : PLANE_DEGAMMA_SIZE_SDR ``` This is correct. Note that the post-CSC LUT size (`PLANE_GAMMA_SIZE = 32`) is the same for both HDR and SDR, but the pre-CSC differs (128 vs 32). No issues. --- Generated by Claude Code Patch Reviewer