Hello Ard and Leif, I saw Ming Huang validated the patch (OS boot): Hisilicon/D0x: Use StatusCode Router & Handler in MdeModulePkg
https://edk2.groups.io/g/devel/message/41419 (Thanks a lot for the validation effort.) Do you have remaining concerns for review this series? Thanks in advance. Best Regards, Hao Wu > -----Original Message----- > From: Wu, Hao A > Sent: Monday, May 27, 2019 10:43 AM > To: devel@edk2.groups.io; Ard Biesheuvel; Leif Lindholm > Cc: Kinney, Michael D; Andrew Fish; Laszlo Ersek; Ni, Ray; > guoh...@huawei.com > Subject: RE: [edk2-devel] [edk2-platforms][PATCH v2 00/18] Drop > IntelFramework[Module]Pkg dependency > > 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@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > > Wu, Hao A > > Sent: Monday, May 20, 2019 4:41 PM > > To: devel@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.biesheu...@linaro.org> > > Cc: Leif Lindholm <leif.lindh...@linaro.org> > > Cc: Michael D Kinney <michael.d.kin...@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 (#42053): https://edk2.groups.io/g/devel/message/42053 Mute This Topic: https://groups.io/mt/31683278/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-