Claude review: dt-bindings: display: rockchip: analogix-dp: Fix hclk as third clock for RK3588

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

Patch Review

Clean approach. The top-level `clock-names` allows both `grf` and `hclk` via enum, with per-SoC allOf constraints controlling the count:

```yaml
      - enum:
          - grf
          - hclk
```

The per-SoC constraints enforce clock count (2 for RK3288, 3 for RK3399/RK3588) but don't constrain *which* third clock name each SoC must use. In theory, someone could write `"grf"` as the third clock for RK3588 and the binding would still pass. The driver implicitly enforces correctness (it looks for `"hclk"` specifically), but the schema doesn't. This is a minor observation and not blocking -- tightening this would add significant YAML complexity for little practical gain since the canonical DTS uses the right names.

No issues.

---
Generated by Claude Code Patch Reviewer