Claude review: dt-bindings: display: tegra: document Tegra20 HDMI port

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

Patch Review

**COMMIT MESSAGE:**
The commit message is minimal but adequate for a binding addition. It states what is being added without explaining why the change is needed.

**TECHNICAL ISSUES:**

1. **YAML indentation errors (BLOCKING):**
```yaml
+anyOf:
+  - required:
+    - nvidia,ddc-i2c-bus
+    - nvidia,hpd-gpio
+  - required:
+    - port
```

The yamllint warnings indicate lines 107 and 110 have incorrect indentation. The `- required:` entries under `anyOf` should be indented with 6 spaces, not 4. This causes the binding validation to fail.

**Correct format should be:**
```yaml
anyOf:
  - required:
      - nvidia,ddc-i2c-bus
      - nvidia,hpd-gpio
  - required:
      - port
```

2. **Port description clarity:**
```yaml
+  port:
+    description: connection to controller receiving HDMI signals
```

Rob Herring correctly points out this describes the remote endpoint, not the local port. The description should focus on what this HDMI block outputs.

**Suggested improvement:**
```yaml
  port:
    description: HDMI output port for connection to HDMI connector or bridge
```

**STRUCTURAL REVIEW:**

The logic of making `nvidia,ddc-i2c-bus`/`nvidia,hpd-gpio` and `port` mutually exclusive via `anyOf` is correct:
- Old binding: direct DDC/HPD properties (deprecated but still supported)
- New binding: OF graph port (connects to hdmi-connector node)

This maintains backward compatibility while enabling the modern approach.

**RATING:** Needs work (indentation must be fixed, description should be improved)

---

---
Generated by Claude Code Patch Reviewer