Claude review: i.MX LDB DT cleanups

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

Overall Series Review

Subject: i.MX LDB DT cleanups
Author: Marco Felsch <m.felsch@pengutronix.de>
Patches: 5
Reviewed: 2026-05-05T07:44:14.436314

---

This is a clean 4-patch series that removes the unused `reg` and `reg-names` device tree properties from the fsl,ldb (LVDS Display Bridge) binding and associated DTS files. The rationale is sound: the LDB driver (`drivers/gpu/drm/bridge/fsl-ldb.c`) has **never** used these properties. It obtains register access via `syscon_node_to_regmap(dev->of_node->parent)` and uses hardcoded offsets from per-device-type driver data. The parent syscon nodes (IOMUX_GPR, BLK_CTRL) have multi-purpose registers shared across sub-devices, making per-sub-device `reg` properties semantically incorrect.

The ordering is correct: revert the binding schema additions first (patches 1-2), then remove the properties from the DTS files (patches 3-4). The final binding YAML and DTS files are consistent and well-formed.

The cover letter honestly notes "I didn't do runtime tests" — this is acceptable given the driver provably ignores these properties, so there is no functional change.

**Verdict: Series looks good. No blocking issues.**

---

---
Generated by Claude Code Patch Reviewer