]> git.proxmox.com Git - mirror_edk2.git/commitdiff
Code clean up to match with Framework HII 0.92 spec. Add comments for mismatches...
authorrsun3 <rsun3@6f19259b-4bc3-4df7-8a09-765794883524>
Fri, 10 Jul 2009 07:59:23 +0000 (07:59 +0000)
committerrsun3 <rsun3@6f19259b-4bc3-4df7-8a09-765794883524>
Fri, 10 Jul 2009 07:59:23 +0000 (07:59 +0000)
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@8864 6f19259b-4bc3-4df7-8a09-765794883524

IntelFrameworkPkg/Include/Protocol/FrameworkHii.h

index f5169ef9c4fa6bb9b50060691cd591b6e79680fd..f2f212be7d9020ac06c445f892d3fb19e95556bb 100644 (file)
@@ -1,9 +1,11 @@
 /** @file\r
   This file defines the Human Interface Infrastructure protocol which will\r
   be used by resources which want to publish IFR/Font/String data and have it\r
-  collected by the Configuration engine.\r
+  collected by the Configuration engine. This protocol is defined in the.\r
+  Intel Platform Innovation Framework for EFI Human Interface Infrastructure\r
+  Specification Version 0.92.\r
 \r
-  Copyright (c) 2007 - 2008, Intel Corporation\r
+  Copyright (c) 2007 - 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:  FrameworkHii.h\r
-\r
-  @par Revision Reference:\r
-  This protocol is defined in HII spec 0.92.\r
-\r
 **/\r
 \r
 #ifndef _FRAMEWORK_HII_H_\r
 //\r
 #include <Protocol/GraphicsOutput.h>\r
 \r
-//\r
-// In both EDK and EDK II, incompatbile change is done to Framework HII protocol. \r
-// This change should cause a change of GUID in both of code and HII spec. But we \r
-// update the GUID in code in EDK and EDK II. The 0.92 spec is not updated. This\r
-// is a known issue..\r
-//\r
+///\r
+/// In both EDK and EDK II, incompatbile change is done to Framework HII protocol. \r
+/// This change should cause a change of GUID in both of code and HII spec. But we \r
+/// update the GUID in code in EDK and EDK II. The 0.92 spec is not updated. This\r
+/// is a known issue.\r
+///\r
+///\r
+/// Note that EFI_HII_PROTOCOL_GUID is different from that defined in the Framework HII\r
+/// 0.92 spec because the spec changed part of HII interfaces but did not update the protocol\r
+/// GUID.\r
+///\r
 #define EFI_HII_PROTOCOL_GUID \\r
   { \\r
     0xd7ad636e, 0xb997, 0x459b, {0xbf, 0x3f, 0x88, 0x46, 0x89, 0x79, 0x80, 0xe1} \\r
     0x5542cce1, 0xdf5c, 0x4d1b, { 0xab, 0xca, 0x36, 0x4f, 0x77, 0xd3, 0x99, 0xfb } \\r
   }\r
 \r
-\r
 typedef UINT16                    STRING_REF;\r
 typedef UINT32                    RELOFST;\r
 \r
 typedef struct _EFI_HII_PROTOCOL  EFI_HII_PROTOCOL;\r
 \r
+///\r
+/// Note: Name difference between code and the Framework HII 0.92 spec.\r
+///       Add FRAMEWORK_ prefix to avoid name confict with EFI_HII_HANDLE defined in the\r
+///       UEFI 2.1d spec.\r
+///\r
 typedef UINT16                    FRAMEWORK_EFI_HII_HANDLE;\r
 \r
-//\r
-// Global definition\r
-//\r
-#define NARROW_CHAR         0xFFF0\r
-#define WIDE_CHAR           0xFFF1\r
-#define NON_BREAKING_CHAR   0xFFF2\r
-#define GLYPH_WIDTH         8\r
-#define GLYPH_HEIGHT        19\r
-\r
+///\r
+/// HII package type values\r
+///\r
 #define EFI_HII_FONT        1\r
 #define EFI_HII_STRING      2\r
 #define EFI_HII_IFR         3\r
@@ -71,7 +71,7 @@ typedef UINT16                    FRAMEWORK_EFI_HII_HANDLE;
 #define EFI_HII_VARIABLE    6\r
 #define EFI_HII_DEVICE_PATH 7\r
 \r
-\r
+//\r
 // References to string tokens must use this macro to enable scanning for\r
 // token usages.\r
 //\r
@@ -85,75 +85,134 @@ typedef UINT16  EFI_FORM_LABEL;
 \r
 #pragma pack(1)\r
 \r
-//\r
-// The header found at the start of each package.\r
-//\r
+///\r
+/// The header found at the start of each package.\r
+///\r
 typedef struct {\r
-  UINT32  Length;\r
-  UINT16  Type;\r
+  UINT32  Length;  ///< The size of the package in bytes.\r
+  UINT16  Type;    ///< The type of the package.\r
 } EFI_HII_PACK_HEADER;\r
 \r
