Skip to content
This repository has been archived by the owner on Feb 4, 2025. It is now read-only.

Commit

Permalink
Hisilicon: rename OemMiscLib class to HisiOemMiscLib
Browse files Browse the repository at this point in the history
Edk2 commit [1] introduced the <OemMiscLib.h> 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 <[email protected]>
Cc: Wenyi Xie <[email protected]>
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=4564
Signed-off-by: Laszlo Ersek <[email protected]>
Reviewed-by: Ard Biesheuvel <[email protected]>
  • Loading branch information
lersek authored and ardbiesheuvel committed Oct 13, 2023
1 parent 5cfb1a7 commit a77cf41
Show file tree
Hide file tree
Showing 31 changed files with 36 additions and 36 deletions.
2 changes: 1 addition & 1 deletion Platform/Hisilicon/D03/D03.dsc
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@
TimeBaseLib|EmbeddedPkg/Library/TimeBaseLib/TimeBaseLib.inf
RealTimeClockLib|Platform/Hisilicon/D03/Library/DS3231RealTimeClockLib/DS3231RealTimeClockLib.inf

OemMiscLib|Platform/Hisilicon/D03/Library/OemMiscLib2P/OemMiscLib2PHi1610.inf
HisiOemMiscLib|Platform/Hisilicon/D03/Library/HisiOemMiscLib2P/HisiOemMiscLib2PHi1610.inf
OemAddressMapLib|Platform/Hisilicon/D03/Library/OemAddressMap2P/OemAddressMap2PHi1610.inf
PlatformSysCtrlLib|Silicon/Hisilicon/Hi1610/Library/PlatformSysCtrlLibHi1610/PlatformSysCtrlLibHi1610.inf

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
#include <PlatformArch.h>
#include <Library/PlatformSysCtrlLib.h>

#include <Library/OemMiscLib.h>
#include <Library/HisiOemMiscLib.h>
#include <Library/OemAddressMapLib.h>
#include <Library/ArmLib.h>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
#include <IndustryStandard/SmBios.h>

#include <PlatformArch.h>
#include <Library/OemMiscLib.h>
#include <Library/HisiOemMiscLib.h>
#include <Library/I2CLib.h>
#include <Library/HiiLib.h>

