]> git.proxmox.com Git - mirror_edk2.git/blobdiff - IntelFrameworkPkg/Library/FrameworkIfrSupportLib/IfrCommon.c
Fix coding style issue.
[mirror_edk2.git] / IntelFrameworkPkg / Library / FrameworkIfrSupportLib / IfrCommon.c
index d9894c4e6cbfca61c5068d592c9a71d2431daf6a..567eb56a4d3373bd1cf10196d6a5584d290b209a 100644 (file)
@@ -1,4 +1,6 @@
-/*++\r
+/** @file\r
+  Common Library Routines to assist in IFR creation on-the-fly\r
+  \r
 Copyright (c) 2006, 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
@@ -8,51 +10,28 @@ http://opensource.org/licenses/bsd-license.php
 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:\r
-  IfrCommon.c\r
-\r
-Abstract:\r
-\r
-  Common Library Routines to assist in IFR creation on-the-fly\r
-\r
-Revision History:\r
-\r
---*/\r
+**/\r
 \r
 //\r
 // Include common header file for this module.\r
 //\r
 #include "IfrSupportLibInternal.h"\r
 \r
-EFI_STATUS\r
-EFIAPI\r
-IfrLibConstruct (\r
-  IN EFI_HANDLE        ImageHandle,\r
-  IN EFI_SYSTEM_TABLE  *SystemTable\r
-  )\r
-{\r
-  return EFI_SUCCESS;\r
-}\r
-\r
+/**\r
+  Determine what is the current language setting\r
+  The setting is stored in language variable in flash. This routine\r
+  will get setting by accesssing that variable. If failed to access\r
+  language variable, then use default setting that 'eng' as current\r
+  language setting.\r
+  \r
+  @param Lang Pointer of system language\r
+  \r
+  @return whether sucess to get setting from variable\r
+**/\r
 EFI_STATUS\r
 GetCurrentLanguage (\r
   OUT     CHAR16              *Lang\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Determine what is the current language setting\r
-\r
-Arguments:\r
-\r
-  Lang      - Pointer of system language\r
-\r
-Returns:\r
-\r
-  Status code\r
-\r
---*/\r
 {\r
   EFI_STATUS  Status;\r
   UINTN       Size;\r
@@ -92,7 +71,17 @@ Returns:
   return Status;\r
 }\r
 \r
-\r
+/**\r
+  Add a string to the incoming buffer and return the token and offset data\r
+  \r
+  @param StringBuffer      The incoming buffer\r
+  @param Language          Currrent language\r
+  @param String            The string to be added\r
+  @param StringToken       The index where the string placed  \r
+  \r
+  @retval EFI_OUT_OF_RESOURCES No enough buffer to allocate\r
+  @retval EFI_SUCCESS          String successfully added to the incoming buffer\r
+**/\r
 EFI_STATUS\r
 AddString (\r
   IN      VOID                *StringBuffer,\r
@@ -100,29 +89,6 @@ AddString (
   IN      CHAR16              *String,\r
   IN OUT  STRING_REF          *StringToken\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Add a string to the incoming buffer and return the token and offset data\r
-\r
-Arguments:\r
-\r
-  StringBuffer      - The incoming buffer\r
-\r
-  Language          - Currrent language\r
-\r
-  String            - The string to be added\r
-\r
-  StringToken       - The index where the string placed\r
-\r
-Returns:\r
-\r
-  EFI_OUT_OF_RESOURCES    - No enough buffer to allocate\r
-\r
-  EFI_SUCCESS             - String successfully added to the incoming buffer\r
-\r
---*/\r
 {\r
   EFI_HII_STRING_PACK *StringPack;\r
   EFI_HII_STRING_PACK *StringPackBuffer;\r
@@ -315,31 +281,20 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
+/**\r
+  Add op-code data to the FormBuffer\r
+  \r
+  @param FormBuffer      Form buffer to be inserted to\r
+  @param OpCodeData      Op-code data to be inserted  \r
+  \r
+  @retval EFI_OUT_OF_RESOURCES    No enough buffer to allocate\r
+  @retval EFI_SUCCESS             Op-code data successfully inserted  \r
+**/\r
 EFI_STATUS\r
 AddOpCode (\r
   IN      VOID                *FormBuffer,\r
   IN OUT  VOID                *OpCodeData\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Add op-code data to the FormBuffer\r
-\r
-Arguments:\r
-\r
-  FormBuffer      - Form buffer to be inserted to\r
-\r
-  OpCodeData      - Op-code data to be inserted\r
-\r
-Returns:\r
-\r
-  EFI_OUT_OF_RESOURCES    - No enough buffer to allocate\r
-\r
-  EFI_SUCCESS             - Op-code data successfully inserted\r
-\r
---*/\r
 {\r
   EFI_HII_PACK_HEADER *NewBuffer;\r
   UINT8               *Source;\r
@@ -372,52 +327,52 @@ Returns:
   //\r
   // Copy data to the new buffer until we run into the end_form\r
   //\r
-  for (; ((EFI_IFR_OP_HEADER *) Source)->OpCode != EFI_IFR_END_FORM_OP;) {\r
+  for (; ((FRAMEWORK_EFI_IFR_OP_HEADER *) Source)->OpCode != FRAMEWORK_EFI_IFR_END_FORM_OP;) {\r
     //\r
     // If the this opcode is an end_form_set we better be creating and endform\r
     // Nonetheless, we will add data before the end_form_set.  This also provides\r
     // for interesting behavior in the code we will run, but has no bad side-effects\r
     // since we will possibly do a 0 byte copy in this particular end-case.\r
     //\r
-    if (((EFI_IFR_OP_HEADER *) Source)->OpCode == EFI_IFR_END_FORM_SET_OP) {\r
+    if (((FRAMEWORK_EFI_IFR_OP_HEADER *) Source)->OpCode == FRAMEWORK_EFI_IFR_END_FORM_SET_OP) {\r
       break;\r
     }\r
 \r
     //\r
     // Copy data to new buffer\r
     //\r
-    CopyMem (Destination, Source, ((EFI_IFR_OP_HEADER *) Source)->Length);\r
+    CopyMem (Destination, Source, ((FRAMEWORK_EFI_IFR_OP_HEADER *) Source)->Length);\r
 \r
     //\r
     // Adjust Source/Destination to next op-code location\r
     //\r
-    Destination = Destination + (UINTN) ((EFI_IFR_OP_HEADER *) Source)->Length;\r
-    Source      = Source + (UINTN) ((EFI_IFR_OP_HEADER *) Source)->Length;\r
+    Destination = Destination + (UINTN) ((FRAMEWORK_EFI_IFR_OP_HEADER *) Source)->Length;\r
+    Source      = Source + (UINTN) ((FRAMEWORK_EFI_IFR_OP_HEADER *) Source)->Length;\r
   }\r
 \r
   //\r
   // Prior to the end_form is where we insert the new op-code data\r
   //\r
-  CopyMem (Destination, OpCodeData, ((EFI_IFR_OP_HEADER *) OpCodeData)->Length);\r
-  Destination       = Destination + (UINTN) ((EFI_IFR_OP_HEADER *) OpCodeData)->Length;\r
+  CopyMem (Destination, OpCodeData, ((FRAMEWORK_EFI_IFR_OP_HEADER *) OpCodeData)->Length);\r
+  Destination       = Destination + (UINTN) ((FRAMEWORK_EFI_IFR_OP_HEADER *) OpCodeData)->Length;\r
 \r
-  NewBuffer->Length = (UINT32) (NewBuffer->Length + (UINT32) (((EFI_IFR_OP_HEADER *) OpCodeData)->Length));\r
+  NewBuffer->Length = (UINT32) (NewBuffer->Length + (UINT32) (((FRAMEWORK_EFI_IFR_OP_HEADER *) OpCodeData)->Length));\r
 \r
   //\r
   // Copy end-form data to new buffer\r
   //\r
-  CopyMem (Destination, Source, ((EFI_IFR_OP_HEADER *) Source)->Length);\r
+  CopyMem (Destination, Source, ((FRAMEWORK_EFI_IFR_OP_HEADER *) Source)->Length);\r
 \r
   //\r
   // Adjust Source/Destination to next op-code location\r
   //\r
-  Destination = Destination + (UINTN) ((EFI_IFR_OP_HEADER *) Source)->Length;\r
-  Source      = Source + (UINTN) ((EFI_IFR_OP_HEADER *) Source)->Length;\r
+  Destination = Destination + (UINTN) ((FRAMEWORK_EFI_IFR_OP_HEADER *) Source)->Length;\r
+  Source      = Source + (UINTN) ((FRAMEWORK_EFI_IFR_OP_HEADER *) Source)->Length;\r
 \r
   //\r
   // Copy end-formset data to new buffer\r
   //\r
-  CopyMem (Destination, Source, ((EFI_IFR_OP_HEADER *) Source)->Length);\r
+  CopyMem (Destination, Source, ((FRAMEWORK_EFI_IFR_OP_HEADER *) Source)->Length);\r
 \r
   //\r
   // Zero out the original buffer and copy the updated data in the new buffer to the old buffer\r
@@ -432,26 +387,18 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+  Get the HII protocol interface\r
+  \r
+  @param Hii     HII protocol interface\r
+  \r
+  @return the statue of locating HII protocol\r
+**/\r
 STATIC\r
 EFI_STATUS\r
 GetHiiInterface (\r
   OUT     EFI_HII_PROTOCOL    **Hii\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Get the HII protocol interface\r
-\r
-Arguments:\r
-\r
-  Hii     - HII protocol interface\r
-\r
-Returns:\r
-\r
-  Status code\r
-\r
---*/\r
 {\r
   EFI_STATUS  Status;\r
 \r
@@ -467,41 +414,26 @@ Returns:
   return Status;;\r
 }\r
 \r
-\r
+/**\r
+  Extract information pertaining to the HiiHandle\r
+  \r
+  @param HiiHandle       Hii handle\r
+  @param ImageLength     For input, length of DefaultImage;\r
+                         For output, length of actually required\r
+  @param DefaultImage    Image buffer prepared by caller\r
+  @param Guid            Guid information about the form \r
+  \r
+  @retval EFI_OUT_OF_RESOURCES    No enough buffer to allocate\r
+  @retval EFI_BUFFER_TOO_SMALL    DefualtImage has no enough ImageLength\r
+  @retval EFI_SUCCESS             Successfully extract data from Hii database.\r
+**/\r
 EFI_STATUS\r
 ExtractDataFromHiiHandle (\r
-  IN      EFI_HII_HANDLE      HiiHandle,\r
+  IN      FRAMEWORK_EFI_HII_HANDLE       HiiHandle,\r
   IN OUT  UINT16              *ImageLength,\r
   OUT     UINT8               *DefaultImage,\r
   OUT     EFI_GUID            *Guid\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Extract information pertaining to the HiiHandle\r
-\r
-Arguments:\r
-\r
-  HiiHandle       - Hii handle\r
-\r
-  ImageLength     - For input, length of DefaultImage;\r
-                    For output, length of actually required\r
-\r
-  DefaultImage    - Image buffer prepared by caller\r
-\r
-  Guid            - Guid information about the form\r
-\r
-Returns:\r
-\r
-  EFI_OUT_OF_RESOURCES    - No enough buffer to allocate\r
-\r
-  EFI_BUFFER_TOO_SMALL    - DefualtImage has no enough ImageLength\r
-\r
-  EFI_SUCCESS             - Successfully extract data from Hii database.\r
-\r
-\r
---*/\r
 {\r
   EFI_STATUS        Status;\r
   EFI_HII_PROTOCOL  *Hii;\r
@@ -560,31 +492,31 @@ Returns:
   //\r
   RawData = (UINT8 *) ((UINTN) RawData + sizeof (EFI_HII_PACK_HEADER));\r
 \r
-  for (Index = 0; RawData[Index] != EFI_IFR_END_FORM_SET_OP;) {\r
+  for (Index = 0; RawData[Index] != FRAMEWORK_EFI_IFR_END_FORM_SET_OP;) {\r
     switch (RawData[Index]) {\r
-    case EFI_IFR_FORM_SET_OP:\r
+    case FRAMEWORK_EFI_IFR_FORM_SET_OP:\r
       //\r
       // Copy the GUID information from this handle\r
       //\r
-      CopyMem (Guid, &((EFI_IFR_FORM_SET *) &RawData[Index])->Guid, sizeof (EFI_GUID));\r
+      CopyMem (Guid, &((FRAMEWORK_EFI_IFR_FORM_SET *) &RawData[Index])->Guid, sizeof (EFI_GUID));\r
       break;\r
 \r
-    case EFI_IFR_ONE_OF_OP:\r
-    case EFI_IFR_CHECKBOX_OP:\r
-    case EFI_IFR_NUMERIC_OP:\r
-    case EFI_IFR_DATE_OP:\r
-    case EFI_IFR_TIME_OP:\r
-    case EFI_IFR_PASSWORD_OP:\r
-    case EFI_IFR_STRING_OP:\r
+    case FRAMEWORK_EFI_IFR_ONE_OF_OP:\r
+    case FRAMEWORK_EFI_IFR_CHECKBOX_OP:\r
+    case FRAMEWORK_EFI_IFR_NUMERIC_OP:\r
+    case FRAMEWORK_EFI_IFR_DATE_OP:\r
+    case FRAMEWORK_EFI_IFR_TIME_OP:\r
+    case FRAMEWORK_EFI_IFR_PASSWORD_OP:\r
+    case FRAMEWORK_EFI_IFR_STRING_OP:\r
       //\r
       // Remember, multiple op-codes may reference the same item, so let's keep a running\r
       // marker of what the highest QuestionId that wasn't zero length.  This will accurately\r
       // maintain the Size of the NvStore\r
       //\r
-      if (((EFI_IFR_ONE_OF *) &RawData[Index])->Width != 0) {\r
-        Temp = ((EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId + ((EFI_IFR_ONE_OF *) &RawData[Index])->Width;\r
+      if (((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->Width != 0) {\r
+        Temp = ((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId + ((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->Width;\r
         if (SizeOfNvStore < Temp) {\r
-          SizeOfNvStore = ((EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId + ((EFI_IFR_ONE_OF *) &RawData[Index])->Width;\r
+          SizeOfNvStore = ((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId + ((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->Width;\r
         }\r
       }\r
     }\r
@@ -608,26 +540,26 @@ Returns:
   //\r
   // Copy the default image information to the user's buffer\r
   //\r
-  for (Index = 0; RawData[Index] != EFI_IFR_END_FORM_SET_OP;) {\r
+  for (Index = 0; RawData[Index] != FRAMEWORK_EFI_IFR_END_FORM_SET_OP;) {\r
     switch (RawData[Index]) {\r
-    case EFI_IFR_ONE_OF_OP:\r
-      CachedStart = ((EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId;\r
+    case FRAMEWORK_EFI_IFR_ONE_OF_OP:\r
+      CachedStart = ((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId;\r
       break;\r
 \r
-    case EFI_IFR_ONE_OF_OPTION_OP:\r
-      if (((EFI_IFR_ONE_OF_OPTION *) &RawData[Index])->Flags & EFI_IFR_FLAG_DEFAULT) {\r
-        CopyMem (&DefaultImage[CachedStart], &((EFI_IFR_ONE_OF_OPTION *) &RawData[Index])->Value, 2);\r
+    case FRAMEWORK_EFI_IFR_ONE_OF_OPTION_OP:\r
+      if (((FRAMEWORK_EFI_IFR_ONE_OF_OPTION *) &RawData[Index])->Flags & FRAMEWORK_EFI_IFR_FLAG_DEFAULT) {\r
+        CopyMem (&DefaultImage[CachedStart], &((FRAMEWORK_EFI_IFR_ONE_OF_OPTION *) &RawData[Index])->Value, 2);\r
       }\r
       break;\r
 \r
-    case EFI_IFR_CHECKBOX_OP:\r
-      DefaultImage[((EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId] = ((EFI_IFR_CHECKBOX *) &RawData[Index])->Flags;\r
+    case FRAMEWORK_EFI_IFR_CHECKBOX_OP:\r
+      DefaultImage[((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId] = ((FRAMEWORK_EFI_IFR_CHECKBOX *) &RawData[Index])->Flags;\r
       break;\r
 \r
-    case EFI_IFR_NUMERIC_OP:\r
+    case FRAMEWORK_EFI_IFR_NUMERIC_OP:\r
       CopyMem (\r
-        &DefaultImage[((EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId],\r
-        &((EFI_IFR_NUMERIC *) &RawData[Index])->Default,\r
+        &DefaultImage[((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId],\r
+        &((FRAMEWORK_EFI_IFR_NUMERIC *) &RawData[Index])->Default,\r
         2\r
         );\r
       break;\r
@@ -647,33 +579,27 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
-EFI_HII_HANDLE\r
+/**\r
+  Finds HII handle for given pack GUID previously registered with the HII.\r
+  \r
+  @param HiiProtocol pointer to pointer to HII protocol interface.\r
+                     If NULL, the interface will be found but not returned.\r
+                     If it points to NULL, the interface will be found and\r
+                     written back to the pointer that is pointed to.\r
+  @param Guid        The GUID of the pack that registered with the HII.\r
+\r
+  @return  Handle to the HII pack previously registered by the memory driver.\r
+**/\r
+FRAMEWORK_EFI_HII_HANDLE \r
 FindHiiHandle (\r
   IN OUT EFI_HII_PROTOCOL    **HiiProtocol, OPTIONAL\r
   IN     EFI_GUID            *Guid\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Finds HII handle for given pack GUID previously registered with the HII.\r
-\r
-Arguments:\r
-  HiiProtocol - pointer to pointer to HII protocol interface.\r
-                If NULL, the interface will be found but not returned.\r
-                If it points to NULL, the interface will be found and\r
-                written back to the pointer that is pointed to.\r
-  Guid        - The GUID of the pack that registered with the HII.\r
-\r
-Returns:\r
-  Handle to the HII pack previously registered by the memory driver.\r
-\r
---*/\r
 {\r
   EFI_STATUS        Status;\r
 \r
-  EFI_HII_HANDLE    *HiiHandleBuffer;\r
-  EFI_HII_HANDLE    HiiHandle;\r
+  FRAMEWORK_EFI_HII_HANDLE     *HiiHandleBuffer;\r
+  FRAMEWORK_EFI_HII_HANDLE     HiiHandle;\r
   UINT16            HiiHandleBufferLength;\r
   UINT32            NumberOfHiiHandles;\r
   EFI_GUID          HiiGuid;\r
@@ -741,7 +667,7 @@ Returns:
     goto lbl_exit;\r
   }\r
 \r
-  NumberOfHiiHandles = HiiHandleBufferLength / sizeof (EFI_HII_HANDLE);\r
+  NumberOfHiiHandles = HiiHandleBufferLength / sizeof (FRAMEWORK_EFI_HII_HANDLE );\r
 \r
   //\r
   // Iterate Hii handles and look for the one that matches our Guid\r
@@ -763,34 +689,24 @@ lbl_exit:
   return HiiHandle;\r
 }\r
 \r
-\r
-EFI_STATUS\r
-ValidateDataFromHiiHandle (\r
-  IN      EFI_HII_HANDLE      HiiHandle,\r
-  OUT     BOOLEAN             *Results\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Validate that the data associated with the HiiHandle in NVRAM is within\r
   the reasonable parameters for that FormSet.  Values for strings and passwords\r
   are not verified due to their not having the equivalent of valid range settings.\r
 \r
-Arguments:\r
-\r
-  HiiHandle -   Handle of the HII database entry to query\r
-\r
-  Results -     If return Status is EFI_SUCCESS, Results provides valid data\r
-                TRUE  = NVRAM Data is within parameters\r
-                FALSE = NVRAM Data is NOT within parameters\r
-\r
-Returns:\r
-\r
-  EFI_OUT_OF_RESOURCES      - No enough buffer to allocate\r
+  @param HiiHandle    Handle of the HII database entry to query\r
 \r
-  EFI_SUCCESS               - Data successfully validated\r
---*/\r
+  @param Results      If return Status is EFI_SUCCESS, Results provides valid data\r
+                      TRUE  = NVRAM Data is within parameters\r
+                      FALSE = NVRAM Data is NOT within parameters\r
+  @retval EFI_OUT_OF_RESOURCES      No enough buffer to allocate\r
+  @retval EFI_SUCCESS               Data successfully validated\r
+**/\r
+EFI_STATUS\r
+ValidateDataFromHiiHandle (\r
+  IN      FRAMEWORK_EFI_HII_HANDLE       HiiHandle,\r
+  OUT     BOOLEAN             *Results\r
+  )\r
 {\r
   EFI_STATUS        Status;\r
   EFI_HII_PROTOCOL  *Hii;\r
@@ -854,36 +770,36 @@ Returns:
   //\r
   RawData = (UINT8 *) ((UINTN) RawData + sizeof (EFI_HII_PACK_HEADER));\r
 \r
-  for (Index = 0; RawData[Index] != EFI_IFR_END_FORM_SET_OP;) {\r
-    if (RawData[Index] == EFI_IFR_FORM_SET_OP) {\r
-      CopyMem (&Guid, &((EFI_IFR_FORM_SET *) &RawData[Index])->Guid, sizeof (EFI_GUID));\r
+  for (Index = 0; RawData[Index] != FRAMEWORK_EFI_IFR_END_FORM_SET_OP;) {\r
+    if (RawData[Index] == FRAMEWORK_EFI_IFR_FORM_SET_OP) {\r
+      CopyMem (&Guid, &((FRAMEWORK_EFI_IFR_FORM_SET *) &RawData[Index])->Guid, sizeof (EFI_GUID));\r
       break;\r
     }\r
 \r
     Index = RawData[Index + 1] + Index;\r
   }\r
 \r
-  for (Index = 0; RawData[Index] != EFI_IFR_END_FORM_SET_OP;) {\r
+  for (Index = 0; RawData[Index] != FRAMEWORK_EFI_IFR_END_FORM_SET_OP;) {\r
     switch (RawData[Index]) {\r
-    case EFI_IFR_FORM_SET_OP:\r
+    case FRAMEWORK_EFI_IFR_FORM_SET_OP:\r
       break;\r
 \r
-    case EFI_IFR_ONE_OF_OP:\r
-    case EFI_IFR_CHECKBOX_OP:\r
-    case EFI_IFR_NUMERIC_OP:\r
-    case EFI_IFR_DATE_OP:\r
-    case EFI_IFR_TIME_OP:\r
-    case EFI_IFR_PASSWORD_OP:\r
-    case EFI_IFR_STRING_OP:\r
+    case FRAMEWORK_EFI_IFR_ONE_OF_OP:\r
+    case FRAMEWORK_EFI_IFR_CHECKBOX_OP:\r
+    case FRAMEWORK_EFI_IFR_NUMERIC_OP:\r
+    case FRAMEWORK_EFI_IFR_DATE_OP:\r
+    case FRAMEWORK_EFI_IFR_TIME_OP:\r
+    case FRAMEWORK_EFI_IFR_PASSWORD_OP:\r
+    case FRAMEWORK_EFI_IFR_STRING_OP:\r
       //\r
       // Remember, multiple op-codes may reference the same item, so let's keep a running\r
       // marker of what the highest QuestionId that wasn't zero length.  This will accurately\r
       // maintain the Size of the NvStore\r
       //\r
-      if (((EFI_IFR_ONE_OF *) &RawData[Index])->Width != 0) {\r
-        Temp = ((EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId + ((EFI_IFR_ONE_OF *) &RawData[Index])->Width;\r
+      if (((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->Width != 0) {\r
+        Temp = ((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId + ((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->Width;\r
         if (SizeOfNvStore < Temp) {\r
-          SizeOfNvStore = ((EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId + ((EFI_IFR_ONE_OF *) &RawData[Index])->Width;\r
+          SizeOfNvStore = ((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId + ((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->Width;\r
         }\r
       }\r
     }\r
@@ -942,25 +858,25 @@ Returns:
   // This allows for the possibility of stale (obsoleted) data in the variable\r
   // can be overlooked without causing an error\r
   //\r
-  for (Index = 0; RawData[Index] != EFI_IFR_END_FORM_SET_OP;) {\r
+  for (Index = 0; RawData[Index] != FRAMEWORK_EFI_IFR_END_FORM_SET_OP;) {\r
     switch (RawData[Index]) {\r
-    case EFI_IFR_ONE_OF_OP:\r
+    case FRAMEWORK_EFI_IFR_ONE_OF_OP:\r
       //\r
       // A one_of has no data, its the option that does - cache the storage Id\r
       //\r
-      CachedStart = ((EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId;\r
+      CachedStart = ((FRAMEWORK_EFI_IFR_ONE_OF *) &RawData[Index])->QuestionId;\r
       break;\r
 \r
-    case EFI_IFR_ONE_OF_OPTION_OP:\r
+    case FRAMEWORK_EFI_IFR_ONE_OF_OPTION_OP:\r
       //\r
       // A one_of_option can be any value\r
       //\r
-      if (VariableData[CachedStart] == ((EFI_IFR_ONE_OF_OPTION *) &RawData[Index])->Value) {\r
+      if (VariableData[CachedStart] == ((FRAMEWORK_EFI_IFR_ONE_OF_OPTION *) &RawData[Index])->Value) {\r
         GotMatch = TRUE;\r
       }\r
       break;\r
 \r
-    case EFI_IFR_END_ONE_OF_OP:\r
+    case FRAMEWORK_EFI_IFR_END_ONE_OF_OP:\r
       //\r
       // At this point lets make sure that the data value in the NVRAM matches one of the options\r
       //\r
@@ -970,20 +886,20 @@ Returns:
       }\r
       break;\r
 \r
-    case EFI_IFR_CHECKBOX_OP:\r
+    case FRAMEWORK_EFI_IFR_CHECKBOX_OP:\r
       //\r
       // A checkbox is a boolean, so 0 and 1 are valid\r
       // Remember, QuestionId corresponds to the offset location of the data in the variable\r
       //\r
-      if (VariableData[((EFI_IFR_CHECKBOX *) &RawData[Index])->QuestionId] > 1) {\r
+      if (VariableData[((FRAMEWORK_EFI_IFR_CHECKBOX *) &RawData[Index])->QuestionId] > 1) {\r
         *Results = FALSE;\r
         return EFI_SUCCESS;\r
       }\r
       break;\r
 \r
-    case EFI_IFR_NUMERIC_OP:\r
-        if ((VariableData[((EFI_IFR_NUMERIC *)&RawData[Index])->QuestionId] < ((EFI_IFR_NUMERIC *)&RawData[Index])->Minimum) ||\r
-            (VariableData[((EFI_IFR_NUMERIC *)&RawData[Index])->QuestionId] > ((EFI_IFR_NUMERIC *)&RawData[Index])->Maximum)) {\r
+    case FRAMEWORK_EFI_IFR_NUMERIC_OP:\r
+        if ((VariableData[((FRAMEWORK_EFI_IFR_NUMERIC *)&RawData[Index])->QuestionId] < ((FRAMEWORK_EFI_IFR_NUMERIC *)&RawData[Index])->Minimum) ||\r
+            (VariableData[((FRAMEWORK_EFI_IFR_NUMERIC *)&RawData[Index])->QuestionId] > ((FRAMEWORK_EFI_IFR_NUMERIC *)&RawData[Index])->Maximum)) {\r
         *Results = FALSE;\r
         return EFI_SUCCESS;\r
       }\r
@@ -1003,3 +919,4 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+\r