Claude review: drm/imagination: simplify pvr_fw_trace_seq_data

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

Patch Review

**Correctness: Good**

The conversion is mechanically correct:

- The `u32 *buffer` pointer is replaced with `u32 buffer[]` flexible array member, properly placed at the end of the struct.
- `kzalloc_obj(*trace_seq_data)` + `kcalloc(ROGUE_FW_TRACE_BUF_DEFAULT_SIZE_IN_DWORDS, ...)` is replaced with a single `kzalloc_flex(*trace_seq_data, buffer, ROGUE_FW_TRACE_BUF_DEFAULT_SIZE_IN_DWORDS)`, which will compute the correct total size via `struct_size`.
- The `err_free_buffer` goto label and `kfree(trace_seq_data->buffer)` in both the error path and `fw_trace_release()` are correctly removed, since one `kfree(trace_seq_data)` now frees everything.

**All existing users of `trace_seq_data->buffer` remain valid** — the FAM is accessed with the same `trace_seq_data->buffer[idx]` syntax (confirmed at line 276 of the patched file in `read_fw_trace()`).

**No issues found.** This is a clean, correct simplification.

**Reviewed-by worthy:** Yes, this patch is ready to apply.

---
Generated by Claude Code Patch Reviewer