On 11/29/17 05:58, Star Zeng wrote:
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=540
> 
> To consume FIT table for Microcode update,
> UefiCpuPkg/Feature/Capsule/MicrocodeUpdateDxe
> needs to be updated to consume
> IntelSiliconPkg/Include/IndustryStandard/FirmwareInterfaceTable.h,
> but UefiCpuPkg could not depend on IntelSiliconPkg.
> 
> Since the Microcode update feature is specific to Intel,
> we can first move the Microcode update feature code from
> UefiCpuPkg to IntelSiliconPkg [first step], then update
> the code to consume FIT table [second step].
> 
> This patch series is for the first step.
> 
> Cc: Jiewen Yao <jiewen....@intel.com>
> Cc: Michael D Kinney <michael.d.kin...@intel.com>
> Cc: Eric Dong <eric.d...@intel.com>
> Cc: Laszlo Ersek <ler...@redhat.com>
> Regression-tested-by: Yonghong Zhu <yonghong....@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Star Zeng <star.z...@intel.com>
> 
> Star Zeng (2):
>   IntelSiliconPkg: Move MicrocodeUpdate from UefiCpuPkg
>   IntelSiliconPkg: Update MicrocodeUpdate to build with the package
> 
>  .../MicrocodeFlashAccessLibNull/MicrocodeFlashAccessLibNull.c  |  0
>  .../MicrocodeFlashAccessLibNull.inf                            |  6 +++---
>  .../MicrocodeFlashAccessLibNull.uni                            |  0
>  .../Capsule/MicrocodeCapsulePdb/MicrocodeCapsulePdb.dsc        |  0
>  .../Capsule/MicrocodeCapsulePdb/MicrocodeCapsulePdb.fdf        |  0
>  .../Feature/Capsule/MicrocodeCapsulePdb/Readme.md              |  0
>  .../Capsule/MicrocodeCapsuleTxt/Microcode/Microcode.inf        |  0
>  .../Capsule/MicrocodeCapsuleTxt/MicrocodeCapsuleTxt.dsc        |  0
>  .../Capsule/MicrocodeCapsuleTxt/MicrocodeCapsuleTxt.fdf        |  0
>  .../Feature/Capsule/MicrocodeCapsuleTxt/Readme.md              |  0
>  .../Feature/Capsule/MicrocodeUpdateDxe/MicrocodeFmp.c          |  0
>  .../Feature/Capsule/MicrocodeUpdateDxe/MicrocodeUpdate.c       |  0
>  .../Feature/Capsule/MicrocodeUpdateDxe/MicrocodeUpdate.h       |  0
>  .../Feature/Capsule/MicrocodeUpdateDxe/MicrocodeUpdateDxe.inf  |  3 ++-
>  .../Feature/Capsule/MicrocodeUpdateDxe/MicrocodeUpdateDxe.uni  |  0
>  .../Capsule/MicrocodeUpdateDxe/MicrocodeUpdateDxeExtra.uni     |  0
>  {UefiCpuPkg => IntelSiliconPkg}/Include/Guid/MicrocodeFmp.h    |  0
>  .../Include/Library/MicrocodeFlashAccessLib.h                  |  0
>  IntelSiliconPkg/IntelSiliconPkg.dec                            | 10 
> +++++++++-
>  IntelSiliconPkg/IntelSiliconPkg.dsc                            |  2 ++
>  UefiCpuPkg/UefiCpuPkg.dec                                      |  7 -------
>  UefiCpuPkg/UefiCpuPkg.dsc                                      |  2 --
>  22 files changed, 16 insertions(+), 14 deletions(-)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/Library/MicrocodeFlashAccessLibNull/MicrocodeFlashAccessLibNull.c
>  (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/Library/MicrocodeFlashAccessLibNull/MicrocodeFlashAccessLibNull.inf
>  (85%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/Library/MicrocodeFlashAccessLibNull/MicrocodeFlashAccessLibNull.uni
>  (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/MicrocodeCapsulePdb/MicrocodeCapsulePdb.dsc 
> (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/MicrocodeCapsulePdb/MicrocodeCapsulePdb.fdf 
> (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/MicrocodeCapsulePdb/Readme.md (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/MicrocodeCapsuleTxt/Microcode/Microcode.inf 
> (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/MicrocodeCapsuleTxt/MicrocodeCapsuleTxt.dsc 
> (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/MicrocodeCapsuleTxt/MicrocodeCapsuleTxt.fdf 
> (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/MicrocodeCapsuleTxt/Readme.md (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/MicrocodeUpdateDxe/MicrocodeFmp.c (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/MicrocodeUpdateDxe/MicrocodeUpdate.c (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/MicrocodeUpdateDxe/MicrocodeUpdate.h (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/MicrocodeUpdateDxe/MicrocodeUpdateDxe.inf 
> (91%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/MicrocodeUpdateDxe/MicrocodeUpdateDxe.uni 
> (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Feature/Capsule/MicrocodeUpdateDxe/MicrocodeUpdateDxeExtra.uni
>  (100%)
>  rename {UefiCpuPkg => IntelSiliconPkg}/Include/Guid/MicrocodeFmp.h (100%)
>  rename {UefiCpuPkg => 
> IntelSiliconPkg}/Include/Library/MicrocodeFlashAccessLib.h (100%)
> 

Acked-by: Laszlo Ersek <ler...@redhat.com>

Thanks
Laszlo
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to