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: Fix HDR pre-CSC LUT programming loop Date: Mon, 09 Mar 2026 08:26:53 +1000 Message-ID: In-Reply-To: <20260306165307.3233194-6-chaitanya.kumar.borah@intel.com> References: <20260306165307.3233194-1-chaitanya.kumar.borah@intel.com> <20260306165307.3233194-6-chaitanya.kumar.borah@intel.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review Correct fix =E2=80=94 `i++ > 130` was a clear typo that should be `i++ < 13= 0`, causing the do-while to exit immediately (only writing one entry instea= d of the intended 3). The `do { ... } while (i++ < 130)` with `i` starting = at 128 writes entries 128, 129, 130 =E2=80=94 3 clamp values, which is corr= ect. **Ordering issue**: This patch has `Cc: stable` and `Fixes:` tags but depen= ds on no other patch in the series. It should be patch 1 to enable clean ba= ckporting. Currently being at position 5 means backporting it requires eith= er cherry-picking (which works since it's independent) or backporting patch= es 1-4 too (which are features, not fixes). --- Generated by Claude Code Patch Reviewer