1. Patchew
  2. EDK2
  3. View series

[edk2-devel] [edk2-platform patch 0/6] Add tools FMMT FCE and FitGen

Zhang, Shenglei posted 6 patches 4 years, 10 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/edk2 tags/patchew/20190621012643.9352-1-shenglei.zhang@intel.com
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
[edk2-devel] [edk2-platform patch 0/6] Add tools FMMT FCE and FitGen
Posted by Zhang, Shenglei 4 years, 10 months ago 
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.feng@intel.com>
Cc: Liming Gao <liming.gao@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 (#42668): https://edk2.groups.io/g/devel/message/42668
Mute This Topic: https://groups.io/mt/32154333/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [edk2-platform patch 0/6] Add tools FMMT FCE and FitGen
Posted by Yao, Jiewen 4 years, 10 months ago 
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.feng@intel.com>; Gao, Liming
> <liming.gao@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.feng@intel.com>
> Cc: Liming Gao <liming.gao@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 (#42680): https://edk2.groups.io/g/devel/message/42680
Mute This Topic: https://groups.io/mt/32154333/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [edk2-platform patch 0/6] Add tools FMMT FCE and FitGen
Posted by Liming Gao 4 years, 10 months ago 
Jiewen:
  I give the comments https://edk2.groups.io/g/devel/message/42545?p=,,,20,0,0,0::Created,,FMMT,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.zhang@intel.com>
> Cc: Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming <liming.gao@intel.com>; Yao, Jiewen <jiewen.yao@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.feng@intel.com>; Gao, Liming
> > <liming.gao@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.feng@intel.com>
> > Cc: Liming Gao <liming.gao@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 (#42681): https://edk2.groups.io/g/devel/message/42681
Mute This Topic: https://groups.io/mt/32154333/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [edk2-platform patch 0/6] Add tools FMMT FCE and FitGen
Posted by Yao, Jiewen 4 years, 10 months ago 
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.yao@intel.com>; devel@edk2.groups.io; Zhang,
> Shenglei <shenglei.zhang@intel.com>
> Cc: Feng, Bob C <bob.c.feng@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.zhang@intel.com>
> > Cc: Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming
> <liming.gao@intel.com>; Yao, Jiewen <jiewen.yao@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.feng@intel.com>; Gao, Liming
> > > <liming.gao@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.feng@intel.com>
> > > Cc: Liming Gao <liming.gao@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 (#42688): https://edk2.groups.io/g/devel/message/42688
Mute This Topic: https://groups.io/mt/32154333/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [edk2-platform patch 0/6] Add tools FMMT FCE and FitGen
Posted by Liming Gao 4 years, 9 months ago 
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.gao@intel.com>; devel@edk2.groups.io; Zhang, Shenglei <shenglei.zhang@intel.com>
> Cc: Feng, Bob C <bob.c.feng@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.yao@intel.com>; devel@edk2.groups.io; Zhang,
> > Shenglei <shenglei.zhang@intel.com>
> > Cc: Feng, Bob C <bob.c.feng@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.zhang@intel.com>
> > > Cc: Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming
> > <liming.gao@intel.com>; Yao, Jiewen <jiewen.yao@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.feng@intel.com>; Gao, Liming
> > > > <liming.gao@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.feng@intel.com>
> > > > Cc: Liming Gao <liming.gao@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 (#42824): https://edk2.groups.io/g/devel/message/42824
Mute This Topic: https://groups.io/mt/32154333/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [edk2-platform patch 0/6] Add tools FMMT FCE and FitGen
Posted by Yao, Jiewen 4 years, 9 months ago 
Thanks Liming.

I would treat that as *a general help request* to validate the BaseTool update on ARM/AARCH64 Linux OS.

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?


Thank you
Yao Jiewen

> -----Original Message-----
> From: Gao, Liming
> Sent: Tuesday, June 25, 2019 10:09 PM
> To: Yao, Jiewen <jiewen.yao@intel.com>; devel@edk2.groups.io; Zhang,
> Shenglei <shenglei.zhang@intel.com>; ard.biesheuvel@linaro.org;
> leif.lindholm@linaro.org
> Cc: Feng, Bob C <bob.c.feng@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.gao@intel.com>; devel@edk2.groups.io; Zhang,
> Shenglei <shenglei.zhang@intel.com>
> > Cc: Feng, Bob C <bob.c.feng@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.yao@intel.com>; devel@edk2.groups.io; Zhang,
> > > Shenglei <shenglei.zhang@intel.com>
> > > Cc: Feng, Bob C <bob.c.feng@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.zhang@intel.com>
> > > > Cc: Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming
> > > <liming.gao@intel.com>; Yao, Jiewen <jiewen.yao@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.feng@intel.com>; Gao, Liming
> > > > > <liming.gao@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.feng@intel.com>
> > > > > Cc: Liming Gao <liming.gao@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 (#42827): https://edk2.groups.io/g/devel/message/42827
Mute This Topic: https://groups.io/mt/32154333/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [edk2-platform patch 0/6] Add tools FMMT FCE and FitGen
Posted by Liming Gao 4 years, 9 months ago 
Jiewen:

Thanks
Liming
> -----Original Message-----
> From: Yao, Jiewen
> Sent: Tuesday, June 25, 2019 10:15 PM
> To: Gao, Liming <liming.gao@intel.com>; devel@edk2.groups.io; Zhang, Shenglei <shenglei.zhang@intel.com>; ard.biesheuvel@linaro.org;
> leif.lindholm@linaro.org
> Cc: Feng, Bob C <bob.c.feng@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.yao@intel.com>; devel@edk2.groups.io; Zhang,
> > Shenglei <shenglei.zhang@intel.com>; ard.biesheuvel@linaro.org;
> > leif.lindholm@linaro.org
> > Cc: Feng, Bob C <bob.c.feng@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.gao@intel.com>; devel@edk2.groups.io; Zhang,
> > Shenglei <shenglei.zhang@intel.com>
> > > Cc: Feng, Bob C <bob.c.feng@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.yao@intel.com>; devel@edk2.groups.io; Zhang,
> > > > Shenglei <shenglei.zhang@intel.com>
> > > > Cc: Feng, Bob C <bob.c.feng@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.zhang@intel.com>
> > > > > Cc: Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming
> > > > <liming.gao@intel.com>; Yao, Jiewen <jiewen.yao@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.feng@intel.com>; Gao, Liming
> > > > > > <liming.gao@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.feng@intel.com>
> > > > > > Cc: Liming Gao <liming.gao@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/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Patchew 2.1-devel-0870f84

© 2016 - 2024 Red Hat, Inc.