Update header file comments.

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

diff --git a/IntelFrameworkModulePkg/Include/Library/GenericBdsLib.h b/IntelFrameworkModulePkg/Include/Library/GenericBdsLib.h
index c1cc82de2bba23b8597287c11cd24860db4614cc..34393728537ef96a36f9d0ca9bdd3f36d1deb34a 100644 (file)
--- a/IntelFrameworkModulePkg/Include/Library/GenericBdsLib.h
+++ b/IntelFrameworkModulePkg/Include/Library/GenericBdsLib.h
@@ -1,7 +1,10 @@
 /** @file\r
 /** @file\r

-  Generic BDS library definition, include the data structure and function.\r
+  Generic BDS library defines general interfaces for BDS driver including:\r
+    1) BDS boot policy interface;\r
+    2) BDS boot device connect interface;\r
+    3) BDS Misc interfaces for mainting boot variable, ouput string.\r

 \r
 \r

-Copyright (c) 2004 - 2008, Intel Corporation. <BR>\r
+Copyright (c) 2004 - 2009, Intel Corporation. <BR>\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
 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

diff --git a/IntelFrameworkModulePkg/Include/Library/OemHookStatusCodeLib.h b/IntelFrameworkModulePkg/Include/Library/OemHookStatusCodeLib.h
index 5f5726ea8a36608907adce8b477b8357b29fd442..da44d822d29b5ef79862e39985682ef7593cc033 100644 (file)
--- a/IntelFrameworkModulePkg/Include/Library/OemHookStatusCodeLib.h
+++ b/IntelFrameworkModulePkg/Include/Library/OemHookStatusCodeLib.h
@@ -1,7 +1,8 @@
 /** @file\r
 /** @file\r

-  OEM hook status code library functions with no library constructor/destructor\r
+  OEM hook status code library. Platform can implement an instance to \r
+  initialize the OEM devices to report status code information.\r

 \r
 \r

-  Copyright (c) 2006, Intel Corporation\r
+  Copyright (c) 2006 - 2009, 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
   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

diff --git a/IntelFrameworkModulePkg/Include/Library/PciIncompatibleDeviceSupportLib.h b/IntelFrameworkModulePkg/Include/Library/PciIncompatibleDeviceSupportLib.h
index e29b27c65fe5f3038118e840e903689deebcd721..a518991a79af123da136a26ce03266a34e3bdecc 100644 (file)
--- a/IntelFrameworkModulePkg/Include/Library/PciIncompatibleDeviceSupportLib.h
+++ b/IntelFrameworkModulePkg/Include/Library/PciIncompatibleDeviceSupportLib.h
@@ -1,7 +1,8 @@
 /** @file\r
 /** @file\r

-  PCI Incompatible device support Libary.\r
+  PCI Incompatible device support Libary. Platform can implement an \r
+  instance to support the incompatible PCI devices.\r

 \r
 \r

-Copyright (c) 2006 - 2007, Intel Corporation                                                         \r
+Copyright (c) 2006 - 2009, 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
 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


@@ -74,8 +75,8 @@ typedef struct {
   @param  PciDeviceInfo       A pointer to PCI device information.\r
   @param  Configuration       Returned information.\r
 \r
   @param  PciDeviceInfo       A pointer to PCI device information.\r
   @param  Configuration       Returned information.\r
 \r

-  @retval returns EFI_SUCCESS if check incompatible device ok.\r
-          Otherwise return EFI_UNSUPPORTED.\r
+  @retval EFI_SUCCESS         The incompatible device is supported.\r
+  @retval EFI_UNSUPPORTED     The incompatible device is not supported.\r

 **/\r
 RETURN_STATUS\r
 EFIAPI\r
 **/\r
 RETURN_STATUS\r
 EFIAPI\r


