Claude review: gpu: nova-core: move lifetime to `Bar0`

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

Patch Review

**Commit message typo:**

```
and thuis the owned MMIO type would be removed
```

"thuis" → "thus"

**Core type change (driver.rs):**

```rust
-pub(crate) type Bar0 = kernel::io::Mmio<BAR0_SIZE>;
+pub(crate) type Bar0<'a> = &'a pci::Bar<'a, BAR0_SIZE>;
```

This is the key line. `Bar0<'a>` collapses what was `&'a Mmio<BAR0_SIZE>` into a single type alias that is itself a reference. Since references are `Copy`, all downstream usage sites that previously passed `&Bar0` (implicitly copyable) now pass `Bar0<'_>` (still copyable). Semantically equivalent.

One observation: tying both the reference lifetime and the `Bar`'s internal lifetime to the same `'a` could in theory be more restrictive than having them separate. In practice, every usage in the codebase already had both lifetimes unified (e.g., `&'gpu Bar0` where `Bar0` was `Mmio` which internally held the same `'gpu`-bounded mapping), so this is fine.

**Mechanical signature changes (falcon.rs, fb.rs, gsp/, fsp/, etc.):**

All 30+ files follow the same pattern consistently:

```rust
// Before
pub(crate) fn dma_reset(&self, bar: &Bar0) {
// After
pub(crate) fn dma_reset(&self, bar: Bar0<'_>) {
```

and for struct fields / lifetime-carrying positions:

```rust
// Before
bar: &'a Bar0,
// After
bar: Bar0<'a>,
```

I checked a representative sample across `falcon.rs`, `falcon/hal.rs`, `falcon/hal/ga102.rs`, `falcon/hal/tu102.rs`, `fb.rs`, `fb/hal.rs`, all `fb/hal/*.rs`, `gsp/boot.rs`, `gsp/cmdq.rs`, `gsp/hal.rs`, `gsp/hal/tu102.rs`, `gsp/sequencer.rs`, `regs.rs`, `vbios.rs`, `firmware/booter.rs`, `firmware/fwsec.rs`, `fsp.rs`, and `gpu/hal.rs`. All transformations are consistent and correct.

**Non-mechanical change (gpu.rs) — field reordering:**

```rust
// Before (within try_pin_init! macro):
-            bar,
             sysmem_flush: SysmemFlush::register(pdev.as_ref(), bar, spec.chipset)?,
             ...
             unload_bundle: gsp.boot(pdev, bar, spec.chipset, gsp_falcon, sec2_falcon)?,
+            bar,
```

The `bar` field initialization was moved from before `sysmem_flush` to after `unload_bundle`. Since `Bar0<'_>` is `Copy`, this has no effect on the value or on borrowing. However, in `try_pin_init!`, fields are initialized in written order, and on error the already-initialized fields are dropped in reverse. Moving `bar` to last means it's dropped first on error — but since it's a reference with no destructor, this is a no-op. Still, it would be helpful if the commit message briefly noted **why** the reorder was needed or desired. If it's purely a style preference (group the reference at the end), that's fine — but if it's to satisfy a borrow-checker constraint from the type change, that should be documented.

**Trait impls (falcon/hal.rs, fb/hal.rs, gpu/hal.rs, fsp/hal.rs, gsp/hal.rs):**

All trait definitions and their implementations were updated in lockstep. No trait method was missed. Default implementations (e.g., `select_core` returning `Ok(())`) were also updated. Consistent and complete.

**Overall:** Clean, correct mechanical refactoring with clear motivation. The only actionable items are the "thuis" typo and optionally a note about the `gpu.rs` field reordering.

---
Generated by Claude Code Patch Reviewer