[edk2-devel] [edk2-platforms][PATCH v2 00/18] Drop IntelFramework[Module]Pkg dependency

Wu, Hao A hao.a.wu at intel.com
Mon May 27 02:43:26 UTC 2019


Hello Ard and Leif,

Do you have comments on this series?

Also, for the below patch to Hisilicon platforms:
[PATCH v2 06/18] Hisilicon/D0x: Use StatusCode Router & Handler in MdeModulePkg

it is suggested to be verified on actual hardware:
https://edk2.groups.io/g/devel/topic/31614327#41188

However, it seems that there is no accessible hardware for such verification
at this moment. Do you think it is fine to push this change anyway?

(Also CCed the stewards here for comments.)

Best Regards,
Hao Wu


> -----Original Message-----
> From: devel at edk2.groups.io [mailto:devel at edk2.groups.io] On Behalf Of
> Wu, Hao A
> Sent: Monday, May 20, 2019 4:41 PM
> To: devel at edk2.groups.io
> Cc: Wu, Hao A; Ard Biesheuvel; Leif Lindholm; Kinney, Michael D
> Subject: [edk2-devel] [edk2-platforms][PATCH v2 00/18] Drop
> IntelFramework[Module]Pkg dependency
> 
> The series is also available at:
> https://github.com/hwu25/edk2-
> platforms/commits/drop_intelframework_v2
> 
> V2 changes:
> A. For platforms BeagleBoard, CelloBoard and Overdrive1000Board, update
>    their DSC file to consume the null ReportStatusCodeLib in MdePkg. The
>    change is done since these platforms do not have a StatusCode Router
>    module;
> B. Add the change for platform RaspberryPi/RPi3 to drop its dependency on
>    the framework packages;
> C. Update the commit log message for patch
>    'Silicon/Hisilicon: Use ReportStatusCodeLib (DXE) in MdeModulePkg'
>    to better reflect the commit impact.
> 
> 
> This series will drop the IntelFramework[Module]Pkg dependency for
> platforms within the edk2-platforms repo.
> packages.
> 
> Tests done for the series:
> Platforms build verified with GCC5 toolchain:
> * Platform/AMD/OverdriveBoard
> * Platform/ARM/JunoPkg
> * Platform/ARM/SgiPkg
> * Platform/ARM/VExpressPkg
> * Platform/BeagleBoard
> * Platform/Hisilicon/D0x
> * Platform/Hisilicon/HiKey
> * Platform/Hisilicon/HiKey960
> * Platform/LeMaker/CelloBoard
> * Platform/Marvell/Armada70x0Db (only AARCH64)
> * Platform/Marvell/Armada80x0Db (only AARCH64)
> * Platform/RaspberryPi/RPi3
> * Platform/Socionext/DeveloperBox
> * Platform/Socionext/SynQuacerEvalBoard (only AARCH64)
> * Platform/SoftIron/Overdrive1000Board
> * Platform/SolidRun/Armada80x0McBin (only AARCH64)
> 
> Platforms originally cannot be built via GCC5 toolchain (thus, not
> verified):
> * Platform/Comcast/RDKQemu (ARM, AARCH64)
> * Platform/Marvell/Armada70x0Db (ARM)
> * Platform/Marvell/Armada80x0Db (ARM)
> * Platform/Socionext/SynQuacerEvalBoard (ARM, AARCH64)
> * Platform/SolidRun/Armada80x0McBin (ARM)
> 
> Cc: Ard Biesheuvel <ard.biesheuvel at linaro.org>
> Cc: Leif Lindholm <leif.lindholm at linaro.org>
> Cc: Michael D Kinney <michael.d.kinney at intel.com>
> 
> Hao A Wu (18):
>   Platform/AMD: Use Lzma decompress lib in MdeModulePkg
>   Platform/BeagleBoard: Use ReportStatusCodeLib null instance
>   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 null instance
>   Platform/SoftIron: Use Lzma decompress lib in MdeModulePkg
>   Platform/SoftIron: Use ReportStatusCodeLib null instance
>   Platform/RPi3: Use ReportStatusCodeLib null instance
>   Platform/RaspberryPi: Drop the consume of PcdShellFile
> 
>  Silicon/Hisilicon/Hisilicon.dsc.inc                                                       |  10 +-
>  Platform/AMD/OverdriveBoard/OverdriveBoard.dsc                                            |
> 2 +-
>  Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.dsc
> |  12 +-
>  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/RaspberryPi/RPi3/RPi3.dsc                                                        |   7 +-
>  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/ProcessorSubClassDx
> e.inf            |   1 -
>  Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMiscDxe.inf
> |   2 -
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/PlatformIntelBdsLib.inf
> |  77 --
>  Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
> |   1 -
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatform.h
> |  55 --
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatformCommon.h
> |  21 -
>  Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.c
> |   4 +-
> 
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFu
> nction.c            |   4 +-
> 
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type01/MiscSystemManuf
> acturerFunction.c    |   4 +-
> 
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type02/MiscBaseBoardMa
> nufacturerFunction.c |   4 +-
> 
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type03/MiscChassisManufa
> cturerFunction.c   |   2 +-
> 
> Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/Type09/MiscSystemSlotDes
> ignationFunction.c |   4 +-
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatform.c
> | 957 --------------------
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatformCommon.c
> | 112 ---
>  27 files changed, 48 insertions(+), 1299 deletions(-)
>  delete mode 100644
> Silicon/Hisilicon/Library/PlatformIntelBdsLib/PlatformIntelBdsLib.inf
>  delete mode 100644
> Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatform.h
>  delete mode 100644
> Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatformCommon.h
>  delete mode 100644
> Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatform.c
>  delete mode 100644
> Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatformCommon.c
> 
> --
> 2.12.0.windows.1
> 
> 
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41383): https://edk2.groups.io/g/devel/message/41383
Mute This Topic: https://groups.io/mt/31683278/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-





More information about the edk2-devel-archive mailing list