Claude review: drm/bridge: imx8mp-hdmi-tx: add an hdmi-connector when missing using a DT overlay at boot time

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

Patch Review

**Dead variable:** `hdmi_conn` is declared with `__free(device_node)` but is never assigned or used anywhere in the function. This should be removed.

```c
struct device_node *hdmi_conn __free(device_node) = NULL;  // unused
```

**Minor typos in commit message:** "device tre" should be "device tree", "isntantiating" should be "instantiating", "abd" should be "and".

**Error handling for `of_overlay_fdt_apply`:** The function returns `of_overlay_fdt_apply(...)` directly. If the overlay application fails, this returns an error from a `subsys_initcall`, which will be logged but won't prevent boot. This seems acceptable — the HDMI output simply won't work, which is a degraded but not catastrophic failure.

**Hardcoded paths:** The code uses hardcoded DT paths like `/soc@0/bus@32c00000/hdmi@32fd8000`. These are specific to the i.MX8MP SoC and the compatible check (`fsl,imx8mp-soc`) guards against running on other hardware. This is reasonable for a fixup module.

**Kconfig:** The `DRM_IMX8MP_DW_HDMI_BRIDGE_CONNECTOR_FIXUP` is a `bool` with `default y` — it's always built when both LCDIF and HDMI bridge are enabled. The `select OF_OVERLAY` and `select DRM_DISPLAY_CONNECTOR` on the parent config are appropriate.

**The overlay itself** looks correct — it adds the `fixup-hdmi-connector` node at root level and adds a `port@1/endpoint` to the existing hdmi node with proper remote-endpoint phandle linkage.

---
Generated by Claude Code Patch Reviewer