Claude review: gpu: nova: require little endian

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

Overall Series Review

Subject: gpu: nova: require little endian
Author: Eliot Courtney <ecourtney@nvidia.com>
Patches: 7
Reviewed: 2026-04-12T14:07:45.623021

---

This is a single-patch series that adds `depends on !CPU_BIG_ENDIAN` to both `nova-core` and `nova-drm` Kconfig entries. The change is straightforward, well-motivated, and correct in intent: the nova driver code assumes little-endian byte ordering (e.g., directly interpreting GPU command queue bytes), and this makes that assumption explicit at the config level rather than leaving it implicit.

The author notes that `RUST` transitively depends on `!CPU_BIG_ENDIAN`. Checking the current tree, this is *practically* true but not *technically guaranteed*: arm64 selects `HAVE_RUST` unconditionally, but arm64's `CPU_BIG_ENDIAN` currently depends on `BROKEN` (so it's effectively unreachable). The other arch that could theoretically enable both is RISC-V, but it doesn't define `CPU_BIG_ENDIAN` at all. So while the transitive dependency holds today through indirect means, adding the explicit `depends on !CPU_BIG_ENDIAN` is the right thing to do -- it documents the driver's actual requirement rather than relying on an accidental property of the Rust toolchain's arch support.

**Verdict:** The patch is clean, minimal, and correct. No issues found.

---

---
Generated by Claude Code Patch Reviewer