]> git.proxmox.com Git - mirror_edk2.git/blobdiff - IntelFrameworkPkg/Include/Ppi/ReadOnlyVariable.h
Minor code enhancement.
[mirror_edk2.git] / IntelFrameworkPkg / Include / Ppi / ReadOnlyVariable.h
index 6e6c0873a52764e49f656ecbe71d78d50347abb9..87d2dc35e665d17b7b04cf76ad5102f8cd87dcb7 100644 (file)
@@ -1,12 +1,13 @@
 /** @file\r
-  This file declares Read-only Variable Service PPI\r
+  This file declares the Read-only Variable Service PPI, which is required by the framework spec.\r
+\r
   These services provide a lightweight, read-only variant of the full EFI variable services. The\r
   reason that these services are read-only is to reduce the complexity of flash management. Also,\r
   some implementation of the PEI may use the same physical flash part for variable and PEIM\r
   storage; as such, a write command to certain technologies would alter the contents of the entire part,\r
   thus making the in situ PEIM execution not follow the required flow.\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
   THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
   WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 \r
-  Module Name:  ReadOnlyVariable.h\r
-\r
   @par Revision Reference:\r
   This PPI is defined in PEI CIS\r
   Version 0.91.\r
-\r
 **/\r
 \r
 #ifndef __PEI_READ_ONLY_VARIABLE_PPI_H__\r
 #define __PEI_READ_ONLY_VARIABLE_PPI_H__\r
 \r
