Claude review: arm64: dts: ti: k3-am62-verdin: Add Toradex OV5640 CSI Cameras

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

Patch Review

Good use of a shared `.dtsi` file with two thin `.dtso` wrappers that only override `clock-frequency` (27 MHz vs 24 MHz). Clean factoring.

- The `clk_ov5640_osc` fixed-clock node is defined without a `clock-frequency` — it must be set by the including `.dtso`. This works because overlays are compiled as complete units, and the `.dtsi` is included into the `.dtso` before compilation.

- The `regulator_camera` uses `gpio = <&main_gpio0 42 GPIO_ACTIVE_HIGH>` with `enable-active-high` — correct for a GPIO-controlled power switch.

- All three OV5640 supply rails (`AVDD`, `DOVDD`, `DVDD`) point to `&regulator_camera`. This is a simplification — the OV5640 datasheet specifies different voltages for each rail (2.8V analog, 1.8V IO, 1.5V digital core). This is common in DTS for modules where a single power control GPIO gates all rails through the module's on-board regulators.

- The camera endpoint specifies `data-lanes = <1 2>` (2-lane CSI-2) and `bus-type = <4>` (CSI-2 D-PHY) on both sides of the link. Correct.

**No issues.**

---
Generated by Claude Code Patch Reviewer