Tianocore/edk2-platforms.git/Platform/Hisilicon/D06, branch CodeCleanup

Platform,Silicon: drop redundant uses of PcdArmArchTimerFreqInHz

2024-06-21T11:31:20+00:00

PcdArmArchTimerFreqInHz is about to be removed, as it is now obsolete. Some platforms already explicitly set it to 0, which is the default. And some modules reference it in their .inf without actually ever using it. Drop these redundant uses first. Cc: Ard Biesheuvel Cc: Chuong Tran Cc: Graeme Gregory Cc: Marcin Juszkiewicz Cc: Meenakshi Aggarwal Cc: Nhi Pham Cc: Rebecca Cran Cc: Sami Mujawar Cc: Thomas Abraham Cc: Wenyi Xie Signed-off-by: Leif Lindholm Reviewed-by: Ard Biesheuvel Reviewed-by: Nhi Pham

Hisilicon: rename OemMiscLib class to HisiOemMiscLib

2023-10-13T10:11:01+00:00

Edk2 commit [1] introduced the library class header to ArmPkg; commit [2] introduced the first OemMiscLib instance to ArmPkg; and commit [3] introduced (retroactively) the library class itself to ArmPkg. Alas, when all these edk2 commits were made, edk2-platforms already had a (Hisilicon-specific) library class called OemMiscLib -- and many of the Hisilicon modules would use both "HisiPkg.dec" and "ArmPkg.dec". Therefore the edk2 OemMiscLib brought about a library class conflict, breaking both edk2-platforms and edk2-non-osi content. Rename the original OemMiscLib class to HisiOemMiscLib, and update all references. This patch is necessary for building the D03, D05, D06, HiKey, and HiKey960 platforms. [1] 2c7c64fc04a0 ("ArmPkg: Add Library/OemMiscLib.h", 2021-02-08) [2] 0e51d7e445e0 ("ArmPkg: Add Universal/Smbios/OemMiscLibNull", 2021-02-08) [3] d03f71dd8be6 ("ArmPkg: Add missing library headers to ArmPkg.dec", 2021-04-28) Cc: Leif Lindholm Cc: Wenyi Xie Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=4564 Signed-off-by: Laszlo Ersek Reviewed-by: Ard Biesheuvel

Hisilicon: enable NOOPT builds

2023-10-13T10:10:16+00:00

There's no reason for not supporting NOOPT in the Hisilicon platform DSCs, and there is a reason for supporting them: build-only testing. Add NOOPT wherever it's missing, and stick to a consistent order wherever NOOPT already exists. Cc: Leif Lindholm Cc: Wenyi Xie Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=4564 Signed-off-by: Laszlo Ersek Reviewed-by: Ard Biesheuvel

Platform/Hisilicon: Use PcdPciIoTranslation PCD from MdePkg

2021-10-14T04:10:08+00:00

Compliant with BZ: #3665 https://bugzilla.tianocore.org/show_bug.cgi?id=3665 PcdPciIoTranslation PCD is relocated to MdePkg that leveraged by both ARM and RISC-V arch. This patch uses the one from MdePkg instead the one under ArmVirtPkg. Signed-off-by: Abner Chang Cc: Leif Lindholm Cc: Ard Biesheuvel Cc: Wenyi Xie Cc: Daniel Schaefer Acked-by: Leif Lindholm

Platform/Hisilicon: Consume MdeLibs.dsc.inc for RegisterFilterLib

2021-03-31T06:20:15+00:00

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246 MdeLibs.dsc.inc was added for some basic/default library instances provided by MdePkg and RegisterFilterLibNull Library was also added into it as the first version of MdeLibs.dsc.inc. So update platform dsc to consume MdeLibs.dsc.inc for RegisterFilterLibNull which will be consumed by IoLib and BaseLib. Cc: Leif Lindholm Cc: Ard Biesheuvel Cc: Wenyi Xie Signed-off-by: Dandan Bi Reviewed-by: Michael D Kinney

