On Thu, 2017-06-08 at 03:32 +0000, Gao, Liming wrote: > Supreeth: > Thanks for your contribution. I review this patch set, and have two > comments. Thanks Liming for reviewing the Patch set.
> 1) This is added in PI1.5. So, its version check should be with > 0x00010032 (1 and 50), not 0x0001000A (1 and 10). Smm standalone > module INF needs to specify PI_SPECIFICATION_VERSION = > 0x00010032. I will rectify it and upload v2 version. > 2) Does the change conform to doc change recorded in below bugzilla? > https://bugzilla.tianocore.org/show_bug.cgi?id=550 Not sure about the [Depex] sections (page 34). Will check this and confirm before uploading v2. > https://bugzilla.tianocore.org/show_bug.cgi?id=551 Yes. It conforms with this, though I will recheck once again. > https://bugzilla.tianocore.org/show_bug.cgi?id=552 Yes. It conforms with this, though I will recheck once again. > > Thanks > Liming > > > > -----Original Message----- > > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf > > Of > > Supreeth Venkatesh > > Sent: Tuesday, June 06, 2017 5:44 AM > > To: edk2-devel@lists.01.org > > Cc: leif.lindh...@arm.com; ard.biesheu...@linaro.org > > Subject: [edk2] [PATCH 00/10] *** BaseTools: Tools change to > > support PI v1.5 > > Specification. *** > > > > *** > > PI v1.5 Specification Volume 4 defines Management Mode Core > > Interface. > > In order to support Management Mode Core Interface, Module Types > > MM_STANDALONE, MM_CORE_STANDALONE are needed. > > PI specification v1.5 defines the following new file types: > > #define EFI_FV_FILETYPE_MM 0x0A > > #define EFI_FV_FILETYPE_COMBINED_MM_DXE 0x0C > > #define EFI_FV_FILETYPE_MM_STANDALONE 0x0E > > The following changes are necessitated in order to support the > > above changes > > to the specification. > > *** > > > > Supreeth Venkatesh (10): > > BaseTools: Tools change to support PI v1.5 Specification. > > BaseTools: Tools change to support PI v1.5 Specification. > > BaseTools: Tools change to support PI v1.5 Specification. > > BaseTools: Tools change to support PI v1.5 Specification. > > BaseTools: Tools change to support PI v1.5 Specification. > > BaseTools: Tools change to support PI v1.5 Specification. > > BaseTools: Tools change to support PI v1.5 Specification. > > BaseTools: Tools change to support PI v1.5 Specification. > > BaseTools: Tools change to support PI v1.5 Specification. > > BaseTools: Tools change to support PI v1.5 Specification. > > > > BaseTools/Source/C/GenFfs/GenFfs.c | 6 +- > > BaseTools/Source/C/GenFw/GenFw.c | 5 +- > > BaseTools/Source/C/Include/Common/PiFirmwareFile.h | 1 + > > BaseTools/Source/Python/AutoGen/GenC.py | 148 > > ++++++++++++++++++++- > > BaseTools/Source/Python/AutoGen/GenDepex.py | 17 ++- > > BaseTools/Source/Python/Common/DataType.py | 4 +- > > BaseTools/Source/Python/Common/FdfParserLite.py | 4 +- > > .../Source/Python/CommonDataClass/CommonClass.py | 6 +- > > .../Source/Python/CommonDataClass/PackageClass.py | 2 +- > > BaseTools/Source/Python/Eot/FvImage.py | 1 + > > BaseTools/Source/Python/GenFds/FdfParser.py | 4 +- > > BaseTools/Source/Python/GenFds/Ffs.py | 7 +- > > BaseTools/Source/Python/GenFds/FfsInfStatement.py | 3 + > > .../Source/Python/Workspace/WorkspaceDatabase.py | 4 + > > BaseTools/Source/Python/build/BuildReport.py | 4 +- > > BaseTools/Source/Python/build/build.py | 2 +- > > 16 files changed, 199 insertions(+), 19 deletions(-) > > > > -- > > 2.7.4 > > > > _______________________________________________ > > edk2-devel mailing list > > edk2-devel@lists.01.org > > https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel