Claude review: dt-bindings: display: bridge: lt9211: Require data-lanes on DSI input ports

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

Overall Series Review

Subject: dt-bindings: display: bridge: lt9211: Require data-lanes on DSI input ports
Author: Marek Vasut <marex@nabladev.com>
Patches: 2
Reviewed: 2026-04-12T13:19:34.040015

---

This is a single-patch v2 series adding `data-lanes` property definitions to the DSI input ports of the Lontium LT9211 devicetree binding. The pattern follows established conventions used by similar bridge bindings (e.g., `ti,sn65dsi83.yaml`). The switch from `$ref: /schemas/graph.yaml#/properties/port` to `$ref: /schemas/graph.yaml#/$defs/port-base` with `unevaluatedProperties: false` is the correct approach to allow additional endpoint properties.

The main concern is that the patch adds `required: - data-lanes` to port@0 and port@1, but these ports can serve multiple protocols (DSI, LVDS, or DPI). The `data-lanes` property is only meaningful for DSI, so unconditionally requiring it would make the binding reject valid LVDS/DPI devicetree descriptions. Notably, the ti,sn65dsi83 binding—which appears to be the template for this change—does **not** mark `data-lanes` as required, even though its ports are DSI-only.

---
Generated by Claude Code Patch Reviewer