Expand Down Expand Up @@ -181,7 +181,7 @@ OemGetPackages (
return HiiAddPackages (
&gEfiCallerIdGuid,
NULL,
OemMiscLib2PStrings,
HisiOemMiscLib2PStrings,
NULL,
NULL
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,11 @@

[Defines]
INF_VERSION = 0x00010005
BASE_NAME = OemMiscLib2P
BASE_NAME = HisiOemMiscLib2P
FILE_GUID = B9CE7465-21A2-4ecd-B347-BBDDBD098CEE
MODULE_TYPE = BASE
VERSION_STRING = 1.0
LIBRARY_CLASS = OemMiscLib
LIBRARY_CLASS = HisiOemMiscLib

[Sources.common]
BoardFeature2PHi1610.c
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
#include <Library/IoLib.h>
#include <Library/TimerLib.h>
#include <Library/SerialPortLib.h>
#include <Library/OemMiscLib.h>
#include <Library/HisiOemMiscLib.h>
#include <PlatformArch.h>
#include <Library/PlatformSysCtrlLib.h>
#include <Library/OemAddressMapLib.h>
Expand Down
2 changes: 1 addition & 1 deletion Platform/Hisilicon/D05/D05.dsc
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@
#D05 RTC hardware is same as D03
RealTimeClockLib|Platform/Hisilicon/D03/Library/DS3231RealTimeClockLib/DS3231RealTimeClockLib.inf

OemMiscLib|Platform/Hisilicon/D05/Library/OemMiscLibD05/OemMiscLibD05.inf
HisiOemMiscLib|Platform/Hisilicon/D05/Library/HisiOemMiscLibD05/HisiOemMiscLibD05.inf
OemAddressMapLib|Platform/Hisilicon/D05/Library/OemAddressMapD05/OemAddressMapD05.inf
PlatformSysCtrlLib|Silicon/Hisilicon/Hi1616/Library/PlatformSysCtrlLibHi1616/PlatformSysCtrlLibHi1616.inf

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@
#include <Library/BaseMemoryLib.h>
#include <Library/DebugLib.h>
#include <Library/HiiLib.h>
#include <Library/HisiOemMiscLib.h>
#include <Library/I2CLib.h>
#include <Library/IoLib.h>
#include <Library/OemMiscLib.h>
#include <Protocol/Smbios.h>


Expand Down Expand Up @@ -209,7 +209,7 @@ OemGetPackages (
return HiiAddPackages (
&gEfiCallerIdGuid,
NULL,
OemMiscLibHi1616EvbStrings,
HisiOemMiscLibHi1616EvbStrings,
NULL,
NULL
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,11 @@

[Defines]
INF_VERSION = 0x00010019
BASE_NAME = OemMiscLibHi1616Evb
BASE_NAME = HisiOemMiscLibHi1616Evb
FILE_GUID = 751C7627-D5F8-499C-AEEEE-C87858759612
MODULE_TYPE = BASE
VERSION_STRING = 1.0
LIBRARY_CLASS = OemMiscLib
LIBRARY_CLASS = HisiOemMiscLib

[Sources.common]
BoardFeatureD05.c
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,10 @@

#include <Library/BaseMemoryLib.h>
#include <Library/DebugLib.h>
#include <Library/HisiOemMiscLib.h>
#include <Library/IoLib.h>
#include <Library/LpcLib.h>
#include <Library/OemAddressMapLib.h>
#include <Library/OemMiscLib.h>
#include <Library/PcdLib.h>
#include <Library/PlatformPciLib.h>
#include <Library/PlatformSysCtrlLib.h>
Expand Down
2 changes: 1 addition & 1 deletion Platform/Hisilicon/D06/D06.dsc
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@
TimeBaseLib|EmbeddedPkg/Library/TimeBaseLib/TimeBaseLib.inf
RtcHelperLib|Silicon/Hisilicon/Library/RtcHelperLib/RtcHelperLib.inf
RealTimeClockLib|Silicon/Hisilicon/Library/M41T83RealTimeClockLib/M41T83RealTimeClockLib.inf
OemMiscLib|Platform/Hisilicon/D06/Library/OemMiscLibD06/OemMiscLibD06.inf
HisiOemMiscLib|Platform/Hisilicon/D06/Library/HisiOemMiscLibD06/HisiOemMiscLibD06.inf
OemAddressMapLib|Platform/Hisilicon/D06/Library/OemAddressMapD06/OemAddressMapD06.inf
PlatformSysCtrlLib|Silicon/Hisilicon/Hi1620/Library/PlatformSysCtrlLibHi1620/PlatformSysCtrlLibHi1620.inf

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,9 @@
#include <Library/BaseMemoryLib.h>
#include <Library/DebugLib.h>
#include <Library/HiiLib.h>
#include <Library/HisiOemMiscLib.h>
#include <Library/I2CLib.h>
#include <Library/IoLib.h>
#include <Library/OemMiscLib.h>
#include <Protocol/Smbios.h>

#include <PlatformArch.h>
Expand Down Expand Up @@ -416,7 +416,7 @@ OemGetPackages (
return HiiAddPackages (
&gEfiCallerIdGuid,
NULL,
OemMiscLibStrings,
HisiOemMiscLibStrings,
NULL,
NULL
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,11 @@

[Defines]
INF_VERSION = 0x0001001A
BASE_NAME = OemMiscLib
BASE_NAME = HisiOemMiscLib
FILE_GUID = 3002911C-C160-4C46-93BB-782846673EEA
MODULE_TYPE = BASE
VERSION_STRING = 1.0
LIBRARY_CLASS = OemMiscLib
LIBRARY_CLASS = HisiOemMiscLib

[Sources.common]
BoardFeatureD06.c
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,10 @@
#include <Library/BaseMemoryLib.h>
#include <Library/CpldD06.h>
#include <Library/DebugLib.h>
#include <Library/HisiOemMiscLib.h>
#include <Library/IoLib.h>
#include <Library/LpcLib.h>
#include <Library/OemAddressMapLib.h>
#include <Library/OemMiscLib.h>
#include <Library/PcdLib.h>
#include <Library/PlatformSysCtrlLib.h>
#include <Library/SerialPortLib.h>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,9 @@
#include <Library/BaseMemoryLib.h>
#include <Library/DebugLib.h>
#include <Library/DevicePathLib.h>
#include <Library/HisiOemMiscLib.h>
#include <Library/IoLib.h>
#include <Library/MemoryAllocationLib.h>
#include <Library/OemMiscLib.h>
#include <Library/PcdLib.h>
#include <Library/PciHostBridgeLib.h>
#include <Library/PlatformPciLib.h>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,8 @@
BaseMemoryLib
DebugLib
DevicePathLib
HisiOemMiscLib
MemoryAllocationLib
OemMiscLib

[Pcd]
gHisiTokenSpaceGuid.PcdPcieRootBridgeMask
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,13 @@
#include <Library/BaseLib.h>
#include <Library/BaseMemoryLib.h>
#include <Library/DebugLib.h>
#include <Library/HisiOemMiscLib.h>
#include <Library/HobLib.h>
#include <Library/HwMemInitLib.h>
#include <Library/OemConfigData.h>
#include <Library/OemMiscLib.h>
#include <Library/UefiBootServicesTableLib.h>
#include <Library/UefiRuntimeServicesTableLib.h>
#include <Library/UefiLib.h>
#include <Library/UefiRuntimeServicesTableLib.h>

#define CORECOUNT(X) ((X) * CORE_NUM_PER_SOCKET)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,6 @@
#include <Library/PrintLib.h>
#include <Library/UefiBootServicesTableLib.h>
#include <Library/MemoryAllocationLib.h>
#include <Library/OemMiscLib.h>
#include <Library/HisiOemMiscLib.h>

#endif
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
DebugLib
UefiLib
UefiDriverEntryPoint
OemMiscLib
HisiOemMiscLib

[Protocols]
gEfiSmbiosProtocolGuid
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
#include <Guid/DebugMask.h>
#include <Guid/MemoryMapData.h>
#include <Library/PlatformSysCtrlLib.h>
#include <Library/OemMiscLib.h>
#include <Library/HisiOemMiscLib.h>

//
// This is the generated header file which includes whatever needs to be exported (strings + IFR)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
#include <Library/PcdLib.h>
#include <PlatformArch.h>
#include <Library/PlatformSysCtrlLib.h>
#include <Library/OemMiscLib.h>
#include <Library/HisiOemMiscLib.h>
#include <Library/ArmLib.h>

//
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
PcdLib

PlatformSysCtrlLib
OemMiscLib
HisiOemMiscLib

[Protocols]
gEfiSmbiosProtocolGuid # PROTOCOL ALWAYS_CONSUMED
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
BaseMemoryLib
BaseLib
DebugLib
OemMiscLib
HisiOemMiscLib
UefiBootServicesTableLib
UefiRuntimeServicesTableLib
UefiDriverEntryPoint
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@

#include "SmbiosMisc.h"

#include <Library/OemMiscLib.h>
#include <Library/HisiOemMiscLib.h>

extern SMBIOS_TABLE_TYPE9 MiscSystemSlotDesignationPcie0Data;
extern SMBIOS_TABLE_TYPE9 MiscSystemSlotDesignationPcie1Data;
Expand Down
2 changes: 1 addition & 1 deletion Silicon/Hisilicon/Hi1610/Drivers/PcieInit1610/PcieInit.c
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
#include "PcieInit.h"
#include <Library/UefiBootServicesTableLib.h>
#include <Library/PcdLib.h>
#include <Library/OemMiscLib.h>
#include <Library/HisiOemMiscLib.h>
#include <Library/PlatformPciLib.h>


Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
TimerLib
PcdLib
IoLib
OemMiscLib
HisiOemMiscLib

[Protocols]
#gEfiPcieRootBridgeProtocolGuid
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@
#include <Uefi.h>
#include <IndustryStandard/Acpi.h>
#include <Library/DebugLib.h>
#include <Library/HisiOemMiscLib.h>
#include <Library/IoLib.h>
#include <Library/OemMiscLib.h>
#include <Library/PcdLib.h>
#include <Library/PciExpressLib.h>
#include <Library/PlatformPciLib.h>
Expand Down
2 changes: 1 addition & 1 deletion Silicon/Hisilicon/HisiPkg.dec
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@
PlatformSysCtrlLib|Include/Library/PlatformSysCtrlLib.h
CpldIoLib|Include/Library/CpldIoLib.h
OemAddressMapLib|Include/Library/OemAddressMapLib.h
OemMiscLib|Include/Library/OemMiscLib.h
HisiOemMiscLib|Include/Library/HisiOemMiscLib.h
I2CLib|Include/Library/I2CLib.h
PlatformPciLib|Include/Library/PlatformPciLib.h
FdtUpdateLib|Include/Library/FdtUpdateLib.h
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@
**/


#ifndef _OEM_MISC_LIB_H_
#define _OEM_MISC_LIB_H_
#ifndef HISI_OEM_MISC_LIB_H_
#define HISI_OEM_MISC_LIB_H_

#include <Uefi.h>

Expand Down

0 comments on commit a77cf41

Please sign in to comment.