Claude review: drm/rockchip: analogix_dp: Enable hclk for RK3588

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

Patch Review

Acquires the hclk using `devm_clk_get_optional_enabled()`:

```c
+	clk = devm_clk_get_optional_enabled(dev, "hclk");
+	if (IS_ERR(clk))
+		return dev_err_probe(dev, PTR_ERR(clk),
+				     "failed to get hclk property\n");
```

Good use of `devm_clk_get_optional_enabled()` -- it handles both the "not present" case (for RK3288/RK3399) and the error case cleanly. The clock is managed by devres so it stays enabled for the device lifetime, which matches the commit message's statement that hclk "must remain enabled during probe." The local `clk` variable is unused after the call, which is fine since devm handles the lifecycle.

One thing to note: this means hclk is always-on once probed. The existing `pclk` is toggled in `rockchip_dp_poweron`/`rockchip_dp_powerdown`, but hclk is not. Per the commit message this is intentional ("must remain enabled during probe") and the hardware requires it to stay on for datapath gating. This seems correct.

---
Generated by Claude Code Patch Reviewer