Claude review: gpu: nova-core: convert PMC registers to kernel register macro

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

Patch Review

The core PMC register conversion. The `Architecture` type changes from `TryFrom<u8>` to `TryFrom<Bounded<u32, 6>>` and the `Revision` struct members change to `Bounded<u8, 4>`, with `.cast()` calls to convert between bounded types.

One inconsistency at line 829:
```rust
-            self.0,
+            self.inner,
```
The `Display` impl for `NV_PMC_BOOT_42` directly accesses `self.inner` to get the raw value, while elsewhere the series consistently uses `self.into_raw()` (e.g., line 819: `self.into_raw() >> Self::ARCHITECTURE_RANGE.start()`). Consider using `self.into_raw()` here too for consistency, or verify that `self.inner` is the intended public API of the kernel register macro (it may be — the `inner` field might be `pub` on the generated struct).

The `From<Architecture> for Bounded<u32, 6>` implementation at lines 700-703 maps each variant explicitly via `Bounded::<u32, _>::new::<0x16>()` etc., which is correct and avoids the previous `repr(u8)` cast.

The removal of `Default` from `Architecture` and `Revision` is good — those defaults were artifacts of the old macro.

---
Generated by Claude Code Patch Reviewer