Claude review: drm/tyr: move clock cleanup into Clocks Drop impl

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

Patch Review

**Author:** Deborah Brouwer

Moves clock disable/unprepare from `TyrDrmDeviceData::PinnedDrop` into a `Drop` impl on `Clocks` itself. This ensures clocks are disabled last (after other fields that may depend on them are dropped), following Rust's field drop order (fields drop in declaration order).

```rust
+impl Drop for Clocks {
+    fn drop(&mut self) {
+        self.core.disable_unprepare();
+        self.stacks.disable_unprepare();
+        self.coregroup.disable_unprepare();
+    }
+}
```

This is correct because `Clocks` is declared after other fields in `TyrDrmDeviceData`, so it drops last. Also removes `#[pin_data]` from `Clocks` and `Regulators` since they don't need structural pinning.

Clean and well-motivated change.

---
Generated by Claude Code Patch Reviewer