-//\r
-// A form list consists of a large variety of structure\r
-// possibilities so to represent the binary blob of data\r
-// associated with a package of forms, we will assume a\r
-// pointer to a self-describing data buffer.\r
-//\r
+///\r
+/// IFR package structure.\r
+/// Immediately following the EFI_HII_IFR_PACK structure will be a series of IFR opcodes. \r
+///\r
 typedef struct {\r
-  EFI_HII_PACK_HEADER Header;\r
+  EFI_HII_PACK_HEADER Header; ///< Header of the IFR package.\r
 } EFI_HII_IFR_PACK;\r
 \r
+///\r
+/// HII Handle package structure.\r
+/// \r
 typedef struct {\r
+  ///\r
+  /// Header of the package.\r
+  ///\r
   EFI_HII_PACK_HEADER Header;           // Must be filled in\r
+  ///\r
+  /// The image handle of the driver to which the package is referring.\r
+  ///\r
   EFI_HANDLE          ImageHandle;      // Must be filled in\r
+  ///\r
+  /// The handle of the device that is being described by this package.\r
+  ///\r
   EFI_HANDLE          DeviceHandle;     // Optional\r
+  ///\r
+  /// The handle of the parent of the device that is being described by this package.\r
+  ///\r
   EFI_HANDLE          ControllerHandle; // Optional\r
+  ///\r
+  /// The handle that was registered to receive EFI_FORM_CALLBACK_PROTOCOL calls from other drivers.\r
+  ///\r
   EFI_HANDLE          CallbackHandle;   // Optional\r
+  ///\r
+  /// Note this field is not defined in the Framework HII 0.92 spec.\r
+  /// Unused. Reserved for source code compatibility.\r
+  ///\r
   EFI_HANDLE          COBExportHandle;  // Optional\r
 } EFI_HII_HANDLE_PACK;\r
 \r
-//\r
-// ********************************************************\r
-// EFI_VARIABLE_CONTENTS\r
-// ********************************************************\r
-//\r
+///\r
+/// Variable package structure.\r
+/// \r
 typedef struct {\r
+  ///\r
+  /// Header of the package.\r
+  ///\r
   EFI_HII_PACK_HEADER Header;\r
+  ///\r
+  /// GUID of the EFI variable\r
+  ///\r
   EFI_GUID            VariableGuid;\r
+  ///\r
+  /// Length in bytes of the EFI variable\r
+  ///\r
   UINT32              VariableNameLength;\r
+  ///\r
+  /// The unique value for this variable\r
+  ///\r
   UINT16              VariableId;\r
   //\r
   //  CHAR16                VariableName[]; //Null-terminated\r
   //\r
 } EFI_HII_VARIABLE_PACK;\r
 \r
-//\r
-// ********************************************************\r
-// EFI_DEVICE_PATH_PACK\r
-// ********************************************************\r
-//\r
+///\r
+/// Device path package structure.\r
+///\r
 typedef struct {\r
+  ///\r
+  /// Header of the package.\r
+  ///\r
   EFI_HII_PACK_HEADER Header;\r
   //\r
   //  EFI_DEVICE_PATH       DevicePath[];\r
   //\r
 } EFI_HII_DEVICE_PATH_PACK;\r
 \r
-//\r
-// ********************************************************\r
-// EFI_HII_DATA_TABLE\r
-// ********************************************************\r
-//\r
 typedef struct {\r
+  ///\r
+  /// Unique value that correlates to the original HII handle. \r
+  ///\r
   FRAMEWORK_EFI_HII_HANDLE  HiiHandle;\r
-  EFI_GUID        PackageGuid;\r
-  UINT32          DataTableSize;\r
-  UINT32          IfrDataOffset;\r
-  UINT32          StringDataOffset;\r
-  UINT32          VariableDataOffset;\r
-  UINT32          DevicePathOffset;\r
-  UINT32          NumberOfVariableData;\r
-  UINT32          NumberOfLanguages;\r
+  ///\r
+  /// If an IFR pack exists in a data table that does not contain strings,\r
+  /// then the strings for that IFR pack are located in another data table \r
+  /// that contains a string pack and has a matching HiiDataTable.PackageGuid. \r
+  ///\r
+  EFI_GUID                  PackageGuid;\r
+  ///\r
+  /// Size of the EFI_HII_DATA_TABLE in bytes.\r
+  ///\r
+  UINT32                    DataTableSize;\r
+  ///\r
+  /// Byte offset from the start of this structure to the IFR data.\r
+  /// If the offset value is 0, then no IFR data is enclosed.\r
+  ///\r
+  UINT32                    IfrDataOffset;\r
+  ///\r
+  /// Byte offset from the start of this structure to the string data. \r
+  /// If the offset value is 0, then no string data is enclosed.\r
+  ///\r
+  UINT32                    StringDataOffset;\r
+  ///\r
+  /// Byte offset from the start of this structure to the variable data.\r
+  /// If the offset value is 0, then no variable data is enclosed.\r
+  ///\r
+  UINT32                    VariableDataOffset;\r
+  ///\r
+  /// Byte offset from the start of this structure to the device path data.\r
+  /// If the offset value is 0, then no DevicePath data is enclosed.\r
+  ///\r
+  UINT32                    DevicePathOffset;\r
+  ///\r
+  /// Number of VariableData[] elements in the array.\r
+  ///\r
+  UINT32                    NumberOfVariableData;\r
+  ///\r
+  /// The number of language string packages.\r
+  ///\r
+  UINT32                    NumberOfLanguages;\r
   //\r
   // EFI_HII_DEVICE_PATH_PACK DevicePath[];\r
   // EFI_HII_VARIABLE_PACK VariableData[];\r
@@ -162,50 +221,101 @@ typedef struct {
   //\r
 } EFI_HII_DATA_TABLE;\r
 \r
-//\r
-// ********************************************************\r
-// EFI_HII_EXPORT_TABLE\r
-// ********************************************************\r
-//\r
+///\r
+/// Structure defining format for exporting data from the HII Database.\r
+///\r
 typedef struct {\r
+  ///\r
+  /// Number of EFI_HII_DATA_TABLE entries.\r
+  ///\r
   UINT32    NumberOfHiiDataTables;\r
+  ///\r
+  /// Defines the revision of the EFI_HII_DATA_TABLE structure.\r
+  ///\r
   EFI_GUID  Revision;\r
   //\r
   // EFI_HII_DATA_TABLE HiiDataTable[];\r
   //\r
 } EFI_HII_EXPORT_TABLE;\r
 \r
+///\r
+/// Structure used to pass data to update a form or form package\r
+/// that has previously been registered with the EFI HII database.\r
+///\r
 typedef struct {\r
-  BOOLEAN               FormSetUpdate;      // If TRUE, next variable is significant\r
-  EFI_PHYSICAL_ADDRESS  FormCallbackHandle; // If not 0, will update Formset with this info\r
-  BOOLEAN               FormUpdate;         // If TRUE, next variable is significant\r
-  UINT16                FormValue;          // specify which form is to be updated if FormUpdate value is TRUE.\r
-  STRING_REF            FormTitle;          // If not 0, will update Form with this info\r
-  UINT16                DataCount;          // The number of Data entries in this structure\r
-  UINT8                 *Data;              // An array of 1+ op-codes, specified by DataCount\r
-} FRAMEWORK_EFI_HII_UPDATE_DATA;\r
+  ///\r
+  /// If TRUE, indicates that the FormCallbackHandle value will \r
+  /// be used to update the contents of the CallBackHandle entry in the form set.\r
+  ///\r
+  BOOLEAN               FormSetUpdate;\r
+  ///\r
+  /// This parameter is valid only when FormSetUpdate is TRUE.\r
+  /// The value in this parameter will be used to update the contents \r
+  /// of the CallbackHandle entry in the form set.\r
+  ///\r
+  EFI_PHYSICAL_ADDRESS  FormCallbackHandle;\r
+  ///\r
+  /// If TRUE, indicates that the FormTitle contents will be \r
+  /// used to update the FormValue's title.\r
+  ///\r
+  BOOLEAN               FormUpdate;\r
+  ///\r
+  /// Specifies which form is to be updated if the FormUpdate value is TRUE.\r
+  ///\r
+  UINT16                FormValue;\r
+  ///\r
+  /// This parameter is valid only when the FormUpdate parameter is TRUE.\r
+  /// The value in this parameter will be used to update the contents of the form title. \r
+  ///\r
+  STRING_REF            FormTitle;\r
+  ///\r
+  /// The number of Data entries in this structure.\r
+  UINT16                DataCount;\r
+  ///\r
+  /// An array of 1+ opcodes, specified by DataCount.\r
+  ///\r
+  UINT8                 *Data;\r
+} EFI_HII_UPDATE_DATA;\r
 \r
 //\r
 // String attributes\r
 //\r
 #define LANG_RIGHT_TO_LEFT  0x00000001\r
 \r
