Claude review: arm64: dts: ti: beagley-ai: Enable HDMI display and audio

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

Patch Review

Enables HDMI via the IT66122 bridge chip on I2C1, with DSS1 VP0 driving 24-bit parallel video to the bridge. Also sets up MCASP1 for HDMI audio via simple-audio-card.

**Observations:**

- The DT uses DSS1 VP0 for HDMI with a comment noting VP choice. Good documentation.
- The `bootph-all` on `main_i2c1_pins_default` seems unnecessary for a display path that shouldn't be needed in early boot, but it's harmless.
- The `auxclk-fs-ratio = <2177>` for MCASP is an unusual value. The cover letter mentions "Sample-rate is off by 400 PPM" which suggests this ratio doesn't perfectly divide the master clock for standard audio sample rates, but 400 PPM is within typical acceptable limits.
- The pin mux for MCASP1_AXR0 is set as `PIN_OUTPUT` which is correct for TX.

No blocking issues.

**Summary:** The series is well-prepared for v3 with proper error handling improvements from v2. The main area I'd suggest looking at is the unconditional `clk_put(oldi->serial)` in the OLDI aux driver remove/error paths (patch 12) — while it's likely safe due to zeroed allocation, it would be cleaner to guard with a NULL check or mirror the conditional from probe. The rest looks solid and ready for merging.

---
Generated by Claude Code Patch Reviewer