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: Sun, 12 Apr 2026 13:07:23 +1000 Message-ID: In-Reply-To: <20260408051514.608781-6-chaitanya.kumar.borah@intel.com> References: <20260408051514.608781-1-chaitanya.kumar.borah@intel.com> <20260408051514.608781-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 **Overall**: Genuine bug fix. The `>` should be `<` in the do-while conditi= on. ```c - } while (i++ > 130); // never true when i starts at 128 =E2=86=92 0 ite= rations + } while (i++ < 130); // 3 iterations (i=3D128,129,130) =E2=86=92 correct ``` The original condition `i++ > 130` is false when `i=3D128`, so the do-while= body executes exactly once instead of the intended 3 times. The fix correc= tly programs entries 128-130 (3 clamp entries for values > 1.0). Correctly tagged with `Cc: stable` and `Fixes:`. This should be sent separa= tely/earlier in the series rather than mixed in with new features, as it's = a standalone fix that affects existing HDR plane functionality. --- Generated by Claude Code Patch Reviewer