Re: [edk2-devel] [edk2-platforms][PATCH v1 00/16] Drop IntelFramework[Module]Pkg dependency

2019-05-15 Thread Leif Lindholm
Hi,

For patches 1,3-5,7-8,10-13,15:
Reviewed-by: Leif Lindholm 

However, since we have not yet changed the license for edk2-platforms,
I still need Contributed-under on all of these before I push.
I have a branch prepared, so all I need is a confirmation by an email
to the list that the set is intended to be
Contributed-under: TianoCore Contribution Agreement 1.1
.

Best Regards,

Leif

On Tue, May 14, 2019 at 10:08:13AM +0800, Hao A Wu wrote:
> This series will drop the IntelFramework[Module]Pkg dependency for
> platforms within the edk2-platforms repo. This is the 1st part series,
> which will handle platforms only with explicit dependency on the framework
> packages.
> 
> The affected platforms include:
> * Platform/AMD/OverdriveBoard
> * Platform/BeagleBoard
> * Platform/Hisilicon/D0x
> * Platform/LeMaker/CelloBoard
> * Platform/SoftIron/Overdrive1000Board
> 
> Tests done:
> Build verified for the above affected platforms.
> 
> Please note that for:
> * Platform/Intel/QuarkPlatformPkg
> * Vlv2TbltDevicePkg
> 
> They will handle by some one else with another patch sereis.
> 
> And for:
> * Platform/RaspberryPi/RPi3
> 
> It has implicit dependency on the framework packages, and will be handled
> in the part 2 series.
> 
> Cc: Ard Biesheuvel 
> Cc: Leif Lindholm 
> Cc: Michael D Kinney 
> 
> Hao A Wu (16):
>   Platform/AMD: Use Lzma decompress lib in MdeModulePkg
>   Platform/BeagleBoard: Use ReportStatusCodeLib in MdeModulePkg
>   Platform/BeagleBoard: Drop the consume of PcdShellFile
>   Hisilicon/D0x: Use Lzma decompress lib in MdeModulePkg
>   Hisilicon/D0x: Drop the consume of GenericBdsLib & PlatformBdsLib
>   Hisilicon/D0x: Use StatusCode Router & Handler in MdeModulePkg
>   Hisilicon/D06: Drop the consume of PcdShellFile
>   Silicon/Hisilicon: Use Lzma decompress lib in MdeModulePkg
>   Silicon/Hisilicon: Use ReportStatusCodeLib (DXE) in MdeModulePkg
>   Silicon/Hisilicon: Use DebugLib (DXE_RUNTIME_DRIVER) in MdeModulePkg
>   Silicon/Hisilicon/Smbios: Drop IntelFramework[Module]Pkg dependency
>   Silicon/Hisilicon/PlatformIntelBdsLib: Remove Hisilicon PlatformBdsLib
>   Platform/LeMaker: Use Lzma decompress lib in MdeModulePkg
>   Platform/LeMaker: Use ReportStatusCodeLib in MdeModulePkg
>   Platform/SoftIron: Use Lzma decompress lib in MdeModulePkg
>   Platform/SoftIron: Use ReportStatusCodeLib in MdeModulePkg
> 
>  Silicon/Hisilicon/Hisilicon.dsc.inc  
>  |  10 +-
>  Platform/AMD/OverdriveBoard/OverdriveBoard.dsc   
>  |   2 +-
>  Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.dsc   
>  |  13 +-
>  Platform/Hisilicon/D03/D03.dsc   
>  |  10 +-
>  Platform/Hisilicon/D05/D05.dsc   
>  |  10 +-
>  Platform/Hisilicon/D06/D06.dsc   
>  |   8 +-
>  Platform/LeMaker/CelloBoard/CelloBoard.dsc   
>  |  10 +-
>  Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc  
>  |  10 +-
>  Platform/Hisilicon/D03/D03.fdf   
>  |   6 +-
>  Platform/Hisilicon/D05/D05.fdf   
>  |   6 +-
>  Platform/Hisilicon/D06/D06.fdf   
>  |   6 +-
>  Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClassDxe.inf 
>  |   2 -
>  
> Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClassDxe.inf
> |   1 -
>  Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMiscDxe.inf 
>  |   2 -
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/PlatformIntelBdsLib.inf
>  |  84 --
>  Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h  
>  |   1 -
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatform.h 
>  |  61 --
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatformCommon.h   
>  |  27 -
>  Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.c
>  |   4 +-
>  
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c
> |   4 +-
>  
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type01/MiscSystemManufacturerFunction.c
> |   4 +-
>  
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type02/MiscBaseBoardManufacturerFunction.c
>  |   4 +-
>  
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type03/MiscChassisManufacturerFunction.c
>|   2 +-
>  
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type09/MiscSystemSlotDesignationFunction.c
>  |   4 +-
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatform.c 
>  | 9

Re: [edk2-devel] [edk2-platforms][PATCH v1 00/16] Drop IntelFramework[Module]Pkg dependency

