From: "Wu, Hao A" <hao.a.wu@intel.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
Ard Biesheuvel <ard.biesheuvel@linaro.org>,
Leif Lindholm <leif.lindholm@linaro.org>
Cc: "Kinney, Michael D" <michael.d.kinney@intel.com>,
Andrew Fish <afish@apple.com>, Laszlo Ersek <lersek@redhat.com>,
"Ni, Ray" <ray.ni@intel.com>,
"guoheyi@huawei.com" <guoheyi@huawei.com>
Subject: Re: [edk2-devel] [edk2-platforms][PATCH v2 00/18] Drop IntelFramework[Module]Pkg dependency
Date: Mon, 27 May 2019 02:43:26 +0000 [thread overview]
Message-ID: <B80AF82E9BFB8E4FBD8C89DA810C6A093C8D6909@SHSMSX104.ccr.corp.intel.com> (raw)
In-Reply-To: <20190520084105.9028-1-hao.a.wu@intel.com>
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.biesheuvel@linaro.org>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Michael D Kinney <michael.d.kinney@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
>
>
>
next prev parent reply other threads:[~2019-05-27 2:43 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-20 8:40 [edk2-platforms][PATCH v2 00/18] Drop IntelFramework[Module]Pkg dependency Wu, Hao A
2019-05-20 8:40 ` [edk2-platforms][PATCH v2 01/18] Platform/AMD: Use Lzma decompress lib in MdeModulePkg Wu, Hao A
2019-05-20 8:40 ` [edk2-platforms][PATCH v2 02/18] Platform/BeagleBoard: Use ReportStatusCodeLib null instance Wu, Hao A
2019-05-20 8:40 ` [edk2-platforms][PATCH v2 03/18] Platform/BeagleBoard: Drop the consume of PcdShellFile Wu, Hao A
2019-05-20 8:40 ` [edk2-platforms][PATCH v2 04/18] Hisilicon/D0x: Use Lzma decompress lib in MdeModulePkg Wu, Hao A
2019-05-20 8:40 ` [edk2-platforms][PATCH v2 05/18] Hisilicon/D0x: Drop the consume of GenericBdsLib & PlatformBdsLib Wu, Hao A
2019-05-20 8:40 ` [edk2-platforms][PATCH v2 06/18] Hisilicon/D0x: Use StatusCode Router & Handler in MdeModulePkg Wu, Hao A
2019-05-20 8:40 ` [edk2-platforms][PATCH v2 07/18] Hisilicon/D06: Drop the consume of PcdShellFile Wu, Hao A
2019-05-20 8:40 ` [edk2-platforms][PATCH v2 08/18] Silicon/Hisilicon: Use Lzma decompress lib in MdeModulePkg Wu, Hao A
2019-05-20 8:40 ` [edk2-platforms][PATCH v2 09/18] Silicon/Hisilicon: Use ReportStatusCodeLib (DXE) " Wu, Hao A
2019-05-20 8:40 ` [edk2-platforms][PATCH v2 10/18] Silicon/Hisilicon: Use DebugLib (DXE_RUNTIME_DRIVER) " Wu, Hao A
2019-05-20 8:40 ` [edk2-platforms][PATCH v2 11/18] Silicon/Hisilicon/Smbios: Drop IntelFramework[Module]Pkg dependency Wu, Hao A
2019-05-20 8:40 ` [edk2-platforms][PATCH v2 12/18] Silicon/Hisilicon/PlatformIntelBdsLib: Remove Hisilicon PlatformBdsLib Wu, Hao A
2019-05-20 8:41 ` [edk2-platforms][PATCH v2 13/18] Platform/LeMaker: Use Lzma decompress lib in MdeModulePkg Wu, Hao A
2019-05-20 8:41 ` [edk2-platforms][PATCH v2 14/18] Platform/LeMaker: Use ReportStatusCodeLib null instance Wu, Hao A
2019-05-20 8:41 ` [edk2-platforms][PATCH v2 15/18] Platform/SoftIron: Use Lzma decompress lib in MdeModulePkg Wu, Hao A
2019-05-20 8:41 ` [edk2-platforms][PATCH v2 16/18] Platform/SoftIron: Use ReportStatusCodeLib null instance Wu, Hao A
2019-05-20 8:41 ` [edk2-platforms][PATCH v2 17/18] Platform/RPi3: " Wu, Hao A
2019-05-20 8:41 ` [edk2-platforms][PATCH v2 18/18] Platform/RaspberryPi: Drop the consume of PcdShellFile Wu, Hao A
2019-05-27 2:43 ` Wu, Hao A [this message]
2019-06-10 5:13 ` [edk2-devel] [edk2-platforms][PATCH v2 00/18] Drop IntelFramework[Module]Pkg dependency Wu, Hao A
2019-06-10 15:12 ` Leif Lindholm
2019-06-11 6:19 ` Wu, Hao A
2019-06-11 9:38 ` Leif Lindholm
2019-06-12 0:21 ` Wu, Hao A
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=B80AF82E9BFB8E4FBD8C89DA810C6A093C8D6909@SHSMSX104.ccr.corp.intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox