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/fourcc: document format naming scheme Date: Tue, 28 Apr 2026 15:16:43 +1000 Message-ID: In-Reply-To: <20260426151641.93763-1-contact@emersion.fr> References: <20260426151641.93763-1-contact@emersion.fr> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Overall Series Review Subject: drm/fourcc: document format naming scheme Author: Simon Ser Patches: 3 Reviewed: 2026-04-28T15:16:43.486341 --- This is a single documentation-only patch from Simon Ser that adds a naming= scheme reference to `drm_fourcc.h`, explaining how format names and fourcc= s are structured. The motivation is solid =E2=80=94 the existing header has= hundreds of format definitions with no documentation explaining the naming= conventions, making it hard for contributors to choose names for new forma= ts. The documentation is largely accurate for RGB formats but has a factual err= or in the YUV digit-reversal rule and a typo. The "for the most part" quali= fier on the YUV section is appropriate given the many exceptions, but the N= V family =E2=80=94 arguably the most widely used multi-planar YUV format fa= mily =E2=80=94 doesn't follow the described scheme at all, which deserves a= cknowledgment. --- Generated by Claude Code Patch Reviewer