Claude review: dt-bindings: display: waveshare,dsp2dpi: describe DSI2LVDS setup

Claude Code Review Bot
2026-04-13T08:54:38Z

Patch Review

**Subject line typo**: The subject says `dsp2dpi` but the binding file is `dsi2dpi`. This is a minor cosmetic issue in the commit subject only (the actual file path in the diff is correct).

The binding changes are straightforward:

1. **Compatible changed from `const` to `enum`** — correctly adds `waveshare,dsi2lvds` alongside the existing `waveshare,dsi2dpi`:
```yaml
compatible:
- const: waveshare,dsi2dpi
+ enum:
+ - waveshare,dsi2dpi
+ - waveshare,dsi2lvds
```
This is the standard approach for extending a single-compatible binding.

2. **Description updates** — appropriately broadened to mention "or LVDS":
```yaml
- Waveshare bridge board is part of Waveshare panel which converts DSI to DPI.
+ Waveshare bridge board is part of Waveshare panel which converts DSI to DPI
+ or LVDS.
```
and:
```yaml
- Video port for MIPI DPI output panel.
+ Video port for MIPI DPI or LVDS output to the panel.
```

The commit message has a useful note explaining why the ICN6202/ICN6211 bridges are not itemized in the compatible strings (they're handled entirely on the board, not by the host). This is good context for DT reviewers. Minor nit: "itmized" in the commit body is a typo for "itemized".

No issues with the binding schema itself.