Claude review: arm64: dts: ti: k3-am62-verdin: Add display and peripheral overlays

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

Overall Series Review

Subject: arm64: dts: ti: k3-am62-verdin: Add display and peripheral overlays
Author: Vitor Soares <ivitro@gmail.com>
Patches: 24
Reviewed: 2026-05-25T19:56:37.725096

---

This is a well-structured device tree overlay series from Toradex for the Verdin AM62 SoM, adding support for multiple displays (LVDS, DSI, DSI-to-LVDS bridge), cameras, audio, CAN, and UART peripherals. The series is largely straightforward DTS work with a few binding changes.

**Positive aspects:**
- Good use of overlays (`.dtso`) for optional accessories — the right approach for carrier-board-specific peripherals
- Clean factoring of common OV5640 code into a shared `.dtsi`
- Dual-channel LVDS support for the LG panel is well-handled with the `lvds-dual-ports` schema reference
- Proper SPDX license headers, Signed-off-by, and documentation links throughout

**Concerns:**
- Heavy use of `Assisted-by: Claude:claude-sonnet-4.6` — not a standard kernel tag. Should use `Co-developed-by:` or drop it; DT maintainers may push back.
- Several patches use endpoint label names (`panel_lvds_bridge_in`, `dsi_lvds_bridge_out`, etc.) that will conflict if multiple overlays are ever applied simultaneously — inherent to overlays but worth noting.
- Patch 1 removes `logictechno,lt170410-2whc` from `panel-simple.yaml` and adds to `panel-lvds.yaml`, but there may be existing users relying on the panel-simple driver; this could be a regression.
- Patch 13 (dual-channel LVDS binding) changes the `required:` section in a way that affects all existing users of `panel-lvds` — removing `port` from `required` and using `oneOf` with `port` or `ports` is correct, but needs careful review by DT binding maintainers.

---

---
Generated by Claude Code Patch Reviewer