@@ -96,8 +97,8 @@ PciResourceUpdateCheck (
   @param  Offset              The address within the PCI configuration space.\r
   @param  Configuration       Returned information.\r
 \r
   @param  Offset              The address within the PCI configuration space.\r
   @param  Configuration       Returned information.\r
 \r

-  @retval returns EFI_SUCCESS if check incompatible device ok.\r
-          Otherwise return EFI_UNSUPPORTED.\r
+  @retval EFI_SUCCESS         The incompatible device is supported.\r
+  @retval EFI_UNSUPPORTED     The incompatible device is not supported.\r

 **/\r
 RETURN_STATUS\r
 EFIAPI\r
 **/\r
 RETURN_STATUS\r
 EFIAPI\r


@@ -123,8 +124,8 @@ PciRegisterUpdateCheck (
   @param  AccessWidth         Access width needs to check incompatibility.\r
   @param  Configuration       Returned information.\r
 \r
   @param  AccessWidth         Access width needs to check incompatibility.\r
   @param  Configuration       Returned information.\r
 \r

-  @retval returns EFI_SUCCESS if check incompatible device ok.\r
-          Otherwise return EFI_UNSUPPORTED.\r
+  @retval EFI_SUCCESS         The incompatible device is supported.\r
+  @retval EFI_UNSUPPORTED     The incompatible device is not supported.\r

 **/\r
 RETURN_STATUS\r
 EFIAPI\r
 **/\r
 RETURN_STATUS\r
 EFIAPI\r

diff --git a/IntelFrameworkModulePkg/Include/Library/PlatformBdsLib.h b/IntelFrameworkModulePkg/Include/Library/PlatformBdsLib.h
index 5a85deb5fe7ea21af0bd8550f51aa3758e2873bb..2e4b0d86c80a88570f3adbcebcd8f6653e7dcb5b 100644 (file)
--- a/IntelFrameworkModulePkg/Include/Library/PlatformBdsLib.h
+++ b/IntelFrameworkModulePkg/Include/Library/PlatformBdsLib.h
@@ -1,8 +1,8 @@
 /** @file\r
 /** @file\r

-  Platform BDS library definition. Platform package can provide hook library\r
-  instances to implement platform specific behavior.\r
+  Platform BDS library definition. Platform can implement an\r
+  instances to support the platform specific behavior.\r

 \r
 \r

-Copyright (c) 2008, Intel Corporation. <BR>\r
+Copyright (c) 2008 - 2009, Intel Corporation. <BR>\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
 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

diff --git a/IntelFrameworkModulePkg/Library/GenericBdsLib/GenericBdsLib.inf b/IntelFrameworkModulePkg/Library/GenericBdsLib/GenericBdsLib.inf
index 52467067cd9c14ccb291a78c48520e404e6df61d..f2a258b4dc4576f1e04940eac7699b3d8302cff5 100644 (file)
--- a/IntelFrameworkModulePkg/Library/GenericBdsLib/GenericBdsLib.inf
+++ b/IntelFrameworkModulePkg/Library/GenericBdsLib/GenericBdsLib.inf
@@ -107,7 +107,7 @@
   gEfiUgaDrawProtocolGuid |PcdUgaConsumeSupport # PROTOCOL SOMETIMES_CONSUMES\r
   gEfiOEMBadgingProtocolGuid                    # PROTOCOL CONSUMES\r
   gEfiHiiFontProtocolGuid                       # PROTOCOL CONSUMES\r
   gEfiUgaDrawProtocolGuid |PcdUgaConsumeSupport # PROTOCOL SOMETIMES_CONSUMES\r
   gEfiOEMBadgingProtocolGuid                    # PROTOCOL CONSUMES\r
   gEfiHiiFontProtocolGuid                       # PROTOCOL CONSUMES\r

-  gEfiSecurityArchProtocolGuid\r
+  gEfiSecurityArchProtocolGuid                  # PROTOCOL CONSUMES\r

 \r
 [FeaturePcd.common]\r
   gEfiMdePkgTokenSpaceGuid.PcdUgaConsumeSupport\r
 \r
 [FeaturePcd.common]\r
   gEfiMdePkgTokenSpaceGuid.PcdUgaConsumeSupport\r

diff --git a/IntelFrameworkPkg/ChangeLog.txt b/IntelFrameworkPkg/ChangeLog.txt
index 08d46cebd25866ab1bf14c0bdd9f8d9534b2b135..3044ada56e31bcc9a3f8188c9721bf341ce9dac1 100644 (file)
--- a/IntelFrameworkPkg/ChangeLog.txt
+++ b/IntelFrameworkPkg/ChangeLog.txt
@@ -68,3 +68,11 @@ EDK_5216: Non-Compatible: qwang12
 \r
                Possible Impacts:\r
                1) Modules that may reference this definitions need to be updated.\r
 \r
                Possible Impacts:\r
                1) Modules that may reference this definitions need to be updated.\r

+EDK_8583: Non-Compatible: lgao4\r
+      Retire the following framework HII opcodes, which are not defined in FrameworkHii specification.\r
+         #define FRAMEWORK_EFI_IFR_TRUE_OP                 0x27\r
+         #define FRAMEWORK_EFI_IFR_FALSE_OP                0x28\r
+         #define FRAMEWORK_EFI_IFR_GT_OP                   0x29\r
+         #define FRAMEWORK_EFI_IFR_GE_OP                   0x2A\r
+         #define FRAMEWORK_EFI_IFR_OEM_DEFINED_OP          0x2B\r
+         
\ No newline at end of file

diff --git a/IntelFrameworkPkg/Include/Framework/FrameworkInternalFormRepresentation.h b/IntelFrameworkPkg/Include/Framework/FrameworkInternalFormRepresentation.h
index 7f7512a59f10fe1203834a00b18e6829f829dd2e..6ea3d39ef6978eb7a1620a91de30172e23fe8171 100644 (file)
--- a/IntelFrameworkPkg/Include/Framework/FrameworkInternalFormRepresentation.h
+++ b/IntelFrameworkPkg/Include/Framework/FrameworkInternalFormRepresentation.h
@@ -62,12 +62,7 @@
 #define FRAMEWORK_EFI_IFR_VARSTORE_OP             0x24\r
 #define FRAMEWORK_EFI_IFR_VARSTORE_SELECT_OP      0x25\r
 #define FRAMEWORK_EFI_IFR_VARSTORE_SELECT_PAIR_OP 0x26\r
 #define FRAMEWORK_EFI_IFR_VARSTORE_OP             0x24\r
 #define FRAMEWORK_EFI_IFR_VARSTORE_SELECT_OP      0x25\r
 #define FRAMEWORK_EFI_IFR_VARSTORE_SELECT_PAIR_OP 0x26\r

-#define FRAMEWORK_EFI_IFR_TRUE_OP                 0x27\r
-#define FRAMEWORK_EFI_IFR_FALSE_OP                0x28\r
-#define FRAMEWORK_EFI_IFR_GT_OP                   0x29\r
-#define FRAMEWORK_EFI_IFR_GE_OP                   0x2A\r
-#define FRAMEWORK_EFI_IFR_OEM_DEFINED_OP          0x2B\r
-#define FRAMEWORK_EFI_IFR_LAST_OPCODE             FRAMEWORK_EFI_IFR_OEM_DEFINED_OP\r
+#define FRAMEWORK_EFI_IFR_LAST_OPCODE             FRAMEWORK_EFI_IFR_VARSTORE_SELECT_PAIR_OP\r

 #define FRAMEWORK_EFI_IFR_OEM_OP                  0xFE\r
 #define FRAMEWORK_EFI_IFR_NV_ACCESS_COMMAND       0xFF\r
 \r
 #define FRAMEWORK_EFI_IFR_OEM_OP                  0xFE\r
 #define FRAMEWORK_EFI_IFR_NV_ACCESS_COMMAND       0xFF\r
 \r


@@ -378,22 +373,6 @@ typedef struct {
   UINT16                            SecondaryVarId; // variable store ID, as referenced elsewhere in the form\r
 } FRAMEWORK_EFI_IFR_VARSTORE_SELECT_PAIR;\r
 \r
   UINT16                            SecondaryVarId; // variable store ID, as referenced elsewhere in the form\r
 } FRAMEWORK_EFI_IFR_VARSTORE_SELECT_PAIR;\r
 \r

-typedef struct {\r
-  FRAMEWORK_EFI_IFR_OP_HEADER       Header;\r
-} FRAMEWORK_EFI_IFR_TRUE;\r
-\r
-typedef struct {\r
-  FRAMEWORK_EFI_IFR_OP_HEADER       Header;\r
-} FRAMEWORK_EFI_IFR_FALSE;\r
-\r
-typedef struct {\r
-  FRAMEWORK_EFI_IFR_OP_HEADER       Header;\r
-} FRAMEWORK_EFI_IFR_GT;\r
-\r
-typedef struct {\r
-  FRAMEWORK_EFI_IFR_OP_HEADER       Header;\r
-} FRAMEWORK_EFI_IFR_GE;\r
-\r

 //\r
 // Save defaults and restore defaults have same structure\r
 //\r
 //\r
 // Save defaults and restore defaults have same structure\r
 //\r

diff --git a/IntelFrameworkPkg/Include/Protocol/FirmwareVolume.h b/IntelFrameworkPkg/Include/Protocol/FirmwareVolume.h
index 98f2909b188812050c46c1c1796dd2aa1dcf7ae6..86bcc8c215f53910f58023ba2f146bfeeb7ed097 100644 (file)
--- a/IntelFrameworkPkg/Include/Protocol/FirmwareVolume.h
+++ b/IntelFrameworkPkg/Include/Protocol/FirmwareVolume.h
@@ -188,12 +188,13 @@ EFI_STATUS
                                 required to complete the read\r
   @param  AuthenticationStatus  pointer to the authentication status of the data\r
 \r
                                 required to complete the read\r
   @param  AuthenticationStatus  pointer to the authentication status of the data\r
 \r

-  @retval EFI_SUCCESS\r
-  @retval EFI_WARN_BUFFER_TOO_SMALL\r
-  @retval EFI_OUT_OF_RESOURCES\r
-  @retval EFI_NOT_FOUND\r
-  @retval EFI_DEVICE_ERROR\r
-  @retval EFI_ACCESS_DENIED\r
+  @retval EFI_SUCCESS                The call completed successfully.\r
+  @retval EFI_WARN_BUFFER_TOO_SMALL  The buffer is too small to contain the requested output. \r
+                                     The buffer is filled and the output is truncated.\r
+  @retval EFI_OUT_OF_RESOURCES       An allocation failure occurred.\r
+  @retval EFI_NOT_FOUND              Name was not found in the firmware volume.\r
+  @retval EFI_DEVICE_ERROR           A hardware error occurred when attempting to access the firmware volume.\r
+  @retval EFI_ACCESS_DENIED          The firmware volume is configured to disallow reads.\r

 \r
 **/\r
 typedef\r
 \r
 **/\r
 typedef\r


@@ -232,13 +233,16 @@ typedef struct {
                                 element in the array indicates a file to write, and there are\r
                                 NumberOfFiles elements in the input array.\r
 \r
                                 element in the array indicates a file to write, and there are\r
                                 NumberOfFiles elements in the input array.\r
 \r

-  @retval EFI_SUCCESS\r
-  @retval EFI_OUT_OF_RESOURCES\r
-  @retval EFI_DEVICE_ERROR\r
-  @retval EFI_WRITE_PROTECTED\r
-  @retval EFI_NOT_FOUND\r
-  @retval EFI_INVALID_PARAMETER\r
-\r
+  @retval EFI_SUCCESS           The write completed successfully.\r
+  @retval EFI_OUT_OF_RESOURCES  The firmware volume does not have enough free space to store file(s).\r
+  @retval EFI_DEVICE_ERROR      A hardware error occurred when attempting to access the firmware volume.\r
+  @retval EFI_WRITE_PROTECTED   The firmware volume is configured to disallow writes.\r
+  @retval EFI_NOT_FOUND         A delete was requested, but the requested file was not \r
+                                found in the firmware volume.\r
+  @retval EFI_INVALID_PARAMETER A delete was requested with a multiple file write.\r
+                                An unsupported WritePolicy was requested.\r
+                                An unknown file type was specified.\r
+                                A file system specific error has occurred.\r

 **/\r
 typedef\r
 EFI_STATUS\r
 **/\r
 typedef\r
 EFI_STATUS\r


@@ -261,10 +265,12 @@ EFI_STATUS
   @param  Attributes            pointer to Attributes of the file found\r
   @param  Size                  pointer to Size in bytes of the file found\r
 \r
   @param  Attributes            pointer to Attributes of the file found\r
   @param  Size                  pointer to Size in bytes of the file found\r
 \r

-  @retval EFI_SUCCESS\r
-  @retval EFI_NOT_FOUND\r
-  @retval EFI_DEVICE_ERROR\r
-  @retval EFI_ACCESS_DENIED\r
+  @retval EFI_SUCCESS           The output parameters are filled with data obtained from \r
+                                the first matching file that was found.\r
+  @retval EFI_NOT_FOUND         No files of type FileType were found.\r
+  @retval EFI_DEVICE_ERROR      A hardware error occurred when attempting to access \r
+                                the firmware volume.\r
+  @retval EFI_ACCESS_DENIED     The firmware volume is configured to disallow reads.\r

 \r
 **/\r
 typedef\r
 \r
 **/\r
 typedef\r

diff --git a/IntelFrameworkPkg/Include/Protocol/SectionExtraction.h b/IntelFrameworkPkg/Include/Protocol/SectionExtraction.h
index 18074ec588d86e2a4149983e8683dc5a5b9302c7..a20f0df064c986dd8ed6dce91d6ba84ee6bd67e9 100644 (file)
--- a/IntelFrameworkPkg/Include/Protocol/SectionExtraction.h
+++ b/IntelFrameworkPkg/Include/Protocol/SectionExtraction.h
@@ -26,6 +26,7 @@
 #define _SECTION_EXTRACTION_PROTOCOL_H_\r
 \r
 #include <PiDxe.h>\r
 #define _SECTION_EXTRACTION_PROTOCOL_H_\r
 \r
 #include <PiDxe.h>\r

+#include <Framework/FirmwareVolumeImageFormat.h>\r

 \r
 //\r
 // Protocol GUID definition\r
 \r
 //\r
 // Protocol GUID definition\r


@@ -101,8 +102,8 @@ EFI_STATUS
   @retval EFI_OUT_OF_RESOURCES  The system has insufficient resources to process\r
                                 the request.\r
   @retval EFI_INVALID_PARAMETER The SectionStreamHandle does not exist.\r
   @retval EFI_OUT_OF_RESOURCES  The system has insufficient resources to process\r
                                 the request.\r
   @retval EFI_INVALID_PARAMETER The SectionStreamHandle does not exist.\r

-  @retval EFI_BUFFER_TOO_SMALL  The size of the input buffer is insufficient to\r
-                                contain the requested section.\r
+  @retval EFI_WARN_BUFFER_TOO_SMALL The size of the input buffer is insufficient to contain the requested\r
+                                    section. The input buffer is filled and section contents are truncated.\r

 \r
 **/\r
 typedef\r
 \r
 **/\r
 typedef\r