2019-05-13 Thread Wu, Hao A
Sorry to mention, this series is also available at:
https://github.com/hwu25/edk2-platforms/tree/drop_intelframework_part1_v1

Best Regards,
Hao Wu


> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Wu,
> Hao A
> Sent: Tuesday, May 14, 2019 10:08 AM
> To: devel@edk2.groups.io
> Cc: Wu, Hao A; Ard Biesheuvel; Leif Lindholm; Kinney, Michael D
> Subject: [edk2-devel] [edk2-platforms][PATCH v1 00/16] Drop
> IntelFramework[Module]Pkg dependency
> 
> This series will drop the IntelFramework[Module]Pkg dependency for
> platforms within the edk2-platforms repo. This is the 1st part series,
> which will handle platforms only with explicit dependency on the framework
> packages.
> 
> The affected platforms include:
> * Platform/AMD/OverdriveBoard
> * Platform/BeagleBoard
> * Platform/Hisilicon/D0x
> * Platform/LeMaker/CelloBoard
> * Platform/SoftIron/Overdrive1000Board
> 
> Tests done:
> Build verified for the above affected platforms.
> 
> Please note that for:
> * Platform/Intel/QuarkPlatformPkg
> * Vlv2TbltDevicePkg
> 
> They will handle by some one else with another patch sereis.
> 
> And for:
> * Platform/RaspberryPi/RPi3
> 
> It has implicit dependency on the framework packages, and will be handled
> in the part 2 series.
> 
> Cc: Ard Biesheuvel 
> Cc: Leif Lindholm 
> Cc: Michael D Kinney 
> 
> Hao A Wu (16):
>   Platform/AMD: Use Lzma decompress lib in MdeModulePkg
>   Platform/BeagleBoard: Use ReportStatusCodeLib in MdeModulePkg
>   Platform/BeagleBoard: Drop the consume of PcdShellFile
>   Hisilicon/D0x: Use Lzma decompress lib in MdeModulePkg
>   Hisilicon/D0x: Drop the consume of GenericBdsLib & PlatformBdsLib
>   Hisilicon/D0x: Use StatusCode Router & Handler in MdeModulePkg
>   Hisilicon/D06: Drop the consume of PcdShellFile
>   Silicon/Hisilicon: Use Lzma decompress lib in MdeModulePkg
>   Silicon/Hisilicon: Use ReportStatusCodeLib (DXE) in MdeModulePkg
>   Silicon/Hisilicon: Use DebugLib (DXE_RUNTIME_DRIVER) in MdeModulePkg
>   Silicon/Hisilicon/Smbios: Drop IntelFramework[Module]Pkg dependency
>   Silicon/Hisilicon/PlatformIntelBdsLib: Remove Hisilicon PlatformBdsLib
>   Platform/LeMaker: Use Lzma decompress lib in MdeModulePkg
>   Platform/LeMaker: Use ReportStatusCodeLib in MdeModulePkg
>   Platform/SoftIron: Use Lzma decompress lib in MdeModulePkg
>   Platform/SoftIron: Use ReportStatusCodeLib in MdeModulePkg
> 
>  Silicon/Hisilicon/Hisilicon.dsc.inc  
>  |  10 +-
>  Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
> |   2 +-
>  Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.dsc
> |  13 +-
>  Platform/Hisilicon/D03/D03.dsc   
>  |  10 +-
>  Platform/Hisilicon/D05/D05.dsc   
>  |  10 +-
>  Platform/Hisilicon/D06/D06.dsc   
>  |   8 +-
>  Platform/LeMaker/CelloBoard/CelloBoard.dsc   
>  |
> 10 +-
>  Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc
> |  10 +-
>  Platform/Hisilicon/D03/D03.fdf   
>  |   6 +-
>  Platform/Hisilicon/D05/D05.fdf   
>  |   6 +-
>  Platform/Hisilicon/D06/D06.fdf   
>  |   6 +-
>  Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClassDxe.inf
> |   2 -
> 
> Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClassDxe.i
> nf|   1 -
>  Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMiscDxe.inf
> |   2 -
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/PlatformIntelBdsLib.inf
> |  84 --
>  Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
> |   1 -
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatform.h
> |  61 --
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatformCommon.h
> |  27 -
>  Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.c
> |   4 +-
> 
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFuncti
> on.c|   4 +-
> 
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type01/MiscSystemManufact
> urerFunction.c|   4 +-
> 
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type02/MiscBaseBoardManuf
> acturerFunction.c |   4 +-
> 
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type03/MiscChassisManufact
> urerFunction.c   |   2 +-
> 
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type09/MiscSystemSlotDesig
> nationFunction.c |   4 +-
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatform.c
> | 963 
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatformCommon.c
> | 118 ---
>  26 files changed, 57 insertions(+), 1315 deletions(-)
>  delete mode 100644
> Silicon/Hisilicon/Library/PlatformIntelBdsLib/PlatformIntelBd