From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: Re: [PATCH] staging: fbtft: fix macro whitespace errors Date: Fri, 27 Feb 2026 11:42:54 +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 Overall Series Review Subject: Re: [PATCH] staging: fbtft: fix macro whitespace errors Author: Andy Shevchenko Patches: 4 Reviewed: 2026-02-27T11:42:54.570254 --- **NAK =E2=80=94 This patch is incorrect and will break compilation.** This is a single patch that claims to fix "macro whitespace errors" reporte= d by checkpatch.pl in the staging fbtft driver. However, the patch fundamen= tally misunderstands the code: what appears to be trailing whitespace befor= e the closing parenthesis is actually an **intentionally empty macro argume= nt**. Removing it changes a 4-argument macro invocation into a 3-argument o= ne, which will cause a preprocessor error. --- Generated by Claude Code Patch Reviewer