Retire TCG platform protocol, which will be replaced by PI Security Architecture...

[mirror_edk2.git] / MdeModulePkg / Include / Protocol / GenericMemoryTest.h

diff --git a/MdeModulePkg/Include/Protocol/GenericMemoryTest.h b/MdeModulePkg/Include/Protocol/GenericMemoryTest.h
index 87959ed73a1cf8e885041dcebb47d013c2c4b072..68905159d620510c6be52845d38940e45003e7e8 100644 (file)
--- a/MdeModulePkg/Include/Protocol/GenericMemoryTest.h
+++ b/MdeModulePkg/Include/Protocol/GenericMemoryTest.h
@@ -1,5 +1,5 @@
 /** @file\r
-  This protocol defines the EFI generic memory test interfaces in Dxe phase.\r
+  This protocol defines the generic memory test interfaces in Dxe phase.\r
 \r
 Copyright (c) 2006 - 2008, Intel Corporation. <BR>\r
 All rights reserved. This program and the accompanying materials\r
@@ -20,6 +20,10 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 \r
 typedef struct _EFI_GENERIC_MEMORY_TEST_PROTOCOL  EFI_GENERIC_MEMORY_TEST_PROTOCOL;\r
 \r
+///\r
+/// Memory test coverage level\r
+/// Ignore op not test memory, Quick and Sparse op test memory quickly, Extensive op test memory detailedly.\r
+///\r
 typedef enum {\r
   IGNORE,\r
   QUICK,\r
@@ -36,11 +40,10 @@ typedef enum {
   @param  Level               The coverage level of the memory test. \r
   @param  RequireSoftECCInit  Indicate if the memory need software ECC init. \r
 \r
-  @retval EFI_SUCCESS         The generic memory test initialized correctly. \r
-  @retval EFI_NO_MEDIA        There is not any non-tested memory found, in this \r
-                              function if not any non-tesed memory found means  \r
+  @retval EFI_SUCCESS         The generic memory test is initialized correctly. \r
+  @retval EFI_NO_MEDIA        There is not any non-tested memory found, which means  \r
                               that the memory test driver have not detect any \r
-                              non-tested extended memory of current system. \r
+                              non-tested extended memory in current system. \r
 \r
 **/\r
 typedef\r
@@ -49,8 +52,7 @@ EFI_STATUS
   IN EFI_GENERIC_MEMORY_TEST_PROTOCOL          *This,\r
   IN  EXTENDMEM_COVERAGE_LEVEL                 Level,\r
   OUT BOOLEAN                                  *RequireSoftECCInit\r
-  )\r
-;\r
+  );\r
 \r
 \r
 /**\r
@@ -59,59 +61,54 @@ EFI_STATUS
   @param  This                Protocol instance pointer. \r
   @param  TestedMemorySize    Return the tested extended memory size. \r
   @param  TotalMemorySize     Return the whole system physical memory size, this  \r
-                              value may be changed if in some case some error  \r
-                              DIMMs be disabled. \r
-  @param  ErrorOut            Any time the memory error occurs, this will be \r
-                              TRUE. \r
-  @param  IfTestAbort         Indicate if the user press "ESC" to skip the memory \r
-                              test. \r
-\r
-  @retval EFI_SUCCESS         One block of memory test ok, the block size is hide \r
-                              internally. \r
-  @retval EFI_NOT_FOUND       Indicate all the non-tested memory blocks have  \r
-                              already go through. \r
-  @retval EFI_DEVICE_ERROR    Mis-compare error, and no agent can handle it\r
+                              value may be changed if some error DIMMs is disabled in some case. \r
+  @param  ErrorOut            TRUE if the memory error occurs.\r
+  @param  IfTestAbort         Indicate if the user press "ESC" to skip the memory test. \r
+\r
+  @retval EFI_SUCCESS         One block of memory pass test.\r
+  @retval EFI_NOT_FOUND       Indicate all the non-tested memory blocks have been\r
+                              already gone through.\r
+  @retval EFI_DEVICE_ERROR    Memory device error occurs and no agent can handle it.\r
 \r
 **/\r
 typedef\r
 EFI_STATUS\r
 (EFIAPI *EFI_PERFORM_MEMORY_TEST)(\r
-  IN EFI_GENERIC_MEMORY_TEST_PROTOCOL *This,\r
+  IN EFI_GENERIC_MEMORY_TEST_PROTOCOL          *This,\r
   OUT UINT64                                   *TestedMemorySize,\r
   OUT UINT64                                   *TotalMemorySize,\r
   OUT BOOLEAN                                  *ErrorOut,\r
   IN BOOLEAN                                   IfTestAbort\r
-  )\r
-;\r
+  );\r
 \r
 \r
 /**\r
-  The memory test finished.\r
+  Finish the memory test.\r
 \r
   @param  This                Protocol instance pointer. \r
 \r
-  @retval EFI_SUCCESS         Successful free all the generic memory test driver \r
-                              allocated resource and notify to platform memory \r
-                              test driver that memory test finished. \r
+  @retval EFI_SUCCESS         Success. Then free all the generic memory test driver\r
+                              allocated resource and notify to platform memory\r
+                              test driver that memory test finished.\r
 \r
 **/\r
 typedef\r
 EFI_STATUS\r
 (EFIAPI *EFI_MEMORY_TEST_FINISHED)(\r
   IN EFI_GENERIC_MEMORY_TEST_PROTOCOL *This\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Provide capability to test compatible range which used by some sepcial\r
-  driver required using memory range before BDS perform memory test.\r
+  Provide capability to test compatible range used by some sepcial\r
+  driver before BDS perform memory test.\r
 \r
   @param  This                Protocol instance pointer. \r
-  @param  StartAddress        The start address of the memory range. \r
-  @param  Length              The memory range's length. \r
+  @param  StartAddress        The start address of the compatible memory range that\r
+                              must be below 16M.\r
+  @param  Length              The compatible memory range's length. \r
   \r
   @retval EFI_SUCCESS           The compatible memory range pass the memory test. \r
-  @retval EFI_INVALID_PARAMETER The compatible memory range must be below 16M.\r
+  @retval EFI_INVALID_PARAMETER The compatible memory range are not below Low 16M.\r
 \r
 **/\r
 typedef\r
@@ -120,8 +117,7 @@ EFI_STATUS
   IN EFI_GENERIC_MEMORY_TEST_PROTOCOL          *This,\r
   IN  EFI_PHYSICAL_ADDRESS                     StartAddress,\r
   IN  UINT64                                   Length\r
-  )\r
-;\r
+  );\r
 \r
 struct _EFI_GENERIC_MEMORY_TEST_PROTOCOL {\r
   EFI_MEMORY_TEST_INIT              MemoryTestInit;\r