Claude review: drm/bridge: imx93-mipi-dsi: Fix mode validation

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

Overall Series Review

Subject: drm/bridge: imx93-mipi-dsi: Fix mode validation
Author: Liu Ying <victor.liu@nxp.com>
Patches: 1
Reviewed: 2026-05-16T09:49:36.532814

---

This is a single-patch fix for mode validation in the i.MX93 MIPI DSI bridge driver. The problem: the old code checked only the **last** bridge in the chain for `DRM_BRIDGE_OP_DETECT | DRM_BRIDGE_OP_EDID`, but when the last bridge is a connector bridge (with no ops set), the check always fails, and the +/-0.5% pixel clock deviation tolerance is never applied. The fix iterates over all bridges after the DSI bridge, looking for the first one that has both ops set.

The patch is well-reasoned and correct. The refcounting is handled properly in all paths (normal exit, early return on `MODE_NOCLOCK`, loop exhaustion). The manual iteration (instead of using `drm_for_each_bridge_in_chain_from()`) is the right approach because `bridge_chain_mutex` is already held by the caller `drm_bridge_chain_mode_valid()` — using the macro would deadlock. This was identified in v2 review and fixed in v3.

**Verdict: Looks good.** One minor observation below but nothing blocking.

---

---
Generated by Claude Code Patch Reviewer