Claude review: arm64: dts: imx8mq: Add DCSS + HDMI/DP display pipeline

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

Patch Review

**PHY and bridge share the same reg address:**
```dts
mdhp_phy: phy@32c00000 {
reg = <0x32c00000 0x100000>;
};

mhdp: bridge@32c00000 {
reg = <0x32c00000 0x100000>;
};
```
Two nodes with the same `reg` will fail `dtbs_check` and is technically incorrect. If they truly share the same MMIO space, the typical pattern is either a syscon node, or one node with sub-resources. This needs resolution.

**Typo in node name**: `mdhp_phy` should be `mhdp_phy` (transposed 'd' and 'h').

**Missing `cdns,bridge-type`**: The `mhdp` node in the SoC dtsi doesn't set `cdns,bridge-type`, relying on board files to provide it. But the binding marks it as required, so this DTS will fail validation. It should either be set per-board (which it is in patch 8) via an override, or the property should not be required in the binding.

**`mhdp_out` endpoint is empty:**
```dts
port@1 {
reg = <1>;
mhdp_out: endpoint {
};
};
```
An empty endpoint without `remote-endpoint` or `data-lanes` is unusual. Board-level DTS overrides it in patch 8 but the SoC-level node still has an incomplete endpoint that won't pass binding validation (missing required `data-lanes`).

---
Generated by Claude Code Patch Reviewer