Claude review: dt-bindings: display: rockchip: analogix-dp: Add support for RK3576

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

Patch Review

**Clock-names binding change is problematic.** The patch changes the third clock-name from a fixed `const: grf` to an `enum: [grf, hclk]`:

```yaml
-      - const: grf
+      - enum:
+          - grf
+          - hclk
```

This is a semantic change that affects **all** existing compatibles, not just the new RK3576 one. The RK3288 uses a `grf` clock, and now the schema would also allow `hclk` for RK3288, which doesn't make sense. This should instead be handled with a per-compatible conditional (in the `allOf` section), similar to how the reset constraints are already conditional. For example, add an `if`/`then` block that constrains clock-names for RK3576 to `[dp, pclk, hclk]` and keeps the existing behavior for other compatibles.

Alternatively, since RK3576 uses `hclk` and RK3288 uses `grf` as the third clock, consider restructuring the items list to use `oneOf` or per-compatible `clock-names` constraints.

**The cover letter says "The newly added clock 'hclk' is the video datapath clock"** — this description should be in the binding document itself (in the items description or the property description), not just the commit message.

---
Generated by Claude Code Patch Reviewer