Claude review: drm/imagination: Switch reset_reason fields from enum to u32

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

Patch Review

**Verdict: Good with a minor note.**

The rationale is correct — `enum` is not a fixed-size type in C and using it in firmware interface structures that must match a specific binary layout is fragile. The existing `pvr_rogue_fwif_check.h` offset checks (offset 4, next field at offset 8) confirm the field is expected to be 4 bytes, which `u32` guarantees.

The `Fixes:` tag referencing the original FWIF headers commit is appropriate.

One minor observation: `struct rogue_context_reset_reason_data` in `pvr_rogue_fwif_shared.h` is also changed, but I don't see a corresponding offset/size check for that struct in the check headers. This is not a problem with the patch itself, just a pre-existing gap.

---

---
Generated by Claude Code Patch Reviewer