-//\r
-// A string package is used to localize strings to a particular\r
-// language.  The package is associated with a particular driver\r
-// or set of drivers.  Tools are used to associate tokens with\r
-// string references in forms and in programs.  These tokens are\r
-// language agnostic.  When paired with a language pack (directly\r
-// or indirectly), the string token resolves into an actual\r
-// UNICODE string.  The NumStringPointers determines how many\r
-// StringPointers (offset values) there are as well as the total\r
-// number of Strings that are defined.\r
-//\r
+///\r
+/// A string package is used to localize strings to a particular\r
+/// language.  The package is associated with a particular driver\r
+/// or set of drivers.  Tools are used to associate tokens with\r
+/// string references in forms and in programs.  These tokens are\r
+/// language agnostic.  When paired with a language pack (directly\r
+/// or indirectly), the string token resolves into an actual\r
+/// UNICODE string.  The NumStringPointers determines how many\r
+/// StringPointers (offset values) there are as well as the total\r
+/// number of Strings that are defined.\r
+///\r
 typedef struct {\r
+  ///\r
+  /// Header of the package.\r
+  ///\r
   EFI_HII_PACK_HEADER Header;\r
+  ///\r
+  /// The string containing one or more ISO 639-2 three-character designator(s)\r
+  /// of the language or languages whose translations are contained in this language pack.\r
+  /// The first designator indicates the primary language while the others are secondary languages. \r
+  ///\r
   RELOFST             LanguageNameString;\r
+  ///\r
+  /// Contains the offset into this structure of a printable name of the language\r
+  /// for use when prompting the user. The language printed is to be the primary language.\r
+  ///\r
   RELOFST             PrintableLanguageName;\r
+  ///\r
+  /// The number of Strings and StringPointers contained within the string package.\r
+  ///\r
   UINT32              NumStringPointers;\r
+  ///\r
+  /// Indicates the direction the language is to be printed.\r
+  ///\r
   UINT32              Attributes;\r
   //\r
   //  RELOFST               StringPointers[];\r
@@ -214,67 +324,104 @@ typedef struct {
 } EFI_HII_STRING_PACK;\r
 \r
 \r
-//\r
-// A font list consists of a font header followed by a series\r
-// of glyph structures.  Note that fonts are not language specific.\r
-//\r
+///\r
+/// A font list consists of a font header followed by a series\r
+/// of glyph structures.  Note that fonts are not language specific.\r
+///\r
 typedef struct {\r
+  ///\r
+  /// Header of the package.\r
+  ///\r
   EFI_HII_PACK_HEADER Header;\r
+  ///\r
+  /// The number of NarrowGlyphs that are included in the font package.\r
+  ///\r
   UINT16              NumberOfNarrowGlyphs;\r
+  ///\r
+  /// The number of WideGlyphs that are included in the font package.\r
+  ///\r
   UINT16              NumberOfWideGlyphs;\r
+  //EFI_NARROW_GLYPH  NarrowGlyphs[];\r
+  //EFI_WIDE_GLYPH    WideGlyphs[];\r
 } EFI_HII_FONT_PACK;\r
 \r
-//\r
-// The IfrData in the EFI_HII_IFR_PACK structure definition\r
-// is variable length, and not really part of the header. To\r
-// simplify from code the size of the header, define an\r
-// identical structure that does not include the IfrData field.\r
-// Then use sizeof() this new structure to determine the\r
-// actual size of the header.\r
-//\r
-typedef struct {\r
-  EFI_HII_PACK_HEADER Header;\r
-} EFI_HII_IFR_PACK_HEADER;\r
-\r
+///\r
+/// The definition of a specific physical key\r
+///\r
+/// Note: Name difference between code and the Framework HII 0.92 spec.\r
+///       Add FRAMEWORK_ prefix to avoid name confict with EFI_KEY_DESCRIPTOR defined in the\r
+///       UEFI 2.1d spec.\r
+///\r
 typedef struct {\r
+  ///\r
+  /// Used to describe a physical key on a keyboard.\r
+  ///\r
   EFI_KEY Key;\r
+  ///\r
+  /// Unicode value for the Key.\r
   CHAR16  Unicode;\r
+  ///\r
+  /// Unicode value for the key with the shift key being held down.\r
+  ///\r
   CHAR16  ShiftedUnicode;\r
+  ///\r
+  /// Unicode value for the key with the Alt-GR being held down.\r
+  ///\r
   CHAR16  AltGrUnicode;\r
+  ///\r
+  /// Unicode value for the key with the Alt-GR and shift keys being held down.\r
+  ///\r
   CHAR16  ShiftedAltGrUnicode;\r
+  ///\r
+  /// Modifier keys are defined to allow for special functionality that \r
+  /// is not necessarily accomplished by a printable character. \r
+  ///\r
   UINT16  Modifier;\r
 } FRAMEWORK_EFI_KEY_DESCRIPTOR;\r
 \r
-//\r
-// This structure allows a sparse set of keys to be redefined\r
-// or a complete redefinition of the keyboard layout.  Most\r
-// keyboards have a lot of commonality in their layouts, therefore\r
-// only defining those keys that need to change from the default\r
-// minimizes the passed in information.\r
-//\r
-// Additionally, when an update occurs, the active keyboard layout\r
-// will be switched to the newly updated keyboard layout.  This\r
-// allows for situations that when a keyboard layout driver is\r
-// loaded as part of system initialization, the system will default\r
-// the keyboard behavior to the new layout.\r
-//\r
-// Each call to update the keyboard mapping should contain the\r
-// complete set of key descriptors to be updated, since every\r
-// call to the HII which contains an EFI_HII_KEYBOARD_PACK will\r
-// wipe the previous set of overrides.  A call to\r
-//\r
+///\r
+/// This structure allows a sparse set of keys to be redefined\r
+/// or a complete redefinition of the keyboard layout.  Most\r
+/// keyboards have a lot of commonality in their layouts, therefore\r
+/// only defining those keys that need to change from the default\r
+/// minimizes the passed in information.\r
+///\r
+/// Additionally, when an update occurs, the active keyboard layout\r
+/// will be switched to the newly updated keyboard layout.  This\r
+/// allows for situations that when a keyboard layout driver is\r
+/// loaded as part of system initialization, the system will default\r
+/// the keyboard behavior to the new layout.\r
+///\r
 typedef struct {\r
-  EFI_HII_PACK_HEADER Header;\r
+  ///\r
+  /// Header of the package.\r
+  EFI_HII_PACK_HEADER           Header;\r
+  ///\r
+  /// A pointer to a buffer containing an array of EFI_KEY_DESCRIPTOR entries.\r
+  /// Each entry will reflect the definition of a specific physical key.\r
+  ///\r
   FRAMEWORK_EFI_KEY_DESCRIPTOR  *Descriptor;\r
-  UINT8               DescriptorCount;\r
+  ///\r
+  /// The number of Descriptor entries being described.\r
+  ///\r
+  UINT8                         DescriptorCount;\r
 } EFI_HII_KEYBOARD_PACK;\r
 \r
-//\r
-// The EFI_HII_PACKAGES can contain different types of packages just\r
-// after the structure as inline data.\r
-//\r
+///\r
+/// The packages structure that will be used to pass contents into the HII database.\r
+///\r
+/// The EFI_HII_PACKAGES can contain various number of packages of different types just\r
+/// after the structure as inline data.\r
+///\r
 typedef struct {\r
+  ///\r
+  /// The number of packages being defined in this structure.\r
+  ///\r
   UINTN     NumberOfPackages;\r
+  ///\r
+  /// The GUID to be used to identify this set of packages that are being exported\r
+  /// to the HII database.\r
+  ///\r
   EFI_GUID  *GuidId;\r
   //\r
   // EFI_HII_HANDLE_PACK    *HandlePack;        // Only one pack.\r
@@ -285,9 +432,21 @@ typedef struct {
   //\r
 } EFI_HII_PACKAGES;\r
 \r
+///\r
+/// Packed link list that contains all the discernable defaults of variables\r
+/// for the opcodes that are defined in this Handle's domain of data.\r
+///\r
 typedef struct _EFI_HII_VARIABLE_PACK_LIST {\r
+  ///\r
+  /// A pointer points to the next data structure of type \r
+  /// EFI_HII_VARIABLE_PACK_LIST in the packed link list.\r
+  ///\r
   struct _EFI_HII_VARIABLE_PACK_LIST   *NextVariablePack;\r
+  ///\r
+  /// A pointer points to the content of the variable entry defined by GUID/name/data.\r
+  ///\r
   EFI_HII_VARIABLE_PACK                *VariablePack;\r
+  //EFI_HII_VARIABLE_PACK              Content\r
 } EFI_HII_VARIABLE_PACK_LIST;\r
 \r
 \r
@@ -307,10 +466,10 @@ typedef struct _EFI_HII_VARIABLE_PACK_LIST {
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *FRAMEWORK_EFI_HII_NEW_PACK)(\r
-  IN  EFI_HII_PROTOCOL    *This,\r
-  IN  EFI_HII_PACKAGES    *Packages,\r
-  OUT FRAMEWORK_EFI_HII_HANDLE       *Handle\r
+(EFIAPI *EFI_HII_NEW_PACK)(\r
+  IN  EFI_HII_PROTOCOL          *This,\r
+  IN  EFI_HII_PACKAGES          *Packages,\r
+  OUT FRAMEWORK_EFI_HII_HANDLE  *Handle\r
   );\r
 \r
 /**\r
@@ -327,9 +486,9 @@ EFI_STATUS
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *FRAMEWORK_EFI_HII_REMOVE_PACK)(\r
-  IN EFI_HII_PROTOCOL    *This,\r
-  IN FRAMEWORK_EFI_HII_HANDLE       Handle\r
+(EFIAPI *EFI_HII_REMOVE_PACK)(\r
+  IN EFI_HII_PROTOCOL          *This,\r
+  IN FRAMEWORK_EFI_HII_HANDLE  Handle\r
   );\r
 \r
 /**\r
@@ -348,10 +507,10 @@ EFI_STATUS
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *FRAMEWORK_EFI_HII_FIND_HANDLES)(\r
-  IN     EFI_HII_PROTOCOL *This,\r
-  IN OUT UINT16           *HandleBufferLength,\r
-  OUT    FRAMEWORK_EFI_HII_HANDLE    *Handle\r
+(EFIAPI *EFI_HII_FIND_HANDLES)(\r
+  IN     EFI_HII_PROTOCOL          *This,\r
+  IN OUT UINT16                    *HandleBufferLength,\r
+  OUT    FRAMEWORK_EFI_HII_HANDLE  *Handle\r
   );\r
 \r
 /**\r
@@ -373,11 +532,11 @@ EFI_STATUS
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *FRAMEWORK_EFI_HII_EXPORT)(\r
-  IN     EFI_HII_PROTOCOL *This,\r
-  IN     FRAMEWORK_EFI_HII_HANDLE    Handle,\r
-  IN OUT UINTN            *BufferSize,\r
-  OUT    VOID             *Buffer\r
+(EFIAPI *EFI_HII_EXPORT)(\r
+  IN     EFI_HII_PROTOCOL          *This,\r
+  IN     FRAMEWORK_EFI_HII_HANDLE  Handle,\r
+  IN OUT UINTN                     *BufferSize,\r
+  OUT    VOID                      *Buffer\r
   );\r
 \r
 /**\r
@@ -393,9 +552,9 @@ EFI_STATUS
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *FRAMEWORK_EFI_HII_RESET_STRINGS)(\r
-  IN     EFI_HII_PROTOCOL   *This,\r
-  IN     FRAMEWORK_EFI_HII_HANDLE      Handle\r
+(EFIAPI *EFI_HII_RESET_STRINGS)(\r
+  IN EFI_HII_PROTOCOL          *This,\r
+  IN FRAMEWORK_EFI_HII_HANDLE  Handle\r
   );\r
 \r
 /**\r
@@ -419,7 +578,7 @@ EFI_STATUS
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *FRAMEWORK_EFI_HII_TEST_STRING) (\r
+(EFIAPI *EFI_HII_TEST_STRING) (\r
   IN     EFI_HII_PROTOCOL  *This,\r
   IN     CHAR16            *StringToTest,\r
   IN OUT UINT32            *FirstMissing,\r
@@ -429,6 +588,9 @@ EFI_STATUS
 /**\r
   Translates a Unicode character into the corresponding font glyph.\r
 \r
+  Note that this function prototype name is different from that in the Framework HII 0.92 spec\r
+  to avoid name confict with EFI_HII_GET_GLYPH defined in the UEFI 2.1d spec.\r
+\r
   @param  This                  A pointer to the EFI_HII_PROTOCOL instance.\r
   @param  Source                A pointer to a Unicode string.\r
   @param  Index                 On input, the offset into the string from which to fetch\r
@@ -482,7 +644,7 @@ EFI_STATUS
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *FRAMEWORK_EFI_HII_GLYPH_TO_BLT)(\r
+(EFIAPI *EFI_HII_GLYPH_TO_BLT)(\r
   IN     EFI_HII_PROTOCOL             *This,\r
   IN     UINT8                        *GlyphBuffer,\r
   IN     EFI_GRAPHICS_OUTPUT_BLT_PIXEL Foreground,\r
@@ -496,6 +658,9 @@ EFI_STATUS
 /**\r
   Allows a new string to be added to an already existing string package.\r
 \r
+  Note that this function prototype name is different from that in the Framework HII 0.92 spec\r
+  to avoid name confict with EFI_HII_NEW_STRING defined in the UEFI 2.1d spec.\r
+\r
   @param  This                  A pointer to the EFI_HII_PROTOCOL instance.\r
   @param  Pointer               to a NULL-terminated string containing a single ISO 639-2\r
                                 language identifier, indicating the language in which the string is translated.\r
@@ -513,11 +678,11 @@ EFI_STATUS
 typedef\r
 EFI_STATUS\r
 (EFIAPI *FRAMEWORK_EFI_HII_NEW_STRING)(\r
-  IN     EFI_HII_PROTOCOL      *This,\r
-  IN     CHAR16                *Language,\r
-  IN     FRAMEWORK_EFI_HII_HANDLE         Handle,\r
-  IN OUT STRING_REF            *Reference,\r
-  IN     CHAR16                *NewString\r
+  IN     EFI_HII_PROTOCOL          *This,\r
+  IN     CHAR16                    *Language,\r
+  IN     FRAMEWORK_EFI_HII_HANDLE  Handle,\r
+  IN OUT STRING_REF                *Reference,\r
+  IN     CHAR16                    *NewString\r
   );\r
 \r
 /**\r
@@ -535,10 +700,10 @@ EFI_STATUS
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *FRAMEWORK_EFI_HII_GET_PRI_LANGUAGES)(\r
-  IN  EFI_HII_PROTOCOL    *This,\r
-  IN  FRAMEWORK_EFI_HII_HANDLE       Handle,\r
-  OUT EFI_STRING          *LanguageString\r
+(EFIAPI *EFI_HII_GET_PRI_LANGUAGES)(\r
+  IN  EFI_HII_PROTOCOL          *This,\r
+  IN  FRAMEWORK_EFI_HII_HANDLE  Handle,\r
+  OUT EFI_STRING                *LanguageString\r
   );\r
 \r
 /**\r
@@ -558,21 +723,24 @@ EFI_STATUS
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *FRAMEWORK_EFI_HII_GET_SEC_LANGUAGES)(\r
-  IN  EFI_HII_PROTOCOL    *This,\r
-  IN  FRAMEWORK_EFI_HII_HANDLE       Handle,\r
-  IN  CHAR16              *PrimaryLanguage,\r
-  OUT EFI_STRING          *LanguageString\r
+(EFIAPI *EFI_HII_GET_SEC_LANGUAGES)(\r
+  IN  EFI_HII_PROTOCOL          *This,\r
+  IN  FRAMEWORK_EFI_HII_HANDLE  Handle,\r
+  IN  CHAR16                    *PrimaryLanguage,\r
+  OUT EFI_STRING                *LanguageString\r
   );\r
 \r
 /**\r
   Extracts a string from a package already registered with the EFI HII database.\r
 \r
+  Note that this function prototype name is different from that in the Framework HII 0.92 spec\r
+  to avoid name confict with EFI_HII_GET_STRING defined in the UEFI 2.1d spec.\r
+\r
   @param  This                  A pointer to the EFI_HII_PROTOCOL instance.\r
   @param  Handle                The handle on which the string resides.\r
   @param  Token                 The string token assigned to the string.\r
   @param  Raw                   If TRUE, the string is returned unedited in the internal\r
-                                storage format described above. If false, the string returned is edited\r
+                                storage format. If false, the string returned is edited\r
                                 by replacing <cr> with <space> and by removing special characters such\r
                                 as the <wide> prefix.\r
   @param  LanguageString        Pointer to a NULL-terminated string containing a\r
@@ -591,13 +759,13 @@ EFI_STATUS
 typedef\r
 EFI_STATUS\r
 (EFIAPI *FRAMEWORK_EFI_HII_GET_STRING)(\r
-  IN     EFI_HII_PROTOCOL  *This,\r
-  IN     FRAMEWORK_EFI_HII_HANDLE     Handle,\r
-  IN     STRING_REF        Token,\r
-  IN     BOOLEAN           Raw,\r
-  IN     CHAR16            *LanguageString,\r
-  IN OUT UINTN             *BufferLength,\r
-  OUT    EFI_STRING        StringBuffer\r
+  IN     EFI_HII_PROTOCOL          *This,\r
+  IN     FRAMEWORK_EFI_HII_HANDLE  Handle,\r
+  IN     STRING_REF                Token,\r
+  IN     BOOLEAN                   Raw,\r
+  IN     CHAR16                    *LanguageString,\r
+  IN OUT UINTN                     *BufferLength,\r
+  OUT    EFI_STRING                StringBuffer\r
   );\r
 \r
 /**\r
@@ -624,15 +792,15 @@ EFI_STATUS
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *FRAMEWORK_EFI_HII_GET_LINE)(\r
-  IN     EFI_HII_PROTOCOL  *This,\r
-  IN     FRAMEWORK_EFI_HII_HANDLE     Handle,\r
-  IN     STRING_REF        Token,\r
-  IN OUT UINT16            *Index,\r
-  IN     UINT16            LineWidth,\r
-  IN     CHAR16            *LanguageString,\r
-  IN OUT UINT16            *BufferLength,\r
-  OUT    EFI_STRING        StringBuffer\r
+(EFIAPI *EFI_HII_GET_LINE)(\r
+  IN     EFI_HII_PROTOCOL          *This,\r
+  IN     FRAMEWORK_EFI_HII_HANDLE  Handle,\r
+  IN     STRING_REF                Token,\r
+  IN OUT UINT16                    *Index,\r
+  IN     UINT16                    LineWidth,\r
+  IN     CHAR16                    *LanguageString,\r
+  IN OUT UINT16                    *BufferLength,\r
+  OUT    EFI_STRING                StringBuffer\r
   );\r
 \r
 /**\r
@@ -658,12 +826,12 @@ EFI_STATUS
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *FRAMEWORK_EFI_HII_GET_FORMS)(\r
-  IN     EFI_HII_PROTOCOL  *This,\r
-  IN     FRAMEWORK_EFI_HII_HANDLE     Handle,\r
-  IN     EFI_FORM_ID       FormId,\r
-  IN OUT UINTN             *BufferLength,\r
-  OUT    UINT8             *Buffer\r
+(EFIAPI *EFI_HII_GET_FORMS)(\r
+  IN     EFI_HII_PROTOCOL          *This,\r
+  IN     FRAMEWORK_EFI_HII_HANDLE  Handle,\r
+  IN     EFI_FORM_ID               FormId,\r
+  IN OUT UINTN                     *BufferLength,\r
+  OUT    UINT8                     *Buffer\r
   );\r
 \r
 /**\r
@@ -685,7 +853,7 @@ EFI_STATUS
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *FRAMEWORK_EFI_HII_GET_DEFAULT_IMAGE)(\r
+(EFIAPI *EFI_HII_GET_DEFAULT_IMAGE)(\r
   IN     EFI_HII_PROTOCOL           *This,\r
   IN     FRAMEWORK_EFI_HII_HANDLE   Handle,\r
   IN     UINTN                      DefaultMask,\r
@@ -711,17 +879,20 @@ EFI_STATUS
 **/\r
 typedef\r
 EFI_STATUS\r
-(EFIAPI *FRAMEWORK_EFI_HII_UPDATE_FORM)(\r
-  IN EFI_HII_PROTOCOL     *This,\r
-  IN FRAMEWORK_EFI_HII_HANDLE        Handle,\r
-  IN EFI_FORM_LABEL       Label,\r
-  IN BOOLEAN              AddData,\r
-  IN FRAMEWORK_EFI_HII_UPDATE_DATA  *Data\r
+(EFIAPI *EFI_HII_UPDATE_FORM)(\r
+  IN EFI_HII_PROTOCOL          *This,\r
+  IN FRAMEWORK_EFI_HII_HANDLE  Handle,\r
+  IN EFI_FORM_LABEL            Label,\r
+  IN BOOLEAN                   AddData,\r
+  IN EFI_HII_UPDATE_DATA       *Data\r
   );\r
 \r
 /**\r
   Retrieves the current keyboard layout.\r
 \r
+  Note that this function prototype name is different from that in the Framework HII 0.92 spec\r
+  to avoid name confict with EFI_HII_GET_KEYBOARD_LAYOUT defined in the UEFI 2.1d spec.\r
+\r
   @param  This                  A pointer to the EFI_HII_PROTOCOL instance.\r
   @param  DescriptorCount       A pointer to the number of Descriptor entries being\r
                                 described in the keyboard layout being retrieved.\r
@@ -734,41 +905,41 @@ EFI_STATUS
 typedef\r
 EFI_STATUS\r
 (EFIAPI *FRAMEWORK_EFI_HII_GET_KEYBOARD_LAYOUT)(\r
-  IN     EFI_HII_PROTOCOL    *This,\r
-  OUT    UINT16              *DescriptorCount,\r
+  IN     EFI_HII_PROTOCOL              *This,\r
+  OUT    UINT16                        *DescriptorCount,\r
   OUT    FRAMEWORK_EFI_KEY_DESCRIPTOR  *Descriptor\r
   );\r
 \r
-/**\r
-  The HII Protocol manages the HII database, which is a repository for data\r
-  having to do with fonts, strings, forms, keyboards, and other future human\r
-  interface items.\r
-**/\r
+///\r
+///  The HII Protocol manages the HII database, which is a repository for data\r
+///  having to do with fonts, strings, forms, keyboards, and other future human\r
+///  interface items.\r
+///\r
 struct _EFI_HII_PROTOCOL {\r
   ///\r
   /// Extracts the various packs from a package list.\r
   ///\r
-  FRAMEWORK_EFI_HII_NEW_PACK            NewPack;\r
+  EFI_HII_NEW_PACK                      NewPack;\r
   \r
   ///\r
   /// Removes a package from the HII database.\r
   ///\r
-  FRAMEWORK_EFI_HII_REMOVE_PACK         RemovePack;\r
+  EFI_HII_REMOVE_PACK                   RemovePack;\r
   \r
   ///\r
   /// Determines the handles that are currently active in the database.\r
   /// \r
-  FRAMEWORK_EFI_HII_FIND_HANDLES        FindHandles;\r
+  EFI_HII_FIND_HANDLES                  FindHandles;\r
   \r
   ///\r
   /// Export the entire contents of the database to a buffer.\r
   ///\r
-  FRAMEWORK_EFI_HII_EXPORT              ExportDatabase;\r
+  EFI_HII_EXPORT                        ExportDatabase;\r
 \r
   ///\r
   /// Tests if all of the characters in a string have corresponding font characters.\r
   ///\r
-  FRAMEWORK_EFI_HII_TEST_STRING         TestString;\r
+  EFI_HII_TEST_STRING                   TestString;\r
   \r
   ///\r
   /// Translates a Unicode character into the corresponding font glyph.\r
@@ -778,7 +949,7 @@ struct _EFI_HII_PROTOCOL {
   ///\r
   /// Converts a glyph value into a format that is ready for a UGA BLT command.\r
   ///\r
-  FRAMEWORK_EFI_HII_GLYPH_TO_BLT        GlyphToBlt;\r
+  EFI_HII_GLYPH_TO_BLT                  GlyphToBlt;\r
 \r
   ///\r
   /// Allows a new string to be added to an already existing string package.\r
@@ -789,13 +960,13 @@ struct _EFI_HII_PROTOCOL {
   /// Allows a program to determine the primary languages that are supported\r
   /// on a given handle.  \r
   ///\r
-  FRAMEWORK_EFI_HII_GET_PRI_LANGUAGES   GetPrimaryLanguages;\r
+  EFI_HII_GET_PRI_LANGUAGES             GetPrimaryLanguages;\r
   \r
   ///\r
   /// Allows a program to determine which secondary languages are supported\r
   /// on a given handle for a given primary language.  \r
   ///\r
-  FRAMEWORK_EFI_HII_GET_SEC_LANGUAGES   GetSecondaryLanguages;\r
+  EFI_HII_GET_SEC_LANGUAGES             GetSecondaryLanguages;\r
   \r
   ///\r
   /// Extracts a string from a package that is already registered with the\r
@@ -806,28 +977,30 @@ struct _EFI_HII_PROTOCOL {
   ///\r
   /// Remove any new strings that were added after the initial string export\r
   /// for this handle.  \r
+  /// \r
+  /// Note this function is not defined in the Framework HII 0.92 spec.\r
   ///\r
-  FRAMEWORK_EFI_HII_RESET_STRINGS       ResetStrings;\r
+  EFI_HII_RESET_STRINGS                 ResetStrings;\r
   \r
   ///\r
   /// Allows a program to extract a part of a string of not more than a given width.\r
   ///\r
-  FRAMEWORK_EFI_HII_GET_LINE            GetLine;\r
+  EFI_HII_GET_LINE                      GetLine;\r
   \r
   ///\r
   /// Allows a program to extract a form or form package that has been previously registered.\r
   ///\r
-  FRAMEWORK_EFI_HII_GET_FORMS           GetForms;\r
+  EFI_HII_GET_FORMS                     GetForms;\r
   \r
   ///\r
   /// Allows a program to extract the nonvolatile image that represents the default storage image.\r
   ///\r
-  FRAMEWORK_EFI_HII_GET_DEFAULT_IMAGE   GetDefaultImage;\r
+  EFI_HII_GET_DEFAULT_IMAGE             GetDefaultImage;\r
   \r
   ///\r
   /// Allows a program to update a previously registered form.\r
   ///\r
-  FRAMEWORK_EFI_HII_UPDATE_FORM         UpdateForm;\r
+  EFI_HII_UPDATE_FORM                   UpdateForm;\r
 \r
   ///\r
   /// Allows a program to extract the current keyboard layout.\r
@@ -838,5 +1011,5 @@ struct _EFI_HII_PROTOCOL {
 extern EFI_GUID gEfiHiiProtocolGuid;\r
 extern EFI_GUID gEfiHiiCompatibilityProtocolGuid;\r
 \r
-\r
 #endif\r
+\r