Hisilicon/Library: Move two functions to RtcHelperLib

2020-05-20T11:29:51+00:00

The functions of acquiring ownership of RTC will be used for other RTC library, so move them to RtcHelperLib. Rename them by add leading Oem for uniform. Signed-off-by: Ming Huang Reviewed-by: Leif Lindholm

Hisilicon/D06: Drop the consume of PcdShellFile

2019-06-12T00:19:04+00:00

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=1800 After the drop of the consume of GenericBdsLib, there is no module that will use gEfiIntelFrameworkModulePkgTokenSpaceGuid.PcdShellFile. This commit will drop the consume of this PCD. Cc: Michael D Kinney Signed-off-by: Hao A Wu Reviewed-by: Leif Lindholm Reviewed-by: Ard Biesheuvel

Hisilicon/D0x: Use StatusCode Router & Handler in MdeModulePkg

2019-06-12T00:19:01+00:00

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=1800 This commit adopts a similar approach to edk2 commit a6d594c5fabd8da2273d2794826ec086cf9c3c04. Currently, Hisilicon platforms use modules from under "IntelFrameworkModulePkg/Universal/StatusCode/", which produce EFI_PEI_PROGRESS_CODE_PPI and EFI_STATUS_CODE_PROTOCOL directly, and write the status codes, as they are reported, to the serial port or to a memory buffer. This is called "handling" the status codes. MdeModulePkg offers a PEIM under "MdeModulePkg/Universal/ReportStatusCodeRouter/Pei" that produces both EFI_PEI_PROGRESS_CODE_PPI and EFI_PEI_RSC_HANDLER_PPI, and a runtime DXE driver under "MdeModulePkg/Universal/ReportStatusCodeRouter/RuntimeDxe" that produces both EFI_STATUS_CODE_PROTOCOL and EFI_RSC_HANDLER_PROTOCOL. MdeModulePkg also offers status code handler modules under MdeModulePkg/Universal/StatusCodeHandler/ that depend on EFI_PEI_RSC_HANDLER_PPI and EFI_RSC_HANDLER_PROTOCOL, respectively. The StatusCodeHandler modules register themselves with ReportStatusCodeRouter through EFI_PEI_RSC_HANDLER_PPI / EFI_RSC_HANDLER_PROTOCOL. When another module reports a status code through EFI_PEI_PROGRESS_CODE_PPI / EFI_STATUS_CODE_PROTOCOL, it reaches the phase-matching ReportStatusCodeRouter module first, which in turn passes the status code to the pre-registered, phase-matching StatusCodeHandler module. The status code handling in the StatusCodeHandler modules is identical to the one currently provided by the IntelFrameworkModulePkg modules. Replace the IntelFrameworkModulePkg modules with the MdeModulePkg ones, so we can decrease our dependency on IntelFrameworkModulePkg. OS boot verified by Ming Huang : https://edk2.groups.io/g/devel/message/41419 Cc: Ard Biesheuvel Cc: Michael D Kinney Signed-off-by: Hao A Wu Acked-by: Leif Lindholm

Hisilicon/D0x: Drop the consume of GenericBdsLib & PlatformBdsLib

2019-06-12T00:18:47+00:00

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=1800 Hisilicon platforms have already switched to the BDS driver in MdeModulePkg by commit 5845a5cde9d6bd51d77067b2594654005887a434. Thus, the consume of GenericBdsLib and PlatformBdsLib is no longer needed. Cc: Michael D Kinney Signed-off-by: Hao A Wu Reviewed-by: Leif Lindholm Reviewed-by: Ard Biesheuvel

Hisilicon: Update D06.dsc to remove unnecessary Network libs

2019-06-10T10:44:27+00:00

Update D06.dsc to remove a conditional include which is redundant since the NetworkPkg.dsc include file will already include it if necessary. Signed-off-by: Liming Gao [ardb: expand commit log] Reviewed-by: Ard Biesheuvel