Minor update for PI include files according to code review comments.

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

diff --git a/MdePkg/Include/Pi/PiFirmwareFile.h b/MdePkg/Include/Pi/PiFirmwareFile.h
index febebe7314382052bf6021edd27de6d356de6baa..39c4def36d45e7df962587a22723e0fa0e4313be 100644 (file)
--- a/MdePkg/Include/Pi/PiFirmwareFile.h
+++ b/MdePkg/Include/Pi/PiFirmwareFile.h
@@ -156,7 +156,7 @@ typedef struct {
 typedef EFI_COMMON_SECTION_HEADER EFI_DXE_DEPEX_SECTION;\r
 \r
 ///\r
-/// Leaf section witch contains a PI FV.\r
+/// Leaf section which contains a PI FV.\r
 /// \r
 typedef EFI_COMMON_SECTION_HEADER EFI_FIRMWARE_VOLUME_IMAGE_SECTION;\r
 \r
@@ -220,7 +220,7 @@ typedef struct {
 \r
 ///\r
 /// Leaf section which contains a numeric build number and\r
-/// an optional unicode string that represent the file revision. \r
+/// an optional unicode string that represents the file revision. \r
 /// \r
 typedef struct {\r
   EFI_COMMON_SECTION_HEADER   CommonHeader;\r

diff --git a/MdePkg/Include/Pi/PiMultiPhase.h b/MdePkg/Include/Pi/PiMultiPhase.h
index b49971f8138fab2dd3dfd449d2915b107e567ef8..a88258e583c91645c5a52b737e928a1b4180169c 100644 (file)
--- a/MdePkg/Include/Pi/PiMultiPhase.h
+++ b/MdePkg/Include/Pi/PiMultiPhase.h
@@ -1,7 +1,7 @@
 /** @file\r
   Include file matches things in PI for multiple module types.\r
 \r
-  Copyright (c) 2006 - 2007, Intel Corporation                                                         \r
+  Copyright (c) 2006 - 2008, Intel Corporation                                                         \r
   All rights reserved. This program and the accompanying materials                          \r
   are licensed and made available under the terms and conditions of the BSD License         \r
   which accompanies this distribution.  The full text of the license may be found at        \r
@@ -93,7 +93,7 @@ typedef struct {
 \r
 \r
 //\r
-// Bit values for AuthenticationStatus\r
+// Bit values for Authentication Status\r
 //\r
 #define EFI_AUTH_STATUS_PLATFORM_OVERRIDE   0x01\r
 #define EFI_AUTH_STATUS_IMAGE_SIGNED        0x02\r

diff --git a/MdePkg/Include/Pi/PiPeiCis.h b/MdePkg/Include/Pi/PiPeiCis.h
index 34691a085173efa7f6534564f579995d38310314..5efa75e15089f60f70ac318a168fef8cb69207f0 100644 (file)
--- a/MdePkg/Include/Pi/PiPeiCis.h
+++ b/MdePkg/Include/Pi/PiPeiCis.h
@@ -613,7 +613,8 @@ typedef struct {
 } EFI_FV_INFO;\r
 \r
 /**\r
-   \r
+  Returns information about the specified volume.\r
+  \r
   This function returns information about a specific firmware\r
   volume, including its name, type, attributes, starting address\r
   and size.\r
@@ -637,7 +638,8 @@ EFI_STATUS
   );\r
 \r
 /**\r
-   \r
+  Register a PEIM so that it will be shadowed and called again.\r
+  \r
   This service registers a file handle so that after memory is\r
   available, the PEIM will be re-loaded into permanent memory and\r
   re-initialized. The PEIM registered this way will always be\r
@@ -648,8 +650,8 @@ EFI_STATUS
   initialized after permanent memory is installed, even the first\r
   time.\r
 \r
-  @param FileHandle   PEIM's file handle. Must be the currently\r
-                      executing PEIM.\r
+  @param FileHandle             PEIM's file handle. Must be the currently\r
+                                executing PEIM.\r
   \r
   @retval EFI_SUCCESS           The PEIM was successfully registered for\r
                                 shadowing.\r
@@ -722,6 +724,7 @@ struct _EFI_PEI_SERVICES {
   EFI_PEI_SET_MEM             SetMem;\r
   //\r
   // Status Code\r
+  //\r
   EFI_PEI_REPORT_STATUS_CODE  ReportStatusCode;\r
   //\r
   // Reset\r
@@ -736,6 +739,7 @@ struct _EFI_PEI_SERVICES {
   EFI_PEI_PCI_CFG2_PPI        *PciCfg;\r
   //\r
   // Future Installed Services\r
+  //\r
   EFI_PEI_FFS_FIND_BY_NAME    FfsFindFileByName;\r
   EFI_PEI_FFS_GET_FILE_INFO   FfsGetFileInfo;\r
   EFI_PEI_FFS_GET_VOLUME_INFO FfsGetVolumeInfo;\r
@@ -809,6 +813,7 @@ typedef struct _EFI_SEC_PEI_HAND_OFF {
 \r
 \r
 /**\r
+  Entry point of PEI Foundation.\r
 \r
   This function is the entry point for the PEI Foundation, which\r
   allows the SEC phase to pass information about the stack,\r
@@ -826,8 +831,7 @@ typedef struct _EFI_SEC_PEI_HAND_OFF {
                         information about the PEI core's\r
                         operating environment, such as the size\r
                         and location of temporary RAM, the stack\r
-                        location and the BFV location. The type\r
-                        EFI_SEC_PEI_HAND_OFF is\r
+                        location and the BFV location.\r
 \r
   @param PpiList        Points to a list of one or more PPI\r
                         descriptors to be installed initially by\r