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/color-mgmt: Typo s/R332/RGB332/ Date: Thu, 23 Apr 2026 08:43:29 +1000 Message-ID: In-Reply-To: References: X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review **Status: Reviewed-by worthy** The change is a one-character fix in a kerneldoc comment: ``` - * drm_crtc_fill_palette_332 - Programs a default palette for R332-like fo= rmats + * drm_crtc_fill_palette_332 - Programs a default palette for RGB332-like = formats ``` The typo is clearly unintentional =E2=80=94 the description body on line 83= 8 already correctly says "Programs an RGB332 palette to hardware", and the = function name itself references "332" (the RGB332 pixel format where R=3D3 = bits, G=3D3 bits, B=3D2 bits). "R332" would imply a single red channel with= 332... something, which doesn't make sense. The `Fixes:` tag correctly points to commit `7ff61177b7116825` ("drm/color-= mgmt: Prepare for RGB332 palettes") which introduced the typo. No concerns with this patch. --- Generated by Claude Code Patch Reviewer