On Wed, 2025-04-30 at 09:39 +1000, Ben Skeggs wrote:
> This commit enables basic support for Hopper GPUs, and is intended
> primarily as a base supporting Blackwell GPUs, which reuse most of
> the code added here.

...

> --- /dev/null
> +++ b/drivers/gpu/drm/nouveau/nvkm/subdev/fb/gh100.c
> @@ -0,0 +1,47 @@
> +/*
> + * Copyright (c) 2025, NVIDIA CORPORATION. All rights reserved.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be included in
> + * all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> + * DEALINGS IN THE SOFTWARE.

Shouldn't this just be an SPDX comment?

> +#ifndef SHF_STRINGS
> +/* These might not be defined in elf.h */
> +#define SHF_STRINGS          0x20
> +#define SHF_OS_NONCONFORMING 0x100
> +#define SHF_MASKOS           0x0FF00000
> +#endif

This is no longer necessary.

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/include/uapi/linux/elf.h?id=b0db1ed17645b53993bae3dd2c4be7013600084f

> +/**
> + * elf_validate_sections - validate each section in the FMC ELF image
> + * @elf: ELF iamge

@elf: ELF image
@length: size of the entire ELF image


> +/**
> + * elf_section - return a pointer to the data for a given section
> + * @elf: ELF image
> + * @name: section name to search for
> + * @psize: pointer to section header for found section

Replace this last line with:

@len: pointer to returned length of found section

Reply via email to