Re: [PATCH v12 02/69] vmlinux.lds.h: move BOUNDED_SECTION_* macros to reuse later
| Newsgroups | gmane.comp.freedesktop.xorg.drivers.intel,gmane.comp.video.dri.devel |
|---|---|
| Message-ID | <[email protected]> |
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: - [Low] The newly created file `include/asm-generic/bounded_sections.lds.h` is missing an SPDX license identifier. -- --- Patch [2]: [PATCH v12 02/69] vmlinux.lds.h: move BOUNDED_SECTION_* macros to reuse later --- commit 66f82d3de735bc79fa17c5cd892641b6f5cecb8c Author: Jim Cromie <[email protected]> vmlinux.lds.h: move BOUNDED_SECTION_* macros to reuse later This commit moves the BOUNDED_SECTION_* macros into a newly created header file, include/asm-generic/bounded_sections.lds.h, and includes it back into vmlinux.lds.h. This change sets up the macros to be reused later. > diff --git a/include/asm-generic/bounded_sections.lds.h b/include/asm-generic/bounded_sections.lds.h > new file mode 100644 > index 0000000000000..280a893bbe29f > --- /dev/null > +++ b/include/asm-generic/bounded_sections.lds.h > @@ -0,0 +1,36 @@ > +#ifndef _ASM_GENERIC_BOUNDED_SECTIONS_H [Severity: Low] Should this newly created file include an SPDX license identifier at the top? Kernel policy generally requires new source files to have one. > +#define _ASM_GENERIC_BOUNDED_SECTIONS_H > + > +#define BOUNDED_SECTION_PRE_LABEL(_sec_, _label_, _BEGIN_, _END_) \ [ ... ] -- Sashiko AI review · https://sashiko.dev/#/patchset/[email protected]?part=2