Re: [PATCH v2] gpu: nova: require little endian

[John Hubbard <jhubbard@nvidia.com>]

On 4/6/26 8:59 PM, Eliot Courtney wrote:
> The driver already assumes little endian in a lot of locations. For
> example, all the code that reads RPCs out of the command queue just
> directly interprets the bytes.
> 
> Make this explicit in Kconfig.
> 
> Signed-off-by: Eliot Courtney <ecourtney@nvidia.com>
> ---
> The current code assumes little endian in a bunch of places. I think we
> should either explicitly decide to be generic on endianness or explicitly
> decide not to - having some handling sprinkled around in various
> locations seems confusing to me.
> 
> I believe that currently e.g. `RUST` transitively depends on
> !CPU_BIG_ENDIAN, so this is more about making the decision explicit for
> nova-core rather than fixing any kind of hole.
> ---
> Changes in v2:
> - Also add to nova-drm (Thanks Danilo)
> - Did not take Reviewed-by + Acked-by because commit changed
>    substantively
> - Link to v1: https://patch.msgid.link/20260406-fix-kconfig-v1-1-8f634499f7f0@nvidia.com
> ---
>   drivers/gpu/drm/nova/Kconfig  | 1 +
>   drivers/gpu/nova-core/Kconfig | 1 +
>   2 files changed, 2 insertions(+)

Reviewed-by: John Hubbard <jhubbard@nvidia.com>

thanks,
-- 
John Hubbard

> 
> diff --git a/drivers/gpu/drm/nova/Kconfig b/drivers/gpu/drm/nova/Kconfig
> index 3e637ad7b5ba..a2028b8539d7 100644
> --- a/drivers/gpu/drm/nova/Kconfig
> +++ b/drivers/gpu/drm/nova/Kconfig
> @@ -4,6 +4,7 @@ config DRM_NOVA
>   	depends on DRM=y
>   	depends on PCI
>   	depends on RUST
> +	depends on !CPU_BIG_ENDIAN
>   	select AUXILIARY_BUS
>   	select NOVA_CORE
>   	default n
> diff --git a/drivers/gpu/nova-core/Kconfig b/drivers/gpu/nova-core/Kconfig
> index a4f2380654e2..d8456f8eaa05 100644
> --- a/drivers/gpu/nova-core/Kconfig
> +++ b/drivers/gpu/nova-core/Kconfig
> @@ -3,6 +3,7 @@ config NOVA_CORE
>   	depends on 64BIT
>   	depends on PCI
>   	depends on RUST
> +	depends on !CPU_BIG_ENDIAN
>   	select AUXILIARY_BUS
>   	select RUST_FW_LOADER_ABSTRACTIONS
>   	default n
> 
> ---
> base-commit: a7a080bb4236ebe577b6776d940d1717912ff6dd
> change-id: 20260406-fix-kconfig-3a059f622697
> 
> Best regards,
> --
> Eliot Courtney <ecourtney@nvidia.com>
>