From 4c617fa92b48cd64f0b16de8044914e49f13a43a Mon Sep 17 00:00:00 2001
From: eric_tian <eric_tian@6f19259b-4bc3-4df7-8a09-765794883524>
Date: Wed, 8 Jul 2009 04:32:40 +0000
Subject: [PATCH] 1. add comments for those fields not defined by
 MemorySubclass spec. 2. change name MemorySpeed to MemoryTypeSpeed, which
 follows spec.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@8804 6f19259b-4bc3-4df7-8a09-765794883524
---
 .../Include/Framework/DataHubRecords.h         | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/IntelFrameworkPkg/Include/Framework/DataHubRecords.h b/IntelFrameworkPkg/Include/Framework/DataHubRecords.h
index db777e58b0..38423d1507 100644
--- a/IntelFrameworkPkg/Include/Framework/DataHubRecords.h
+++ b/IntelFrameworkPkg/Include/Framework/DataHubRecords.h
@@ -997,6 +997,10 @@ typedef enum _EFI_MEMORY_FORM_FACTOR {
   EfiMemoryFormFactorRimm                     = 0x0C,
   EfiMemoryFormFactorSodimm                   = 0x0D,
   EfiMemoryFormFactorSrimm                    = 0x0E,
+  ///
+  /// Inconsistent with specification here:  
+  /// This field is NOT defined in MemSubClass spec0.9. It's introduced for SMBIOS2.6 spec.
+  ///
   EfiMemoryFormFactorFbDimm                   = 0x0F
 } EFI_MEMORY_FORM_FACTOR;
 
@@ -1019,7 +1023,15 @@ typedef enum _EFI_MEMORY_ARRAY_TYPE {
   EfiMemoryTypeSgram                          = 0x10,
   EfiMemoryTypeRdram                          = 0x11,
   EfiMemoryTypeDdr                            = 0x12,
+  ///
+  /// Inconsistent with specification here:  
+  /// This field is NOT defined in MemSubClass spec0.9. It's introduced for SMBIOS2.6 spec.
+  ///
   EfiMemoryTypeDdr2                           = 0x13,
+  ///
+  /// Inconsistent with specification here:  
+  /// This field is NOT defined in MemSubClass spec0.9. It's introduced for SMBIOS2.6 spec.
+  ///
   EfiMemoryTypeDdr2FbDimm                     = 0x14
 } EFI_MEMORY_ARRAY_TYPE;
 
@@ -1047,6 +1059,10 @@ typedef enum {
   EfiMemoryStateError        = 3,
   EfiMemoryStateAbsent       = 4,
   EfiMemoryStateDisabled     = 5,
+  ///
+  /// Inconsistent with specification here:  
+  /// This field is NOT defined in MemSubClass spec0.9. It's introduced for SMBIOS2.6 spec.
+  ///
   EfiMemoryStatePartial      = 6
 } EFI_MEMORY_STATE;
 
@@ -1126,7 +1142,7 @@ typedef struct {
   /// The memory speed in megahertz (MHz). A value of 0x00 denotes that 
   /// the speed is unknown.
   ///
-  EFI_EXP_BASE10_DATA               MemorySpeed;
+  EFI_EXP_BASE10_DATA               MemoryTypeSpeed;
   ///
   /// The memory state. 
   ///