Jiewen: Thanks Liming > -----Original Message----- > From: Yao, Jiewen > Sent: Tuesday, June 25, 2019 10:15 PM > To: Gao, Liming <liming....@intel.com>; devel@edk2.groups.io; Zhang, Shenglei > <shenglei.zh...@intel.com>; ard.biesheu...@linaro.org; > leif.lindh...@linaro.org > Cc: Feng, Bob C <bob.c.f...@intel.com> > Subject: RE: [edk2-devel] [edk2-platform patch 0/6] Add tools FMMT FCE and > FitGen > > Thanks Liming. > > I would treat that as *a general help request* to validate the BaseTool > update on ARM/AARCH64 Linux OS. >
Yes. This is a help request. > If we can have Ard or any other ARM person to help validate a new base tool > patch, that will be great. > > In this case, it is about the new added FCE or FMMT. > In the future, maybe we have other C-tool update. > > Should we add them to CC list in any tool patch? > FCE and FMMT are new tools with lots of source code. If they meet with the compile failure, it may take more effort to fix them. So, I want to verify them before submit the patches. If the code change is not big, the developer can fix the error quickly after the code is submitted. That's also fine to verify the change after submit. Thanks Liming > > Thank you > Yao Jiewen > > > -----Original Message----- > > From: Gao, Liming > > Sent: Tuesday, June 25, 2019 10:09 PM > > To: Yao, Jiewen <jiewen....@intel.com>; devel@edk2.groups.io; Zhang, > > Shenglei <shenglei.zh...@intel.com>; ard.biesheu...@linaro.org; > > leif.lindh...@linaro.org > > Cc: Feng, Bob C <bob.c.f...@intel.com> > > Subject: RE: [edk2-devel] [edk2-platform patch 0/6] Add tools FMMT FCE > > and FitGen > > > > Jiewen: > > For FCE/FMMT C tools, I only compile them on X64 Linux OS. I don't > > compile them on ARM or AARCH64 Linux OS for ARM native build. Now, > > edk2 BaseTools C tools top level Makefile supports ARM, AARCH64, IA32 and > > X64. If they don't pass build on ARM native Linux OS, this change will bring > > the break for some user. If we can avoid the break, I agree to add them into > > edk2 BaseTools. > > > > Ard: > > I have no ARM or AARCH64 Linux OS. Could you help compile FCE/FMMT > > tools in ARM or AARCH64 Linux OS? These patches are in > > https://github.com/shenglei10/edk2/commits/movetool. After pick those > > patches, you just need to type make in BaseTools directory and compile all C > > tools. FCE/FMMT are common tools to update FD/FV image binaries. FMMT > > is a tool to enable removal, addition and replacement of FFS files in FD > > image binaries. FCE is a tool to retrieve and change HII configuration data > > as > > the default NV variable in Firmware Device(*.fd) files. They are used in > > post > > build step to replace FFS or insert the default NV storage in FD image. > > > > Thanks > > Liming > > > -----Original Message----- > > > From: Yao, Jiewen > > > Sent: Friday, June 21, 2019 11:34 AM > > > To: Gao, Liming <liming....@intel.com>; devel@edk2.groups.io; Zhang, > > Shenglei <shenglei.zh...@intel.com> > > > Cc: Feng, Bob C <bob.c.f...@intel.com> > > > Subject: RE: [edk2-devel] [edk2-platform patch 0/6] Add tools FMMT FCE > > and FitGen > > > > > > Sorry, I missed the first mail. > > > I saw there is no V2 tag in this series, so I think it is the first mail. > > > My > > apology. > > > > > > I have no concern on FitGen - it is Intel only. You can check in at first. > > > FMMT/FCE is common. I think BaseTools is more proper place. > > > > > > "It is not validated" - it seems a weird justification to me. > > > Or does it mean, for any future feature Intel/ARM people contribute, we > > should treat Platform package as the first choice, just because the > > > it is cross validated? > > > > > > May I understand more on the rule we have today on what goes to EDK2 > > repo and what goes to EDK2-PLATFORM repo? > > > > > > Thank you > > > Yao Jiewen > > > > > > > > > > -----Original Message----- > > > > From: Gao, Liming > > > > Sent: Friday, June 21, 2019 10:31 AM > > > > To: Yao, Jiewen <jiewen....@intel.com>; devel@edk2.groups.io; Zhang, > > > > Shenglei <shenglei.zh...@intel.com> > > > > Cc: Feng, Bob C <bob.c.f...@intel.com> > > > > Subject: RE: [edk2-devel] [edk2-platform patch 0/6] Add tools FMMT FCE > > > > and FitGen > > > > > > > > Jiewen: > > > > I give the comments > > > > > > https://edk2.groups.io/g/devel/message/42545?p=,,,20,0,0,0::Created,,FM > > > > MT,20,2,0,32013143 for the patch BaseTools/FCE: Add a tool FCE > > > > > > > > I give new proposal https://edk2.groups.io/g/devel/message/42546 > > for > > > > those tools. > > > > > > > > Thanks > > > > Liming > > > > > -----Original Message----- > > > > > From: Yao, Jiewen > > > > > Sent: Friday, June 21, 2019 10:26 AM > > > > > To: devel@edk2.groups.io; Zhang, Shenglei > > <shenglei.zh...@intel.com> > > > > > Cc: Feng, Bob C <bob.c.f...@intel.com>; Gao, Liming > > > > <liming....@intel.com>; Yao, Jiewen <jiewen....@intel.com> > > > > > Subject: RE: [edk2-devel] [edk2-platform patch 0/6] Add tools FMMT > > FCE > > > > and FitGen > > > > > > > > > > HI > > > > > I saw some conflict: > > > > > ========= > > > > > BaseTools/FCE: Add a tool FCE > > > > > Add FMMT and FCE into Platform/Intel/Tools. > > > > > ========= > > > > > > > > > > I agree that FitGet goes to Intel silicon. > > > > > > > > > > Should we put both FCE/FMMT to BaseTools? > > > > > > > > > > Thank you > > > > > Yao Jiewen > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On > > Behalf > > > > Of > > > > > > Zhang, Shenglei > > > > > > Sent: Friday, June 21, 2019 9:27 AM > > > > > > To: devel@edk2.groups.io > > > > > > Cc: Feng, Bob C <bob.c.f...@intel.com>; Gao, Liming > > > > > > <liming....@intel.com> > > > > > > Subject: [edk2-devel] [edk2-platform patch 0/6] Add tools FMMT FCE > > and > > > > > > FitGen > > > > > > > > > > > > Add FMMT and FCE into Platform/Intel/Tools. > > > > > > Add FitGen into Silicon/Intel/Tools. > > > > > > https://bugzilla.tianocore.org/show_bug.cgi?id=1847 > > > > > > https://bugzilla.tianocore.org/show_bug.cgi?id=1848 > > > > > > https://bugzilla.tianocore.org/show_bug.cgi?id=1849 > > > > > > > > > > > > Cc: Bob Feng <bob.c.f...@intel.com> > > > > > > Cc: Liming Gao <liming....@intel.com> > > > > > > Shenglei Zhang (6): > > > > > > Platform/Tools: Add a tool FMMT > > > > > > Platform/Tools: Add a tool BfmLib > > > > > > BaseTools/FCE: Add a tool FCE > > > > > > Platform/Tools: Add top level Makefile and GNUMakefile > > > > > > Silicon/Tools: Add a tool FitGen > > > > > > Silicon/Tools: Add top level Makefile and GNUMakefile > > > > > > > > > > > > Platform/Intel/Tools/BfmLib/BfmLib.c | 4355 > > +++++++++++ > > > > > > Platform/Intel/Tools/BfmLib/BinFileManager.c | 1024 +++ > > > > > > Platform/Intel/Tools/BfmLib/BinFileManager.h | 439 ++ > > > > > > Platform/Intel/Tools/BfmLib/GNUmakefile | 15 + > > > > > > Platform/Intel/Tools/BfmLib/Makefile | 17 + > > > > > > Platform/Intel/Tools/FCE/BinaryCreate.c | 216 + > > > > > > Platform/Intel/Tools/FCE/BinaryCreate.h | 157 + > > > > > > Platform/Intel/Tools/FCE/BinaryParse.c | 1326 ++++ > > > > > > Platform/Intel/Tools/FCE/BinaryParse.h | 187 + > > > > > > Platform/Intel/Tools/FCE/Common.c | 2183 ++++++ > > > > > > Platform/Intel/Tools/FCE/Common.h | 999 +++ > > > > > > Platform/Intel/Tools/FCE/Expression.c | 2367 ++++++ > > > > > > Platform/Intel/Tools/FCE/Fce.c | 6449 > > > > > > +++++++++++++++++ > > > > > > Platform/Intel/Tools/FCE/Fce.h | 447 ++ > > > > > > Platform/Intel/Tools/FCE/GNUmakefile | 22 + > > > > > > Platform/Intel/Tools/FCE/IfrParse.c | 4836 > > ++++++++++++ > > > > > > Platform/Intel/Tools/FCE/IfrParse.h | 789 ++ > > > > > > Platform/Intel/Tools/FCE/Makefile | 19 + > > > > > > .../Intel/Tools/FCE/MonotonicBasedVariable.c | 874 +++ > > > > > > .../Intel/Tools/FCE/MonotonicBasedVariable.h | 162 + > > > > > > Platform/Intel/Tools/FCE/TimeBasedVariable.c | 878 +++ > > > > > > Platform/Intel/Tools/FCE/TimeBasedVariable.h | 166 + > > > > > > Platform/Intel/Tools/FCE/Variable.c | 1091 +++ > > > > > > Platform/Intel/Tools/FCE/Variable.h | 154 + > > > > > > Platform/Intel/Tools/FCE/VariableCommon.h | 55 + > > > > > > .../Tools/FMMT/FirmwareModuleManagement.c | 2559 > > +++++++ > > > > > > .../Tools/FMMT/FirmwareModuleManagement.h | 479 ++ > > > > > > Platform/Intel/Tools/FMMT/FmmtConf.ini | 6 + > > > > > > Platform/Intel/Tools/FMMT/FmmtLib.c | 5051 > > > > > > +++++++++++++ > > > > > > Platform/Intel/Tools/FMMT/GNUmakefile | 16 + > > > > > > Platform/Intel/Tools/FMMT/Makefile | 17 + > > > > > > Platform/Intel/Tools/FMMT/Rebase.c | 846 +++ > > > > > > Platform/Intel/Tools/FMMT/Rebase.h | 31 + > > > > > > Platform/Intel/Tools/GNUmakefile | 30 + > > > > > > Platform/Intel/Tools/Makefile | 33 + > > > > > > Silicon/Intel/Tools/FitGen/FitGen.c | 3137 ++++++++ > > > > > > Silicon/Intel/Tools/FitGen/FitGen.h | 48 + > > > > > > Silicon/Intel/Tools/FitGen/GNUmakefile | 16 + > > > > > > Silicon/Intel/Tools/FitGen/Makefile | 17 + > > > > > > Silicon/Intel/Tools/GNUmakefile | 34 + > > > > > > Silicon/Intel/Tools/Makefile | 31 + > > > > > > 41 files changed, 41578 insertions(+) > > > > > > create mode 100644 Platform/Intel/Tools/BfmLib/BfmLib.c > > > > > > create mode 100644 > > Platform/Intel/Tools/BfmLib/BinFileManager.c > > > > > > create mode 100644 > > Platform/Intel/Tools/BfmLib/BinFileManager.h > > > > > > create mode 100644 Platform/Intel/Tools/BfmLib/GNUmakefile > > > > > > create mode 100644 Platform/Intel/Tools/BfmLib/Makefile > > > > > > create mode 100644 Platform/Intel/Tools/FCE/BinaryCreate.c > > > > > > create mode 100644 Platform/Intel/Tools/FCE/BinaryCreate.h > > > > > > create mode 100644 Platform/Intel/Tools/FCE/BinaryParse.c > > > > > > create mode 100644 Platform/Intel/Tools/FCE/BinaryParse.h > > > > > > create mode 100644 Platform/Intel/Tools/FCE/Common.c > > > > > > create mode 100644 Platform/Intel/Tools/FCE/Common.h > > > > > > create mode 100644 Platform/Intel/Tools/FCE/Expression.c > > > > > > create mode 100644 Platform/Intel/Tools/FCE/Fce.c > > > > > > create mode 100644 Platform/Intel/Tools/FCE/Fce.h > > > > > > create mode 100644 Platform/Intel/Tools/FCE/GNUmakefile > > > > > > create mode 100644 Platform/Intel/Tools/FCE/IfrParse.c > > > > > > create mode 100644 Platform/Intel/Tools/FCE/IfrParse.h > > > > > > create mode 100644 Platform/Intel/Tools/FCE/Makefile > > > > > > create mode 100644 > > > > Platform/Intel/Tools/FCE/MonotonicBasedVariable.c > > > > > > create mode 100644 > > > > Platform/Intel/Tools/FCE/MonotonicBasedVariable.h > > > > > > create mode 100644 > > Platform/Intel/Tools/FCE/TimeBasedVariable.c > > > > > > create mode 100644 > > Platform/Intel/Tools/FCE/TimeBasedVariable.h > > > > > > create mode 100644 Platform/Intel/Tools/FCE/Variable.c > > > > > > create mode 100644 Platform/Intel/Tools/FCE/Variable.h > > > > > > create mode 100644 Platform/Intel/Tools/FCE/VariableCommon.h > > > > > > create mode 100644 > > > > > > Platform/Intel/Tools/FMMT/FirmwareModuleManagement.c > > > > > > create mode 100644 > > > > > > Platform/Intel/Tools/FMMT/FirmwareModuleManagement.h > > > > > > create mode 100644 Platform/Intel/Tools/FMMT/FmmtConf.ini > > > > > > create mode 100644 Platform/Intel/Tools/FMMT/FmmtLib.c > > > > > > create mode 100644 Platform/Intel/Tools/FMMT/GNUmakefile > > > > > > create mode 100644 Platform/Intel/Tools/FMMT/Makefile > > > > > > create mode 100644 Platform/Intel/Tools/FMMT/Rebase.c > > > > > > create mode 100644 Platform/Intel/Tools/FMMT/Rebase.h > > > > > > create mode 100644 Platform/Intel/Tools/GNUmakefile > > > > > > create mode 100644 Platform/Intel/Tools/Makefile > > > > > > create mode 100644 Silicon/Intel/Tools/FitGen/FitGen.c > > > > > > create mode 100644 Silicon/Intel/Tools/FitGen/FitGen.h > > > > > > create mode 100644 Silicon/Intel/Tools/FitGen/GNUmakefile > > > > > > create mode 100644 Silicon/Intel/Tools/FitGen/Makefile > > > > > > create mode 100644 Silicon/Intel/Tools/GNUmakefile > > > > > > create mode 100644 Silicon/Intel/Tools/Makefile > > > > > > > > > > > > -- > > > > > > 2.18.0.windows.1 > > > > > > > > > > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#42834): https://edk2.groups.io/g/devel/message/42834 Mute This Topic: https://groups.io/mt/32154333/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
