Claude review: drm/panel: jadard-jd9365da-h3: support Waveshare WXGA DSI panels

Claude Code Review Bot
2026-04-11T23:06:56Z

Patch Review

Adds Waveshare 8.0" and 10.1" WXGA (800x1280) panels with lane-conditional init sequences.

1. **Bug**: `waveshare_8_0_a_init()` returns `0` instead of `dsi_ctx.accum_err`:
```c
msleep(60);

return 0;
```
Any DSI write errors during init will be silently ignored.

2. **Readability**: `waveshare_10_1_a_init()` has extensive inline `if (jadard->dsi->lanes == 4) ... else ...` conditionals scattered throughout the gamma and GIP configuration, making it very hard to audit. Consider splitting into two separate init sequences or at minimum using lookup tables indexed by lane count.

3. **Style**: Inconsistent braces -- `if` branch without braces, `else` branch with braces in several places, violating kernel coding style.

---

---
Generated by Claude Code Patch Reviewer