[edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'

public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed

* [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
@ 2021-01-11  3:31 Rebecca Cran
  2021-01-11  3:31 ` [edk2-platforms PATCH 1/1] " Rebecca Cran
  2021-01-11  4:42 ` [edk2-platforms PATCH 0/1] " Michael D Kinney
  0 siblings, 2 replies; 10+ messages in thread
From: Rebecca Cran @ 2021-01-11  3:31 UTC (permalink / raw)
  To: devel
  Cc: Rebecca Cran, Sai Chaganty, Liming Gao, Michael D Kinney,
	Kelly Steele, Agyeman Prince, Zailiang Sun, Yi Qian,
	Leif Lindholm, Ard Biesheuvel, Wenyi Xie, Chasel Chiu, Ray Ni,
	Abner Chang, Daniel Schaefer, Gilbert Chen, Thomas Abraham,
	Pete Batard, Marcin Wojtas

Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'

This patch depends on the corresponding edk2 change with subject
"Various packages: rename SmBios.h to Smbios.h in MdePkg and update references through edk2 tree".

Rebecca Cran (1):
  Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'

 Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec               | 10 +++++-----
 Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h       |  2 +-
 Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h         |  2 +-
 Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h            |  2 +-
 Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h             |  4 ++--
 Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h       |  4 ++--
 Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h              |  2 +-
 Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h                        |  2 +-
 Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h                     |  2 +-
 Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h                  |  2 +-
 Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h            |  2 +-
 Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h                          |  2 +-
 Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h |  2 +-
 Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h         |  2 +-
 Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h   |  2 +-
 Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h                       |  2 +-
 Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h                    |  2 +-
 Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c                           |  2 +-
 Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c                   |  2 +-
 Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c                       |  2 +-
 Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c                       |  2 +-
 Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c                 |  2 +-
 Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                   |  4 ++--
 Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c                |  2 +-
 Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                       |  4 ++--
 Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c      |  2 +-
 Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c                        |  2 +-
 27 files changed, 35 insertions(+), 35 deletions(-)

-- 
2.26.2


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [edk2-platforms PATCH 1/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
  2021-01-11  3:31 [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h' Rebecca Cran
@ 2021-01-11  3:31 ` Rebecca Cran
  2021-01-11  3:37   ` Abner Chang
  2021-01-11 12:35   ` Pete Batard
  2021-01-11  4:42 ` [edk2-platforms PATCH 0/1] " Michael D Kinney
  1 sibling, 2 replies; 10+ messages in thread
From: Rebecca Cran @ 2021-01-11  3:31 UTC (permalink / raw)
  To: devel
  Cc: Rebecca Cran, Sai Chaganty, Liming Gao, Michael D Kinney,
	Kelly Steele, Agyeman Prince, Zailiang Sun, Yi Qian,
	Leif Lindholm, Ard Biesheuvel, Wenyi Xie, Chasel Chiu, Ray Ni,
	Abner Chang, Daniel Schaefer, Gilbert Chen, Thomas Abraham,
	Pete Batard, Marcin Wojtas

A corresponding edk2 changeset renamed SmBios.h to Smbios.h in
MdePkg/Include/IndustryStandard and MdePkg/Include/Guid. Update all
references through the edk2-platform tree to follow the change.

Signed-off-by: Rebecca Cran <rebecca@nuviainc.com>
---
 Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec               | 10 +++++-----
 Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h       |  2 +-
 Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h         |  2 +-
 Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h            |  2 +-
 Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h             |  4 ++--
 Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h       |  4 ++--
 Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h              |  2 +-
 Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h                        |  2 +-
 Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h                     |  2 +-
 Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h                  |  2 +-
 Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h            |  2 +-
 Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h                          |  2 +-
 Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h |  2 +-
 Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h         |  2 +-
 Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h   |  2 +-
 Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h                       |  2 +-
 Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h                    |  2 +-
 Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c                           |  2 +-
 Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c                   |  2 +-
 Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c                       |  2 +-
 Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c                       |  2 +-
 Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c                 |  2 +-
 Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                   |  4 ++--
 Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c                |  2 +-
 Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                       |  4 ++--
 Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c      |  2 +-
 Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c                        |  2 +-
 27 files changed, 35 insertions(+), 35 deletions(-)

diff --git a/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec b/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
index 6170f379691a..4b6442ed0ebf 100644
--- a/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
+++ b/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
@@ -35,7 +35,7 @@ [PcdsDynamic, PcdsDynamicEx]
   #
   gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType0BiosInformation|{0x0}|SMBIOS_TABLE_TYPE0|0xD0000001 {
     <HeaderFiles>
-      IndustryStandard/SmBios.h
+      IndustryStandard/Smbios.h
     <Packages>
       MdePkg/MdePkg.dec
       SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
@@ -68,7 +68,7 @@ [PcdsDynamic, PcdsDynamicEx]
   #
   gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType1SystemInformation|{0x0}|SMBIOS_TABLE_TYPE1|0xD0000002 {
     <HeaderFiles>
-      IndustryStandard/SmBios.h
+      IndustryStandard/Smbios.h
     <Packages>
       MdePkg/MdePkg.dec
       SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
@@ -84,7 +84,7 @@ [PcdsDynamic, PcdsDynamicEx]
 
   gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType2BaseBoardInformation|{0x0}|SMBIOS_TABLE_TYPE2|0xD0000003 {
     <HeaderFiles>
-      IndustryStandard/SmBios.h
+      IndustryStandard/Smbios.h
     <Packages>
       MdePkg/MdePkg.dec
       SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
@@ -108,7 +108,7 @@ [PcdsDynamic, PcdsDynamicEx]
   #
   gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType3SystemEnclosureChassis|{0x0}|SMBIOS_TABLE_TYPE3|0xD0000004 {
     <HeaderFiles>
-      IndustryStandard/SmBios.h
+      IndustryStandard/Smbios.h
     <Packages>
       MdePkg/MdePkg.dec
       SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
@@ -128,7 +128,7 @@ [PcdsDynamic, PcdsDynamicEx]
   #
   gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType32SystemBootInformation|{0x0}|SMBIOS_TABLE_TYPE32|0xD0000005 {
     <HeaderFiles>
-      IndustryStandard/SmBios.h
+      IndustryStandard/Smbios.h
     <Packages>
       MdePkg/MdePkg.dec
       SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
diff --git a/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h b/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h
index 2a349bcd3716..85ab757b4437 100644
--- a/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h
+++ b/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h
@@ -11,7 +11,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 
 #include <PiDxe.h>
 #include <Protocol/Smbios.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Library/UefiBootServicesTableLib.h>
 #include <Library/DebugLib.h>
 #include <Library/BaseLib.h>
diff --git a/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h b/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h
index 5d36c872d510..9532da7b3d82 100644
--- a/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h
+++ b/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h
@@ -17,7 +17,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 //
 // The protocols, PPI and GUID definitions for this module
 //
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Protocol/Smbios.h>
 #include <Protocol/PciRootBridgeIo.h>
 #include <Protocol/SmbusHc.h>
diff --git a/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h b/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h
index ab7537043525..b913ccc52ab5 100644
--- a/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h
+++ b/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h
@@ -14,7 +14,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 
 
 #include <PiDxe.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Protocol/Smbios.h>
 #include <Guid/MdeModuleHii.h>
 #include <DataHubRecords.h>
diff --git a/Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h b/Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h
index 566b6a48c65d..0b3777cf3803 100644
--- a/Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h
+++ b/Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h
@@ -14,7 +14,7 @@
 
 #include <IndustryStandard/Pci.h>
 #include <IndustryStandard/Acpi.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <IndustryStandard/PeImage.h>
 
 #include <Library/DebugLib.h>
@@ -45,7 +45,7 @@
 #include <Protocol/LoadedImage.h>
 
 #include <Guid/Acpi.h>
-#include <Guid/SmBios.h>
+#include <Guid/Smbios.h>
 #include <Guid/Mps.h>
 #include <Guid/HobList.h>
 #include <Guid/GlobalVariable.h>
diff --git a/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h b/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h
index 01ba20d2dad7..ccbaf31449d5 100644
--- a/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h
+++ b/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h
@@ -14,7 +14,7 @@
 
 #include <IndustryStandard/Pci.h>
 #include <IndustryStandard/Acpi.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <IndustryStandard/PeImage.h>
 
 #include <Library/DebugLib.h>
@@ -44,7 +44,7 @@
 #include <Protocol/LoadedImage.h>
 
 #include <Guid/Acpi.h>
-#include <Guid/SmBios.h>
+#include <Guid/Smbios.h>
 #include <Guid/Mps.h>
 #include <Guid/HobList.h>
 #include <Guid/GlobalVariable.h>
diff --git a/Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h b/Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h
index ccd35e2924ff..7a8a71ad5aa5 100644
--- a/Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h
+++ b/Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h
@@ -13,7 +13,7 @@
 #include <PiDxe.h>
 
 #include <Protocol/Smbios.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Library/DebugLib.h>
 #include <Library/BaseLib.h>
 #include <Library/BaseMemoryLib.h>
diff --git a/Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h b/Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h
index f95e6bbc2ddd..2bce59df7785 100644
--- a/Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h
+++ b/Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h
@@ -15,7 +15,7 @@
 
 
 #include <PiDxe.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Protocol/Smbios.h>
 
 #include <Guid/MdeModuleHii.h>
diff --git a/Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h b/Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h
index 7194ae589042..0e0dda54d591 100644
--- a/Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h
+++ b/Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h
@@ -14,7 +14,7 @@
 #include <Uefi.h>
 #include <PiDxe.h>
 #include <IndustryStandard/Pci.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Protocol/Smbios.h>
 #include <Protocol/PciIo.h>
 #include <Protocol/PciRootBridgeIo.h>
diff --git a/Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h b/Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h
index 22c7f6797168..ee8440c7c126 100644
--- a/Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h
+++ b/Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h
@@ -14,7 +14,7 @@
 #include <Uefi.h>
 #include <PiDxe.h>
 #include <Protocol/Smbios.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Library/HiiLib.h>
 #include <Library/DebugLib.h>
 #include <Library/MemoryAllocationLib.h>
diff --git a/Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h b/Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h
index 3ce180d21df3..86c42f4920aa 100644
--- a/Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h
+++ b/Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h
@@ -12,7 +12,7 @@
 
 #include <Uefi.h>
 #include <Protocol/Smbios.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Library/HiiLib.h>
 #include <Library/DebugLib.h>
 #include <Library/MemoryAllocationLib.h>
diff --git a/Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h b/Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
index 4692eb71334e..f6e658c7c0df 100644
--- a/Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
+++ b/Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
@@ -20,7 +20,7 @@ Based on files under Nt32Pkg/MiscSubClassPlatformDxe/
 #define _SMBIOS_MISC_DRIVER_H
 
 #include <Protocol/Smbios.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Library/HiiLib.h>
 #include <Library/PcdLib.h>
 #include <Library/DebugLib.h>
diff --git a/Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h b/Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h
index 6e993053fce7..be92c86ecff3 100644
--- a/Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h
+++ b/Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h
@@ -19,7 +19,7 @@
 #include <Ppi/MasterBootMode.h>
 #include <Ppi/SiPolicy.h>
 #include <Library/CpuPolicyLib.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Library/SiConfigBlockLib.h>
 #include <Library/CpuPlatformLib.h>
 #include <Register/Cpuid.h>
diff --git a/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h b/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h
index b30bc3f9e7ee..457e792c6271 100644
--- a/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h
+++ b/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h
@@ -15,7 +15,7 @@
 #ifndef __FIRMWARE_VERSION_INFO_H__
 #define __FIRMWARE_VERSION_INFO_H__
 
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 
 #define INTEL_FIRMWARE_VERSION_INFO_GROUP_NAME    "Firmware Version Info"
 
diff --git a/Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h b/Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h
index 23321d6432c5..a76328745f77 100644
--- a/Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h
+++ b/Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h
@@ -18,7 +18,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 #include <Ppi/MasterBootMode.h>
 #include <Ppi/SiPolicy.h>
 #include <Library/CpuPolicyLib.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Library/SiConfigBlockLib.h>
 #include <Library/CpuPlatformLib.h>
 #include <Register/Cpuid.h>
diff --git a/Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h b/Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h
index 2f5847e53e07..8a6b6b2f0632 100644
--- a/Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h
+++ b/Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h
@@ -10,7 +10,7 @@
 #define RISC_V_PROCESSOR_SPECIFIC_HOB_DATA_H_
 
 #include <Uefi.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <SmbiosProcessorSpecificData.h>
 
 #define TO_BE_FILLED 0
diff --git a/Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h b/Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h
index 81e48cd0685c..c5bbf41ece05 100644
--- a/Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h
+++ b/Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h
@@ -11,7 +11,7 @@
 #ifndef SMBIOS_RISC_V_PROCESSOR_SPECIFIC_DATA_H_
 #define SMBIOS_RISC_V_PROCESSOR_SPECIFIC_DATA_H_
 
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <RiscVImpl.h>
 
 #pragma pack(1)
diff --git a/Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c b/Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
index ac18b16ca676..dd7ae0620792 100644
--- a/Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
+++ b/Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
@@ -7,7 +7,7 @@
 
 **/
 #include <ArmPlatform.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Library/ArmLib.h>
 #include <Library/BaseLib.h>
 #include <Library/BaseMemoryLib.h>
diff --git a/Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c b/Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c
index 1670cac1d491..28dc244284d6 100644
--- a/Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c
+++ b/Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c
@@ -11,7 +11,7 @@
 #include <Library/DebugLib.h>
 #include <Library/BaseMemoryLib.h>
 #include <Protocol/Smbios.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 
 #include <PlatformArch.h>
 #include <Library/OemMiscLib.h>
diff --git a/Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c b/Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c
index ac013ca30890..ad692cb6f116 100644
--- a/Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c
+++ b/Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c
@@ -9,7 +9,7 @@
 
 #include <PlatformArch.h>
 #include <Uefi.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Library/BaseMemoryLib.h>
 #include <Library/DebugLib.h>
 #include <Library/HiiLib.h>
diff --git a/Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c b/Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c
index 980eabfe3355..d446f2f01056 100644
--- a/Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c
+++ b/Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c
@@ -8,7 +8,7 @@
 **/
 
 #include <Uefi.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Library/BaseMemoryLib.h>
 #include <Library/DebugLib.h>
 #include <Library/HiiLib.h>
diff --git a/Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c b/Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c
index 5a597d86da04..b32389859121 100644
--- a/Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c
+++ b/Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c
@@ -24,7 +24,7 @@ Revision History:
 #include <Protocol/MpService.h>
 #include <Protocol/PchPlatformPolicy.h>
 #include <Protocol/Smbios.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Library/IoLib.h>
 #include <Guid/GlobalVariable.h>
 #include <Register/Cpuid.h>
diff --git a/Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c b/Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
index 0d0132245aa7..d3fd4b26a536 100644
--- a/Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
+++ b/Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
@@ -34,11 +34,11 @@
  **/
 
 #include <Base.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <IndustryStandard/RpiMbox.h>
 #include <Protocol/Smbios.h>
 #include <Protocol/RpiFirmware.h>
-#include <Guid/SmBios.h>
+#include <Guid/Smbios.h>
 #include <Library/ArmLib.h>
 #include <Library/DebugLib.h>
 #include <Library/UefiDriverEntryPoint.h>
diff --git a/Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c b/Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c
index 6227b7787707..38f886a2c04d 100644
--- a/Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c
+++ b/Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c
@@ -10,7 +10,7 @@
 **/
 
 #include <PiDxe.h>
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Library/BaseLib.h>
 #include <Library/BaseMemoryLib.h>
 #include <Library/DebugLib.h>
diff --git a/Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c b/Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
index aa0e3ff1470e..c4b38f9c47be 100644
--- a/Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
+++ b/Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
@@ -28,8 +28,8 @@
  *----------------------------------------------------------------------------------------
  */
 #include <Protocol/Smbios.h>
-#include <IndustryStandard/SmBios.h>
-#include <Guid/SmBios.h>
+#include <IndustryStandard/Smbios.h>
+#include <Guid/Smbios.h>
 #include <Library/DebugLib.h>
 #include <Library/UefiDriverEntryPoint.h>
 #include <Library/UefiLib.h>
diff --git a/Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c b/Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c
index bcb77eb12209..496f3fef5b09 100644
--- a/Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c
+++ b/Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c
@@ -11,7 +11,7 @@
   http://www.dmtf.org/sites/default/files/standards/documents/DSP0134_3.0.0.pdf
 
 **/
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 #include <Library/UefiLib.h>
 #include <Library/BaseLib.h>
 #include <Library/BaseMemoryLib.h>
diff --git a/Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c b/Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c
index 2ecaec2af515..a419d5ce1fa6 100644
--- a/Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c
+++ b/Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c
@@ -18,7 +18,7 @@
 
 #include <Protocol/Smbios.h>
 
-#include <IndustryStandard/SmBios.h>
+#include <IndustryStandard/Smbios.h>
 
 //
 // SMBIOS tables often reference each other using
-- 
2.26.2


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [edk2-platforms PATCH 1/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
  2021-01-11  3:31 ` [edk2-platforms PATCH 1/1] " Rebecca Cran
@ 2021-01-11  3:37   ` Abner Chang
  2021-01-11 12:35   ` Pete Batard
  1 sibling, 0 replies; 10+ messages in thread
From: Abner Chang @ 2021-01-11  3:37 UTC (permalink / raw)
  To: Rebecca Cran, devel@edk2.groups.io
  Cc: Sai Chaganty, Liming Gao, Michael D Kinney, Kelly Steele,
	Agyeman Prince, Zailiang Sun, Yi Qian, Leif Lindholm,
	Ard Biesheuvel, Wenyi Xie, Chasel Chiu, Ray Ni, Schaefer, Daniel,
	Chen, Gilbert, Thomas Abraham, Pete Batard, Marcin Wojtas

For RISC-V,
Reviewed-by: Abner Chang <abner.chang@hpe.com>

> -----Original Message-----
> From: Rebecca Cran [mailto:rebecca@nuviainc.com]
> Sent: Monday, January 11, 2021 11:31 AM
> To: devel@edk2.groups.io
> Cc: Rebecca Cran <rebecca@nuviainc.com>; Sai Chaganty
> <rangasai.v.chaganty@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>;
> Michael D Kinney <michael.d.kinney@intel.com>; Kelly Steele
> <kelly.steele@intel.com>; Agyeman Prince <prince.agyeman@intel.com>;
> Zailiang Sun <zailiang.sun@intel.com>; Yi Qian <yi.qian@intel.com>; Leif
> Lindholm <leif@nuviainc.com>; Ard Biesheuvel
> <ardb+tianocore@kernel.org>; Wenyi Xie <xiewenyi2@huawei.com>; Chasel
> Chiu <chasel.chiu@intel.com>; Ray Ni <ray.ni@intel.com>; Chang, Abner
> (HPS SW/FW Technologist) <abner.chang@hpe.com>; Schaefer, Daniel
> <daniel.schaefer@hpe.com>; Chen, Gilbert <gilbert.chen@hpe.com>;
> Thomas Abraham <thomas.abraham@arm.com>; Pete Batard
> <pete@akeo.ie>; Marcin Wojtas <mw@semihalf.com>
> Subject: [edk2-platforms PATCH 1/1] Update SMBIOS includes to use
> capitalization 'Smbios.h' vs 'SmBios.h'
> 
> A corresponding edk2 changeset renamed SmBios.h to Smbios.h in
> MdePkg/Include/IndustryStandard and MdePkg/Include/Guid. Update all
> references through the edk2-platform tree to follow the change.
> 
> Signed-off-by: Rebecca Cran <rebecca@nuviainc.com>
> ---
> 
> Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.d
> ec               | 10 +++++-----
> 
> Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/Smb
> iosBasic.h       |  2 +-
> 
> Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/Memory
> SubClass.h         |  2 +-
> 
> Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonH
> eader.h            |  2 +-
> 
> Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHo
> ok.h             |  4 ++--
> 
> Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPl
> atform.h       |  4 ++--
> 
> Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDx
> e.h              |  2 +-
>  Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h
> |  2 +-
>  Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h
> |  2 +-
>  Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h
> |  2 +-
>  Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h
> |  2 +-
>  Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
> |  2 +-
> 
> Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicy
> Library.h |  2 +-
> 
> Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.
> h         |  2 +-
> 
> Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLi
> brary.h   |  2 +-
>  Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h
> |  2 +-
>  Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h
> |  2 +-
>  Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> |  2 +-
>  Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c
> |  2 +-
>  Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c
> |  2 +-
>  Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c
> |  2 +-
>  Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c
> |  2 +-
>  Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> |  4 ++--
> 
> Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.
> c                |  2 +-
>  Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> |  4 ++--
> 
> Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataH
> obLib.c      |  2 +-
>  Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> |  2 +-
>  27 files changed, 35 insertions(+), 35 deletions(-)
> 
> diff --git
> a/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.
> dec
> b/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.
> dec
> index 6170f379691a..4b6442ed0ebf 100644
> ---
> a/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.
> dec
> +++
> b/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg
> +++ .dec
> @@ -35,7 +35,7 @@ [PcdsDynamic, PcdsDynamicEx]
>    #
> 
> gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType0BiosInformation|{0x0}
> |SMBIOS_TABLE_TYPE0|0xD0000001 {
>      <HeaderFiles>
> -      IndustryStandard/SmBios.h
> +      IndustryStandard/Smbios.h
>      <Packages>
>        MdePkg/MdePkg.dec
>        SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> @@ -68,7 +68,7 @@ [PcdsDynamic, PcdsDynamicEx]
>    #
> 
> gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType1SystemInformation|{0
> x0}|SMBIOS_TABLE_TYPE1|0xD0000002 {
>      <HeaderFiles>
> -      IndustryStandard/SmBios.h
> +      IndustryStandard/Smbios.h
>      <Packages>
>        MdePkg/MdePkg.dec
>        SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> @@ -84,7 +84,7 @@ [PcdsDynamic, PcdsDynamicEx]
> 
> 
> gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType2BaseBoardInformation
> |{0x0}|SMBIOS_TABLE_TYPE2|0xD0000003 {
>      <HeaderFiles>
> -      IndustryStandard/SmBios.h
> +      IndustryStandard/Smbios.h
>      <Packages>
>        MdePkg/MdePkg.dec
>        SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> @@ -108,7 +108,7 @@ [PcdsDynamic, PcdsDynamicEx]
>    #
> 
> gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType3SystemEnclosureChass
> is|{0x0}|SMBIOS_TABLE_TYPE3|0xD0000004 {
>      <HeaderFiles>
> -      IndustryStandard/SmBios.h
> +      IndustryStandard/Smbios.h
>      <Packages>
>        MdePkg/MdePkg.dec
>        SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> @@ -128,7 +128,7 @@ [PcdsDynamic, PcdsDynamicEx]
>    #
> 
> gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType32SystemBootInformati
> on|{0x0}|SMBIOS_TABLE_TYPE32|0xD0000005 {
>      <HeaderFiles>
> -      IndustryStandard/SmBios.h
> +      IndustryStandard/Smbios.h
>      <Packages>
>        MdePkg/MdePkg.dec
>        SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> diff --git
> a/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/S
> mbiosBasic.h
> b/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/S
> mbiosBasic.h
> index 2a349bcd3716..85ab757b4437 100644
> ---
> a/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/S
> mbiosBasic.h
> +++
> b/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/S
> +++ mbiosBasic.h
> @@ -11,7 +11,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> 
>  #include <PiDxe.h>
>  #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Library/UefiBootServicesTableLib.h>
>  #include <Library/DebugLib.h>
>  #include <Library/BaseLib.h>
> diff --git
> a/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/Memo
> rySubClass.h
> b/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/Memo
> rySubClass.h
> index 5d36c872d510..9532da7b3d82 100644
> ---
> a/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/Memo
> rySubClass.h
> +++
> b/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/Memo
> ry
> +++ SubClass.h
> @@ -17,7 +17,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent  //  // The
> protocols, PPI and GUID definitions for this module  // -#include
> <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Protocol/Smbios.h>
>  #include <Protocol/PciRootBridgeIo.h>
>  #include <Protocol/SmbusHc.h>
> diff --git
> a/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/Commo
> nHeader.h
> b/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/Commo
> nHeader.h
> index ab7537043525..b913ccc52ab5 100644
> ---
> a/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/Commo
> nHeader.h
> +++
> b/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/Commo
> nH
> +++ eader.h
> @@ -14,7 +14,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> 
> 
>  #include <PiDxe.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Protocol/Smbios.h>
>  #include <Guid/MdeModuleHii.h>
>  #include <DataHubRecords.h>
> diff --git
> a/Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsH
> ook.h
> b/Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsH
> ook.h
> index 566b6a48c65d..0b3777cf3803 100644
> ---
> a/Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsH
> ook.h
> +++
> b/Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBds
> +++ Hook.h
> @@ -14,7 +14,7 @@
> 
>  #include <IndustryStandard/Pci.h>
>  #include <IndustryStandard/Acpi.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <IndustryStandard/PeImage.h>
> 
>  #include <Library/DebugLib.h>
> @@ -45,7 +45,7 @@
>  #include <Protocol/LoadedImage.h>
> 
>  #include <Guid/Acpi.h>
> -#include <Guid/SmBios.h>
> +#include <Guid/Smbios.h>
>  #include <Guid/Mps.h>
>  #include <Guid/HobList.h>
>  #include <Guid/GlobalVariable.h>
> diff --git
> a/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/Bds
> Platform.h
> b/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/Bd
> sPlatform.h
> index 01ba20d2dad7..ccbaf31449d5 100644
> ---
> a/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/Bds
> Platform.h
> +++
> b/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/B
> +++ dsPlatform.h
> @@ -14,7 +14,7 @@
> 
>  #include <IndustryStandard/Pci.h>
>  #include <IndustryStandard/Acpi.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <IndustryStandard/PeImage.h>
> 
>  #include <Library/DebugLib.h>
> @@ -44,7 +44,7 @@
>  #include <Protocol/LoadedImage.h>
> 
>  #include <Guid/Acpi.h>
> -#include <Guid/SmBios.h>
> +#include <Guid/Smbios.h>
>  #include <Guid/Mps.h>
>  #include <Guid/HobList.h>
>  #include <Guid/GlobalVariable.h>
> diff --git
> a/Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatform
> Dxe.h
> b/Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatform
> Dxe.h
> index ccd35e2924ff..7a8a71ad5aa5 100644
> ---
> a/Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatform
> Dxe.h
> +++
> b/Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatform
> +++ Dxe.h
> @@ -13,7 +13,7 @@
>  #include <PiDxe.h>
> 
>  #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Library/DebugLib.h>
>  #include <Library/BaseLib.h>
>  #include <Library/BaseMemoryLib.h>
> diff --git
> a/Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h
> b/Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h
> index f95e6bbc2ddd..2bce59df7785 100644
> --- a/Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h
> +++ b/Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h
> @@ -15,7 +15,7 @@
> 
> 
>  #include <PiDxe.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Protocol/Smbios.h>
> 
>  #include <Guid/MdeModuleHii.h>
> diff --git
> a/Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h
> b/Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h
> index 7194ae589042..0e0dda54d591 100644
> --- a/Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h
> +++ b/Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h
> @@ -14,7 +14,7 @@
>  #include <Uefi.h>
>  #include <PiDxe.h>
>  #include <IndustryStandard/Pci.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Protocol/Smbios.h>
>  #include <Protocol/PciIo.h>
>  #include <Protocol/PciRootBridgeIo.h>
> diff --git
> a/Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h
> b/Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h
> index 22c7f6797168..ee8440c7c126 100644
> ---
> a/Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h
> +++
> b/Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.
> +++ h
> @@ -14,7 +14,7 @@
>  #include <Uefi.h>
>  #include <PiDxe.h>
>  #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Library/HiiLib.h>
>  #include <Library/DebugLib.h>
>  #include <Library/MemoryAllocationLib.h> diff --git
> a/Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass
> .h
> b/Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass
> .h
> index 3ce180d21df3..86c42f4920aa 100644
> ---
> a/Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass
> .h
> +++ b/Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSub
> +++ Class.h
> @@ -12,7 +12,7 @@
> 
>  #include <Uefi.h>
>  #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Library/HiiLib.h>
>  #include <Library/DebugLib.h>
>  #include <Library/MemoryAllocationLib.h> diff --git
> a/Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
> b/Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
> index 4692eb71334e..f6e658c7c0df 100644
> --- a/Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
> +++ b/Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
> @@ -20,7 +20,7 @@ Based on files under
> Nt32Pkg/MiscSubClassPlatformDxe/  #define _SMBIOS_MISC_DRIVER_H
> 
>  #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Library/HiiLib.h>
>  #include <Library/PcdLib.h>
>  #include <Library/DebugLib.h>
> diff --git
> a/Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPoli
> cyLibrary.h
> b/Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPoli
> cyLibrary.h
> index 6e993053fce7..be92c86ecff3 100644
> ---
> a/Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPoli
> cyLibrary.h
> +++ b/Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/Pei
> +++ CpuPolicyLibrary.h
> @@ -19,7 +19,7 @@
>  #include <Ppi/MasterBootMode.h>
>  #include <Ppi/SiPolicy.h>
>  #include <Library/CpuPolicyLib.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Library/SiConfigBlockLib.h>
>  #include <Library/CpuPlatformLib.h>
>  #include <Register/Cpuid.h>
> diff --git
> a/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInf
> o.h
> b/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionIn
> fo.h
> index b30bc3f9e7ee..457e792c6271 100644
> ---
> a/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInf
> o.h
> +++ b/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVer
> +++ sionInfo.h
> @@ -15,7 +15,7 @@
>  #ifndef __FIRMWARE_VERSION_INFO_H__
>  #define __FIRMWARE_VERSION_INFO_H__
> 
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
> 
>  #define INTEL_FIRMWARE_VERSION_INFO_GROUP_NAME    "Firmware
> Version Info"
> 
> diff --git
> a/Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicy
> Library.h
> b/Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicy
> Library.h
> index 23321d6432c5..a76328745f77 100644
> ---
> a/Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicy
> Library.h
> +++ b/Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCp
> +++ uPolicyLibrary.h
> @@ -18,7 +18,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent  #include
> <Ppi/MasterBootMode.h>  #include <Ppi/SiPolicy.h>  #include
> <Library/CpuPolicyLib.h> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Library/SiConfigBlockLib.h>
>  #include <Library/CpuPlatformLib.h>
>  #include <Register/Cpuid.h>
> diff --git a/Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h
> b/Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h
> index 2f5847e53e07..8a6b6b2f0632 100644
> --- a/Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h
> +++ b/Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h
> @@ -10,7 +10,7 @@
>  #define RISC_V_PROCESSOR_SPECIFIC_HOB_DATA_H_
> 
>  #include <Uefi.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <SmbiosProcessorSpecificData.h>
> 
>  #define TO_BE_FILLED 0
> diff --git a/Silicon/RISC-
> V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h b/Silicon/RISC-
> V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h
> index 81e48cd0685c..c5bbf41ece05 100644
> --- a/Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h
> +++ b/Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h
> @@ -11,7 +11,7 @@
>  #ifndef SMBIOS_RISC_V_PROCESSOR_SPECIFIC_DATA_H_
>  #define SMBIOS_RISC_V_PROCESSOR_SPECIFIC_DATA_H_
> 
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <RiscVImpl.h>
> 
>  #pragma pack(1)
> diff --git
> a/Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> b/Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> index ac18b16ca676..dd7ae0620792 100644
> --- a/Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> +++ b/Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> @@ -7,7 +7,7 @@
> 
>  **/
>  #include <ArmPlatform.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Library/ArmLib.h>
>  #include <Library/BaseLib.h>
>  #include <Library/BaseMemoryLib.h>
> diff --git
> a/Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c
> b/Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c
> index 1670cac1d491..28dc244284d6 100644
> --- a/Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c
> +++
> b/Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c
> @@ -11,7 +11,7 @@
>  #include <Library/DebugLib.h>
>  #include <Library/BaseMemoryLib.h>
>  #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
> 
>  #include <PlatformArch.h>
>  #include <Library/OemMiscLib.h>
> diff --git
> a/Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c
> b/Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c
> index ac013ca30890..ad692cb6f116 100644
> --- a/Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c
> +++ b/Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c
> @@ -9,7 +9,7 @@
> 
>  #include <PlatformArch.h>
>  #include <Uefi.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Library/BaseMemoryLib.h>
>  #include <Library/DebugLib.h>
>  #include <Library/HiiLib.h>
> diff --git
> a/Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c
> b/Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c
> index 980eabfe3355..d446f2f01056 100644
> --- a/Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c
> +++ b/Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c
> @@ -8,7 +8,7 @@
>  **/
> 
>  #include <Uefi.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Library/BaseMemoryLib.h>
>  #include <Library/DebugLib.h>
>  #include <Library/HiiLib.h>
> diff --git
> a/Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c
> b/Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c
> index 5a597d86da04..b32389859121 100644
> ---
> a/Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c
> +++
> b/Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords
> +++ .c
> @@ -24,7 +24,7 @@ Revision History:
>  #include <Protocol/MpService.h>
>  #include <Protocol/PchPlatformPolicy.h>  #include <Protocol/Smbios.h> -
> #include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Library/IoLib.h>
>  #include <Guid/GlobalVariable.h>
>  #include <Register/Cpuid.h>
> diff --git
> a/Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> b/Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> index 0d0132245aa7..d3fd4b26a536 100644
> ---
> a/Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> +++
> b/Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> @@ -34,11 +34,11 @@
>   **/
> 
>  #include <Base.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <IndustryStandard/RpiMbox.h>
>  #include <Protocol/Smbios.h>
>  #include <Protocol/RpiFirmware.h>
> -#include <Guid/SmBios.h>
> +#include <Guid/Smbios.h>
>  #include <Library/ArmLib.h>
>  #include <Library/DebugLib.h>
>  #include <Library/UefiDriverEntryPoint.h> diff --git
> a/Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformD
> xe.c
> b/Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformD
> xe.c
> index 6227b7787707..38f886a2c04d 100644
> ---
> a/Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformD
> xe.c
> +++
> b/Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformD
> x
> +++ e.c
> @@ -10,7 +10,7 @@
>  **/
> 
>  #include <PiDxe.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Library/BaseLib.h>
>  #include <Library/BaseMemoryLib.h>
>  #include <Library/DebugLib.h>
> diff --git
> a/Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> b/Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> index aa0e3ff1470e..c4b38f9c47be 100644
> --- a/Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> +++ b/Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> @@ -28,8 +28,8 @@
>   *----------------------------------------------------------------------------------------
>   */
>  #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> -#include <Guid/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
> +#include <Guid/Smbios.h>
>  #include <Library/DebugLib.h>
>  #include <Library/UefiDriverEntryPoint.h>  #include <Library/UefiLib.h> diff
> --git
> a/Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDat
> aHobLib.c
> b/Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDat
> aHobLib.c
> index bcb77eb12209..496f3fef5b09 100644
> ---
> a/Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDat
> aHobLib.c
> +++ b/Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbio
> +++ sDataHobLib.c
> @@ -11,7 +11,7 @@
>    INVALID URI REMOVED
> 3A__www.dmtf.org_sites_default_files_standards_documents_DSP0134-
> 5F3.0.0.pdf&d=DwIDAg&c=C5b8zRQO1miGmBeVZ2LFWg&r=_SN6FZBN4Vgi4
> Ulkskz6qU3NYRO03nHp9P7Z5q59A3E&m=b9acthSLb5ZJA8bE-
> 9QgbS1tYFKr8ukhiAW3JkqgjVc&s=vwGkkgIBRJEoMJc1DUhEwH84iDxcYWGpF
> 7xf-E4dRTU&e=
> 
>  **/
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>  #include <Library/UefiLib.h>
>  #include <Library/BaseLib.h>
>  #include <Library/BaseMemoryLib.h>
> diff --git a/Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> b/Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> index 2ecaec2af515..a419d5ce1fa6 100644
> --- a/Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> +++ b/Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> @@ -18,7 +18,7 @@
> 
>  #include <Protocol/Smbios.h>
> 
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
> 
>  //
>  // SMBIOS tables often reference each other using
> --
> 2.26.2


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
  2021-01-11  3:31 [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h' Rebecca Cran
  2021-01-11  3:31 ` [edk2-platforms PATCH 1/1] " Rebecca Cran
@ 2021-01-11  4:42 ` Michael D Kinney
  2021-01-11  5:11   ` 回复: [edk2-devel] " gaoliming
                     ` (2 more replies)
  1 sibling, 3 replies; 10+ messages in thread
From: Michael D Kinney @ 2021-01-11  4:42 UTC (permalink / raw)
  To: Rebecca Cran, devel@edk2.groups.io, Kinney, Michael D
  Cc: Chaganty, Rangasai V, Liming Gao, Steele, Kelly, Agyeman, Prince,
	Sun, Zailiang, Qian, Yi, Leif Lindholm, Ard Biesheuvel, Wenyi Xie,
	Chiu, Chasel, Ni, Ray, Abner Chang, Daniel Schaefer, Gilbert Chen,
	Thomas Abraham, Pete Batard, Marcin Wojtas

Hi Rebecca,

Where is the review of the change to rename SmBios.h to Smbios.h in MdePkg?

I do not think that change is a good idea because it would impact every downstream platform
that produces SMBIOS records.

Mike

> -----Original Message-----
> From: Rebecca Cran <rebecca@nuviainc.com>
> Sent: Sunday, January 10, 2021 7:31 PM
> To: devel@edk2.groups.io
> Cc: Rebecca Cran <rebecca@nuviainc.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Liming Gao
> <gaoliming@byosoft.com.cn>; Kinney, Michael D <michael.d.kinney@intel.com>; Steele, Kelly <kelly.steele@intel.com>;
> Agyeman, Prince <prince.agyeman@intel.com>; Sun, Zailiang <zailiang.sun@intel.com>; Qian, Yi <yi.qian@intel.com>; Leif
> Lindholm <leif@nuviainc.com>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Wenyi Xie <xiewenyi2@huawei.com>; Chiu, Chasel
> <chasel.chiu@intel.com>; Ni, Ray <ray.ni@intel.com>; Abner Chang <abner.chang@hpe.com>; Daniel Schaefer
> <daniel.schaefer@hpe.com>; Gilbert Chen <gilbert.chen@hpe.com>; Thomas Abraham <thomas.abraham@arm.com>; Pete Batard
> <pete@akeo.ie>; Marcin Wojtas <mw@semihalf.com>
> Subject: [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> 
> Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> 
> This patch depends on the corresponding edk2 change with subject
> "Various packages: rename SmBios.h to Smbios.h in MdePkg and update references through edk2 tree".
> 
> Rebecca Cran (1):
>   Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> 
>  Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec               | 10 +++++-----
>  Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h       |  2 +-
>  Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h         |  2 +-
>  Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h            |  2 +-
>  Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h             |  4 ++--
>  Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h       |  4 ++--
>  Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h              |  2 +-
>  Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h                        |  2 +-
>  Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h                     |  2 +-
>  Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h                  |  2 +-
>  Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h            |  2 +-
>  Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h                          |  2 +-
>  Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h |  2 +-
>  Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h         |  2 +-
>  Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h   |  2 +-
>  Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h                       |  2 +-
>  Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h                    |  2 +-
>  Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c                           |  2 +-
>  Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c                   |  2 +-
>  Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c                       |  2 +-
>  Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c                       |  2 +-
>  Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c                 |  2 +-
>  Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                   |  4 ++--
>  Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c                |  2 +-
>  Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                       |  4 ++--
>  Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c      |  2 +-
>  Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c                        |  2 +-
>  27 files changed, 35 insertions(+), 35 deletions(-)
> 
> --
> 2.26.2


^ permalink raw reply	[flat|nested] 10+ messages in thread

* 回复: [edk2-devel] [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
  2021-01-11  4:42 ` [edk2-platforms PATCH 0/1] " Michael D Kinney
@ 2021-01-11  5:11   ` gaoliming
  2021-01-11  6:02   ` Rebecca Cran
  2021-01-11 11:47   ` Leif Lindholm
  2 siblings, 0 replies; 10+ messages in thread
From: gaoliming @ 2021-01-11  5:11 UTC (permalink / raw)
  To: devel, michael.d.kinney, 'Rebecca Cran'
  Cc: 'Chaganty, Rangasai V', 'Steele, Kelly',
	'Agyeman, Prince', 'Sun, Zailiang',
	'Qian, Yi', 'Leif Lindholm',
	'Ard Biesheuvel', 'Wenyi Xie',
	'Chiu, Chasel', 'Ni, Ray', 'Abner Chang',
	'Daniel Schaefer', 'Gilbert Chen',
	'Thomas Abraham', 'Pete Batard',
	'Marcin Wojtas'

Mike:
  I see the discussion about renaming file name in https://edk2.groups.io/g/devel/message/70095. 

  I also think this change impact is big. I suggest to discuss it more before we make the change. 

Thanks
Liming
> -----邮件原件-----
> 发件人: bounce+27952+70102+4905953+8761045@groups.io
> <bounce+27952+70102+4905953+8761045@groups.io> 代表 Michael D
> Kinney
> 发送时间: 2021年1月11日 12:43
> 收件人: Rebecca Cran <rebecca@nuviainc.com>; devel@edk2.groups.io;
> Kinney, Michael D <michael.d.kinney@intel.com>
> 抄送: Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Liming Gao
> <gaoliming@byosoft.com.cn>; Steele, Kelly <kelly.steele@intel.com>;
> Agyeman, Prince <prince.agyeman@intel.com>; Sun, Zailiang
> <zailiang.sun@intel.com>; Qian, Yi <yi.qian@intel.com>; Leif Lindholm
> <leif@nuviainc.com>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Wenyi
> Xie <xiewenyi2@huawei.com>; Chiu, Chasel <chasel.chiu@intel.com>; Ni,
> Ray <ray.ni@intel.com>; Abner Chang <abner.chang@hpe.com>; Daniel
> Schaefer <daniel.schaefer@hpe.com>; Gilbert Chen <gilbert.chen@hpe.com>;
> Thomas Abraham <thomas.abraham@arm.com>; Pete Batard
> <pete@akeo.ie>; Marcin Wojtas <mw@semihalf.com>
> 主题: Re: [edk2-devel] [edk2-platforms PATCH 0/1] Update SMBIOS includes
> to use capitalization 'Smbios.h' vs 'SmBios.h'
> 
> Hi Rebecca,
> 
> Where is the review of the change to rename SmBios.h to Smbios.h in
> MdePkg?
> 
> I do not think that change is a good idea because it would impact every
> downstream platform
> that produces SMBIOS records.
> 
> Mike
> 
> > -----Original Message-----
> > From: Rebecca Cran <rebecca@nuviainc.com>
> > Sent: Sunday, January 10, 2021 7:31 PM
> > To: devel@edk2.groups.io
> > Cc: Rebecca Cran <rebecca@nuviainc.com>; Chaganty, Rangasai V
> <rangasai.v.chaganty@intel.com>; Liming Gao
> > <gaoliming@byosoft.com.cn>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Steele, Kelly <kelly.steele@intel.com>;
> > Agyeman, Prince <prince.agyeman@intel.com>; Sun, Zailiang
> <zailiang.sun@intel.com>; Qian, Yi <yi.qian@intel.com>; Leif
> > Lindholm <leif@nuviainc.com>; Ard Biesheuvel
> <ardb+tianocore@kernel.org>; Wenyi Xie <xiewenyi2@huawei.com>; Chiu,
> Chasel
> > <chasel.chiu@intel.com>; Ni, Ray <ray.ni@intel.com>; Abner Chang
> <abner.chang@hpe.com>; Daniel Schaefer
> > <daniel.schaefer@hpe.com>; Gilbert Chen <gilbert.chen@hpe.com>;
> Thomas Abraham <thomas.abraham@arm.com>; Pete Batard
> > <pete@akeo.ie>; Marcin Wojtas <mw@semihalf.com>
> > Subject: [edk2-platforms PATCH 0/1] Update SMBIOS includes to use
> capitalization 'Smbios.h' vs 'SmBios.h'
> >
> > Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> >
> > This patch depends on the corresponding edk2 change with subject
> > "Various packages: rename SmBios.h to Smbios.h in MdePkg and update
> references through edk2 tree".
> >
> > Rebecca Cran (1):
> >   Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> >
> >
> Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> | 10 +++++-----
> >
> Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/Smbio
> sBasic.h       |  2 +-
> >
> Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySu
> bClass.h         |  2 +-
> >
> Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHea
> der.h            |  2 +-
> >
> Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHoo
> k.h             |  4 ++--
> >
> Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPla
> tform.h       |  4 ++--
> >
> Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.
> h              |  2 +-
> >  Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h
> |  2 +-
> >  Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h
> |  2 +-
> >  Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h
> |  2 +-
> >
> Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h
> |  2 +-
> >  Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
> |  2 +-
> >
> Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyL
> ibrary.h |  2 +-
> >
> Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h
> |  2 +-
> >
> Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLib
> rary.h   |  2 +-
> >  Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h
> |  2 +-
> >  Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h
> |  2 +-
> >  Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> |  2 +-
> >  Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c
> |  2 +-
> >  Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c
> |  2 +-
> >  Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c
> |  2 +-
> >  Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c
> |  2 +-
> >  Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> |  4 ++--
> >
> Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> |  2 +-
> >  Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> |  4 ++--
> >
> Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHo
> bLib.c      |  2 +-
> >  Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> |  2 +-
> >  27 files changed, 35 insertions(+), 35 deletions(-)
> >
> > --
> > 2.26.2
> 
> 
> 
> 
> 




^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
  2021-01-11  4:42 ` [edk2-platforms PATCH 0/1] " Michael D Kinney
  2021-01-11  5:11   ` 回复: [edk2-devel] " gaoliming
@ 2021-01-11  6:02   ` Rebecca Cran
  2021-01-11 11:47   ` Leif Lindholm
  2 siblings, 0 replies; 10+ messages in thread
From: Rebecca Cran @ 2021-01-11  6:02 UTC (permalink / raw)
  To: Kinney, Michael D, devel@edk2.groups.io
  Cc: Chaganty, Rangasai V, Liming Gao, Steele, Kelly, Agyeman, Prince,
	Sun, Zailiang, Qian, Yi, Leif Lindholm, Ard Biesheuvel, Wenyi Xie,
	Chiu, Chasel, Ni, Ray, Abner Chang, Daniel Schaefer, Gilbert Chen,
	Thomas Abraham, Pete Batard, Marcin Wojtas

On 1/10/21 9:42 PM, Kinney, Michael D wrote:
> Where is the review of the change to rename SmBios.h to Smbios.h in MdePkg?

I've sent it, but it's got stalled. It should get delivered sometime 
tomorrow.

> I do not think that change is a good idea because it would impact every downstream platform
> that produces SMBIOS records.

Okay. I wouldn't mind too much abandoning the change and sticking with 
"SmBios" for my ArmPkg patch series then.

-- 
Rebecca Cran

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
  2021-01-11  4:42 ` [edk2-platforms PATCH 0/1] " Michael D Kinney
  2021-01-11  5:11   ` 回复: [edk2-devel] " gaoliming
  2021-01-11  6:02   ` Rebecca Cran
@ 2021-01-11 11:47   ` Leif Lindholm
  2021-01-11 22:28     ` Michael D Kinney
  2 siblings, 1 reply; 10+ messages in thread
From: Leif Lindholm @ 2021-01-11 11:47 UTC (permalink / raw)
  To: Kinney, Michael D
  Cc: Rebecca Cran, devel@edk2.groups.io, Chaganty, Rangasai V,
	Liming Gao, Steele, Kelly, Agyeman, Prince, Sun, Zailiang,
	Qian, Yi, Ard Biesheuvel, Wenyi Xie, Chiu, Chasel, Ni, Ray,
	Abner Chang, Daniel Schaefer, Gilbert Chen, Thomas Abraham,
	Pete Batard, Marcin Wojtas

On Mon, Jan 11, 2021 at 04:42:49 +0000, Kinney, Michael D wrote:
> Hi Rebecca,
> 
> Where is the review of the change to rename SmBios.h to Smbios.h in MdePkg?
> 
> I do not think that change is a good idea because it would impact
> every downstream platform that produces SMBIOS records.

If those platforms are tracking master, this sort of thing should be
expected.

If they are jumping between stable tags, they are likely to have more
things than this to worry about on each rebase.

If they are "randomly" cherry-picking patches, they will already be
used to fixing up much worse things like this.

Not to mention that platforms that build only on Windows and Os X are
unlikely to even notice due to case insensitive filesystems. And in my
experience, if they ever move to build on a case-sensitive filesystem,
they will as a result find many other case typos to fix.

As Sean said:
"If a platform dev model can't handle this type of breaking change
then this project is paralyzed and bigger things should be addressed."

/
    Leif

> 
> Mike
> 
> > -----Original Message-----
> > From: Rebecca Cran <rebecca@nuviainc.com>
> > Sent: Sunday, January 10, 2021 7:31 PM
> > To: devel@edk2.groups.io
> > Cc: Rebecca Cran <rebecca@nuviainc.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Liming Gao
> > <gaoliming@byosoft.com.cn>; Kinney, Michael D <michael.d.kinney@intel.com>; Steele, Kelly <kelly.steele@intel.com>;
> > Agyeman, Prince <prince.agyeman@intel.com>; Sun, Zailiang <zailiang.sun@intel.com>; Qian, Yi <yi.qian@intel.com>; Leif
> > Lindholm <leif@nuviainc.com>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Wenyi Xie <xiewenyi2@huawei.com>; Chiu, Chasel
> > <chasel.chiu@intel.com>; Ni, Ray <ray.ni@intel.com>; Abner Chang <abner.chang@hpe.com>; Daniel Schaefer
> > <daniel.schaefer@hpe.com>; Gilbert Chen <gilbert.chen@hpe.com>; Thomas Abraham <thomas.abraham@arm.com>; Pete Batard
> > <pete@akeo.ie>; Marcin Wojtas <mw@semihalf.com>
> > Subject: [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> > 
> > Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> > 
> > This patch depends on the corresponding edk2 change with subject
> > "Various packages: rename SmBios.h to Smbios.h in MdePkg and update references through edk2 tree".
> > 
> > Rebecca Cran (1):
> >   Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> > 
> >  Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec               | 10 +++++-----
> >  Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h       |  2 +-
> >  Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h         |  2 +-
> >  Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h            |  2 +-
> >  Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h             |  4 ++--
> >  Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h       |  4 ++--
> >  Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h              |  2 +-
> >  Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h                        |  2 +-
> >  Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h                     |  2 +-
> >  Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h                  |  2 +-
> >  Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h            |  2 +-
> >  Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h                          |  2 +-
> >  Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h |  2 +-
> >  Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h         |  2 +-
> >  Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h   |  2 +-
> >  Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h                       |  2 +-
> >  Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h                    |  2 +-
> >  Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c                           |  2 +-
> >  Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c                   |  2 +-
> >  Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c                       |  2 +-
> >  Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c                       |  2 +-
> >  Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c                 |  2 +-
> >  Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                   |  4 ++--
> >  Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c                |  2 +-
> >  Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                       |  4 ++--
> >  Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c      |  2 +-
> >  Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c                        |  2 +-
> >  27 files changed, 35 insertions(+), 35 deletions(-)
> > 
> > --
> > 2.26.2
> 

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-platforms PATCH 1/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
  2021-01-11  3:31 ` [edk2-platforms PATCH 1/1] " Rebecca Cran
  2021-01-11  3:37   ` Abner Chang
@ 2021-01-11 12:35   ` Pete Batard
  1 sibling, 0 replies; 10+ messages in thread
From: Pete Batard @ 2021-01-11 12:35 UTC (permalink / raw)
  To: Rebecca Cran, devel
  Cc: Sai Chaganty, Liming Gao, Michael D Kinney, Kelly Steele,
	Agyeman Prince, Zailiang Sun, Yi Qian, Leif Lindholm,
	Ard Biesheuvel, Wenyi Xie, Chasel Chiu, Ray Ni, Abner Chang,
	Daniel Schaefer, Gilbert Chen, Thomas Abraham, Marcin Wojtas

For Raspberry Pi,
Reviewed-by: Pete Batard <pete@akeo.ie>

On 2021.01.11 03:31, Rebecca Cran wrote:
> A corresponding edk2 changeset renamed SmBios.h to Smbios.h in
> MdePkg/Include/IndustryStandard and MdePkg/Include/Guid. Update all
> references through the edk2-platform tree to follow the change.
> 
> Signed-off-by: Rebecca Cran <rebecca@nuviainc.com>
> ---
>   Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec               | 10 +++++-----
>   Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h       |  2 +-
>   Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h         |  2 +-
>   Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h            |  2 +-
>   Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h             |  4 ++--
>   Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h       |  4 ++--
>   Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h              |  2 +-
>   Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h                        |  2 +-
>   Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h                     |  2 +-
>   Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h                  |  2 +-
>   Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h            |  2 +-
>   Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h                          |  2 +-
>   Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h |  2 +-
>   Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h         |  2 +-
>   Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h   |  2 +-
>   Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h                       |  2 +-
>   Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h                    |  2 +-
>   Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c                           |  2 +-
>   Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c                   |  2 +-
>   Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c                       |  2 +-
>   Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c                       |  2 +-
>   Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c                 |  2 +-
>   Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                   |  4 ++--
>   Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c                |  2 +-
>   Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                       |  4 ++--
>   Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c      |  2 +-
>   Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c                        |  2 +-
>   27 files changed, 35 insertions(+), 35 deletions(-)
> 
> diff --git a/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec b/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> index 6170f379691a..4b6442ed0ebf 100644
> --- a/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> +++ b/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> @@ -35,7 +35,7 @@ [PcdsDynamic, PcdsDynamicEx]
>     #
>     gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType0BiosInformation|{0x0}|SMBIOS_TABLE_TYPE0|0xD0000001 {
>       <HeaderFiles>
> -      IndustryStandard/SmBios.h
> +      IndustryStandard/Smbios.h
>       <Packages>
>         MdePkg/MdePkg.dec
>         SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> @@ -68,7 +68,7 @@ [PcdsDynamic, PcdsDynamicEx]
>     #
>     gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType1SystemInformation|{0x0}|SMBIOS_TABLE_TYPE1|0xD0000002 {
>       <HeaderFiles>
> -      IndustryStandard/SmBios.h
> +      IndustryStandard/Smbios.h
>       <Packages>
>         MdePkg/MdePkg.dec
>         SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> @@ -84,7 +84,7 @@ [PcdsDynamic, PcdsDynamicEx]
>   
>     gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType2BaseBoardInformation|{0x0}|SMBIOS_TABLE_TYPE2|0xD0000003 {
>       <HeaderFiles>
> -      IndustryStandard/SmBios.h
> +      IndustryStandard/Smbios.h
>       <Packages>
>         MdePkg/MdePkg.dec
>         SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> @@ -108,7 +108,7 @@ [PcdsDynamic, PcdsDynamicEx]
>     #
>     gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType3SystemEnclosureChassis|{0x0}|SMBIOS_TABLE_TYPE3|0xD0000004 {
>       <HeaderFiles>
> -      IndustryStandard/SmBios.h
> +      IndustryStandard/Smbios.h
>       <Packages>
>         MdePkg/MdePkg.dec
>         SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> @@ -128,7 +128,7 @@ [PcdsDynamic, PcdsDynamicEx]
>     #
>     gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosType32SystemBootInformation|{0x0}|SMBIOS_TABLE_TYPE32|0xD0000005 {
>       <HeaderFiles>
> -      IndustryStandard/SmBios.h
> +      IndustryStandard/Smbios.h
>       <Packages>
>         MdePkg/MdePkg.dec
>         SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec
> diff --git a/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h b/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h
> index 2a349bcd3716..85ab757b4437 100644
> --- a/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h
> +++ b/Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h
> @@ -11,7 +11,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>   
>   #include <PiDxe.h>
>   #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Library/UefiBootServicesTableLib.h>
>   #include <Library/DebugLib.h>
>   #include <Library/BaseLib.h>
> diff --git a/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h b/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h
> index 5d36c872d510..9532da7b3d82 100644
> --- a/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h
> +++ b/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h
> @@ -17,7 +17,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>   //
>   // The protocols, PPI and GUID definitions for this module
>   //
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Protocol/Smbios.h>
>   #include <Protocol/PciRootBridgeIo.h>
>   #include <Protocol/SmbusHc.h>
> diff --git a/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h b/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h
> index ab7537043525..b913ccc52ab5 100644
> --- a/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h
> +++ b/Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h
> @@ -14,7 +14,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>   
>   
>   #include <PiDxe.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Protocol/Smbios.h>
>   #include <Guid/MdeModuleHii.h>
>   #include <DataHubRecords.h>
> diff --git a/Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h b/Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h
> index 566b6a48c65d..0b3777cf3803 100644
> --- a/Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h
> +++ b/Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h
> @@ -14,7 +14,7 @@
>   
>   #include <IndustryStandard/Pci.h>
>   #include <IndustryStandard/Acpi.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <IndustryStandard/PeImage.h>
>   
>   #include <Library/DebugLib.h>
> @@ -45,7 +45,7 @@
>   #include <Protocol/LoadedImage.h>
>   
>   #include <Guid/Acpi.h>
> -#include <Guid/SmBios.h>
> +#include <Guid/Smbios.h>
>   #include <Guid/Mps.h>
>   #include <Guid/HobList.h>
>   #include <Guid/GlobalVariable.h>
> diff --git a/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h b/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h
> index 01ba20d2dad7..ccbaf31449d5 100644
> --- a/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h
> +++ b/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h
> @@ -14,7 +14,7 @@
>   
>   #include <IndustryStandard/Pci.h>
>   #include <IndustryStandard/Acpi.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <IndustryStandard/PeImage.h>
>   
>   #include <Library/DebugLib.h>
> @@ -44,7 +44,7 @@
>   #include <Protocol/LoadedImage.h>
>   
>   #include <Guid/Acpi.h>
> -#include <Guid/SmBios.h>
> +#include <Guid/Smbios.h>
>   #include <Guid/Mps.h>
>   #include <Guid/HobList.h>
>   #include <Guid/GlobalVariable.h>
> diff --git a/Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h b/Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h
> index ccd35e2924ff..7a8a71ad5aa5 100644
> --- a/Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h
> +++ b/Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h
> @@ -13,7 +13,7 @@
>   #include <PiDxe.h>
>   
>   #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Library/DebugLib.h>
>   #include <Library/BaseLib.h>
>   #include <Library/BaseMemoryLib.h>
> diff --git a/Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h b/Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h
> index f95e6bbc2ddd..2bce59df7785 100644
> --- a/Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h
> +++ b/Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h
> @@ -15,7 +15,7 @@
>   
>   
>   #include <PiDxe.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Protocol/Smbios.h>
>   
>   #include <Guid/MdeModuleHii.h>
> diff --git a/Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h b/Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h
> index 7194ae589042..0e0dda54d591 100644
> --- a/Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h
> +++ b/Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h
> @@ -14,7 +14,7 @@
>   #include <Uefi.h>
>   #include <PiDxe.h>
>   #include <IndustryStandard/Pci.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Protocol/Smbios.h>
>   #include <Protocol/PciIo.h>
>   #include <Protocol/PciRootBridgeIo.h>
> diff --git a/Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h b/Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h
> index 22c7f6797168..ee8440c7c126 100644
> --- a/Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h
> +++ b/Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h
> @@ -14,7 +14,7 @@
>   #include <Uefi.h>
>   #include <PiDxe.h>
>   #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Library/HiiLib.h>
>   #include <Library/DebugLib.h>
>   #include <Library/MemoryAllocationLib.h>
> diff --git a/Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h b/Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h
> index 3ce180d21df3..86c42f4920aa 100644
> --- a/Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h
> +++ b/Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h
> @@ -12,7 +12,7 @@
>   
>   #include <Uefi.h>
>   #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Library/HiiLib.h>
>   #include <Library/DebugLib.h>
>   #include <Library/MemoryAllocationLib.h>
> diff --git a/Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h b/Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
> index 4692eb71334e..f6e658c7c0df 100644
> --- a/Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
> +++ b/Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h
> @@ -20,7 +20,7 @@ Based on files under Nt32Pkg/MiscSubClassPlatformDxe/
>   #define _SMBIOS_MISC_DRIVER_H
>   
>   #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Library/HiiLib.h>
>   #include <Library/PcdLib.h>
>   #include <Library/DebugLib.h>
> diff --git a/Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h b/Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h
> index 6e993053fce7..be92c86ecff3 100644
> --- a/Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h
> +++ b/Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h
> @@ -19,7 +19,7 @@
>   #include <Ppi/MasterBootMode.h>
>   #include <Ppi/SiPolicy.h>
>   #include <Library/CpuPolicyLib.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Library/SiConfigBlockLib.h>
>   #include <Library/CpuPlatformLib.h>
>   #include <Register/Cpuid.h>
> diff --git a/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h b/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h
> index b30bc3f9e7ee..457e792c6271 100644
> --- a/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h
> +++ b/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h
> @@ -15,7 +15,7 @@
>   #ifndef __FIRMWARE_VERSION_INFO_H__
>   #define __FIRMWARE_VERSION_INFO_H__
>   
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   
>   #define INTEL_FIRMWARE_VERSION_INFO_GROUP_NAME    "Firmware Version Info"
>   
> diff --git a/Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h b/Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h
> index 23321d6432c5..a76328745f77 100644
> --- a/Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h
> +++ b/Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h
> @@ -18,7 +18,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>   #include <Ppi/MasterBootMode.h>
>   #include <Ppi/SiPolicy.h>
>   #include <Library/CpuPolicyLib.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Library/SiConfigBlockLib.h>
>   #include <Library/CpuPlatformLib.h>
>   #include <Register/Cpuid.h>
> diff --git a/Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h b/Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h
> index 2f5847e53e07..8a6b6b2f0632 100644
> --- a/Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h
> +++ b/Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h
> @@ -10,7 +10,7 @@
>   #define RISC_V_PROCESSOR_SPECIFIC_HOB_DATA_H_
>   
>   #include <Uefi.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <SmbiosProcessorSpecificData.h>
>   
>   #define TO_BE_FILLED 0
> diff --git a/Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h b/Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h
> index 81e48cd0685c..c5bbf41ece05 100644
> --- a/Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h
> +++ b/Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h
> @@ -11,7 +11,7 @@
>   #ifndef SMBIOS_RISC_V_PROCESSOR_SPECIFIC_DATA_H_
>   #define SMBIOS_RISC_V_PROCESSOR_SPECIFIC_DATA_H_
>   
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <RiscVImpl.h>
>   
>   #pragma pack(1)
> diff --git a/Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c b/Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> index ac18b16ca676..dd7ae0620792 100644
> --- a/Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> +++ b/Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> @@ -7,7 +7,7 @@
>   
>   **/
>   #include <ArmPlatform.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Library/ArmLib.h>
>   #include <Library/BaseLib.h>
>   #include <Library/BaseMemoryLib.h>
> diff --git a/Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c b/Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c
> index 1670cac1d491..28dc244284d6 100644
> --- a/Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c
> +++ b/Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c
> @@ -11,7 +11,7 @@
>   #include <Library/DebugLib.h>
>   #include <Library/BaseMemoryLib.h>
>   #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   
>   #include <PlatformArch.h>
>   #include <Library/OemMiscLib.h>
> diff --git a/Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c b/Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c
> index ac013ca30890..ad692cb6f116 100644
> --- a/Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c
> +++ b/Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c
> @@ -9,7 +9,7 @@
>   
>   #include <PlatformArch.h>
>   #include <Uefi.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Library/BaseMemoryLib.h>
>   #include <Library/DebugLib.h>
>   #include <Library/HiiLib.h>
> diff --git a/Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c b/Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c
> index 980eabfe3355..d446f2f01056 100644
> --- a/Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c
> +++ b/Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c
> @@ -8,7 +8,7 @@
>   **/
>   
>   #include <Uefi.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Library/BaseMemoryLib.h>
>   #include <Library/DebugLib.h>
>   #include <Library/HiiLib.h>
> diff --git a/Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c b/Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c
> index 5a597d86da04..b32389859121 100644
> --- a/Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c
> +++ b/Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c
> @@ -24,7 +24,7 @@ Revision History:
>   #include <Protocol/MpService.h>
>   #include <Protocol/PchPlatformPolicy.h>
>   #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Library/IoLib.h>
>   #include <Guid/GlobalVariable.h>
>   #include <Register/Cpuid.h>
> diff --git a/Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c b/Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> index 0d0132245aa7..d3fd4b26a536 100644
> --- a/Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> +++ b/Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> @@ -34,11 +34,11 @@
>    **/
>   
>   #include <Base.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <IndustryStandard/RpiMbox.h>
>   #include <Protocol/Smbios.h>
>   #include <Protocol/RpiFirmware.h>
> -#include <Guid/SmBios.h>
> +#include <Guid/Smbios.h>
>   #include <Library/ArmLib.h>
>   #include <Library/DebugLib.h>
>   #include <Library/UefiDriverEntryPoint.h>
> diff --git a/Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c b/Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> index 6227b7787707..38f886a2c04d 100644
> --- a/Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> +++ b/Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> @@ -10,7 +10,7 @@
>   **/
>   
>   #include <PiDxe.h>
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Library/BaseLib.h>
>   #include <Library/BaseMemoryLib.h>
>   #include <Library/DebugLib.h>
> diff --git a/Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c b/Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> index aa0e3ff1470e..c4b38f9c47be 100644
> --- a/Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> +++ b/Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c
> @@ -28,8 +28,8 @@
>    *----------------------------------------------------------------------------------------
>    */
>   #include <Protocol/Smbios.h>
> -#include <IndustryStandard/SmBios.h>
> -#include <Guid/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
> +#include <Guid/Smbios.h>
>   #include <Library/DebugLib.h>
>   #include <Library/UefiDriverEntryPoint.h>
>   #include <Library/UefiLib.h>
> diff --git a/Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c b/Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c
> index bcb77eb12209..496f3fef5b09 100644
> --- a/Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c
> +++ b/Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c
> @@ -11,7 +11,7 @@
>     http://www.dmtf.org/sites/default/files/standards/documents/DSP0134_3.0.0.pdf
>   
>   **/
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   #include <Library/UefiLib.h>
>   #include <Library/BaseLib.h>
>   #include <Library/BaseMemoryLib.h>
> diff --git a/Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c b/Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> index 2ecaec2af515..a419d5ce1fa6 100644
> --- a/Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> +++ b/Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> @@ -18,7 +18,7 @@
>   
>   #include <Protocol/Smbios.h>
>   
> -#include <IndustryStandard/SmBios.h>
> +#include <IndustryStandard/Smbios.h>
>   
>   //
>   // SMBIOS tables often reference each other using
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
  2021-01-11 11:47   ` Leif Lindholm
@ 2021-01-11 22:28     ` Michael D Kinney
  2021-01-12 18:35       ` Leif Lindholm
  0 siblings, 1 reply; 10+ messages in thread
From: Michael D Kinney @ 2021-01-11 22:28 UTC (permalink / raw)
  To: Leif Lindholm, Kinney, Michael D
  Cc: Rebecca Cran, devel@edk2.groups.io, Chaganty, Rangasai V,
	Liming Gao, Steele, Kelly, Agyeman, Prince, Sun, Zailiang,
	Qian, Yi, Ard Biesheuvel, Wenyi Xie, Chiu, Chasel, Ni, Ray,
	Abner Chang, Daniel Schaefer, Gilbert Chen, Thomas Abraham,
	Pete Batard, Marcin Wojtas

Leif,

I would like to see this proposed filename case change split out.

I am ok with functional changes to follow the SMBIOS Spec.

I am also ok if the SMBIOS Spec decided to make a non-backwards
compatible change and we have to follow the change.

I am also ok if a mistake is made in the translation of 
SMBIOS spec content to EDK II .h files.  We need to fix
those.

The filename case change does not fit into any of the
above that are required to follow specs required for
FW functionality.  Changing the case or name of any
file path in the tree is not related to FW functionality
and needs to be discussed and reviewed on its own.

Since this is about consistence with other usage of the
name "Smbios" or "SmBios" or "SMBios" or "SMBIOS" in
the source code, we need to discuss all the options to
make them consist and then understand what the impact is
to make them consistent.  For example, we could discuss
changing the Protocol/Guid file name instead of the 
IndustryStandard filename.  Or just leaving it alone.

I do not want to see progress slowed on functional
changes for this one.  Please enter a BZ for the filename
inconsistency issue so the issue is not lost and progress
on FW functionality changes can continue.

Thanks,

Mike


> -----Original Message-----
> From: Leif Lindholm <leif@nuviainc.com>
> Sent: Monday, January 11, 2021 3:48 AM
> To: Kinney, Michael D <michael.d.kinney@intel.com>
> Cc: Rebecca Cran <rebecca@nuviainc.com>; devel@edk2.groups.io; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>;
> Liming Gao <gaoliming@byosoft.com.cn>; Steele, Kelly <kelly.steele@intel.com>; Agyeman, Prince <prince.agyeman@intel.com>;
> Sun, Zailiang <zailiang.sun@intel.com>; Qian, Yi <yi.qian@intel.com>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Wenyi
> Xie <xiewenyi2@huawei.com>; Chiu, Chasel <chasel.chiu@intel.com>; Ni, Ray <ray.ni@intel.com>; Abner Chang
> <abner.chang@hpe.com>; Daniel Schaefer <daniel.schaefer@hpe.com>; Gilbert Chen <gilbert.chen@hpe.com>; Thomas Abraham
> <thomas.abraham@arm.com>; Pete Batard <pete@akeo.ie>; Marcin Wojtas <mw@semihalf.com>
> Subject: Re: [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> 
> On Mon, Jan 11, 2021 at 04:42:49 +0000, Kinney, Michael D wrote:
> > Hi Rebecca,
> >
> > Where is the review of the change to rename SmBios.h to Smbios.h in MdePkg?
> >
> > I do not think that change is a good idea because it would impact
> > every downstream platform that produces SMBIOS records.
> 
> If those platforms are tracking master, this sort of thing should be
> expected.
> 
> If they are jumping between stable tags, they are likely to have more
> things than this to worry about on each rebase.
> 
> If they are "randomly" cherry-picking patches, they will already be
> used to fixing up much worse things like this.
> 
> Not to mention that platforms that build only on Windows and Os X are
> unlikely to even notice due to case insensitive filesystems. And in my
> experience, if they ever move to build on a case-sensitive filesystem,
> they will as a result find many other case typos to fix.
> 
> As Sean said:
> "If a platform dev model can't handle this type of breaking change
> then this project is paralyzed and bigger things should be addressed."
> 
> /
>     Leif
> 
> >
> > Mike
> >
> > > -----Original Message-----
> > > From: Rebecca Cran <rebecca@nuviainc.com>
> > > Sent: Sunday, January 10, 2021 7:31 PM
> > > To: devel@edk2.groups.io
> > > Cc: Rebecca Cran <rebecca@nuviainc.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Liming Gao
> > > <gaoliming@byosoft.com.cn>; Kinney, Michael D <michael.d.kinney@intel.com>; Steele, Kelly <kelly.steele@intel.com>;
> > > Agyeman, Prince <prince.agyeman@intel.com>; Sun, Zailiang <zailiang.sun@intel.com>; Qian, Yi <yi.qian@intel.com>; Leif
> > > Lindholm <leif@nuviainc.com>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Wenyi Xie <xiewenyi2@huawei.com>; Chiu,
> Chasel
> > > <chasel.chiu@intel.com>; Ni, Ray <ray.ni@intel.com>; Abner Chang <abner.chang@hpe.com>; Daniel Schaefer
> > > <daniel.schaefer@hpe.com>; Gilbert Chen <gilbert.chen@hpe.com>; Thomas Abraham <thomas.abraham@arm.com>; Pete Batard
> > > <pete@akeo.ie>; Marcin Wojtas <mw@semihalf.com>
> > > Subject: [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> > >
> > > Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> > >
> > > This patch depends on the corresponding edk2 change with subject
> > > "Various packages: rename SmBios.h to Smbios.h in MdePkg and update references through edk2 tree".
> > >
> > > Rebecca Cran (1):
> > >   Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> > >
> > >  Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec               | 10 +++++-----
> > >  Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h       |  2 +-
> > >  Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h         |  2 +-
> > >  Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h            |  2 +-
> > >  Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h             |  4 ++--
> > >  Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h       |  4 ++--
> > >  Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h              |  2 +-
> > >  Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h                        |  2 +-
> > >  Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h                     |  2 +-
> > >  Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h                  |  2 +-
> > >  Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h            |  2 +-
> > >  Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h                          |  2 +-
> > >  Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h |  2 +-
> > >  Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h         |  2 +-
> > >  Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h   |  2 +-
> > >  Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h                       |  2 +-
> > >  Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h                    |  2 +-
> > >  Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c                           |  2 +-
> > >  Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c                   |  2 +-
> > >  Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c                       |  2 +-
> > >  Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c                       |  2 +-
> > >  Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c                 |  2 +-
> > >  Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                   |  4 ++--
> > >  Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c                |  2 +-
> > >  Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                       |  4 ++--
> > >  Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c      |  2 +-
> > >  Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c                        |  2 +-
> > >  27 files changed, 35 insertions(+), 35 deletions(-)
> > >
> > > --
> > > 2.26.2
> >

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
  2021-01-11 22:28     ` Michael D Kinney
@ 2021-01-12 18:35       ` Leif Lindholm
  0 siblings, 0 replies; 10+ messages in thread
From: Leif Lindholm @ 2021-01-12 18:35 UTC (permalink / raw)
  To: Kinney, Michael D
  Cc: Rebecca Cran, devel@edk2.groups.io, Chaganty, Rangasai V,
	Liming Gao, Steele, Kelly, Agyeman, Prince, Sun, Zailiang,
	Qian, Yi, Ard Biesheuvel, Wenyi Xie, Chiu, Chasel, Ni, Ray,
	Abner Chang, Daniel Schaefer, Gilbert Chen, Thomas Abraham,
	Pete Batard, Marcin Wojtas

Hi Mike,

OK, that's fair. I was kicking back at the idea that renaming a header
file was too onerous and should never be done.

https://bugzilla.tianocore.org/show_bug.cgi?id=3164 raised.

Rebecca, please keep the current name SmBios.h for the upstreaming of
this library.

Best Regards,

Leif

On Mon, Jan 11, 2021 at 22:28:19 +0000, Kinney, Michael D wrote:
> Leif,
> 
> I would like to see this proposed filename case change split out.
> 
> I am ok with functional changes to follow the SMBIOS Spec.
> 
> I am also ok if the SMBIOS Spec decided to make a non-backwards
> compatible change and we have to follow the change.
> 
> I am also ok if a mistake is made in the translation of 
> SMBIOS spec content to EDK II .h files.  We need to fix
> those.
> 
> The filename case change does not fit into any of the
> above that are required to follow specs required for
> FW functionality.  Changing the case or name of any
> file path in the tree is not related to FW functionality
> and needs to be discussed and reviewed on its own.
> 
> Since this is about consistence with other usage of the
> name "Smbios" or "SmBios" or "SMBios" or "SMBIOS" in
> the source code, we need to discuss all the options to
> make them consist and then understand what the impact is
> to make them consistent.  For example, we could discuss
> changing the Protocol/Guid file name instead of the 
> IndustryStandard filename.  Or just leaving it alone.
> 
> I do not want to see progress slowed on functional
> changes for this one.  Please enter a BZ for the filename
> inconsistency issue so the issue is not lost and progress
> on FW functionality changes can continue.
> 
> Thanks,
> 
> Mike
> 
> 
> > -----Original Message-----
> > From: Leif Lindholm <leif@nuviainc.com>
> > Sent: Monday, January 11, 2021 3:48 AM
> > To: Kinney, Michael D <michael.d.kinney@intel.com>
> > Cc: Rebecca Cran <rebecca@nuviainc.com>; devel@edk2.groups.io; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>;
> > Liming Gao <gaoliming@byosoft.com.cn>; Steele, Kelly <kelly.steele@intel.com>; Agyeman, Prince <prince.agyeman@intel.com>;
> > Sun, Zailiang <zailiang.sun@intel.com>; Qian, Yi <yi.qian@intel.com>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Wenyi
> > Xie <xiewenyi2@huawei.com>; Chiu, Chasel <chasel.chiu@intel.com>; Ni, Ray <ray.ni@intel.com>; Abner Chang
> > <abner.chang@hpe.com>; Daniel Schaefer <daniel.schaefer@hpe.com>; Gilbert Chen <gilbert.chen@hpe.com>; Thomas Abraham
> > <thomas.abraham@arm.com>; Pete Batard <pete@akeo.ie>; Marcin Wojtas <mw@semihalf.com>
> > Subject: Re: [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> > 
> > On Mon, Jan 11, 2021 at 04:42:49 +0000, Kinney, Michael D wrote:
> > > Hi Rebecca,
> > >
> > > Where is the review of the change to rename SmBios.h to Smbios.h in MdePkg?
> > >
> > > I do not think that change is a good idea because it would impact
> > > every downstream platform that produces SMBIOS records.
> > 
> > If those platforms are tracking master, this sort of thing should be
> > expected.
> > 
> > If they are jumping between stable tags, they are likely to have more
> > things than this to worry about on each rebase.
> > 
> > If they are "randomly" cherry-picking patches, they will already be
> > used to fixing up much worse things like this.
> > 
> > Not to mention that platforms that build only on Windows and Os X are
> > unlikely to even notice due to case insensitive filesystems. And in my
> > experience, if they ever move to build on a case-sensitive filesystem,
> > they will as a result find many other case typos to fix.
> > 
> > As Sean said:
> > "If a platform dev model can't handle this type of breaking change
> > then this project is paralyzed and bigger things should be addressed."
> > 
> > /
> >     Leif
> > 
> > >
> > > Mike
> > >
> > > > -----Original Message-----
> > > > From: Rebecca Cran <rebecca@nuviainc.com>
> > > > Sent: Sunday, January 10, 2021 7:31 PM
> > > > To: devel@edk2.groups.io
> > > > Cc: Rebecca Cran <rebecca@nuviainc.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Liming Gao
> > > > <gaoliming@byosoft.com.cn>; Kinney, Michael D <michael.d.kinney@intel.com>; Steele, Kelly <kelly.steele@intel.com>;
> > > > Agyeman, Prince <prince.agyeman@intel.com>; Sun, Zailiang <zailiang.sun@intel.com>; Qian, Yi <yi.qian@intel.com>; Leif
> > > > Lindholm <leif@nuviainc.com>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Wenyi Xie <xiewenyi2@huawei.com>; Chiu,
> > Chasel
> > > > <chasel.chiu@intel.com>; Ni, Ray <ray.ni@intel.com>; Abner Chang <abner.chang@hpe.com>; Daniel Schaefer
> > > > <daniel.schaefer@hpe.com>; Gilbert Chen <gilbert.chen@hpe.com>; Thomas Abraham <thomas.abraham@arm.com>; Pete Batard
> > > > <pete@akeo.ie>; Marcin Wojtas <mw@semihalf.com>
> > > > Subject: [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> > > >
> > > > Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> > > >
> > > > This patch depends on the corresponding edk2 change with subject
> > > > "Various packages: rename SmBios.h to Smbios.h in MdePkg and update references through edk2 tree".
> > > >
> > > > Rebecca Cran (1):
> > > >   Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h'
> > > >
> > > >  Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec               | 10 +++++-----
> > > >  Features/Intel/SystemInformation/SmbiosFeaturePkg/SmbiosBasicDxe/SmbiosBasic.h       |  2 +-
> > > >  Platform/Intel/QuarkPlatformPkg/Platform/Dxe/MemorySubClass/MemorySubClass.h         |  2 +-
> > > >  Platform/Intel/QuarkPlatformPkg/Platform/Dxe/SmbiosMiscDxe/CommonHeader.h            |  2 +-
> > > >  Platform/Intel/SimicsOpenBoardPkg/Library/BoardBdsHookLib/BoardBdsHook.h             |  4 ++--
> > > >  Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/BdsPlatform.h       |  4 ++--
> > > >  Platform/Intel/SimicsOpenBoardPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h              |  2 +-
> > > >  Platform/Intel/Vlv2TbltDevicePkg/SmBiosMiscDxe/CommonHeader.h                        |  2 +-
> > > >  Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.h                     |  2 +-
> > > >  Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClass.h                  |  2 +-
> > > >  Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.h            |  2 +-
> > > >  Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMisc.h                          |  2 +-
> > > >  Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h |  2 +-
> > > >  Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersionInfo.h         |  2 +-
> > > >  Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/PeiCpuPolicyLibrary.h   |  2 +-
> > > >  Silicon/RISC-V/ProcessorPkg/Include/ProcessorSpecificHobData.h                       |  2 +-
> > > >  Silicon/RISC-V/ProcessorPkg/Include/SmbiosProcessorSpecificData.h                    |  2 +-
> > > >  Platform/ARM/JunoPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c                           |  2 +-
> > > >  Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c                   |  2 +-
> > > >  Platform/Hisilicon/D05/Library/OemMiscLibD05/BoardFeatureD05.c                       |  2 +-
> > > >  Platform/Hisilicon/D06/Library/OemMiscLibD06/BoardFeatureD06.c                       |  2 +-
> > > >  Platform/Intel/Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c                 |  2 +-
> > > >  Platform/RaspberryPi/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                   |  4 ++--
> > > >  Platform/Socionext/DeveloperBox/SmbiosPlatformDxe/SmbiosPlatformDxe.c                |  2 +-
> > > >  Silicon/AMD/Styx/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c                       |  4 ++--
> > > >  Silicon/Intel/IntelSiliconPkg/Library/DxeSmbiosDataHobLib/DxeSmbiosDataHobLib.c      |  2 +-
> > > >  Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c                        |  2 +-
> > > >  27 files changed, 35 insertions(+), 35 deletions(-)
> > > >
> > > > --
> > > > 2.26.2
> > >

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2021-01-12 18:35 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-01-11  3:31 [edk2-platforms PATCH 0/1] Update SMBIOS includes to use capitalization 'Smbios.h' vs 'SmBios.h' Rebecca Cran
2021-01-11  3:31 ` [edk2-platforms PATCH 1/1] " Rebecca Cran
2021-01-11  3:37   ` Abner Chang
2021-01-11 12:35   ` Pete Batard
2021-01-11  4:42 ` [edk2-platforms PATCH 0/1] " Michael D Kinney
2021-01-11  5:11   ` 回复: [edk2-devel] " gaoliming
2021-01-11  6:02   ` Rebecca Cran
2021-01-11 11:47   ` Leif Lindholm
2021-01-11 22:28     ` Michael D Kinney
2021-01-12 18:35       ` Leif Lindholm

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox