On 12/17/25 7:29 PM, Timur Tabi wrote:
> Turing and GA100 share the same GSP-RM firmware binary, and the
> signature ELF section is labeled ".fwsignature_tu10x".
> 
> Signed-off-by: Timur Tabi <[email protected]>
> ---
>  drivers/gpu/nova-core/firmware/gsp.rs | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/nova-core/firmware/gsp.rs 
> b/drivers/gpu/nova-core/firmware/gsp.rs
> index 0549805282ab..112488216bff 100644
> --- a/drivers/gpu/nova-core/firmware/gsp.rs
> +++ b/drivers/gpu/nova-core/firmware/gsp.rs
> @@ -163,9 +163,11 @@ pub(crate) fn new<'a, 'b>(
>          let fw_section = elf::elf64_section(fw.data(), 
> ".fwimage").ok_or(EINVAL)?;
>  
>          let sigs_section = match chipset.arch() {

After applying the fix that I mentioned in my other reply:

+            Architecture::Turing if chipset == Chipset::TU116 || chipset == 
Chipset::TU117 => {
+                ".fwsignature_tu11x"
+            }

, please feel free to add

Reviewed-by: John Hubbard <[email protected]>

thanks,
-- 
John Hubbard

> +            Architecture::Turing => ".fwsignature_tu10x",
> +            // GA100 uses the same firmware as Turing
> +            Architecture::Ampere if chipset == Chipset::GA100 => 
> ".fwsignature_tu10x",
>              Architecture::Ampere => ".fwsignature_ga10x",
>              Architecture::Ada => ".fwsignature_ad10x",
> -            _ => return Err(ENOTSUPP),
>          };
>          let signatures = elf::elf64_section(fw.data(), sigs_section)
>              .ok_or(EINVAL)


Reply via email to