Claude review: staging: fbtft: replace empty macro args with identity converter

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

**Commit message:**
> Remove trailing space and comma before closing parenthesis ')' in define_fbtft_write_reg macro as reported by checkpatch.pl.

**Critical bug — will not compile.** The macro `define_fbtft_write_reg` is defined with 4 parameters:

```c
#define define_fbtft_write_reg(func, buffer_type, data_type, modifier)
```

But the patch changes the invocations from 4 arguments (with an empty 4th) to 3:

```c
-define_fbtft_write_reg(fbtft_write_reg8_bus8, u8, u8, )
+define_fbtft_write_reg(fbtft_write_reg8_bus8, u8, u8)
```

Calling a fixed-arity 4-parameter macro with only 3 arguments is a hard error from the preprocessor. GCC will emit: `error: macro "define_fbtft_write_reg" requires 4 arguments, but only 3 given`.

The existing code with the empty 4th argument (`, )`) is actually well-defined in C99 and GNU C (which the kernel uses via `-std=gnu11`). An empty macro argument is a valid "placemarker" preprocessing token per C99 §6.10.3. When `modifier` is empty, `modifier((data_type)va_arg(args, unsigned int))` expands to `((data_type)va_arg(args, unsigned int))` — a correctly parenthesized identity operation. The checkpatch warning is cosmetic and the existing code is correct.

**Verdict:** NAK. This patch introduces a build failure.

---

---
Generated by Claude Code Patch Reviewer