comments refine

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@8978 6f19259b-4bc3-4df7-8a09-765794883524

diff --git a/IntelFrameworkPkg/Include/Guid/DataHubRecords.h b/IntelFrameworkPkg/Include/Guid/DataHubRecords.h
index cda861a4fbc73e694c501bf825d19abfd4616953..10d33ec9204863272fb593babedcc9f6153e1fac 100644 (file)
--- a/IntelFrameworkPkg/Include/Guid/DataHubRecords.h
+++ b/IntelFrameworkPkg/Include/Guid/DataHubRecords.h
@@ -46,7 +46,7 @@ extern  EFI_GUID  gEfiMiscSubClassGuid;
 \r
 ///\r
 /// Inconsistent with specification here:  \r
-/// In MiscSubclass spec 0.9, the value is 0x0100.\r
+/// In ProcSubclass spec 0.9, the value is 0x0100.\r
 /// Keep it unchanged from the perspective of binary consistency.\r
 ///\r
 #define EFI_PROCESSOR_SUBCLASS_VERSION    0x00010000\r
@@ -696,7 +696,7 @@ typedef STRING_REF                  EFI_PROCESSOR_SOCKET_NAME_DATA;
 \r
 ///\r
 /// Inconsistent with specification here:  \r
-/// In MiscSubclass spec 0.9, the naming is EFI_PROCESSOR_CACHE_ASSOCIATION_DATA.\r
+/// In ProcSubclass spec 0.9, the naming is EFI_PROCESSOR_CACHE_ASSOCIATION_DATA.\r
 /// Keep it unchanged for backward compatibilty.\r
 ///\r
 typedef EFI_INTER_LINK_DATA         EFI_CACHE_ASSOCIATION_DATA;\r
@@ -705,7 +705,7 @@ typedef EFI_INTER_LINK_DATA         EFI_CACHE_ASSOCIATION_DATA;
 /// This data record refers to the health status of the processor. \r
 ///\r
 /// Inconsistent with specification here:  \r
-/// In MiscSubclass spec 0.9, the naming is EFI_PROCESSOR_HEALTH_STATUS_DATA.\r
+/// In ProcSubclass spec 0.9, the naming is EFI_PROCESSOR_HEALTH_STATUS_DATA.\r
 /// Keep it unchanged for backward compatibilty.\r
 ///\r
 typedef enum {\r
@@ -819,7 +819,7 @@ typedef struct {
 typedef EFI_EXP_BASE2_DATA          EFI_CACHE_SIZE_DATA;\r
 ///\r
 /// Inconsistent with specification here:  \r
-/// In MiscSubclass spec 0.9, the naming is EFI_CACHE_MAXIMUM_SIZE_DATA.\r
+/// In CacheSubclass spec 0.9, the naming is EFI_CACHE_MAXIMUM_SIZE_DATA.\r
 /// Keep it unchanged for backward compatibilty.\r
 ///\r
 typedef EFI_EXP_BASE2_DATA          EFI_MAXIMUM_CACHE_SIZE_DATA;\r
@@ -909,7 +909,22 @@ typedef enum {
 } EFI_CACHE_OPERATIONAL_MODE;\r
 \r
 \r
-\r
+///\r
+/// Inconsistent with specification here:\r
+/// In CacheSubclass spec 0.9, the enumeration type data structure is NOT defined.\r
+/// The equivalent in spec is \r
+///      #define EFI_CACHE_SIZE_RECORD_NUMBER                    0x00000001\r
+///      #define EFI_CACHE_MAXIMUM_SIZE_RECORD_NUMBER            0x00000002\r
+///      #define EFI_CACHE_SPEED_RECORD_NUMBER                   0x00000003\r
+///      #define EFI_CACHE_SOCKET_RECORD_NUMBER                  0x00000004\r
+///      #define EFI_CACHE_SRAM_SUPPORT_RECORD_NUMBER            0x00000005 \r
+///      #define EFI_CACHE_SRAM_INSTALL_RECORD_NUMBER            0x00000006 \r
+///      #define EFI_CACHE_ERROR_SUPPORT_RECORD_NUMBER           0x00000007\r
+///      #define EFI_CACHE_TYPE_RECORD_NUMBER                    0x00000008\r
+///      #define EFI_CACHE_ASSOCIATIVITY_RECORD_NUMBER           0x00000009\r
+///      #define EFI_CACHE_CONFIGURATION_RECORD_NUMBER           0x0000000A\r
+/// Keep the definition unchanged for backward compatibility.\r
+///\r
 typedef enum {\r
   CacheSizeRecordType              = 1,\r
   MaximumSizeCacheRecordType       = 2,\r
@@ -1858,7 +1873,7 @@ typedef enum {
 typedef struct {\r
   ///\r
   /// Inconsistent with specification here:  \r
-  /// In MiscSubclass 0.9 spec. It have a wrong field name "EFI_MISC_CHASSIS_TYPE".\r
+  /// In MiscSubclass 0.9 spec, it has a wrong field name "EFI_MISC_CHASSIS_TYPE".\r
   /// Change it to "ChassisType" to pass build.\r
   ///\r
   UINT32                            ChassisType       :16;\r
@@ -2639,7 +2654,7 @@ typedef struct {
 \r
 ///\r
 /// Inconsistent with specification here:  \r
-/// In MiscSubclass spec 0.9, the field name is EFI_MISC_POWER_SUPPLY_UNIT_GROUP_DATA.\r
+/// In MiscSubclass spec 0.9, the structure name is EFI_MISC_POWER_SUPPLY_UNIT_GROUP_DATA.\r
 /// Keep it unchanged for backward compatibilty.\r
 ///\r
 typedef struct {\r