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

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

Patch Review

The commit message mentions introducing `nv_reg` macro, but per v2 changelog this was dropped. The commit message should be updated to remove the `nv_reg` reference:

> nova-core's registers have some constant properties (like a 32-bit size
> and a crate visibility), so introduce the `nv_reg` macro to shorten
> their declaration.

This text is stale from v1.

In `regs.rs`, the `Display` impl for `NV_PMC_BOOT_42` uses `self.inner`:
```rust
            self.inner,
```
while the `architecture_raw` method uses `self.into_raw()`:
```rust
        ((self.into_raw() >> Self::ARCHITECTURE_RANGE.start())
```

This is inconsistent. Since `into_raw()` is the proper accessor pattern for the kernel register macro, `self.inner` should likely also be `self.into_raw()`. However, this depends on whether the kernel macro exposes `inner` as a public field -- it may do so, but using `into_raw()` would be more consistent. Since I can't verify the kernel macro's API (it's on drm-rust-next, not drm-next), this is a potential issue worth flagging.

The `From<Architecture> for Bounded<u32, 6>` impl is more verbose than the old `repr(u8)` cast but is required by the new type system. The `Bounded::new::<CONST>()` pattern is clear.

---
Generated by Claude Code Patch Reviewer