Claude review: dt-bindings: display: bridge: Add Cadence MHDP8501

Claude Code Review Bot
2026-04-12T03:34:45Z

Patch Review

**`cdns,bridge-type` property concern:**
```yaml
cdns,bridge-type:
$ref: /schemas/types.yaml#/definitions/uint32
enum: [0, 1]
description: |
Type of bridge output:
0: DisplayPort
1: HDMI
```
Using a vendor-specific integer property to specify bridge type is questionable. The connector type should ideally be determined by the downstream port's compatible (e.g., `hdmi-connector` vs `dp-connector`). The cover letter mentions "I couldn't find another way to do it..." — the bridge should be able to discover this from its output port graph connection. This was also flagged in previous reviews by Dmitry.

**Missing `Reviewed-by` tags**: No dt-bindings maintainer review tag. Krzysztof's tag was dropped since a new property was added.

**Nit in example DTS:**
```yaml
phys = <&mdhp_phy>;
```
The phandle `mdhp_phy` has a typo (should be `mhdp_phy`, matching standard MHDP naming). This same typo propagates to patch 7.

---
Generated by Claude Code Patch Reviewer