-#include <PiPei.h>\r
-\r
 #define EFI_PEI_READ_ONLY_VARIABLE_ACCESS_PPI_GUID \\r
   { \\r
     0x3cdc90c6, 0x13fb, 0x4a75, {0x9e, 0x79, 0x59, 0xe9, 0xdd, 0x78, 0xb9, 0xfa } \\r
 \r
 typedef struct _EFI_PEI_READ_ONLY_VARIABLE_PPI  EFI_PEI_READ_ONLY_VARIABLE_PPI;\r
 \r
-//\r
-// Variable attributes\r
-//\r
+///\r
+/// Variable attributes\r
+///@{\r
 #define EFI_VARIABLE_NON_VOLATILE       0x00000001\r
 #define EFI_VARIABLE_BOOTSERVICE_ACCESS 0x00000002\r
 #define EFI_VARIABLE_RUNTIME_ACCESS     0x00000004\r
+///\r
+/// Inconsistent with specification here: \r
+/// In Framework Spec, PeiCis0.91, neither the macro or its value is defined.\r
+/// Keeping this inconsistancy for backward compatibility.\r
+/// \r
 #define EFI_VARIABLE_READ_ONLY          0x00000008\r
+///@}\r
 \r
 /**\r
   Get Variable value by Name and GUID pair\r
 \r
-  @param  PeiServices    An indirect pointer to the PEI Services Table published by the PEI Foundation.\r
-  @param  VariableName   A NULL-terminated Unicode string that is the name of the vendor's variable.\r
-  @param  VendorGuid     A unique identifier for the vendor.\r
-  @param  Attributes     If not NULL, a pointer to the memory location to return\r
-                         the attributes bitmask for the variable.\r
-  @param  DataSize       On input, the size in bytes of the return Data buffer.\r
-                         On output, the size of data returned in Data.\r
-  @param  Data           The buffer to return the contents of the variable.\r
+  @param[in]  PeiServices    An indirect pointer to the PEI Services Table published by the PEI Foundation.\r
+  @param[in]  VariableName   A NULL-terminated Unicode string that is the name of the vendor's variable.\r
+  @param[in]  VendorGuid     A unique identifier for the vendor.\r
+  @param[out] Attributes     This OPTIONAL parameter may be either NULL or\r
+                             a pointer to the location in which to return\r
+                             the attributes bitmask for the variable.\r
+  @param[in,out]  DataSize   On input, the size in bytes of the return Data buffer.\r
+                             On output, the size of data returned in Data.\r
+  @param[out] Data           The buffer to return the contents of the variable.\r
 \r
   @retval EFI_SUCCESS           The function completed successfully.\r
   @retval EFI_NOT_FOUND         The variable was not found.\r
@@ -64,30 +67,30 @@ typedef struct _EFI_PEI_READ_ONLY_VARIABLE_PPI  EFI_PEI_READ_ONLY_VARIABLE_PPI;
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *EFI_PEI_GET_VARIABLE) (\r
-  IN EFI_PEI_SERVICES             **PeiServices,\r
-  IN CHAR16                       *VariableName,\r
-  IN EFI_GUID                     *VendorGuid,\r
-  OUT UINT32                      *Attributes OPTIONAL,\r
-  IN OUT UINTN                    *DataSize,\r
-  OUT VOID                        *Data\r
+(EFIAPI *EFI_PEI_GET_VARIABLE)(\r
+  IN     EFI_PEI_SERVICES   **PeiServices,\r
+  IN     CHAR16             *VariableName,\r
+  IN     EFI_GUID           *VendorGuid,\r
+     OUT UINT32             *Attributes OPTIONAL,\r
+  IN OUT UINTN              *DataSize,\r
+     OUT VOID               *Data\r
   );\r
 \r
 /**\r
   This function can be called multiple times to retrieve the VariableName\r
   and VendorGuid of all variables currently available in the system. On each call\r
-  to GetNextVariableName() the previous results are passed into the interface,\r
+  to GetNextVariableName(), the previous results are passed into the interface,\r
   and on output the interface returns the next variable name data.  When the\r
   entire variable list has been returned, the error EFI_NOT_FOUND is returned.\r
 \r
-  @param  PeiServices      An indirect pointer to the PEI Services Table published by the PEI Foundation.\r
-  @param  VariableNameSize The size of the VariableName buffer.\r
-  @param  VariableName     On input, supplies the last VariableName that was\r
-                           returned by GetNextVariableName(). On output, returns the Null-terminated\r
-                           Unicode string of the current variable.\r
-  @param  VendorGuid       On input, supplies the last VendorGuid that was\r
-                           returned by GetNextVariableName(). On output, returns the VendorGuid\r
-                           of the current variable.\r
+  @param[in]  PeiServices       An indirect pointer to the PEI Services Table published by the PEI Foundation.\r
+  @param[in]  VariableNameSize  The size of the VariableName buffer.\r
+  @param[in]  VariableName      On input, supplies the last VariableName that was\r
+                                returned by GetNextVariableName(). On output, returns the Null-terminated\r
+                                Unicode string of the current variable.\r
+  @param[in]  VendorGuid        On input, supplies the last VendorGuid that was\r
+                                returned by GetNextVariableName(). On output, returns the VendorGuid\r
+                                of the current variable.\r
 \r
   @retval EFI_SUCCESS           The function completed successfully.\r
   @retval EFI_NOT_FOUND         The next variable was not found.\r
@@ -98,30 +101,28 @@ EFI_STATUS
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *EFI_PEI_GET_NEXT_VARIABLE_NAME) (\r
+(EFIAPI *EFI_PEI_GET_NEXT_VARIABLE_NAME)(\r
   IN EFI_PEI_SERVICES             **PeiServices,\r
   IN OUT UINTN                    *VariableNameSize,\r
   IN OUT CHAR16                   *VariableName,\r
   IN OUT EFI_GUID                 *VendorGuid\r
   );\r
 \r
-/**\r
-  @par Ppi Description:\r
-  This PPI provides a lightweight, read-only variant of the full EFI\r
-  variable services.\r
-\r
-  @param GetVariable\r
-  A service to ascertain a given variable name.\r
-\r
-  @param GetNextVariableName\r
-  A service to ascertain a variable based upon a given, known variable\r
-\r
-**/\r
+///\r
+/// This PPI provides a lightweight, read-only variant of the full EFI\r
+/// variable services.  \r
+///\r
 struct _EFI_PEI_READ_ONLY_VARIABLE_PPI {\r
-  EFI_PEI_GET_VARIABLE            PeiGetVariable;\r
-  EFI_PEI_GET_NEXT_VARIABLE_NAME  PeiGetNextVariableName;\r
+       ///\r
+  /// Inconsistent with specification here: \r
+  /// In Framework Spec, PeiCis0.91, the field is named as GetVariable and GetNextVariableName.\r
+  /// Keeping this inconsistance is for backward compatibility.\r
+       /// \r
+  EFI_PEI_GET_VARIABLE            PeiGetVariable;         ///< A service to ascertain a given variable name.\r
+  EFI_PEI_GET_NEXT_VARIABLE_NAME  PeiGetNextVariableName; ///< A service to ascertain a variable based upon a given, known variable\r
 };\r
 \r
 extern EFI_GUID gEfiPeiReadOnlyVariablePpiGuid;\r
 \r
-#endif\r
+#endif /* __PEI_READ_ONLY_VARIABLE_PPI_H__ */\r
+\r