Claude review: gpu: nova-core: vbios: compute FWSEC-relative Falcon data offset

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

Patch Review

Good refactoring. Renames `falcon_data_ptr` to `falcon_data_offset` and folds the PCI-AT-length subtraction into the function, making the return value directly usable as a FWSEC-relative offset. The function also gains proper bounds checking via `.get()` and `u32::from_bytes_copy_prefix` instead of the manual slice-to-array conversion.

The new bounds check is correct:
```rust
usize::from_safe_cast(ptr)
    .checked_sub(data.len())
    .ok_or(EINVAL)
```
This replaces the old check `if (usize::from_safe_cast(data_ptr)) < self.base.data.len()` which would error if the pointer was *within* the PCI-AT image (meaning it doesn't point to FWSEC data). The semantics are the same.

No issues.

---
Generated by Claude Code Patch Reviewer