On 3/5/24 13:01, Laszlo Ersek wrote: > Bugzilla: > - https://bugzilla.tianocore.org/show_bug.cgi?id=990 > > This patch series puts the recent BaseTools feature to use in which > AutoGen generates the ProcessLibraryConstructorList() declaration in > "AutoGen.h" for such non-library SEC modules whose INF_VERSION is at > least 1.30. The BaseTools feature is present in both edk2 [1] and > edk2-basetools [2], and has been documented in the Build spec [3] and > the Inf spec [4]. Kudos to Rebecca for tagging a new edk2-basetools > release [5] [6] with the new feature. > > [1] edk2 commit bac9c74080cf > [2] edk2-basetools commit 5b7161de22ee > [3] edk2-BuildSpecification commit range db69f5661cae..7a7165a7d199 > [4] edk2-InfSpecification commit range a31e3c842bee..1ea6546578fe > [5] https://github.com/tianocore/edk2-basetools/releases/tag/v0.1.51 > [6] https://pypi.org/project/edk2-basetools/0.1.51/ > > The edk2-basetools part is adopted in the first patch (for > "pip-requirements.txt") of the edk2 series > > [edk2-devel] [PATCH v2 00/10] > clean up ProcessLibraryConstructorList() declarations in SEC modules > > https://edk2.groups.io/g/devel/message/116367 > msgid <20240305113843.68812-1-ler...@redhat.com> > > The rest of the patches clean up -- superfluous, or even incorrect -- > ProcessLibraryConstructorList() declarations (and, in some cases, > incorrect calls), together with raising the INF_VERSIONs in the related > SEC module INF files to 1.30. > > Comparing this version to v1 is not useful, as the compatibility > approach is different, and so this version is structured differently. > Please review any patches for your subsystem from scratch (they are not > difficult or large). > > Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> > Cc: Leif Lindholm <quic_llind...@quicinc.com> > Cc: Nate DeSimone <nathaniel.l.desim...@intel.com> > Cc: Sai Chaganty <rangasai.v.chaga...@intel.com> > Cc: Bibo Mao <maob...@loongson.cn> > Cc: Chao Li <lic...@loongson.cn> > Cc: Xianglai li <lixiang...@loongson.cn> > Cc: Sunil V L <suni...@ventanamicro.com> > Cc: USER0FISH <libing1...@outlook.com> > Cc: caiyuqing379 <caiyuqing...@outlook.com> > Cc: dahogn <dah...@hotmail.com> > Cc: meng-cz <mengcz1...@gmail.com> > > Thanks, > Laszlo > > Laszlo Ersek (4): > BeagleBoardPkg: auto-generate SEC ProcessLibraryConstructorList() decl > SimicsOpenBoardPkg: auto-gen & fix SEC ProcessLibraryConstructorList() > decl > LoongArchQemuPkg: auto-gen & fix SEC ProcessLibraryConstructorList() > decl > SG2042Pkg/Sec: clean up ProcessLibraryConstructorList() decl > > Platform/BeagleBoard/BeagleBoardPkg/PrePi/PeiUniCore.inf | 2 +- > Platform/BeagleBoard/BeagleBoardPkg/PrePi/PrePi.h | 6 ------ > Platform/Intel/SimicsOpenBoardPkg/SecCore/SecMain.c | 3 +-- > Platform/Intel/SimicsOpenBoardPkg/SecCore/SecMain.inf | 2 +- > Platform/Loongson/LoongArchQemuPkg/Sec/SecMain.c | 3 +-- > Platform/Loongson/LoongArchQemuPkg/Sec/SecMain.inf | 2 +- > Silicon/Sophgo/SG2042Pkg/Sec/Memory.c | 1 - > Silicon/Sophgo/SG2042Pkg/Sec/SecMain.h | 12 ------------ > Silicon/Sophgo/SG2042Pkg/Sec/SecMain.inf | 2 +- > 9 files changed, 6 insertions(+), 27 deletions(-) > > > base-commit: fe41713668d42b20a2370dab27de3269e877e454
Merged by Ard as commit range 4e478301f40e..bb44e786e7a7, via <https://github.com/tianocore/edk2-platforms/pull/130>. Thanks! Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#116497): https://edk2.groups.io/g/devel/message/116497 Mute This Topic: https://groups.io/mt/104742756/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-