Claude review: drm/vkms: Clean up artifacts from old vblank timer

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

Patch Review

**Changes reviewed:**

1. **`vkms_drv.h`: Remove `#include <linux/hrtimer.h>`** — Correct. No struct members or function signatures in this header use `hrtimer_*` types after the conversion. The include is dead.

2. **`vkms_drv.h`: Remove kernel-doc for `vblank_hrtimer` and `period_ns`** — Correct. These fields were removed in `02e2681ffe1a` but their documentation was left behind. Removing orphaned kernel-doc entries is the right cleanup.

3. **`vkms_composer.c` line 597**: Comment update from `vkms_vblank_simulate()` to `vkms_crtc_handle_vblank_timeout()` — Correct. `vkms_crtc_handle_vblank_timeout` exists in `vkms_crtc.c:16` and is the current function name.

4. **`vkms_composer.c` line 639**: Comment update from "vblank hrtimer" to "vblank timer" — Correct.

**Issue — missed stale reference:**

There is a third "hrtimer" comment in `vkms_composer.c` at line 662 that was not updated:

```c
/*
 * The worker can fall behind the vblank hrtimer, make sure we catch up.
 */
```

This should be changed to "vblank timer" for consistency with the other comment fix in this same patch. Since the stated goal of the patch is to "fix comments referring to the original implementation," this seems like an oversight.

**Nit:** The `Fixes:` tag is arguably unnecessary here — this is a comment/include cleanup, not a functional fix. It won't cause backporting confusion since the referenced commit is recent, but `Fixes:` typically implies a behavioral/correctness issue that stable kernels should pick up. This is minor and a matter of project convention.

---
Generated by Claude Code Patch Reviewer