]> git.proxmox.com Git - mirror_edk2.git/commitdiff
BaseTool/VfrCompiler: Support Bit fields in EFI/Buffer VarStore
authorDandan Bi <dandan.bi@intel.com>
Wed, 23 Aug 2017 08:27:26 +0000 (16:27 +0800)
committerEric Dong <eric.dong@intel.com>
Thu, 21 Sep 2017 08:17:40 +0000 (16:17 +0800)
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=545

Enhance VfrCompiler to parse following case:
1. EFI/Buffer VarStore can contain bit fields in their structure.
2. For question Oneof/Checkbox/numeric, their storage can be
   bit fields of an EFI VarStore/Buffer VarStore.

Cc: Eric Dong <eric.dong@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Eric Dong <eric.dong@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
BaseTools/Source/C/Include/Common/MdeModuleHii.h
BaseTools/Source/C/VfrCompile/VfrError.cpp
BaseTools/Source/C/VfrCompile/VfrError.h
BaseTools/Source/C/VfrCompile/VfrFormPkg.cpp
BaseTools/Source/C/VfrCompile/VfrFormPkg.h
BaseTools/Source/C/VfrCompile/VfrSyntax.g
BaseTools/Source/C/VfrCompile/VfrUtilityLib.cpp
BaseTools/Source/C/VfrCompile/VfrUtilityLib.h

index 21c83874748693516bcafec7e77644f420f9739b..e6538e2fdd5230e7d4d8f8c187f3d6ad1b7c6f67 100644 (file)
@@ -1,7 +1,7 @@
 /** @file\r
   EDK II specific HII relative definition.\r
 \r
-  Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.<BR>\r
 \r
   This program and the accompanying materials are licensed and made available\r
   under the terms and conditions of the BSD License which accompanies this\r
@@ -212,10 +212,28 @@ typedef struct _EFI_IFR_GUID_VAREQNAME {
   UINT16              NameId;\r
 } EFI_IFR_GUID_VAREQNAME;\r
 \r
+///\r
+/// EDKII implementation extension GUID, used to indaicate there are bit fields in the varstore.\r
+///\r
+#define EDKII_IFR_BIT_VARSTORE_GUID \\r
+  { 0x82DDD68B, 0x9163, 0x4187, {0x9B, 0x27, 0x20, 0xA8, 0xFD, 0x60 ,0xA7, 0x1D}}\r
+\r
+///\r
+/// EDKII implementation extension flags, used to indaicate the disply style and bit width for bit filed storage.\r
+/// Two high bits for display style and the low six bits for bit width.\r
+///\r
+#define EDKII_IFR_DISPLAY_BIT            0xC0\r
+#define EDKII_IFR_DISPLAY_INT_DEC_BIT    0x00\r
+#define EDKII_IFR_DISPLAY_UINT_DEC_BIT   0x40\r
+#define EDKII_IFR_DISPLAY_UINT_HEX_BIT   0x80\r
+\r
+#define EDKII_IFR_NUMERIC_SIZE_BIT       0x3F\r
+\r
 #pragma pack()\r
 \r
 extern EFI_GUID gEfiIfrTianoGuid;\r
 extern EFI_GUID gEfiIfrFrameworkGuid;\r
+extern EFI_GUID gEdkiiIfrBitVarStoreGuid;\r
 \r
 #endif\r
 \r
index 285e175fa60fc2bc26c637b949ea6078a37c5dee..0f8f7dd891b3d8d6a2b1fe071c531e5d7719e4c6 100644 (file)
@@ -2,7 +2,7 @@
   \r
   VfrCompiler error handler.\r
 \r
-Copyright (c) 2004 - 2016, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2004 - 2017, Intel Corporation. All rights reserved.<BR>\r
 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
@@ -46,6 +46,7 @@ static SVFR_ERROR_HANDLE VFR_ERROR_HANDLE_TABLE [] = {
   { VFR_RETURN_DEFAULT_VALUE_REDEFINED, ": default value re-defined with different value"},\r
   { VFR_RETURN_CONSTANT_ONLY, ": only constant is allowed in the expression"},\r
   { VFR_RETURN_VARSTORE_NAME_REDEFINED_ERROR, ": Varstore name is defined by more than one varstores, it can't be referred as varstore, only varstore strucure name could be used."},\r
+  { VFR_RETURN_BIT_WIDTH_ERROR, ": bit width must be <= sizeof (type) * 8 and the max width can not > 32" },\r
   { VFR_RETURN_CODEUNDEFINED, ": undefined Error Code" }\r
 };\r
 \r
index 4dbc54c504f1aac61390fb9369c1a19313bd0d53..8bae04953191e616137dc0137c59667f00449f6c 100644 (file)
@@ -2,7 +2,7 @@
   \r
   VfrCompiler Error definition\r
 \r
-Copyright (c) 2004 - 2016, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2004 - 2017, Intel Corporation. All rights reserved.<BR>\r
 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
@@ -44,6 +44,7 @@ typedef enum {
   VFR_RETURN_DEFAULT_VALUE_REDEFINED,\r
   VFR_RETURN_CONSTANT_ONLY,\r
   VFR_RETURN_VARSTORE_NAME_REDEFINED_ERROR,\r
+  VFR_RETURN_BIT_WIDTH_ERROR,\r
   VFR_RETURN_CODEUNDEFINED\r
 } EFI_VFR_RETURN_CODE;\r
 \r
index 2be0c385e31371a28ce542ec78f166907301fff5..b8350623f2a06f8c6f3dc6855f43bec5e3018448 100644 (file)
@@ -2,7 +2,7 @@
   \r
   The definition of CFormPkg's member function\r
 \r
-Copyright (c) 2004 - 2016, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2004 - 2017, Intel Corporation. All rights reserved.<BR>\r
 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
@@ -829,6 +829,9 @@ CFormPkg::DeclarePendingQuestion (
   UINT32         ShrinkSize = 0;\r
   EFI_VFR_RETURN_CODE  ReturnCode;\r
   EFI_VFR_VARSTORE_TYPE VarStoreType  = EFI_VFR_VARSTORE_INVALID;\r
+  UINT8    LFlags;\r
+  UINT32   MaxValue;\r
+  CIfrGuid *GuidObj = NULL;\r
 \r
   //\r
   // Declare all questions as Numeric in DisableIf True\r
@@ -844,14 +847,8 @@ CFormPkg::DeclarePendingQuestion (
   // Declare Numeric qeustion for each undefined question.\r
   for (pNode = PendingAssignList; pNode != NULL; pNode = pNode->mNext) {\r
     if (pNode->mFlag == PENDING) {\r
-      CIfrNumeric CNObj;\r
       EFI_VARSTORE_INFO Info; \r
       EFI_QUESTION_ID   QId   = EFI_QUESTION_ID_INVALID;\r
-\r
-      CNObj.SetLineNo (LineNo);\r
-      CNObj.SetPrompt (0x0);\r
-      CNObj.SetHelp (0x0);\r
-\r
       //\r
       // Register this question, assume it is normal question, not date or time question\r
       //\r
@@ -888,7 +885,7 @@ CFormPkg::DeclarePendingQuestion (
       } else {\r
         if (VarStoreType == EFI_VFR_VARSTORE_EFI) {\r
           ReturnCode = lCVfrDataStorage.GetEfiVarStoreInfo (&Info);\r
-        } else if (VarStoreType == EFI_VFR_VARSTORE_BUFFER) {\r
+        } else if (VarStoreType == EFI_VFR_VARSTORE_BUFFER || VarStoreType == EFI_VFR_VARSTORE_BUFFER_BITS) {\r
           VarStr = pNode->mKey;\r
           //convert VarStr with store name to VarStr with structure name\r
           ReturnCode = lCVfrDataStorage.GetBufferVarStoreDataTypeName (Info.mVarStoreId, &SName);\r
@@ -897,7 +894,7 @@ CFormPkg::DeclarePendingQuestion (
             NewStr[0] = '\0';\r
             strcpy (NewStr, SName);\r
             strcat (NewStr, VarStr + strlen (FName));\r
-            ReturnCode = lCVfrVarDataTypeDB.GetDataFieldInfo (NewStr, Info.mInfo.mVarOffset, Info.mVarType, Info.mVarTotalSize);\r
+            ReturnCode = lCVfrVarDataTypeDB.GetDataFieldInfo (NewStr, Info.mInfo.mVarOffset, Info.mVarType, Info.mVarTotalSize, Info.mIsBitVar);\r
             delete[] NewStr;\r
           }\r
         } else {\r
@@ -908,39 +905,63 @@ CFormPkg::DeclarePendingQuestion (
         gCVfrErrorHandle.HandleError (ReturnCode, pNode->mLineNo, pNode->mKey);\r
         return ReturnCode;\r
       }\r
-\r
-      CNObj.SetQuestionId (QId);\r
-      CNObj.SetVarStoreInfo (&Info);\r
       //\r
-      // Numeric doesn't support BOOLEAN data type. \r
-      // BOOLEAN type has the same data size to UINT8. \r
+      // If the storage is bit fields, create Guid opcode to wrap the numeric opcode.\r
       //\r
-      if (Info.mVarType == EFI_IFR_TYPE_BOOLEAN) {\r
-        Info.mVarType = EFI_IFR_TYPE_NUM_SIZE_8;\r
+      if (Info.mIsBitVar) {\r
+        GuidObj = new CIfrGuid(0);\r
+        GuidObj->SetGuid (&gEdkiiIfrBitVarGuid);\r
+        GuidObj->SetLineNo(LineNo);\r
       }\r
-      CNObj.SetFlags (0, Info.mVarType);\r
+\r
+      CIfrNumeric CNObj;\r
+      CNObj.SetLineNo (LineNo);\r
+      CNObj.SetPrompt (0x0);\r
+      CNObj.SetHelp (0x0);\r
+      CNObj.SetQuestionId (QId);\r
+      CNObj.SetVarStoreInfo (&Info);\r
+\r
       //\r
-      // Use maximum value not to limit the vaild value for the undefined question.\r
+      // Set Min/Max/Step Data and flags for the question with bit fields.Min/Max/Step Data are saved as UINT32 type for bit question.\r
       //\r
-      switch (Info.mVarType) {\r
-      case EFI_IFR_TYPE_NUM_SIZE_64:\r
-        CNObj.SetMinMaxStepData ((UINT64) 0, (UINT64) -1 , (UINT64) 0);\r
-        ShrinkSize = 0;\r
-        break;\r
-      case EFI_IFR_TYPE_NUM_SIZE_32:\r
-        CNObj.SetMinMaxStepData ((UINT32) 0, (UINT32) -1 , (UINT32) 0);\r
+      if (Info.mIsBitVar) {\r
+        MaxValue = (1 << Info.mVarTotalSize) -1;\r
+        CNObj.SetMinMaxStepData ((UINT32) 0, MaxValue, (UINT32) 0);\r
         ShrinkSize = 12;\r
-        break;\r
-      case EFI_IFR_TYPE_NUM_SIZE_16:\r
-        CNObj.SetMinMaxStepData ((UINT16) 0, (UINT16) -1 , (UINT16) 0);\r
-        ShrinkSize = 18;\r
-        break;\r
-      case EFI_IFR_TYPE_NUM_SIZE_8:\r
-        CNObj.SetMinMaxStepData ((UINT8) 0, (UINT8) -1 , (UINT8) 0);\r
-        ShrinkSize = 21;\r
-        break;\r
-      default:\r
-        break;\r
+        LFlags = (EDKII_IFR_NUMERIC_SIZE_BIT & Info.mVarTotalSize);\r
+        CNObj.SetFlagsForBitField (0, LFlags);\r
+      } else {\r
+        //\r
+        // Numeric doesn't support BOOLEAN data type.\r
+        // BOOLEAN type has the same data size to UINT8.\r
+        //\r
+        if (Info.mVarType == EFI_IFR_TYPE_BOOLEAN) {\r
+          Info.mVarType = EFI_IFR_TYPE_NUM_SIZE_8;\r
+        }\r
+        CNObj.SetFlags (0, Info.mVarType);\r
+        //\r
+        // Use maximum value not to limit the vaild value for the undefined question.\r
+        //\r
+        switch (Info.mVarType) {\r
+        case EFI_IFR_TYPE_NUM_SIZE_64:\r
+          CNObj.SetMinMaxStepData ((UINT64) 0, (UINT64) -1 , (UINT64) 0);\r
+          ShrinkSize = 0;\r
+          break;\r
+        case EFI_IFR_TYPE_NUM_SIZE_32:\r
+          CNObj.SetMinMaxStepData ((UINT32) 0, (UINT32) -1 , (UINT32) 0);\r
+          ShrinkSize = 12;\r
+          break;\r
+        case EFI_IFR_TYPE_NUM_SIZE_16:\r
+          CNObj.SetMinMaxStepData ((UINT16) 0, (UINT16) -1 , (UINT16) 0);\r
+          ShrinkSize = 18;\r
+          break;\r
+        case EFI_IFR_TYPE_NUM_SIZE_8:\r
+          CNObj.SetMinMaxStepData ((UINT8) 0, (UINT8) -1 , (UINT8) 0);\r
+          ShrinkSize = 21;\r
+          break;\r
+        default:\r
+          break;\r
+        }\r
       }\r
       CNObj.ShrinkBinSize (ShrinkSize);\r
 \r
@@ -956,8 +977,18 @@ CFormPkg::DeclarePendingQuestion (
       //\r
       // End for Numeric\r
       //\r
-      CIfrEnd CEObj; \r
+      CIfrEnd CEObj;\r
       CEObj.SetLineNo (LineNo);\r
+      //\r
+      // End for Guided opcode\r
+      //\r
+      if (GuidObj != NULL) {\r
+        CIfrEnd CEObjGuid;\r
+        CEObjGuid.SetLineNo (LineNo);\r
+        GuidObj->SetScope(1);\r
+        delete GuidObj;\r
+        GuidObj = NULL;\r
+      }\r
     }\r
   }\r
 \r
index 17ab14c021dfacea7f4f1d064bc335dc90306979..8a22cb24e4b7be97f85b6f94b7e765a20f5175aa 100644 (file)
@@ -2,7 +2,7 @@
   \r
   The definition of CFormPkg's member function\r
 \r
-Copyright (c) 2004 - 2016, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2004 - 2017, Intel Corporation. All rights reserved.<BR>\r
 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
@@ -567,8 +567,12 @@ public:
     mMinMaxStepData->u8.Step = Step;\r
   }\r
 \r
-  UINT64 GetMinData (UINT8 VarType) {\r
+  UINT64 GetMinData (UINT8 VarType, BOOLEAN IsBitVar) {\r
     UINT64 MinValue = 0;\r
+    if (IsBitVar) {\r
+      MinValue = mMinMaxStepData->u32.MinValue;\r
+      return MinValue;\r
+    }\r
     switch (VarType) {\r
     case EFI_IFR_TYPE_NUM_SIZE_64:\r
       MinValue = mMinMaxStepData->u64.MinValue;\r
@@ -588,8 +592,12 @@ public:
     return MinValue;\r
   }\r
 \r
-  UINT64 GetMaxData (UINT8 VarType) {\r
+  UINT64 GetMaxData (UINT8 VarType, BOOLEAN IsBitVar) {\r
     UINT64 MaxValue = 0;\r
+    if (IsBitVar) {\r
+      MaxValue = mMinMaxStepData->u32.MaxValue;\r
+      return MaxValue;\r
+    }\r
     switch (VarType) {\r
     case EFI_IFR_TYPE_NUM_SIZE_64:\r
       MaxValue = mMinMaxStepData->u64.MaxValue;\r
@@ -609,8 +617,12 @@ public:
     return MaxValue;\r
   }\r
 \r
-  UINT64 GetStepData (UINT8 VarType) {\r
+  UINT64 GetStepData (UINT8 VarType, BOOLEAN IsBitVar) {\r
     UINT64 MaxValue = 0;\r
+    if (IsBitVar) {\r
+      MaxValue = mMinMaxStepData->u32.Step;\r
+      return MaxValue;\r
+    }\r
     switch (VarType) {\r
     case EFI_IFR_TYPE_NUM_SIZE_64:\r
       MaxValue = mMinMaxStepData->u64.Step;\r
@@ -1407,6 +1419,22 @@ public:
     return VFR_RETURN_SUCCESS;\r
   }\r
 \r
+  EFI_VFR_RETURN_CODE SetFlagsForBitField (IN UINT8 HFlags, IN UINT8 LFlags, BOOLEAN DisplaySettingsSpecified = FALSE) {\r
+    EFI_VFR_RETURN_CODE Ret;\r
+\r
+    Ret = CIfrQuestionHeader::SetFlags (HFlags);\r
+    if (Ret != VFR_RETURN_SUCCESS) {\r
+      return Ret;\r
+    }\r
+\r
+    if (DisplaySettingsSpecified == FALSE) {\r
+      mNumeric->Flags = LFlags | EDKII_IFR_DISPLAY_UINT_DEC_BIT;\r
+    } else {\r
+      mNumeric->Flags = LFlags;\r
+    }\r
+    return VFR_RETURN_SUCCESS;\r
+  }\r
+\r
   UINT8 GetNumericFlags () {\r
     return mNumeric->Flags;\r
   }\r
@@ -1447,6 +1475,22 @@ public:
     return VFR_RETURN_SUCCESS;\r
   }\r
 \r
+  EFI_VFR_RETURN_CODE SetFlagsForBitField (IN UINT8 HFlags, IN UINT8 LFlags) {\r
+    EFI_VFR_RETURN_CODE Ret;\r
+\r
+    Ret = CIfrQuestionHeader::SetFlags (HFlags);\r
+    if (Ret != VFR_RETURN_SUCCESS) {\r
+      return Ret;\r
+    }\r
+\r
+    if (LFlags & EFI_IFR_DISPLAY) {\r
+      mOneOf->Flags = LFlags;\r
+    } else {\r
+      mOneOf->Flags = LFlags | EDKII_IFR_DISPLAY_UINT_DEC_BIT;\r
+    }\r
+    return VFR_RETURN_SUCCESS;\r
+  }\r
+\r
   VOID ShrinkBinSize (IN UINT16 Size) {\r
     //\r
     // Update the buffer size which is truly be used later.\r
index 24e053827bcbe9aa17cad9da36cb7225536e0d0d..fbd5abfacad9d7667ac522e57d46da19e5e37d78 100644 (file)
@@ -363,7 +363,11 @@ vfrDataStructFields [BOOLEAN  FieldInUnion]:
      dataStructFieldDate  [FieldInUnion] |\r
      dataStructFieldTime  [FieldInUnion] |\r
      dataStructFieldRef   [FieldInUnion] |\r
-     dataStructFieldUser [FieldInUnion]\r
+     dataStructFieldUser  [FieldInUnion] |\r
+     dataStructBitField64 [FieldInUnion] |\r
+     dataStructBitField32 [FieldInUnion] |\r
+     dataStructBitField16 [FieldInUnion] |\r
+     dataStructBitField8  [FieldInUnion]\r
   )*\r
   ;\r
 \r
@@ -469,6 +473,83 @@ dataStructFieldUser [BOOLEAN  FieldInUnion]:
   ";"                                               << _PCATCH(gCVfrVarDataTypeDB.DataTypeAddField (N->getText(), T->getText(), ArrayNum, FieldInUnion), T); >>\r
   ;\r
 \r
+dataStructBitField64[BOOLEAN  FieldInUnion]:\r
+  <<\r
+    UINT32 Width = 0;\r
+    BOOLEAN HasBitFieldName = FALSE;\r
+  >>\r
+  D:"UINT64"\r
+  {\r
+    N:StringIdentifier      << HasBitFieldName = TRUE;>>\r
+  }\r
+  ":" I:Number              << Width = _STOU32(I->getText(), I->getLine());>>\r
+\r
+  ";"                       << if (HasBitFieldName) {\r
+                                 _PCATCH(gCVfrVarDataTypeDB.DataTypeAddBitField (N->getText(), D->getText(), Width, FieldInUnion), N);\r
+                               } else {\r
+                                 _PCATCH(gCVfrVarDataTypeDB.DataTypeAddBitField (NULL, D->getText(), Width, FieldInUnion), D);\r
+                               }\r
+                            >>\r
+  ;\r
+\r
+dataStructBitField32[BOOLEAN  FieldInUnion]:\r
+  <<\r
+    UINT32 Width = 0;\r
+    BOOLEAN HasBitFieldName = FALSE;\r
+  >>\r
+  D:"UINT32"\r
+  {\r
+    N:StringIdentifier      << HasBitFieldName = TRUE;>>\r
+  }\r
+\r
+  ":" I:Number              << Width = _STOU32(I->getText(), I->getLine());>>\r
+\r
+  ";"                       << if (HasBitFieldName) {\r
+                                 _PCATCH(gCVfrVarDataTypeDB.DataTypeAddBitField (N->getText(), D->getText(), Width, FieldInUnion), N);\r
+                               } else {\r
+                                 _PCATCH(gCVfrVarDataTypeDB.DataTypeAddBitField (NULL, D->getText(), Width, FieldInUnion), D);\r
+                               }\r
+                            >>\r
+  ;\r
+\r
+dataStructBitField16[BOOLEAN  FieldInUnion]:\r
+  <<\r
+    UINT32 Width = 0;\r
+    BOOLEAN HasBitFieldName = FALSE;\r
+  >>\r
+  D:"UINT16"\r
+  {\r
+    N:StringIdentifier      << HasBitFieldName = TRUE;>>\r
+  }\r
+  ":" I:Number              << Width = _STOU32(I->getText(), I->getLine());>>\r
+\r
+  ";"                       << if (HasBitFieldName) {\r
+                                 _PCATCH(gCVfrVarDataTypeDB.DataTypeAddBitField (N->getText(), D->getText(), Width, FieldInUnion), N);\r
+                               } else {\r
+                                 _PCATCH(gCVfrVarDataTypeDB.DataTypeAddBitField (NULL, D->getText(), Width, FieldInUnion), D);\r
+                               }\r
+                            >>\r
+  ;\r
+\r
+dataStructBitField8[BOOLEAN  FieldInUnion]:\r
+  <<\r
+    UINT32 Width = 0;\r
+    BOOLEAN  HasBitFieldName = FALSE;\r
+  >>\r
+  D:"UINT8"\r
+  {\r
+    N:StringIdentifier      << HasBitFieldName = TRUE;>>\r
+  }\r
+  ":" I:Number              << Width = _STOU32(I->getText(), I->getLine());>>\r
+\r
+  ";"                       << if (HasBitFieldName) {\r
+                                 _PCATCH(gCVfrVarDataTypeDB.DataTypeAddBitField (N->getText(), D->getText(), Width, FieldInUnion), N);\r
+                               } else {\r
+                                 _PCATCH(gCVfrVarDataTypeDB.DataTypeAddBitField (NULL, D->getText(), Width, FieldInUnion), D);\r
+                               }\r
+                            >>\r
+  ;\r
+\r
 //*****************************************************************************\r
 //\r
 // the syntax of GUID definition\r
@@ -751,6 +832,11 @@ vfrExtensionData[UINT8 *DataBuff, UINT32 Size, CHAR8 *TypeName, UINT32 TypeSize,
      EFI_STRING_ID Data_SID = 0;\r
      BOOLEAN  IsArray = FALSE;\r
      UINT8    *ByteOffset = NULL;\r
+     BOOLEAN  BitField = FALSE;\r
+     UINT64   Value;\r
+     UINT64   Mask;\r
+     UINT16   Offset;\r
+     UINT8    PreBits;\r
   >>\r
 (\r
   ("," "data" {OpenBracket IDX1:Number CloseBracket <<IsArray = TRUE;>>}\r
@@ -807,23 +893,69 @@ vfrExtensionData[UINT8 *DataBuff, UINT32 Size, CHAR8 *TypeName, UINT32 TypeSize,
                 memcpy (ByteOffset, &Data_SID, TypeSize);                                                    \r
               }\r
             } else {\r
-              gCVfrVarDataTypeDB.GetDataFieldInfo(TFName, FieldOffset, FieldType, FieldSize);\r
+              gCVfrVarDataTypeDB.GetDataFieldInfo(TFName, FieldOffset, FieldType, FieldSize, BitField);\r
+              if (BitField) {\r
+                Mask = (1 << FieldSize) - 1;\r
+                Offset = FieldOffset / 8;\r
+                PreBits = FieldOffset % 8;\r
+                Mask <<= PreBits;\r
+              }\r
               switch (FieldType) {\r
               case EFI_IFR_TYPE_NUM_SIZE_8:\r
                  Data_U8 = _STOU8(RD->getText(), RD->getLine());\r
-                 memcpy (ByteOffset + FieldOffset, &Data_U8, FieldSize);\r
+                 if (BitField) {\r
+                   //\r
+                   // Set the value to the bit fileds.\r
+                   //\r
+                   Value = *(UINT8*) (ByteOffset + Offset);\r
+                   Data_U8 <<= PreBits;\r
+                   Value = (Value & (~Mask)) | Data_U8;\r
+                   memcpy (ByteOffset + Offset, &Value, sizeof (UINT8));\r
+                 } else {\r
+                   memcpy (ByteOffset + FieldOffset, &Data_U8, FieldSize);\r
+                 }\r
                  break;\r
               case EFI_IFR_TYPE_NUM_SIZE_16:\r
                  Data_U16 = _STOU16(RD->getText(), RD->getLine());\r
-                 memcpy (ByteOffset + FieldOffset, &Data_U16, FieldSize);\r
+                 if (BitField) {\r
+                   //\r
+                   // Set the value to the bit fileds.\r
+                   //\r
+                   Value = *(UINT16*) (ByteOffset + Offset);\r
+                   Data_U16 <<= PreBits;\r
+                   Value = (Value & (~Mask)) | Data_U16;\r
+                   memcpy (ByteOffset + Offset, &Value, sizeof (UINT16));\r
+                 } else {\r
+                   memcpy (ByteOffset + FieldOffset, &Data_U16, FieldSize);\r
+                 }\r
                  break;\r
               case EFI_IFR_TYPE_NUM_SIZE_32:\r
                  Data_U32 = _STOU32(RD->getText(), RD->getLine());\r
-                 memcpy (ByteOffset + FieldOffset, &Data_U32, FieldSize);\r
+                 if (BitField) {\r
+                   //\r
+                   // Set the value to the bit fileds.\r
+                   //\r
+                   Value = *(UINT32*) (ByteOffset + Offset);\r
+                   Data_U32 <<= PreBits;\r
+                   Value = (Value & (~Mask)) | Data_U32;\r
+                   memcpy (ByteOffset + Offset, &Value, sizeof (UINT32));\r
+                 } else {\r
+                   memcpy (ByteOffset + FieldOffset, &Data_U32, FieldSize);\r
+                 }\r
                  break;\r
               case EFI_IFR_TYPE_NUM_SIZE_64:\r
                  Data_U64 = _STOU64(RD->getText(), RD->getLine());\r
-                 memcpy (ByteOffset + FieldOffset, &Data_U64, FieldSize);\r
+                 if (BitField) {\r
+                   //\r
+                   // Set the value to the bit fileds.\r
+                   //\r
+                   Value = *(UINT64*) (ByteOffset + Offset);\r
+                   Data_U64 <<= PreBits;\r
+                   Value = (Value & (~Mask)) | Data_U64;\r
+                   memcpy (ByteOffset + Offset, &Value, sizeof (UINT64));\r
+                 } else {\r
+                   memcpy (ByteOffset + FieldOffset, &Data_U64, FieldSize);\r
+                 }\r
                  break;\r
               case EFI_IFR_TYPE_BOOLEAN:\r
                  Data_BL = _STOU8(RD->getText(), RD->getLine());\r
@@ -874,10 +1006,11 @@ vfrStatementVarStoreLinear :
      UINT32          LineNum;\r
      EFI_VARSTORE_ID VarStoreId = EFI_VARSTORE_ID_INVALID;\r
      UINT32          Size;\r
+     BOOLEAN         IsBitVarStore = FALSE;\r
   >>\r
   V:Varstore                                        << VSObj.SetLineNo(V->getLine()); >>\r
   (\r
-      TN:StringIdentifier ","                       << TypeName = TN->getText(); LineNum = TN->getLine(); >>\r
+      TN:StringIdentifier ","                       << TypeName = TN->getText(); LineNum = TN->getLine(); IsBitVarStore = gCVfrVarDataTypeDB.DataTypeHasBitField (TN->getText());>>\r
     | U8:"UINT8" ","                                << TypeName = U8->getText(); LineNum = U8->getLine(); >>\r
     | U16:"UINT16" ","                              << TypeName = U16->getText(); LineNum = U16->getLine(); >>\r
     | C16:"CHAR16" ","                              << TypeName = (CHAR8 *) "UINT16"; LineNum = C16->getLine(); >>\r
@@ -912,12 +1045,13 @@ vfrStatementVarStoreLinear :
                                                          StoreName = SN->getText();\r
                                                        }\r
                                                        _PCATCH(gCVfrDataStorage.DeclareBufferVarStore (\r
-                                                                                  StoreName,\r
-                                                                                  &Guid,\r
-                                                                                  &gCVfrVarDataTypeDB,\r
-                                                                                  TypeName,\r
-                                                                                  VarStoreId\r
-                                                                                  ), LineNum);\r
+                                                                                StoreName,\r
+                                                                                &Guid,\r
+                                                                                &gCVfrVarDataTypeDB,\r
+                                                                                TypeName,\r
+                                                                                VarStoreId,\r
+                                                                                IsBitVarStore\r
+                                                                                ), LineNum);\r
                                                        VSObj.SetGuid (&Guid);\r
                                                        _PCATCH(gCVfrDataStorage.GetVarStoreId(StoreName, &VarStoreId, &Guid), SN);\r
                                                        VSObj.SetVarStoreId (VarStoreId);\r
@@ -940,10 +1074,11 @@ vfrStatementVarStoreEfi :
      UINT32          LineNum;\r
      CHAR8           *StoreName = NULL;\r
      BOOLEAN         CustomizedName = FALSE;\r
+     BOOLEAN         IsBitVarStore = FALSE;\r
   >>\r
   E:Efivarstore                                     << VSEObj.SetLineNo(E->getLine()); >>\r
   (\r
-      TN:StringIdentifier ","                       << TypeName = TN->getText(); LineNum = TN->getLine(); CustomizedName = TRUE; >>\r
+      TN:StringIdentifier ","                       << TypeName = TN->getText(); LineNum = TN->getLine(); CustomizedName = TRUE; IsBitVarStore = gCVfrVarDataTypeDB.DataTypeHasBitField (TN->getText());>>\r
     | U8:"UINT8" ","                                << TypeName = U8->getText(); LineNum = U8->getLine(); >>\r
     | U16:"UINT16" ","                              << TypeName = U16->getText(); LineNum = U16->getLine(); >>\r
     | C16:"CHAR16" ","                              << TypeName = (CHAR8 *) "UINT16"; LineNum = C16->getLine(); >>\r
@@ -1004,26 +1139,28 @@ vfrStatementVarStoreEfi :
   Uuid "=" guidDefinition[Guid]                     << \r
                                                        if (IsUEFI23EfiVarstore) {\r
                                                        _PCATCH(gCVfrDataStorage.DeclareBufferVarStore (\r
-                                                                                  StoreName,\r
-                                                                                  &Guid,\r
-                                                                                  &gCVfrVarDataTypeDB,\r
-                                                                                  TypeName,\r
-                                                                                  VarStoreId\r
-                                                                                  ), LineNum);                                                        \r
+                                                                                    StoreName,\r
+                                                                                    &Guid,\r
+                                                                                    &gCVfrVarDataTypeDB,\r
+                                                                                    TypeName,\r
+                                                                                    VarStoreId,\r
+                                                                                    IsBitVarStore\r
+                                                                                    ), LineNum);\r
                                                          _PCATCH(gCVfrDataStorage.GetVarStoreId(StoreName, &VarStoreId, &Guid), SN);\r
                                                          _PCATCH(gCVfrVarDataTypeDB.GetDataTypeSize(TypeName, &Size), LineNum);\r
                                                        } else {\r
-                                                        _PCATCH(gCVfrDataStorage.DeclareBufferVarStore (\r
+                                                       _PCATCH(gCVfrDataStorage.DeclareBufferVarStore (\r
                                                                                   TN->getText(),\r
                                                                                   &Guid,\r
                                                                                   &gCVfrVarDataTypeDB,\r
                                                                                   TypeName,\r
-                                                                                  VarStoreId\r
-                                                                                  ), LineNum);                                                      \r
+                                                                                  VarStoreId,\r
+                                                                                  FALSE\r
+                                                                                  ), LineNum);\r
                                                          _PCATCH(gCVfrDataStorage.GetVarStoreId(TN->getText(), &VarStoreId, &Guid), VN);\r
                                                          _PCATCH(gCVfrVarDataTypeDB.GetDataTypeSize(TypeName, &Size), N->getLine());\r
                                                        }\r
-                                                       VSEObj.SetGuid (&Guid);                                                       \r
+                                                       VSEObj.SetGuid (&Guid);\r
                                                        VSEObj.SetVarStoreId (VarStoreId);\r
                                                        \r
                                                        VSEObj.SetSize ((UINT16) Size);\r
@@ -1141,6 +1278,58 @@ vfrStatementHeader[CIfrStatementHeader *SHObj] :
   Help   "=" "STRING_TOKEN" "\(" S2:Number "\)"     << $SHObj->SetHelp (_STOSID(S2->getText(), S2->getLine())); >>\r
   ;\r
 \r
+vfrQuestionBaseInfo[EFI_VARSTORE_INFO & Info, EFI_QUESTION_ID & QId, EFI_QUESION_TYPE QType = QUESTION_NORMAL]:\r
+ <<\r
+    CHAR8             *QName    = NULL;\r
+    CHAR8             *VarIdStr = NULL;\r
+    mUsedDefaultCount           = 0;\r
+ >>\r
+ {\r
+   Name "=" QN:StringIdentifier ","                <<\r
+                                                      QName = QN->getText();\r
+                                                      _PCATCH(mCVfrQuestionDB.FindQuestion (QName), VFR_RETURN_UNDEFINED, QN, "has already been used please used anther name");\r
+                                                   >>\r
+ }\r
+ { V:VarId "=" vfrStorageVarId[Info, VarIdStr] "," }\r
+ {\r
+   QuestionId "=" ID:Number ","                    <<\r
+                                                      QId = _STOQID(ID->getText(), ID->getLine());\r
+                                                      _PCATCH(mCVfrQuestionDB.FindQuestion (QId), VFR_RETURN_UNDEFINED, ID, "has already been used please assign another number");\r
+                                                   >>\r
+ }\r
+                                                   <<\r
+                                                      switch (QType) {\r
+                                                      case QUESTION_NORMAL:\r
+                                                        mCVfrQuestionDB.RegisterQuestion (QName, VarIdStr, QId);\r
+                                                        break;\r
+                                                      case QUESTION_DATE:\r
+                                                        mCVfrQuestionDB.RegisterNewDateQuestion (QName, VarIdStr, QId);\r
+                                                        break;\r
+                                                      case QUESTION_TIME:\r
+                                                        mCVfrQuestionDB.RegisterNewTimeQuestion (QName, VarIdStr, QId);\r
+                                                        break;\r
+                                                      case QUESTION_REF:\r
+                                                        //\r
+                                                        // VarIdStr != NULL stand for question with storagae.\r
+                                                        //\r
+                                                        if (VarIdStr != NULL) {\r
+                                                          mCVfrQuestionDB.RegisterRefQuestion (QName, VarIdStr, QId);\r
+                                                        } else {\r
+                                                          mCVfrQuestionDB.RegisterQuestion (QName, NULL, QId);\r
+                                                        }\r
+                                                        break;\r
+                                                      default:\r
+                                                      _PCATCH(VFR_RETURN_FATAL_ERROR);\r
+                                                      }\r
+                                                   >>\r
+                                                   <<\r
+                                                      if (VarIdStr != NULL) {\r
+                                                        delete VarIdStr;\r
+                                                      }\r
+                                                      _SAVE_CURRQEST_VARINFO (Info);\r
+                                                   >>\r
+  ;\r
+\r
 vfrQuestionHeader[CIfrQuestionHeader & QHObj, EFI_QUESION_TYPE QType = QUESTION_NORMAL]:\r
   <<\r
      EFI_VARSTORE_INFO Info;\r
@@ -1148,60 +1337,16 @@ vfrQuestionHeader[CIfrQuestionHeader & QHObj, EFI_QUESION_TYPE QType = QUESTION_
      Info.mVarTotalSize          = 0;\r
      Info.mInfo.mVarOffset       = EFI_VAROFFSET_INVALID;\r
      Info.mVarStoreId            = EFI_VARSTORE_ID_INVALID;\r
+     Info.mIsBitVar              = FALSE;\r
      EFI_QUESTION_ID   QId       = EFI_QUESTION_ID_INVALID;\r
-     CHAR8             *QName    = NULL;\r
-     CHAR8             *VarIdStr = NULL;\r
-     mUsedDefaultCount           = 0;\r
   >>\r
-  {\r
-    Name "=" QN:StringIdentifier ","                <<\r
-                                                       QName = QN->getText();\r
-                                                       _PCATCH(mCVfrQuestionDB.FindQuestion (QName), VFR_RETURN_UNDEFINED, QN, "has already been used please used anther name");\r
-                                                    >>\r
-  }\r
-  { V:VarId "=" vfrStorageVarId[Info, VarIdStr] "," }\r
-  {\r
-    QuestionId "=" ID:Number ","                    <<\r
-                                                       QId = _STOQID(ID->getText(), ID->getLine());\r
-                                                       _PCATCH(mCVfrQuestionDB.FindQuestion (QId), VFR_RETURN_UNDEFINED, ID, "has already been used please assign another number");\r
-                                                    >>\r
-  }\r
-                                                    <<\r
-                                                       switch (QType) {\r
-                                                       case QUESTION_NORMAL:\r
-                                                         mCVfrQuestionDB.RegisterQuestion (QName, VarIdStr, QId);\r
-                                                         break;\r
-                                                       case QUESTION_DATE:\r
-                                                         mCVfrQuestionDB.RegisterNewDateQuestion (QName, VarIdStr, QId);\r
-                                                         break;\r
-                                                       case QUESTION_TIME:\r
-                                                         mCVfrQuestionDB.RegisterNewTimeQuestion (QName, VarIdStr, QId);\r
-                                                         break;\r
-                                                       case QUESTION_REF:\r
-                                                         //\r
-                                                         // VarIdStr != NULL stand for question with storagae.\r
-                                                         //\r
-                                                         if (VarIdStr != NULL) {\r
-                                                           mCVfrQuestionDB.RegisterRefQuestion (QName, VarIdStr, QId);\r
-                                                         } else {\r
-                                                           mCVfrQuestionDB.RegisterQuestion (QName, NULL, QId);\r
-                                                         }\r
-                                                         break;\r
-                                                       default:\r
-                                                       _PCATCH(VFR_RETURN_FATAL_ERROR);\r
-                                                       }\r
-                                                       $QHObj.SetQuestionId (QId);\r
-                                                       if (VarIdStr != NULL) {\r
-                                                        $QHObj.SetVarStoreInfo (&Info);\r
-                                                       }\r
+  vfrQuestionBaseInfo[Info, QId, QType]\r
+                                                    << $QHObj.SetQuestionId (QId);\r
+                                                        if (Info.mVarStoreId != EFI_VARSTORE_ID_INVALID) {\r
+                                                          $QHObj.SetVarStoreInfo (&Info);\r
+                                                        }\r
                                                     >>\r
   vfrStatementHeader[&$QHObj]\r
-                                                    << \r
-                                                       if (VarIdStr != NULL) {\r
-                                                         delete VarIdStr; \r
-                                                       }\r
-                                                       _SAVE_CURRQEST_VARINFO (Info);\r
-                                                    >>\r
   ;\r
 \r
 questionheaderFlagsField[UINT8 & Flags] :\r
@@ -1270,6 +1415,7 @@ vfrStorageVarId[EFI_VARSTORE_INFO & Info, CHAR8 *&QuestVarIdStr, BOOLEAN CheckFl
                                                                              &gCVfrVarDataTypeDB,\r
                                                                              SName,\r
                                                                              EFI_VARSTORE_ID_INVALID,\r
+                                                                             FALSE,\r
                                                                              FALSE\r
                                                                              );\r
                                                           VfrReturnCode = gCVfrDataStorage.GetVarStoreId(SName, &$Info.mVarStoreId, &mFormsetGuid);\r
@@ -1301,7 +1447,7 @@ vfrStorageVarId[EFI_VARSTORE_INFO & Info, CHAR8 *&QuestVarIdStr, BOOLEAN CheckFl
                                                        if (CheckFlag || VfrReturnCode == VFR_RETURN_SUCCESS) {\r
                                                          _PCATCH(VfrReturnCode, SN2);\r
                                                          VarStoreType = gCVfrDataStorage.GetVarStoreType ($Info.mVarStoreId);\r
-                                                         if (VarStoreType == EFI_VFR_VARSTORE_BUFFER) {\r
+                                                         if (VarStoreType == EFI_VFR_VARSTORE_BUFFER || VarStoreType == EFI_VFR_VARSTORE_BUFFER_BITS) {\r
                                                            _PCATCH(gCVfrDataStorage.GetBufferVarStoreDataTypeName(Info.mVarStoreId, &TName), SN2);\r
                                                            _STRCAT(&VarStr, TName);\r
                                                          }\r
@@ -1311,7 +1457,7 @@ vfrStorageVarId[EFI_VARSTORE_INFO & Info, CHAR8 *&QuestVarIdStr, BOOLEAN CheckFl
     (\r
       "."                                           <<\r
                                                        if (CheckFlag || VfrReturnCode == VFR_RETURN_SUCCESS) {\r
-                                                         _PCATCH(((VarStoreType != EFI_VFR_VARSTORE_BUFFER) ? VFR_RETURN_EFIVARSTORE_USE_ERROR : VFR_RETURN_SUCCESS), SN2);\r
+                                                         _PCATCH((((VarStoreType != EFI_VFR_VARSTORE_BUFFER) && (VarStoreType != EFI_VFR_VARSTORE_BUFFER_BITS))? VFR_RETURN_EFIVARSTORE_USE_ERROR : VFR_RETURN_SUCCESS), SN2);\r
                                                        }\r
                                                        _STRCAT(&VarIdStr, "."); _STRCAT(&VarStr, ".");\r
                                                     >>\r
@@ -1340,7 +1486,8 @@ vfrStorageVarId[EFI_VARSTORE_INFO & Info, CHAR8 *&QuestVarIdStr, BOOLEAN CheckFl
                                                          _PCATCH(gCVfrDataStorage.GetEfiVarStoreInfo (&$Info), SN2);\r
                                                          break;\r
                                                        case EFI_VFR_VARSTORE_BUFFER:\r
-                                                         _PCATCH(gCVfrVarDataTypeDB.GetDataFieldInfo (VarStr, $Info.mInfo.mVarOffset, $Info.mVarType, $Info.mVarTotalSize), SN2->getLine(), VarStr);\r
+                                                       case EFI_VFR_VARSTORE_BUFFER_BITS:\r
+                                                         _PCATCH(gCVfrVarDataTypeDB.GetDataFieldInfo (VarStr, $Info.mInfo.mVarOffset, $Info.mVarType, $Info.mVarTotalSize, $Info.mIsBitVar), SN2->getLine(), VarStr);\r
                                                          VarGuid = gCVfrDataStorage.GetVarStoreGuid($Info.mVarStoreId);\r
                                                          _PCATCH((EFI_VFR_RETURN_CODE)gCVfrBufferConfig.Register (\r
                                                                     SName,\r
@@ -1438,86 +1585,93 @@ vfrConstantValueField[UINT8 Type, EFI_IFR_TYPE_VALUE &Value, BOOLEAN &ListType]
       "\-"                                          << Negative = TRUE;  >>\r
     }\r
     N1:Number                                       <<\r
-                                                       switch ($Type) {\r
-                                                       case EFI_IFR_TYPE_NUM_SIZE_8 :\r
-                                                         $Value.u8 = _STOU8(N1->getText(), N1->getLine());\r
-                                                         if (IntDecStyle) {\r
-                                                           if (Negative) {\r
-                                                             if ($Value.u8 > 0x80) {\r
-                                                               _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT8 type can't big than 0x7F, small than -0x80");\r
-                                                             }\r
-                                                           } else {\r
-                                                             if ($Value.u8 > 0x7F) {\r
-                                                               _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT8 type can't big than 0x7F, small than -0x80");\r
+                                                       //\r
+                                                       // The value stored in bit fields is always set to UINT32 type.\r
+                                                       //\r
+                                                       if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                         $Value.u32 = _STOU32(N1->getText(), N1->getLine());\r
+                                                       } else {\r
+                                                         switch ($Type) {\r
+                                                         case EFI_IFR_TYPE_NUM_SIZE_8 :\r
+                                                           $Value.u8 = _STOU8(N1->getText(), N1->getLine());\r
+                                                           if (IntDecStyle) {\r
+                                                             if (Negative) {\r
+                                                               if ($Value.u8 > 0x80) {\r
+                                                                 _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT8 type can't big than 0x7F, small than -0x80");\r
+                                                               }\r
+                                                             } else {\r
+                                                               if ($Value.u8 > 0x7F) {\r
+                                                                 _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT8 type can't big than 0x7F, small than -0x80");\r
+                                                               }\r
                                                              }\r
                                                            }\r
-                                                         }\r
-                                                         if (Negative) {\r
-                                                           $Value.u8 = ~$Value.u8 + 1;\r
-                                                         }\r
-                                                       break;\r
-                                                       case EFI_IFR_TYPE_NUM_SIZE_16 :\r
-                                                         $Value.u16 = _STOU16(N1->getText(), N1->getLine());\r
-                                                         if (IntDecStyle) {\r
                                                            if (Negative) {\r
-                                                             if ($Value.u16 > 0x8000) {\r
-                                                               _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT16 type can't big than 0x7FFF, small than -0x8000");\r
-                                                             }\r
-                                                           } else {\r
-                                                             if ($Value.u16 > 0x7FFF) {\r
-                                                               _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT16 type can't big than 0x7FFF, small than -0x8000");\r
+                                                             $Value.u8 = ~$Value.u8 + 1;\r
+                                                           }\r
+                                                         break;\r
+                                                         case EFI_IFR_TYPE_NUM_SIZE_16 :\r
+                                                           $Value.u16 = _STOU16(N1->getText(), N1->getLine());\r
+                                                           if (IntDecStyle) {\r
+                                                             if (Negative) {\r
+                                                               if ($Value.u16 > 0x8000) {\r
+                                                                 _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT16 type can't big than 0x7FFF, small than -0x8000");\r
+                                                               }\r
+                                                             } else {\r
+                                                               if ($Value.u16 > 0x7FFF) {\r
+                                                                 _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT16 type can't big than 0x7FFF, small than -0x8000");\r
+                                                               }\r
                                                              }\r
                                                            }\r
-                                                         }\r
-                                                         if (Negative) {\r
-                                                           $Value.u16 = ~$Value.u16 + 1;\r
-                                                         }\r
-                                                       break;\r
-                                                       case EFI_IFR_TYPE_NUM_SIZE_32 :\r
-                                                         $Value.u32    = _STOU32(N1->getText(), N1->getLine());\r
-                                                         if (IntDecStyle) {\r
                                                            if (Negative) {\r
-                                                             if ($Value.u32 > 0x80000000) {\r
-                                                               _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT32 type can't big than 0x7FFFFFFF, small than -0x80000000");\r
-                                                             }\r
-                                                           } else {\r
-                                                             if ($Value.u32 > 0X7FFFFFFF) {\r
-                                                               _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT32 type can't big than 0x7FFFFFFF, small than -0x80000000");\r
+                                                             $Value.u16 = ~$Value.u16 + 1;\r
+                                                           }\r
+                                                         break;\r
+                                                         case EFI_IFR_TYPE_NUM_SIZE_32 :\r
+                                                           $Value.u32    = _STOU32(N1->getText(), N1->getLine());\r
+                                                           if (IntDecStyle) {\r
+                                                             if (Negative) {\r
+                                                               if ($Value.u32 > 0x80000000) {\r
+                                                                 _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT32 type can't big than 0x7FFFFFFF, small than -0x80000000");\r
+                                                               }\r
+                                                             } else {\r
+                                                               if ($Value.u32 > 0X7FFFFFFF) {\r
+                                                                 _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT32 type can't big than 0x7FFFFFFF, small than -0x80000000");\r
+                                                               }\r
                                                              }\r
                                                            }\r
-                                                         }\r
-                                                         if (Negative) {\r
-                                                           $Value.u32 = ~$Value.u32 + 1;\r
-                                                         }\r
-                                                       break;\r
-                                                       case EFI_IFR_TYPE_NUM_SIZE_64 :\r
-                                                         $Value.u64    = _STOU64(N1->getText(), N1->getLine());\r
-                                                         if (IntDecStyle) {\r
                                                            if (Negative) {\r
-                                                             if ($Value.u64 > 0x8000000000000000) {\r
-                                                               _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT64 type can't big than 0x7FFFFFFFFFFFFFFF, small than -0x8000000000000000");\r
-                                                             }\r
-                                                           } else {\r
-                                                             if ($Value.u64 > 0x7FFFFFFFFFFFFFFF) {\r
-                                                               _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT64 type can't big than 0x7FFFFFFFFFFFFFFF, small than -0x8000000000000000");\r
+                                                             $Value.u32 = ~$Value.u32 + 1;\r
+                                                           }\r
+                                                         break;\r
+                                                         case EFI_IFR_TYPE_NUM_SIZE_64 :\r
+                                                           $Value.u64    = _STOU64(N1->getText(), N1->getLine());\r
+                                                           if (IntDecStyle) {\r
+                                                             if (Negative) {\r
+                                                               if ($Value.u64 > 0x8000000000000000) {\r
+                                                                 _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT64 type can't big than 0x7FFFFFFFFFFFFFFF, small than -0x8000000000000000");\r
+                                                               }\r
+                                                             } else {\r
+                                                               if ($Value.u64 > 0x7FFFFFFFFFFFFFFF) {\r
+                                                                 _PCATCH (VFR_RETURN_INVALID_PARAMETER, N1->getLine(), "INT64 type can't big than 0x7FFFFFFFFFFFFFFF, small than -0x8000000000000000");\r
+                                                               }\r
                                                              }\r
                                                            }\r
+                                                           if (Negative) {\r
+                                                             $Value.u64 = ~$Value.u64 + 1;\r
+                                                           }\r
+                                                         break;\r
+                                                         case EFI_IFR_TYPE_BOOLEAN :\r
+                                                           $Value.b      = _STOU8(N1->getText(), N1->getLine());\r
+                                                         break;\r
+                                                         case EFI_IFR_TYPE_STRING :\r
+                                                           $Value.string = _STOU16(N1->getText(), N1->getLine());\r
+                                                         break;\r
+                                                         case EFI_IFR_TYPE_TIME :\r
+                                                         case EFI_IFR_TYPE_DATE :\r
+                                                         case EFI_IFR_TYPE_REF  :\r
+                                                         default :\r
+                                                         break;\r
                                                          }\r
-                                                         if (Negative) {\r
-                                                           $Value.u64 = ~$Value.u64 + 1;\r
-                                                         }\r
-                                                       break;\r
-                                                       case EFI_IFR_TYPE_BOOLEAN :\r
-                                                         $Value.b      = _STOU8(N1->getText(), N1->getLine());\r
-                                                       break;\r
-                                                       case EFI_IFR_TYPE_STRING :\r
-                                                         $Value.string = _STOU16(N1->getText(), N1->getLine());\r
-                                                       break;\r
-                                                       case EFI_IFR_TYPE_TIME :\r
-                                                       case EFI_IFR_TYPE_DATE :\r
-                                                       case EFI_IFR_TYPE_REF  :\r
-                                                       default :\r
-                                                       break;\r
                                                        }\r
                                                     >>\r
   | B1:True                                         << $Value.b      = TRUE; >>\r
@@ -1691,29 +1845,29 @@ vfrStatementDefault :
                                                         if (gCurrentMinMaxData != NULL && gCurrentMinMaxData->IsNumericOpcode()) {\r
                                                           //check default value is valid for Numeric Opcode\r
                                                           NumericQst = (CIfrNumeric *) gCurrentQuestion;\r
-                                                          if ((NumericQst->GetNumericFlags() & EFI_IFR_DISPLAY) == 0) {\r
+                                                          if ((NumericQst->GetNumericFlags() & EFI_IFR_DISPLAY) == 0 && !(_GET_CURRQEST_VARTINFO().mIsBitVar)) {\r
                                                             switch (_GET_CURRQEST_DATATYPE()) {\r
                                                             case EFI_IFR_TYPE_NUM_SIZE_8:\r
-                                                              if (((INT8) Val->u8 < (INT8) gCurrentMinMaxData->GetMinData(_GET_CURRQEST_DATATYPE())) || \r
-                                                                  ((INT8) Val->u8 > (INT8) gCurrentMinMaxData->GetMaxData(_GET_CURRQEST_DATATYPE()))) {\r
+                                                              if (((INT8) Val->u8 < (INT8) gCurrentMinMaxData->GetMinData(_GET_CURRQEST_DATATYPE(), FALSE)) ||\r
+                                                                  ((INT8) Val->u8 > (INT8) gCurrentMinMaxData->GetMaxData(_GET_CURRQEST_DATATYPE(), FALSE))) {\r
                                                                 _PCATCH (VFR_RETURN_INVALID_PARAMETER, D->getLine(), "Numeric default value must be between MinValue and MaxValue.");\r
                                                               }\r
                                                               break;\r
                                                             case EFI_IFR_TYPE_NUM_SIZE_16:\r
-                                                              if (((INT16) Val->u16 < (INT16) gCurrentMinMaxData->GetMinData(_GET_CURRQEST_DATATYPE())) || \r
-                                                                  ((INT16) Val->u16 > (INT16) gCurrentMinMaxData->GetMaxData(_GET_CURRQEST_DATATYPE()))) {\r
+                                                              if (((INT16) Val->u16 < (INT16) gCurrentMinMaxData->GetMinData(_GET_CURRQEST_DATATYPE(), FALSE)) ||\r
+                                                                  ((INT16) Val->u16 > (INT16) gCurrentMinMaxData->GetMaxData(_GET_CURRQEST_DATATYPE(), FALSE))) {\r
                                                                 _PCATCH (VFR_RETURN_INVALID_PARAMETER, D->getLine(), "Numeric default value must be between MinValue and MaxValue.");\r
                                                               }\r
                                                               break;\r
                                                             case EFI_IFR_TYPE_NUM_SIZE_32:\r
-                                                              if (((INT32) Val->u32 < (INT32) gCurrentMinMaxData->GetMinData(_GET_CURRQEST_DATATYPE())) || \r
-                                                                  ((INT32) Val->u32 > (INT32) gCurrentMinMaxData->GetMaxData(_GET_CURRQEST_DATATYPE()))) {\r
+                                                              if (((INT32) Val->u32 < (INT32) gCurrentMinMaxData->GetMinData(_GET_CURRQEST_DATATYPE(), FALSE)) ||\r
+                                                                  ((INT32) Val->u32 > (INT32) gCurrentMinMaxData->GetMaxData(_GET_CURRQEST_DATATYPE(), FALSE))) {\r
                                                                 _PCATCH (VFR_RETURN_INVALID_PARAMETER, D->getLine(), "Numeric default value must be between MinValue and MaxValue.");\r
                                                               }\r
                                                               break;\r
                                                             case EFI_IFR_TYPE_NUM_SIZE_64:\r
-                                                              if (((INT64) Val->u64 < (INT64) gCurrentMinMaxData->GetMinData(_GET_CURRQEST_DATATYPE())) || \r
-                                                                  ((INT64) Val->u64 > (INT64) gCurrentMinMaxData->GetMaxData(_GET_CURRQEST_DATATYPE()))) {\r
+                                                              if (((INT64) Val->u64 < (INT64) gCurrentMinMaxData->GetMinData(_GET_CURRQEST_DATATYPE(), FALSE)) ||\r
+                                                                  ((INT64) Val->u64 > (INT64) gCurrentMinMaxData->GetMaxData(_GET_CURRQEST_DATATYPE(), FALSE))) {\r
                                                                 _PCATCH (VFR_RETURN_INVALID_PARAMETER, D->getLine(), "Numeric default value must be between MinValue and MaxValue.");\r
                                                               }\r
                                                               break;\r
@@ -1721,8 +1875,17 @@ vfrStatementDefault :
                                                               break;\r
                                                             }\r
                                                           } else {\r
-                                                            if (Val->u64 < gCurrentMinMaxData->GetMinData(_GET_CURRQEST_DATATYPE()) || Val->u64 > gCurrentMinMaxData->GetMaxData(_GET_CURRQEST_DATATYPE())) {\r
+                                                            //\r
+                                                            // Value for question stored in bit fields is always set to UINT32 type.\r
+                                                            //\r
+                                                            if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                              if (Val->u32 < gCurrentMinMaxData->GetMinData(_GET_CURRQEST_DATATYPE(), TRUE) || Val->u32 > gCurrentMinMaxData->GetMaxData(_GET_CURRQEST_DATATYPE(), TRUE)) {\r
                                                               _PCATCH (VFR_RETURN_INVALID_PARAMETER, D->getLine(), "Numeric default value must be between MinValue and MaxValue.");\r
+                                                              }\r
+                                                            } else {\r
+                                                              if (Val->u64 < gCurrentMinMaxData->GetMinData(_GET_CURRQEST_DATATYPE(), FALSE) || Val->u64 > gCurrentMinMaxData->GetMaxData(_GET_CURRQEST_DATATYPE(), FALSE)) {\r
+                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, D->getLine(), "Numeric default value must be between MinValue and MaxValue.");\r
+                                                              }\r
                                                             }\r
                                                           }\r
                                                         }\r
@@ -1761,7 +1924,11 @@ vfrStatementDefault :
                                                               break;\r
                                                           }\r
                                                         } else {\r
-                                                          _PCATCH (gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &Size), D->getLine());\r
+                                                          if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            Size = sizeof (UINT32);\r
+                                                          } else {\r
+                                                            _PCATCH (gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &Size), D->getLine());\r
+                                                          }\r
                                                         }\r
                                                         Size += OFFSET_OF (EFI_IFR_DEFAULT, Value);\r
                                                         DObj = new CIfrDefault ((UINT8)Size);\r
@@ -1771,7 +1938,11 @@ vfrStatementDefault :
                                                         } else if (gIsStringOp) {\r
                                                           DObj->SetType (EFI_IFR_TYPE_STRING);\r
                                                         } else {\r
-                                                          DObj->SetType (_GET_CURRQEST_DATATYPE());\r
+                                                          if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            DObj->SetType (EFI_IFR_TYPE_NUM_SIZE_32);\r
+                                                          } else {\r
+                                                            DObj->SetType (_GET_CURRQEST_DATATYPE());\r
+                                                          }\r
                                                         }\r
                                                         DObj->SetValue(*Val);\r
                                                     >>\r
@@ -2178,35 +2349,68 @@ vfrStatementBooleanType :
 //\r
 vfrStatementCheckBox :\r
   <<\r
-     CIfrCheckBox       CBObj;\r
+     CIfrCheckBox       *CBObj = NULL;\r
      EFI_IFR_TYPE_VALUE Val = gZeroEfiIfrTypeValue;\r
      CHAR8              *VarStoreName = NULL;\r
      UINT32             DataTypeSize;\r
      EFI_GUID           *VarStoreGuid = NULL;\r
+     CIfrGuid           *GuidObj = NULL;\r
+     EFI_QUESTION_ID   QId = EFI_QUESTION_ID_INVALID;;\r
+     EFI_VARSTORE_INFO Info;\r
+     Info.mVarType          = EFI_IFR_TYPE_OTHER;\r
+     Info.mVarTotalSize     = 0;\r
+     Info.mInfo.mVarOffset  = EFI_VAROFFSET_INVALID;\r
+     Info.mVarStoreId       = EFI_VARSTORE_ID_INVALID;\r
+     Info.mIsBitVar         = FALSE;\r
   >>\r
-  L:CheckBox                                           << CBObj.SetLineNo(L->getLine()); >>\r
-  vfrQuestionHeader[CBObj] ","                         << //check data type\r
+  L:CheckBox\r
+  vfrQuestionBaseInfo[Info, QId]                       <<\r
+                                                         //\r
+                                                         // Create a GUID opcode to wrap the checkbox opcode, if it refer to bit varstore.\r
+                                                         //\r
+                                                         if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                           GuidObj = new CIfrGuid(0);\r
+                                                           GuidObj->SetGuid (&gEdkiiIfrBitVarGuid);\r
+                                                           GuidObj->SetLineNo(L->getLine());\r
+                                                         }\r
+                                                         CBObj = new CIfrCheckBox;\r
+                                                         CBObj->SetLineNo(L->getLine());\r
+                                                         CBObj->SetQuestionId (QId);\r
+                                                         CBObj->SetVarStoreInfo (&Info);\r
+                                                        >>\r
+  vfrStatementHeader[CBObj]","                           << //check data type\r
                                                           if (_GET_CURRQEST_DATATYPE() == EFI_IFR_TYPE_OTHER) {\r
                                                             _GET_CURRQEST_VARTINFO().mVarType = EFI_IFR_TYPE_BOOLEAN;\r
                                                           }\r
                                                           if (_GET_CURRQEST_VARTINFO().mVarStoreId != EFI_VARSTORE_ID_INVALID) {\r
-                                                            _PCATCH (gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &DataTypeSize), L->getLine(), "CheckBox varid is not the valid data type");\r
-                                                            if (DataTypeSize != 0 && DataTypeSize != _GET_CURRQEST_VARSIZE()) {\r
-                                                              _PCATCH (VFR_RETURN_INVALID_PARAMETER, L->getLine(), "CheckBox varid doesn't support array");\r
-                                                            } else if ((gCVfrDataStorage.GetVarStoreType (_GET_CURRQEST_VARTINFO().mVarStoreId) == EFI_VFR_VARSTORE_BUFFER) &&\r
-                                                                      (_GET_CURRQEST_VARSIZE() != sizeof (BOOLEAN))) {\r
-                                                              _PCATCH (VFR_RETURN_INVALID_PARAMETER, L->getLine(), "CheckBox varid only support BOOLEAN data type");\r
+                                                            //\r
+                                                            // Check whether the question refers to a bit field, if yes. create a Guid to indicate the question refers to a bit field.\r
+                                                            //\r
+                                                            if (_GET_CURRQEST_VARTINFO ().mIsBitVar) {\r
+                                                              _PCATCH (gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &DataTypeSize), L->getLine(), "CheckBox varid is not the valid data type");\r
+                                                              if ((gCVfrDataStorage.GetVarStoreType (_GET_CURRQEST_VARTINFO().mVarStoreId) == EFI_VFR_VARSTORE_BUFFER_BITS) &&\r
+                                                                  (_GET_CURRQEST_VARSIZE() != 1)) {\r
+                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, L->getLine(), "CheckBox varid only occupy 1 bit in Bit Varstore");\r
+                                                              }\r
+                                                            } else {\r
+                                                              _PCATCH (gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &DataTypeSize), L->getLine(), "CheckBox varid is not the valid data type");\r
+                                                              if (DataTypeSize != 0 && DataTypeSize != _GET_CURRQEST_VARSIZE()) {\r
+                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, L->getLine(), "CheckBox varid doesn't support array");\r
+                                                              } else if ((gCVfrDataStorage.GetVarStoreType (_GET_CURRQEST_VARTINFO().mVarStoreId) == EFI_VFR_VARSTORE_BUFFER) &&\r
+                                                                        (_GET_CURRQEST_VARSIZE() != sizeof (BOOLEAN))) {\r
+                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, L->getLine(), "CheckBox varid only support BOOLEAN data type");\r
+                                                              }\r
                                                             }\r
                                                           }\r
                                                        >>\r
   {\r
-    F:FLAGS "=" vfrCheckBoxFlags[CBObj, F->getLine()] ","\r
+    F:FLAGS "=" vfrCheckBoxFlags[*CBObj, F->getLine()] ","\r
                                                        <<\r
                                                          if (_GET_CURRQEST_VARTINFO().mVarStoreId != EFI_VARSTORE_ID_INVALID) {\r
                                                             _PCATCH(gCVfrDataStorage.GetVarStoreName (_GET_CURRQEST_VARTINFO().mVarStoreId, &VarStoreName), VFR_RETURN_SUCCESS, L, "Failed to retrieve varstore name");\r
                                                             VarStoreGuid = gCVfrDataStorage.GetVarStoreGuid(_GET_CURRQEST_VARTINFO().mVarStoreId);\r
                                                             Val.b = TRUE;\r
-                                                            if (CBObj.GetFlags () & 0x01) {\r
+                                                            if (CBObj->GetFlags () & 0x01) {\r
                                                               CheckDuplicateDefaultValue (EFI_HII_DEFAULT_CLASS_STANDARD, F);\r
                                                               _PCATCH(\r
                                                                 gCVfrDefaultStore.BufferVarStoreAltConfigAdd (\r
@@ -2222,7 +2426,7 @@ vfrStatementCheckBox :
                                                                 "No standard default storage found"\r
                                                                 );\r
                                                             }\r
-                                                            if (CBObj.GetFlags () & 0x02) {\r
+                                                            if (CBObj->GetFlags () & 0x02) {\r
                                                               CheckDuplicateDefaultValue (EFI_HII_DEFAULT_CLASS_MANUFACTURING, F);\r
                                                               _PCATCH(\r
                                                                 gCVfrDefaultStore.BufferVarStoreAltConfigAdd (\r
@@ -2242,10 +2446,17 @@ vfrStatementCheckBox :
                                                         >>\r
   }\r
   {\r
-    Key "=" KN:Number  ","                             << AssignQuestionKey (CBObj, KN); >>\r
+    Key "=" KN:Number  ","                             << AssignQuestionKey (*CBObj, KN); >>\r
   }\r
   vfrStatementQuestionOptionList\r
-  E:EndCheckBox                                        << CRT_END_OP (E); >>\r
+  E:EndCheckBox                                        << CRT_END_OP (E);\r
+                                                          if (GuidObj != NULL) {\r
+                                                            GuidObj->SetScope(1);\r
+                                                            CRT_END_OP (E);\r
+                                                            delete GuidObj;\r
+                                                          }\r
+                                                          if (CBObj != NULL) delete CBObj;\r
+                                                       >>\r
   ";"\r
   ;\r
 \r
@@ -2441,7 +2652,8 @@ vfrSetMinMaxStep[CIfrMinMaxStepData & MMSDObj] :
      UINT8  MaxU1 = 0, MinU1 = 0, StepU1 = 0;\r
      BOOLEAN IntDecStyle = FALSE;\r
      CIfrNumeric *NObj = (CIfrNumeric *) (&MMSDObj);\r
-     if ((NObj->GetOpCode() == EFI_IFR_NUMERIC_OP) && ((NObj->GetNumericFlags() & EFI_IFR_DISPLAY) == 0)) {\r
+     if (((_GET_CURRQEST_VARTINFO().mIsBitVar) && (NObj->GetOpCode() == EFI_IFR_NUMERIC_OP) && ((NObj->GetNumericFlags() &  EDKII_IFR_DISPLAY_BIT) == 0)) ||\r
+     (!(_GET_CURRQEST_VARTINFO().mIsBitVar) && (NObj->GetOpCode() == EFI_IFR_NUMERIC_OP) && ((NObj->GetNumericFlags() & EFI_IFR_DISPLAY) == 0))) {\r
        IntDecStyle = TRUE;\r
      }\r
      BOOLEAN MinNegative = FALSE;\r
@@ -2455,75 +2667,85 @@ vfrSetMinMaxStep[CIfrMinMaxStepData & MMSDObj] :
                                                           if (!IntDecStyle &&  MinNegative) {\r
                                                             _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "\"-\" can't be used when not in int decimal type. ");\r
                                                           }\r
-                                                          switch (_GET_CURRQEST_DATATYPE()) {\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_64 :\r
-                                                            MinU8 = _STOU64(I->getText(), I->getLine());\r
-                                                            if (IntDecStyle) {\r
-                                                              if (MinNegative) { \r
-                                                                if (MinU8 > 0x8000000000000000) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT64 type minimum can't small than -0x8000000000000000, big than 0x7FFFFFFFFFFFFFFF");\r
-                                                                }\r
-                                                              } else {\r
-                                                                if (MinU8 > 0x7FFFFFFFFFFFFFFF) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT64 type minimum can't small than -0x8000000000000000, big than 0x7FFFFFFFFFFFFFFF");\r
-                                                                }\r
-                                                              }\r
-                                                            }\r
-                                                            if (MinNegative) {\r
-                                                              MinU8 = ~MinU8 + 1;\r
-                                                            }\r
-                                                            break;\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_32 :\r
+                                                          //\r
+                                                          // Value for question stored in bit fields is always set to UINT32 type.\r
+                                                          //\r
+                                                          if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
                                                             MinU4 = _STOU32(I->getText(), I->getLine());\r
-                                                            if (IntDecStyle) {\r
-                                                              if (MinNegative) { \r
-                                                                if (MinU4 > 0x80000000) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT32 type minimum can't small than -0x80000000, big than 0x7FFFFFFF");\r
-                                                                }\r
-                                                              } else {\r
-                                                                if (MinU4 > 0x7FFFFFFF) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT32 type minimum can't small than -0x80000000, big than 0x7FFFFFFF");\r
-                                                                }\r
-                                                              }\r
-                                                            }\r
-                                                            if (MinNegative) {\r
-                                                              MinU4 = ~MinU4 + 1;\r
+                                                            if (!IntDecStyle && MinU4 > (1<< _GET_CURRQEST_VARTINFO().mVarTotalSize) -1) {\r
+                                                              _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "BIT type minimum can't small than 0, bigger than 2^BitWidth -1");\r
                                                             }\r
-                                                            break;\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_16 :\r
-                                                            MinU2 = _STOU16(I->getText(), I->getLine());\r
-                                                            if (IntDecStyle) {\r
-                                                              if (MinNegative) { \r
-                                                                if (MinU2 > 0x8000) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT16 type minimum can't small than -0x8000, big than 0x7FFF");\r
+                                                          } else {\r
+                                                            switch (_GET_CURRQEST_DATATYPE()) {\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_64 :\r
+                                                              MinU8 = _STOU64(I->getText(), I->getLine());\r
+                                                              if (IntDecStyle) {\r
+                                                                if (MinNegative) {\r
+                                                                  if (MinU8 > 0x8000000000000000) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT64 type minimum can't small than -0x8000000000000000, big than 0x7FFFFFFFFFFFFFFF");\r
+                                                                  }\r
+                                                                } else {\r
+                                                                  if (MinU8 > 0x7FFFFFFFFFFFFFFF) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT64 type minimum can't small than -0x8000000000000000, big than 0x7FFFFFFFFFFFFFFF");\r
+                                                                  }\r
                                                                 }\r
-                                                              } else {\r
-                                                                if (MinU2 > 0x7FFF) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT16 type minimum can't small than -0x8000, big than 0x7FFF");\r
+                                                              }\r
+                                                              if (MinNegative) {\r
+                                                                MinU8 = ~MinU8 + 1;\r
+                                                              }\r
+                                                              break;\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_32 :\r
+                                                              MinU4 = _STOU32(I->getText(), I->getLine());\r
+                                                              if (IntDecStyle) {\r
+                                                                if (MinNegative) {\r
+                                                                  if (MinU4 > 0x80000000) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT32 type minimum can't small than -0x80000000, big than 0x7FFFFFFF");\r
+                                                                  }\r
+                                                                } else {\r
+                                                                  if (MinU4 > 0x7FFFFFFF) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT32 type minimum can't small than -0x80000000, big than 0x7FFFFFFF");\r
+                                                                  }\r
                                                                 }\r
                                                               }\r
-                                                            }\r
-                                                            if (MinNegative) {\r
-                                                              MinU2 = ~MinU2 + 1;\r
-                                                            }\r
-                                                            break;\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_8 :\r
-                                                            MinU1 = _STOU8(I->getText(), I->getLine());\r
-                                                            if (IntDecStyle) {\r
-                                                              if (MinNegative) { \r
-                                                                if (MinU1 > 0x80) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT8 type minimum can't small than -0x80, big than 0x7F");\r
+                                                              if (MinNegative) {\r
+                                                                MinU4 = ~MinU4 + 1;\r
+                                                              }\r
+                                                              break;\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_16 :\r
+                                                              MinU2 = _STOU16(I->getText(), I->getLine());\r
+                                                              if (IntDecStyle) {\r
+                                                                if (MinNegative) {\r
+                                                                  if (MinU2 > 0x8000) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT16 type minimum can't small than -0x8000, big than 0x7FFF");\r
+                                                                  }\r
+                                                                } else {\r
+                                                                  if (MinU2 > 0x7FFF) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT16 type minimum can't small than -0x8000, big than 0x7FFF");\r
+                                                                  }\r
                                                                 }\r
-                                                              } else {\r
-                                                                if (MinU1 > 0x7F) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT8 type minimum can't small than -0x80, big than 0x7F");\r
+                                                              }\r
+                                                              if (MinNegative) {\r
+                                                                MinU2 = ~MinU2 + 1;\r
+                                                              }\r
+                                                              break;\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_8 :\r
+                                                              MinU1 = _STOU8(I->getText(), I->getLine());\r
+                                                              if (IntDecStyle) {\r
+                                                                if (MinNegative) {\r
+                                                                  if (MinU1 > 0x80) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT8 type minimum can't small than -0x80, big than 0x7F");\r
+                                                                  }\r
+                                                                } else {\r
+                                                                  if (MinU1 > 0x7F) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, I->getLine(), "INT8 type minimum can't small than -0x80, big than 0x7F");\r
+                                                                  }\r
                                                                 }\r
                                                               }\r
+                                                              if (MinNegative) {\r
+                                                                MinU1 = ~MinU1 + 1;\r
+                                                              }\r
+                                                              break;\r
                                                             }\r
-                                                            if (MinNegative) {\r
-                                                              MinU1 = ~MinU1 + 1;\r
-                                                            }\r
-                                                            break;\r
                                                           }\r
                                                        >>\r
   Maximum   "=" \r
@@ -2534,178 +2756,236 @@ vfrSetMinMaxStep[CIfrMinMaxStepData & MMSDObj] :
                                                           if (!IntDecStyle && MaxNegative) {\r
                                                             _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "\"-\" can't be used when not in int decimal type. ");\r
                                                           }\r
-\r
-                                                          switch (_GET_CURRQEST_DATATYPE()) {\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_64 : \r
-                                                            MaxU8 = _STOU64(A->getText(), A->getLine()); \r
-                                                            if (IntDecStyle) {\r
+                                                          if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            MaxU4 = _STOU32(A->getText(), A->getLine());\r
+                                                            if (!IntDecStyle && MaxU4 > (1<< _GET_CURRQEST_VARTINFO().mVarTotalSize) -1) {\r
+                                                              _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "BIT type maximum can't bigger than 2^BitWidth -1");\r
+                                                            }\r
+                                                          } else {\r
+                                                            switch (_GET_CURRQEST_DATATYPE()) {\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_64 :\r
+                                                              MaxU8 = _STOU64(A->getText(), A->getLine());\r
+                                                              if (IntDecStyle) {\r
+                                                                if (MaxNegative) {\r
+                                                                  if (MaxU8 > 0x8000000000000000) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT64 type maximum can't small than -0x8000000000000000, big than 0x7FFFFFFFFFFFFFFF");\r
+                                                                  }\r
+                                                                } else {\r
+                                                                  if (MaxU8 > 0x7FFFFFFFFFFFFFFF) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT64 type maximum can't small than -0x8000000000000000, big than 0x7FFFFFFFFFFFFFFF");\r
+                                                                  }\r
+                                                                }\r
+                                                              }\r
                                                               if (MaxNegative) {\r
-                                                                if (MaxU8 > 0x8000000000000000) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT64 type maximum can't small than -0x8000000000000000, big than 0x7FFFFFFFFFFFFFFF");\r
+                                                                MaxU8 = ~MaxU8 + 1;\r
+                                                              }\r
+                                                              if (IntDecStyle) {\r
+                                                                if ((INT64) MaxU8 < (INT64) MinU8) {\r
+                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
                                                                 }\r
                                                               } else {\r
-                                                                if (MaxU8 > 0x7FFFFFFFFFFFFFFF) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT64 type maximum can't small than -0x8000000000000000, big than 0x7FFFFFFFFFFFFFFF");\r
+                                                                if (MaxU8 < MinU8) {\r
+                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
                                                                 }\r
                                                               }\r
-                                                            }\r
-                                                            if (MaxNegative) {\r
-                                                              MaxU8 = ~MaxU8 + 1;\r
-                                                            }\r
-                                                            if (IntDecStyle) {\r
-                                                              if ((INT64) MaxU8 < (INT64) MinU8) {\r
-                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
-                                                              }\r
-                                                            } else {\r
-                                                              if (MaxU8 < MinU8) {\r
-                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
+                                                              break;\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_32 :\r
+                                                              MaxU4 = _STOU32(A->getText(), A->getLine());\r
+                                                              if (IntDecStyle) {\r
+                                                                if (MaxNegative) {\r
+                                                                  if (MaxU4 > 0x80000000) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT32 type maximum can't small than -0x80000000, big than 0x7FFFFFFF");\r
+                                                                  }\r
+                                                                } else {\r
+                                                                  if (MaxU4 > 0x7FFFFFFF) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT32 type maximum can't small than -0x80000000, big than 0x7FFFFFFF");\r
+                                                                  }\r
+                                                                }\r
                                                               }\r
-                                                            }\r
-                                                            break;\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_32 : \r
-                                                            MaxU4 = _STOU32(A->getText(), A->getLine());\r
-                                                            if (IntDecStyle) {\r
                                                               if (MaxNegative) {\r
-                                                                if (MaxU4 > 0x80000000) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT32 type maximum can't small than -0x80000000, big than 0x7FFFFFFF");\r
+                                                                MaxU4 = ~MaxU4 + 1;\r
+                                                              }\r
+                                                              if (IntDecStyle) {\r
+                                                                if ((INT32) MaxU4 < (INT32) MinU4) {\r
+                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
                                                                 }\r
                                                               } else {\r
-                                                                if (MaxU4 > 0x7FFFFFFF) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT32 type maximum can't small than -0x80000000, big than 0x7FFFFFFF");\r
+                                                                if (MaxU4 < MinU4) {\r
+                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
                                                                 }\r
                                                               }\r
-                                                            }\r
-                                                            if (MaxNegative) {\r
-                                                              MaxU4 = ~MaxU4 + 1;\r
-                                                            }\r
-                                                            if (IntDecStyle) {\r
-                                                              if ((INT32) MaxU4 < (INT32) MinU4) {\r
-                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
-                                                              }\r
-                                                            } else {\r
-                                                              if (MaxU4 < MinU4) {\r
-                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
+                                                              break;\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_16 :\r
+                                                              MaxU2 = _STOU16(A->getText(), A->getLine());\r
+                                                              if (IntDecStyle) {\r
+                                                                if (MaxNegative) {\r
+                                                                  if (MaxU2 > 0x8000) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT16 type maximum can't small than -0x8000, big than 0x7FFF");\r
+                                                                  }\r
+                                                                } else {\r
+                                                                  if (MaxU2 > 0x7FFF) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT16 type maximum can't small than -0x8000, big than 0x7FFF");\r
+                                                                  }\r
+                                                                }\r
                                                               }\r
-                                                            }\r
-                                                            break;\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_16 : \r
-                                                            MaxU2 = _STOU16(A->getText(), A->getLine()); \r
-                                                            if (IntDecStyle) {\r
                                                               if (MaxNegative) {\r
-                                                                if (MaxU2 > 0x8000) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT16 type maximum can't small than -0x8000, big than 0x7FFF");\r
+                                                                MaxU2 = ~MaxU2 + 1;\r
+                                                              }\r
+                                                              if (IntDecStyle) {\r
+                                                                if ((INT16) MaxU2 < (INT16) MinU2) {\r
+                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
                                                                 }\r
                                                               } else {\r
-                                                                if (MaxU2 > 0x7FFF) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT16 type maximum can't small than -0x8000, big than 0x7FFF");\r
+                                                                if (MaxU2 < MinU2) {\r
+                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
                                                                 }\r
                                                               }\r
-                                                            }\r
-                                                            if (MaxNegative) {\r
-                                                              MaxU2 = ~MaxU2 + 1;\r
-                                                            }\r
-                                                            if (IntDecStyle) {\r
-                                                              if ((INT16) MaxU2 < (INT16) MinU2) {\r
-                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
-                                                              }\r
-                                                            } else {\r
-                                                              if (MaxU2 < MinU2) {\r
-                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
+                                                              break;\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_8 :\r
+                                                              MaxU1 = _STOU8(A->getText(), A->getLine());\r
+                                                              if (IntDecStyle) {\r
+                                                                if (MaxNegative) {\r
+                                                                  if (MaxU1 > 0x80) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT8 type maximum can't small than -0x80, big than 0x7F");\r
+                                                                  }\r
+                                                                } else {\r
+                                                                  if (MaxU1 > 0x7F) {\r
+                                                                    _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT8 type maximum can't small than -0x80, big than 0x7F");\r
+                                                                  }\r
+                                                                }\r
                                                               }\r
-                                                            }\r
-                                                            break;\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_8 :  \r
-                                                            MaxU1 = _STOU8(A->getText(), A->getLine());\r
-                                                            if (IntDecStyle) {\r
                                                               if (MaxNegative) {\r
-                                                                if (MaxU1 > 0x80) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT8 type maximum can't small than -0x80, big than 0x7F");\r
+                                                                MaxU1 = ~MaxU1 + 1;\r
+                                                              }\r
+                                                              if (IntDecStyle) {\r
+                                                                if ((INT8) MaxU1 < (INT8) MinU1) {\r
+                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
                                                                 }\r
                                                               } else {\r
-                                                                if (MaxU1 > 0x7F) {\r
-                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "INT8 type maximum can't small than -0x80, big than 0x7F");\r
+                                                                if (MaxU1 < MinU1) {\r
+                                                                  _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
                                                                 }\r
                                                               }\r
+                                                              break;\r
                                                             }\r
-                                                            if (MaxNegative) {\r
-                                                              MaxU1 = ~MaxU1 + 1;\r
-                                                            }\r
-                                                            if (IntDecStyle) {\r
-                                                              if ((INT8) MaxU1 < (INT8) MinU1) {\r
-                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
-                                                              }\r
-                                                            } else {\r
-                                                              if (MaxU1 < MinU1) {\r
-                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, A->getLine(), "Maximum can't be less than Minimum");\r
-                                                              }\r
-                                                            }\r
-                                                            break;\r
                                                           }\r
                                                        >>\r
   {\r
     STEP    "=" S:Number ","\r
                                                        <<\r
-                                                          switch (_GET_CURRQEST_DATATYPE()) {\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_64 : StepU8 = _STOU64(S->getText(), S->getLine()); break;\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_32 : StepU4 = _STOU32(S->getText(), S->getLine()); break;\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_16 : StepU2 = _STOU16(S->getText(), S->getLine()); break;\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_8 :  StepU1 = _STOU8(S->getText(), S->getLine());  break;\r
+                                                          if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            StepU4 = _STOU32(S->getText(), S->getLine());\r
+                                                          } else {\r
+                                                            switch (_GET_CURRQEST_DATATYPE()) {\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_64 : StepU8 = _STOU64(S->getText(), S->getLine()); break;\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_32 : StepU4 = _STOU32(S->getText(), S->getLine()); break;\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_16 : StepU2 = _STOU16(S->getText(), S->getLine()); break;\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_8 :  StepU1 = _STOU8(S->getText(), S->getLine());  break;\r
+                                                            }\r
                                                           }\r
                                                        >>\r
   }\r
                                                        <<\r
-                                                          switch (_GET_CURRQEST_DATATYPE()) {\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_64 : $MMSDObj.SetMinMaxStepData (MinU8, MaxU8, StepU8); break;\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_32 : $MMSDObj.SetMinMaxStepData (MinU4, MaxU4, StepU4); break;\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_16 : $MMSDObj.SetMinMaxStepData (MinU2, MaxU2, StepU2); break;\r
-                                                          case EFI_IFR_TYPE_NUM_SIZE_8 :  $MMSDObj.SetMinMaxStepData (MinU1, MaxU1, StepU1);  break;\r
+                                                          if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            $MMSDObj.SetMinMaxStepData (MinU4, MaxU4, StepU4);\r
+                                                          } else {\r
+                                                            switch (_GET_CURRQEST_DATATYPE()) {\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_64 : $MMSDObj.SetMinMaxStepData (MinU8, MaxU8, StepU8); break;\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_32 : $MMSDObj.SetMinMaxStepData (MinU4, MaxU4, StepU4); break;\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_16 : $MMSDObj.SetMinMaxStepData (MinU2, MaxU2, StepU2); break;\r
+                                                            case EFI_IFR_TYPE_NUM_SIZE_8 :  $MMSDObj.SetMinMaxStepData (MinU1, MaxU1, StepU1);  break;\r
+                                                            }\r
                                                           }\r
                                                        >>\r
   ;\r
 \r
 vfrStatementNumeric :\r
   <<\r
-     CIfrNumeric NObj;\r
+     CIfrNumeric *NObj = NULL;\r
      UINT32      DataTypeSize;\r
      BOOLEAN     IsSupported = TRUE;\r
      UINT8       ShrinkSize  = 0;\r
+     CIfrGuid    *GuidObj = NULL;\r
+     UINT8       LFlags = _GET_CURRQEST_DATATYPE() & EFI_IFR_NUMERIC_SIZE;\r
+     EFI_QUESTION_ID   QId = EFI_QUESTION_ID_INVALID;\r
+     EFI_VARSTORE_INFO Info;\r
+     Info.mVarType          = EFI_IFR_TYPE_OTHER;\r
+     Info.mVarTotalSize     = 0;\r
+     Info.mInfo.mVarOffset  = EFI_VAROFFSET_INVALID;\r
+     Info.mVarStoreId       = EFI_VARSTORE_ID_INVALID;\r
+     Info.mIsBitVar         = FALSE;\r
   >>\r
-  L:Numeric                                            << NObj.SetLineNo(L->getLine()); >>\r
-  vfrQuestionHeader[NObj] ","                          << // check data type\r
+  L:Numeric\r
+  vfrQuestionBaseInfo[Info, QId]                       <<\r
+                                                         //\r
+                                                         // Create a GUID opcode to wrap the numeric opcode, if it refer to bit varstore.\r
+                                                         //\r
+                                                         if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                           GuidObj = new CIfrGuid(0);\r
+                                                           GuidObj->SetGuid (&gEdkiiIfrBitVarGuid);\r
+                                                           GuidObj->SetLineNo(L->getLine());\r
+                                                         }\r
+                                                         NObj = new CIfrNumeric;\r
+                                                         NObj->SetLineNo(L->getLine());\r
+                                                         NObj->SetQuestionId (QId);\r
+                                                         NObj->SetVarStoreInfo (&Info);\r
+                                                        >>\r
+  vfrStatementHeader[NObj]","\r
+                                                         <<\r
+                                                          // check data type\r
                                                           if (_GET_CURRQEST_VARTINFO().mVarStoreId != EFI_VARSTORE_ID_INVALID) {\r
-                                                            _PCATCH (gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &DataTypeSize), L->getLine(), "Numeric varid is not the valid data type");\r
-                                                            if (DataTypeSize != 0 && DataTypeSize != _GET_CURRQEST_VARSIZE()) {\r
-                                                              _PCATCH (VFR_RETURN_INVALID_PARAMETER, L->getLine(), "Numeric varid doesn't support array");\r
+                                                            if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                               LFlags = (EDKII_IFR_NUMERIC_SIZE_BIT & (_GET_CURRQEST_VARSIZE()));\r
+                                                              _PCATCH(NObj->SetFlagsForBitField (NObj->FLAGS(), LFlags), L->getLine());\r
+                                                            } else {\r
+                                                              _PCATCH (gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &DataTypeSize), L->getLine(), "Numeric varid is not the valid data type");\r
+                                                              if (DataTypeSize != 0 && DataTypeSize != _GET_CURRQEST_VARSIZE()) {\r
+                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, L->getLine(), "Numeric varid doesn't support array");\r
+                                                              }\r
+                                                              _PCATCH(NObj->SetFlags (NObj->FLAGS(), _GET_CURRQEST_DATATYPE()), L->getLine());\r
                                                             }\r
-                                                            _PCATCH(NObj.SetFlags (NObj.FLAGS(), _GET_CURRQEST_DATATYPE()), L->getLine());\r
                                                           }\r
                                                        >>\r
-  { F:FLAGS "=" vfrNumericFlags[NObj, F->getLine()] "," }\r
+  { F:FLAGS "=" vfrNumericFlags[*NObj, F->getLine()] "," }\r
   {\r
-    Key   "=" KN:Number ","                            << AssignQuestionKey (NObj, KN); >>\r
+    Key   "=" KN:Number ","                            << AssignQuestionKey (*NObj, KN); >>\r
   }\r
-  vfrSetMinMaxStep[NObj]                               <<\r
-                                                          switch (_GET_CURRQEST_DATATYPE()) {\r
+  vfrSetMinMaxStep[*NObj]                               <<\r
+                                                          if (!_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            switch (_GET_CURRQEST_DATATYPE()) {\r
+                                                              //\r
+                                                              // Base on the type to know the actual used size,shrink the buffer\r
+                                                              // size allocate before.\r
+                                                              //\r
+                                                              case EFI_IFR_TYPE_NUM_SIZE_8: ShrinkSize = 21;break;\r
+                                                              case EFI_IFR_TYPE_NUM_SIZE_16:ShrinkSize = 18;break;\r
+                                                              case EFI_IFR_TYPE_NUM_SIZE_32:ShrinkSize = 12;break;\r
+                                                              case EFI_IFR_TYPE_NUM_SIZE_64:break;\r
+                                                              default:\r
+                                                                IsSupported = FALSE;\r
+                                                                break;\r
+                                                            }\r
+                                                          } else {\r
                                                             //\r
-                                                            // Base on the type to know the actual used size,shrink the buffer \r
-                                                            // size allocate before.\r
+                                                            // Question stored in bit fields saved as UINT32 type, so the ShrinkSize same as EFI_IFR_TYPE_NUM_SIZE_32.\r
                                                             //\r
-                                                            case EFI_IFR_TYPE_NUM_SIZE_8: ShrinkSize = 21;break;\r
-                                                            case EFI_IFR_TYPE_NUM_SIZE_16:ShrinkSize = 18;break;\r
-                                                            case EFI_IFR_TYPE_NUM_SIZE_32:ShrinkSize = 12;break;\r
-                                                            case EFI_IFR_TYPE_NUM_SIZE_64:break;\r
-                                                            default: \r
-                                                              IsSupported = FALSE;\r
-                                                              break;\r
+                                                            ShrinkSize = 12;\r
                                                           }\r
-                                                          NObj.ShrinkBinSize (ShrinkSize);\r
+                                                          NObj->ShrinkBinSize (ShrinkSize);\r
+\r
                                                           if (!IsSupported) {\r
                                                             _PCATCH (VFR_RETURN_INVALID_PARAMETER, L->getLine(), "Numeric question only support UINT8, UINT16, UINT32 and UINT64 data type.");\r
                                                           }\r
                                                        >>\r
   vfrStatementQuestionOptionList\r
   E:EndNumeric                                         << \r
-                                                          CRT_END_OP (E); \r
+                                                          CRT_END_OP (E);\r
+                                                          if (GuidObj != NULL) {\r
+                                                            GuidObj->SetScope(1);\r
+                                                            CRT_END_OP (E);\r
+                                                            delete GuidObj;\r
+                                                          }\r
+                                                          if (NObj != NULL) delete NObj;\r
                                                        >>\r
   ";"\r
   ;\r
@@ -2714,88 +2994,173 @@ vfrNumericFlags [CIfrNumeric & NObj, UINT32 LineNum] :
   <<\r
      UINT8 LFlags = _GET_CURRQEST_DATATYPE() & EFI_IFR_NUMERIC_SIZE;\r
      UINT8 HFlags = 0;\r
-     EFI_VFR_VARSTORE_TYPE VarStoreType = EFI_VFR_VARSTORE_INVALID;\r
      BOOLEAN IsSetType = FALSE;\r
      BOOLEAN IsDisplaySpecified = FALSE;\r
+     EFI_VFR_VARSTORE_TYPE VarStoreType = gCVfrDataStorage.GetVarStoreType (_GET_CURRQEST_VARTINFO().mVarStoreId);\r
   >>\r
-  numericFlagsField[HFlags, LFlags, IsSetType, IsDisplaySpecified] ( "\|" numericFlagsField[HFlags, LFlags, IsSetType, IsDisplaySpecified ] )*\r
+  numericFlagsField[HFlags, LFlags, IsSetType, IsDisplaySpecified, LineNum] ( "\|" numericFlagsField[HFlags, LFlags, IsSetType, IsDisplaySpecified, LineNum] )*\r
                                                        <<\r
                                                           //check data type flag\r
-                                                          if (_GET_CURRQEST_VARTINFO().mVarStoreId != EFI_VARSTORE_ID_INVALID) {\r
-                                                            VarStoreType = gCVfrDataStorage.GetVarStoreType (_GET_CURRQEST_VARTINFO().mVarStoreId);\r
-                                                            if (VarStoreType == EFI_VFR_VARSTORE_BUFFER || VarStoreType == EFI_VFR_VARSTORE_EFI) {\r
-                                                              if (_GET_CURRQEST_DATATYPE() != (LFlags & EFI_IFR_NUMERIC_SIZE)) {\r
-                                                                _PCATCH(VFR_RETURN_INVALID_PARAMETER, LineNum, "Numeric Flag is not same to Numeric VarData type");\r
+                                                          if (!_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            if (_GET_CURRQEST_VARTINFO().mVarStoreId != EFI_VARSTORE_ID_INVALID) {\r
+                                                              if (VarStoreType == EFI_VFR_VARSTORE_BUFFER || VarStoreType == EFI_VFR_VARSTORE_EFI) {\r
+                                                                if (_GET_CURRQEST_DATATYPE() != (LFlags & EFI_IFR_NUMERIC_SIZE)) {\r
+                                                                  _PCATCH(VFR_RETURN_INVALID_PARAMETER, LineNum, "Numeric Flag is not same to Numeric VarData type");\r
+                                                                }\r
+                                                              } else {\r
+                                                                // update data type for name/value store\r
+                                                                UINT32 DataTypeSize;\r
+                                                                _GET_CURRQEST_VARTINFO().mVarType = LFlags & EFI_IFR_NUMERIC_SIZE;\r
+                                                                gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &DataTypeSize);\r
+                                                                _GET_CURRQEST_VARTINFO().mVarTotalSize = DataTypeSize;\r
                                                               }\r
-                                                            } else {\r
-                                                              // update data type for name/value store\r
-                                                              UINT32 DataTypeSize;\r
+                                                            } else if (IsSetType){\r
                                                               _GET_CURRQEST_VARTINFO().mVarType = LFlags & EFI_IFR_NUMERIC_SIZE;\r
-                                                              gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &DataTypeSize);\r
-                                                              _GET_CURRQEST_VARTINFO().mVarTotalSize = DataTypeSize;\r
                                                             }\r
-                                                          } else if (IsSetType){\r
-                                                            _GET_CURRQEST_VARTINFO().mVarType = LFlags & EFI_IFR_NUMERIC_SIZE;\r
+                                                            _PCATCH(NObj.SetFlags (HFlags, LFlags, IsDisplaySpecified), LineNum);\r
+                                                          } else if ((_GET_CURRQEST_VARTINFO().mVarStoreId != EFI_VARSTORE_ID_INVALID) && (_GET_CURRQEST_VARTINFO().mIsBitVar)) {\r
+                                                            LFlags |= (EDKII_IFR_NUMERIC_SIZE_BIT & (_GET_CURRQEST_VARSIZE()));\r
+                                                            _PCATCH(NObj.SetFlagsForBitField (HFlags, LFlags, IsDisplaySpecified), LineNum);\r
                                                           }\r
-                                                          _PCATCH(NObj.SetFlags (HFlags, LFlags, IsDisplaySpecified), LineNum);\r
                                                        >>\r
   ;\r
 \r
-numericFlagsField [UINT8 & HFlags, UINT8 & LFlags, BOOLEAN & IsSetType, BOOLEAN & IsDisplaySpecified] :\r
+numericFlagsField [UINT8 & HFlags, UINT8 & LFlags, BOOLEAN & IsSetType, BOOLEAN & IsDisplaySpecified, UINT32 LineNum] :\r
     N:Number                                           << _PCATCH(_STOU8(N->getText(), N->getLine()) == 0 ? VFR_RETURN_SUCCESS : VFR_RETURN_UNSUPPORTED, N->getLine()); >>\r
-  | "NUMERIC_SIZE_1"                                   << $LFlags = ($LFlags & ~EFI_IFR_NUMERIC_SIZE) | EFI_IFR_NUMERIC_SIZE_1; IsSetType = TRUE;>>\r
-  | "NUMERIC_SIZE_2"                                   << $LFlags = ($LFlags & ~EFI_IFR_NUMERIC_SIZE) | EFI_IFR_NUMERIC_SIZE_2; IsSetType = TRUE;>>\r
-  | "NUMERIC_SIZE_4"                                   << $LFlags = ($LFlags & ~EFI_IFR_NUMERIC_SIZE) | EFI_IFR_NUMERIC_SIZE_4; IsSetType = TRUE;>>\r
-  | "NUMERIC_SIZE_8"                                   << $LFlags = ($LFlags & ~EFI_IFR_NUMERIC_SIZE) | EFI_IFR_NUMERIC_SIZE_8; IsSetType = TRUE;>>\r
-  | "DISPLAY_INT_DEC"                                  << $LFlags = ($LFlags & ~EFI_IFR_DISPLAY) | EFI_IFR_DISPLAY_INT_DEC; IsDisplaySpecified = TRUE;>>\r
-  | "DISPLAY_UINT_DEC"                                 << $LFlags = ($LFlags & ~EFI_IFR_DISPLAY) | EFI_IFR_DISPLAY_UINT_DEC; IsDisplaySpecified = TRUE;>>\r
-  | "DISPLAY_UINT_HEX"                                 << $LFlags = ($LFlags & ~EFI_IFR_DISPLAY) | EFI_IFR_DISPLAY_UINT_HEX; IsDisplaySpecified = TRUE;>>\r
+  | "NUMERIC_SIZE_1"                                   << if (!_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            $LFlags = ($LFlags & ~EFI_IFR_NUMERIC_SIZE) | EFI_IFR_NUMERIC_SIZE_1;IsSetType = TRUE;\r
+                                                          } else {\r
+                                                            _PCATCH(VFR_RETURN_INVALID_PARAMETER, LineNum, "Can not specify the size of the numeric value for BIT field");\r
+                                                          }\r
+                                                       >>\r
+  | "NUMERIC_SIZE_2"                                   << if (!_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            $LFlags = ($LFlags & ~EFI_IFR_NUMERIC_SIZE) | EFI_IFR_NUMERIC_SIZE_2;IsSetType = TRUE;\r
+                                                          } else {\r
+                                                            _PCATCH(VFR_RETURN_INVALID_PARAMETER, LineNum, "Can not specify the size of the numeric value for BIT field");\r
+                                                          }\r
+                                                       >>\r
+  | "NUMERIC_SIZE_4"                                   << if (!_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            $LFlags = ($LFlags & ~EFI_IFR_NUMERIC_SIZE) | EFI_IFR_NUMERIC_SIZE_4; IsSetType = TRUE;\r
+                                                          } else {\r
+                                                            _PCATCH(VFR_RETURN_INVALID_PARAMETER, LineNum, "Can not specify the size of the numeric value for BIT field");\r
+                                                          }\r
+                                                       >>\r
+  | "NUMERIC_SIZE_8"                                   << if (!_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            $LFlags = ($LFlags & ~EFI_IFR_NUMERIC_SIZE) | EFI_IFR_NUMERIC_SIZE_8; IsSetType = TRUE;\r
+                                                          } else {\r
+                                                            _PCATCH(VFR_RETURN_INVALID_PARAMETER, LineNum, "Can not specify the size of the numeric value for BIT field");\r
+                                                          }\r
+                                                       >>\r
+  | "DISPLAY_INT_DEC"                                  << if (!_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            $LFlags = ($LFlags & ~EFI_IFR_DISPLAY) | EFI_IFR_DISPLAY_INT_DEC;\r
+                                                          } else {\r
+                                                            $LFlags = ($LFlags & ~EDKII_IFR_DISPLAY_BIT) | EDKII_IFR_DISPLAY_INT_DEC_BIT;\r
+                                                          }\r
+                                                          IsDisplaySpecified = TRUE;\r
+                                                       >>\r
+  | "DISPLAY_UINT_DEC"                                 << if (!_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            $LFlags = ($LFlags & ~EFI_IFR_DISPLAY) | EFI_IFR_DISPLAY_UINT_DEC;\r
+                                                          } else {\r
+                                                            $LFlags = ($LFlags & ~EDKII_IFR_DISPLAY_BIT) | EDKII_IFR_DISPLAY_UINT_DEC_BIT;\r
+                                                          }\r
+                                                          IsDisplaySpecified = TRUE;\r
+                                                       >>\r
+  | "DISPLAY_UINT_HEX"                                 << if (!_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            $LFlags = ($LFlags & ~EFI_IFR_DISPLAY) | EFI_IFR_DISPLAY_UINT_HEX;\r
+                                                          } else {\r
+                                                            $LFlags = ($LFlags & ~EDKII_IFR_DISPLAY_BIT) | EDKII_IFR_DISPLAY_UINT_HEX_BIT;\r
+                                                          }\r
+                                                          IsDisplaySpecified = TRUE;\r
+                                                       >>\r
   | questionheaderFlagsField[HFlags]\r
   ;\r
 \r
 vfrStatementOneOf :\r
   <<\r
-     CIfrOneOf OObj;\r
+     CIfrOneOf *OObj = NULL;\r
      UINT32    DataTypeSize;\r
      BOOLEAN   IsSupported = TRUE;\r
      UINT8     ShrinkSize  = 0;\r
+     CIfrGuid  *GuidObj = NULL;\r
+     UINT8 LFlags = _GET_CURRQEST_DATATYPE() & EFI_IFR_NUMERIC_SIZE;\r
+     EFI_QUESTION_ID   QId = EFI_QUESTION_ID_INVALID;;\r
+     EFI_VARSTORE_INFO Info;\r
+     Info.mVarType               = EFI_IFR_TYPE_OTHER;\r
+     Info.mVarTotalSize          = 0;\r
+     Info.mInfo.mVarOffset       = EFI_VAROFFSET_INVALID;\r
+     Info.mVarStoreId            = EFI_VARSTORE_ID_INVALID;\r
+     Info.mIsBitVar              = FALSE;\r
   >>\r
-  L:OneOf                                              << OObj.SetLineNo(L->getLine()); >>\r
-  vfrQuestionHeader[OObj] ","                          << //check data type\r
+  L:OneOf\r
+  vfrQuestionBaseInfo[Info, QId]                       <<\r
+                                                         //\r
+                                                         // Create a GUID opcode to wrap the oneof opcode, if it refer to bit varstore.\r
+                                                         //\r
+                                                         if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                           GuidObj = new CIfrGuid(0);\r
+                                                           GuidObj->SetGuid (&gEdkiiIfrBitVarGuid);\r
+                                                           GuidObj->SetLineNo(L->getLine());\r
+                                                         }\r
+                                                         OObj = new CIfrOneOf;\r
+                                                         OObj->SetLineNo(L->getLine());\r
+                                                         OObj->SetQuestionId (QId);\r
+                                                         OObj->SetVarStoreInfo (&Info);\r
+                                                        >>\r
+  vfrStatementHeader[OObj]","\r
+                                                        << //check data type\r
                                                           if (_GET_CURRQEST_VARTINFO().mVarStoreId != EFI_VARSTORE_ID_INVALID) {\r
-                                                            _PCATCH (gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &DataTypeSize), L->getLine(), "OneOf varid is not the valid data type");\r
-                                                            if (DataTypeSize != 0 && DataTypeSize != _GET_CURRQEST_VARSIZE()) {\r
-                                                              _PCATCH (VFR_RETURN_INVALID_PARAMETER, L->getLine(), "OneOf varid doesn't support array");\r
+                                                            if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                               LFlags = (EDKII_IFR_NUMERIC_SIZE_BIT & (_GET_CURRQEST_VARSIZE()));\r
+                                                              _PCATCH(OObj->SetFlagsForBitField (OObj->FLAGS(), LFlags), L->getLine());\r
+                                                            } else {\r
+                                                              _PCATCH (gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &DataTypeSize), L->getLine(), "OneOf varid is not the valid data type");\r
+                                                              if (DataTypeSize != 0 && DataTypeSize != _GET_CURRQEST_VARSIZE()) {\r
+                                                                _PCATCH (VFR_RETURN_INVALID_PARAMETER, L->getLine(), "OneOf varid doesn't support array");\r
+                                                              }\r
+                                                              _PCATCH(OObj->SetFlags (OObj->FLAGS(), _GET_CURRQEST_DATATYPE()), L->getLine());\r
                                                             }\r
-                                                            _PCATCH(OObj.SetFlags (OObj.FLAGS(), _GET_CURRQEST_DATATYPE()), L->getLine());\r
                                                           }\r
                                                        >>\r
-  { F:FLAGS "=" vfrOneofFlagsField[OObj, F->getLine()] "," }\r
+  { F:FLAGS "=" vfrOneofFlagsField[*OObj, F->getLine()] "," }\r
   {\r
-    vfrSetMinMaxStep[OObj]\r
+    vfrSetMinMaxStep[*OObj]\r
   }\r
                                                        <<\r
-                                                          switch (_GET_CURRQEST_DATATYPE()) {\r
+                                                          if (!_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            switch (_GET_CURRQEST_DATATYPE()) {\r
+                                                              //\r
+                                                              // Base on the type to know the actual used size,shrink the buffer\r
+                                                              // size allocate before.\r
+                                                              //\r
+                                                              case EFI_IFR_TYPE_NUM_SIZE_8: ShrinkSize = 21;break;\r
+                                                              case EFI_IFR_TYPE_NUM_SIZE_16:ShrinkSize = 18;break;\r
+                                                              case EFI_IFR_TYPE_NUM_SIZE_32:ShrinkSize = 12;break;\r
+                                                              case EFI_IFR_TYPE_NUM_SIZE_64:break;\r
+                                                              default:\r
+                                                                IsSupported = FALSE;\r
+                                                                break;\r
+                                                            }\r
+                                                          } else {\r
                                                             //\r
-                                                            // Base on the type to know the actual used size,shrink the buffer \r
-                                                            // size allocate before.\r
+                                                            // Question stored in bit fields saved as UINT32 type, so the ShrinkSize same as EFI_IFR_TYPE_NUM_SIZE_32.\r
                                                             //\r
-                                                            case EFI_IFR_TYPE_NUM_SIZE_8: ShrinkSize = 21;break;\r
-                                                            case EFI_IFR_TYPE_NUM_SIZE_16:ShrinkSize = 18;break;\r
-                                                            case EFI_IFR_TYPE_NUM_SIZE_32:ShrinkSize = 12;break;\r
-                                                            case EFI_IFR_TYPE_NUM_SIZE_64:break;\r
-                                                            default:\r
-                                                              IsSupported = FALSE;\r
-                                                              break;\r
+                                                            ShrinkSize = 12;\r
                                                           }\r
-                                                          OObj.ShrinkBinSize (ShrinkSize);\r
+                                                          OObj->ShrinkBinSize (ShrinkSize);\r
+\r
                                                           if (!IsSupported) {\r
                                                             _PCATCH (VFR_RETURN_INVALID_PARAMETER, L->getLine(), "OneOf question only support UINT8, UINT16, UINT32 and UINT64 data type.");\r
                                                           }\r
                                                        >>\r
   vfrStatementQuestionOptionList\r
   E:EndOneOf                                           <<\r
-                                                          CRT_END_OP (E); \r
+                                                          CRT_END_OP (E);\r
+                                                          if (GuidObj != NULL) {\r
+                                                            GuidObj->SetScope(1);\r
+                                                            CRT_END_OP (E);\r
+                                                            delete GuidObj;\r
+                                                          }\r
+                                                          if (OObj != NULL) delete OObj;\r
                                                        >>\r
   ";"\r
   ;\r
@@ -2804,30 +3169,33 @@ vfrOneofFlagsField [CIfrOneOf & OObj, UINT32 LineNum] :
   <<\r
      UINT8 LFlags = _GET_CURRQEST_DATATYPE() & EFI_IFR_NUMERIC_SIZE;\r
      UINT8 HFlags = 0;\r
-     EFI_VFR_VARSTORE_TYPE VarStoreType = EFI_VFR_VARSTORE_INVALID;\r
      BOOLEAN IsSetType = FALSE;\r
      BOOLEAN IsDisplaySpecified = FALSE;\r
+     EFI_VFR_VARSTORE_TYPE VarStoreType = gCVfrDataStorage.GetVarStoreType (_GET_CURRQEST_VARTINFO().mVarStoreId);\r
   >>\r
-  numericFlagsField[HFlags, LFlags, IsSetType, IsDisplaySpecified] ( "\|" numericFlagsField[HFlags, LFlags, IsSetType, IsDisplaySpecified] )*\r
+  numericFlagsField[HFlags, LFlags, IsSetType, IsDisplaySpecified, LineNum] ( "\|" numericFlagsField[HFlags, LFlags, IsSetType, IsDisplaySpecified, LineNum] )*\r
                                                        <<\r
                                                           //check data type flag\r
-                                                          if (_GET_CURRQEST_VARTINFO().mVarStoreId != EFI_VARSTORE_ID_INVALID) {\r
-                                                            VarStoreType = gCVfrDataStorage.GetVarStoreType (_GET_CURRQEST_VARTINFO().mVarStoreId);\r
-                                                            if (VarStoreType == EFI_VFR_VARSTORE_BUFFER || VarStoreType == EFI_VFR_VARSTORE_EFI) {\r
-                                                              if (_GET_CURRQEST_DATATYPE() != (LFlags & EFI_IFR_NUMERIC_SIZE)) {\r
-                                                                _PCATCH(VFR_RETURN_INVALID_PARAMETER, LineNum, "Numeric Flag is not same to Numeric VarData type");\r
+                                                          if (!_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                            if (_GET_CURRQEST_VARTINFO().mVarStoreId != EFI_VARSTORE_ID_INVALID) {\r
+                                                              if (VarStoreType == EFI_VFR_VARSTORE_BUFFER || VarStoreType == EFI_VFR_VARSTORE_EFI) {\r
+                                                                if (_GET_CURRQEST_DATATYPE() != (LFlags & EFI_IFR_NUMERIC_SIZE)) {\r
+                                                                 _PCATCH(VFR_RETURN_INVALID_PARAMETER, LineNum, "Numeric Flag is not same to Numeric VarData type");\r
+                                                                }\r
+                                                              } else {\r
+                                                                // update data type for Name/Value store\r
+                                                                UINT32 DataTypeSize;\r
+                                                                _GET_CURRQEST_VARTINFO().mVarType = LFlags & EFI_IFR_NUMERIC_SIZE;\r
+                                                                gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &DataTypeSize);\r
+                                                                _GET_CURRQEST_VARTINFO().mVarTotalSize = DataTypeSize;\r
                                                               }\r
-                                                            } else {\r
-                                                              // update data type for Name/Value store\r
-                                                              UINT32 DataTypeSize;\r
+                                                            } else if (IsSetType){\r
                                                               _GET_CURRQEST_VARTINFO().mVarType = LFlags & EFI_IFR_NUMERIC_SIZE;\r
-                                                              gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &DataTypeSize);\r
-                                                              _GET_CURRQEST_VARTINFO().mVarTotalSize = DataTypeSize;\r
                                                             }\r
-                                                          } else if (IsSetType){\r
-                                                            _GET_CURRQEST_VARTINFO().mVarType = LFlags & EFI_IFR_NUMERIC_SIZE;\r
+                                                            _PCATCH(OObj.SetFlags (HFlags, LFlags), LineNum);\r
+                                                          } else if (_GET_CURRQEST_VARTINFO().mVarStoreId != EFI_VARSTORE_ID_INVALID) {\r
+                                                            _PCATCH(OObj.SetFlagsForBitField (HFlags, LFlags), LineNum);\r
                                                           }\r
-                                                          _PCATCH(OObj.SetFlags (HFlags, LFlags), LineNum);\r
                                                        >>\r
   ;\r
 \r
@@ -3381,22 +3749,26 @@ vfrStatementOneOfOption :
                                                        << \r
                                                           if (gCurrentMinMaxData != NULL) {\r
                                                             //set min/max value for oneof opcode\r
-                                                            UINT64 Step = gCurrentMinMaxData->GetStepData(_GET_CURRQEST_DATATYPE());\r
-                                                            switch (_GET_CURRQEST_DATATYPE()) {\r
-                                                            case EFI_IFR_TYPE_NUM_SIZE_64:\r
-                                                              gCurrentMinMaxData->SetMinMaxStepData(Val->u64, Val->u64, Step);\r
-                                                              break;\r
-                                                            case EFI_IFR_TYPE_NUM_SIZE_32:\r
+                                                            UINT64 Step = gCurrentMinMaxData->GetStepData(_GET_CURRQEST_DATATYPE(), _GET_CURRQEST_VARTINFO().mIsBitVar);\r
+                                                            if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
                                                               gCurrentMinMaxData->SetMinMaxStepData(Val->u32, Val->u32, (UINT32) Step);\r
-                                                              break;\r
-                                                            case EFI_IFR_TYPE_NUM_SIZE_16:\r
-                                                              gCurrentMinMaxData->SetMinMaxStepData(Val->u16, Val->u16, (UINT16) Step);\r
-                                                              break;\r
-                                                            case EFI_IFR_TYPE_NUM_SIZE_8:\r
-                                                              gCurrentMinMaxData->SetMinMaxStepData(Val->u8, Val->u8, (UINT8) Step);\r
-                                                              break;\r
-                                                            default:\r
-                                                              break;\r
+                                                            } else {\r
+                                                              switch (_GET_CURRQEST_DATATYPE()) {\r
+                                                              case EFI_IFR_TYPE_NUM_SIZE_64:\r
+                                                                gCurrentMinMaxData->SetMinMaxStepData(Val->u64, Val->u64, Step);\r
+                                                                break;\r
+                                                              case EFI_IFR_TYPE_NUM_SIZE_32:\r
+                                                                gCurrentMinMaxData->SetMinMaxStepData(Val->u32, Val->u32, (UINT32) Step);\r
+                                                                break;\r
+                                                              case EFI_IFR_TYPE_NUM_SIZE_16:\r
+                                                                gCurrentMinMaxData->SetMinMaxStepData(Val->u16, Val->u16, (UINT16) Step);\r
+                                                                break;\r
+                                                              case EFI_IFR_TYPE_NUM_SIZE_8:\r
+                                                                gCurrentMinMaxData->SetMinMaxStepData(Val->u8, Val->u8, (UINT8) Step);\r
+                                                                break;\r
+                                                              default:\r
+                                                                break;\r
+                                                              }\r
                                                             }\r
                                                           }\r
                                                           if (_GET_CURRQEST_DATATYPE() == EFI_IFR_TYPE_OTHER) {\r
@@ -3430,7 +3802,14 @@ vfrStatementOneOfOption :
                                                                break;\r
                                                             }\r
                                                           } else {\r
-                                                            ReturnCode = gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &Size);\r
+                                                            //\r
+                                                            // For the oneof stored in bit fields, set the option type as UINT32.\r
+                                                            //\r
+                                                            if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                              Size = sizeof (UINT32);\r
+                                                            } else {\r
+                                                              ReturnCode = gCVfrVarDataTypeDB.GetDataTypeSize (_GET_CURRQEST_DATATYPE(), &Size);\r
+                                                            }\r
                                                           }\r
                                                           if (ReturnCode != VFR_RETURN_SUCCESS) {\r
                                                             _PCATCH (ReturnCode, L->getLine());\r
@@ -3441,9 +3820,13 @@ vfrStatementOneOfOption :
                                                           OOOObj->SetLineNo(L->getLine());\r
                                                           OOOObj->SetOption (_STOSID(S->getText(), S->getLine())); \r
                                                           if (ArrayType) {\r
-                                                            OOOObj->SetType (EFI_IFR_TYPE_BUFFER); \r
+                                                            OOOObj->SetType (EFI_IFR_TYPE_BUFFER);\r
                                                           } else {\r
-                                                            OOOObj->SetType (_GET_CURRQEST_DATATYPE()); \r
+                                                            if (_GET_CURRQEST_VARTINFO().mIsBitVar) {\r
+                                                              OOOObj->SetType ( EFI_IFR_TYPE_NUM_SIZE_32);\r
+                                                            } else {\r
+                                                              OOOObj->SetType (_GET_CURRQEST_DATATYPE());\r
+                                                            }\r
                                                           }\r
                                                           OOOObj->SetValue (*Val); \r
                                                        >>\r
@@ -5222,7 +5605,8 @@ EfiVfrParser::_DeclareDefaultLinearVarStore (
                        &mFormsetGuid,\r
                        &gCVfrVarDataTypeDB,\r
                        TypeNameList[Index],\r
-                       EFI_VARSTORE_ID_INVALID\r
+                       EFI_VARSTORE_ID_INVALID,\r
+                       FALSE\r
                        );\r
     gCVfrDataStorage.GetVarStoreId(TypeNameList[Index], &VarStoreId, &mFormsetGuid);\r
     VSObj.SetVarStoreId (VarStoreId);\r
@@ -5247,7 +5631,8 @@ EfiVfrParser::_DeclareDefaultLinearVarStore (
                        &mFormsetGuid,\r
                        &gCVfrVarDataTypeDB,\r
                        (CHAR8 *) DateType,\r
-                       EFI_VARSTORE_ID_INVALID\r
+                       EFI_VARSTORE_ID_INVALID,\r
+                       FALSE\r
                        );\r
     gCVfrDataStorage.GetVarStoreId((CHAR8 *) DateName, &VarStoreId, &mFormsetGuid);\r
     VSObj.SetVarStoreId (VarStoreId);\r
@@ -5268,7 +5653,8 @@ EfiVfrParser::_DeclareDefaultLinearVarStore (
                        &mFormsetGuid,\r
                        &gCVfrVarDataTypeDB,\r
                        (CHAR8 *) TimeType,\r
-                       EFI_VARSTORE_ID_INVALID\r
+                       EFI_VARSTORE_ID_INVALID,\r
+                       FALSE\r
                        );\r
     gCVfrDataStorage.GetVarStoreId((CHAR8 *) TimeName, &VarStoreId, &mFormsetGuid);\r
     VSObj.SetVarStoreId (VarStoreId);\r
index 37b58e2179f0465fe2547cb38a15ac03af321190..b00a926adeb549f465bcf83544dc39fefb50c222 100644 (file)
@@ -600,6 +600,61 @@ CVfrVarDataTypeDB::ExtractStructTypeName (
   return VFR_RETURN_SUCCESS;\r
 }\r
 \r
+/**\r
+  Check whether the DataType contain bit field.\r
+\r
+  @param  TypeName     The name of the type.\r
+\r
+**/\r
+BOOLEAN\r
+CVfrVarDataTypeDB::DataTypeHasBitField (\r
+  IN  CHAR8         *TypeName\r
+  )\r
+{\r
+  SVfrDataType        *pType  = NULL;\r
+  SVfrDataField       *pTmp;\r
+\r
+  GetDataType (TypeName, &pType);\r
+  for (pTmp = pType->mMembers; pTmp!= NULL; pTmp = pTmp->mNext) {\r
+    if (pTmp->mIsBitField) {\r
+       return TRUE;\r
+    }\r
+  }\r
+  return FALSE;\r
+}\r
+\r
+/**\r
+  Check whether the field is bit field or not.\r
+\r
+  @param  VarStr     Point to the field name which may contain the structure name.\r
+\r
+**/\r
+BOOLEAN\r
+CVfrVarDataTypeDB::IsThisBitField (\r
+  IN  CHAR8   *VarStr\r
+  )\r
+{\r
+  CHAR8             FName[MAX_NAME_LEN];\r
+  CHAR8             TName[MAX_NAME_LEN];\r
+  UINT32            ArrayIdx;\r
+  SVfrDataType      *pType  = NULL;\r
+  SVfrDataField     *pField = NULL;\r
+\r
+  CHECK_ERROR_RETURN (ExtractStructTypeName (VarStr, TName), VFR_RETURN_SUCCESS);\r
+  CHECK_ERROR_RETURN (GetDataType (TName, &pType), VFR_RETURN_SUCCESS);\r
+\r
+  while (*VarStr != '\0') {\r
+    CHECK_ERROR_RETURN(ExtractFieldNameAndArrary(VarStr, FName, ArrayIdx), VFR_RETURN_SUCCESS);\r
+    CHECK_ERROR_RETURN(GetTypeField (FName, pType, pField), VFR_RETURN_SUCCESS);\r
+    pType  = pField->mFieldType;\r
+  }\r
+  if (pField->mIsBitField) {\r
+    return TRUE;\r
+  } else {\r
+    return FALSE;\r
+  }\r
+}\r
+\r
 EFI_VFR_RETURN_CODE\r
 CVfrVarDataTypeDB::ExtractFieldNameAndArrary (\r
   IN  CHAR8   *&VarStr,\r
@@ -697,7 +752,8 @@ EFI_VFR_RETURN_CODE
 CVfrVarDataTypeDB::GetFieldOffset (\r
   IN  SVfrDataField *Field,\r
   IN  UINT32        ArrayIdx,\r
-  OUT UINT32        &Offset\r
+  OUT UINT32        &Offset,\r
+  IN  BOOLEAN       IsBitField\r
   )\r
 {\r
   if (Field == NULL) {\r
@@ -729,8 +785,11 @@ CVfrVarDataTypeDB::GetFieldOffset (
   //   return VFR_RETURN_ERROR_ARRARY_NUM;\r
   // }\r
   //\r
-\r
-  Offset = Field->mOffset + Field->mFieldType->mTotalSize * ((ArrayIdx == INVALID_ARRAY_INDEX) ? 0 : ArrayIdx);\r
+  if (IsBitField) {\r
+    Offset = Field->mBitOffset + Field->mFieldType->mTotalSize * ((ArrayIdx == INVALID_ARRAY_INDEX) ? 0 : ArrayIdx) * 8;\r
+  } else {\r
+    Offset = Field->mOffset + Field->mFieldType->mTotalSize * ((ArrayIdx == INVALID_ARRAY_INDEX) ? 0 : ArrayIdx);\r
+  }\r
   return VFR_RETURN_SUCCESS;\r
 }\r
 \r
@@ -749,7 +808,8 @@ CVfrVarDataTypeDB::GetFieldWidth (
 UINT32\r
 CVfrVarDataTypeDB::GetFieldSize (\r
   IN SVfrDataField *Field,\r
-  IN UINT32       ArrayIdx\r
+  IN UINT32       ArrayIdx,\r
+  IN BOOLEAN      BitField\r
   )\r
 {\r
   if (Field == NULL) {\r
@@ -757,9 +817,13 @@ CVfrVarDataTypeDB::GetFieldSize (
   }\r
 \r
   if ((ArrayIdx == INVALID_ARRAY_INDEX) && (Field->mArrayNum != 0)) {\r
-    return Field->mFieldType->mTotalSize * Field->mArrayNum;\r
+      return Field->mFieldType->mTotalSize * Field->mArrayNum;\r
   } else {\r
-    return Field->mFieldType->mTotalSize;\r
+    if (BitField) {\r
+      return Field->mBitWidth;\r
+    } else {\r
+      return Field->mFieldType->mTotalSize;\r
+    }\r
   }\r
 }\r
 \r
@@ -788,18 +852,21 @@ CVfrVarDataTypeDB::InternalTypesListInit (
         pYearField->mOffset      = 0;\r
         pYearField->mNext        = pMonthField;\r
         pYearField->mArrayNum    = 0;\r
+        pYearField->mIsBitField = FALSE;\r
 \r
         strcpy (pMonthField->mFieldName, "Month");\r
         GetDataType ((CHAR8 *)"UINT8", &pMonthField->mFieldType);\r
         pMonthField->mOffset     = 2;\r
         pMonthField->mNext       = pDayField;\r
         pMonthField->mArrayNum   = 0;\r
+        pMonthField->mIsBitField = FALSE;\r
 \r
         strcpy (pDayField->mFieldName, "Day");\r
         GetDataType ((CHAR8 *)"UINT8", &pDayField->mFieldType);\r
         pDayField->mOffset       = 3;\r
         pDayField->mNext         = NULL;\r
         pDayField->mArrayNum     = 0;\r
+        pDayField->mIsBitField = FALSE;\r
 \r
         New->mMembers            = pYearField;\r
       } else if (strcmp (gInternalTypesTable[Index].mTypeName, "EFI_HII_TIME") == 0) {\r
@@ -812,18 +879,21 @@ CVfrVarDataTypeDB::InternalTypesListInit (
         pHoursField->mOffset     = 0;\r
         pHoursField->mNext       = pMinutesField;\r
         pHoursField->mArrayNum   = 0;\r
+        pHoursField->mIsBitField = FALSE;\r
 \r
         strcpy (pMinutesField->mFieldName, "Minutes");\r
         GetDataType ((CHAR8 *)"UINT8", &pMinutesField->mFieldType);\r
         pMinutesField->mOffset   = 1;\r
         pMinutesField->mNext     = pSecondsField;\r
         pMinutesField->mArrayNum = 0;\r
+        pMinutesField->mIsBitField = FALSE;\r
 \r
         strcpy (pSecondsField->mFieldName, "Seconds");\r
         GetDataType ((CHAR8 *)"UINT8", &pSecondsField->mFieldType);\r
         pSecondsField->mOffset   = 2;\r
         pSecondsField->mNext     = NULL;\r
         pSecondsField->mArrayNum = 0;\r
+        pSecondsField->mIsBitField = FALSE;\r
 \r
         New->mMembers            = pHoursField;\r
       } else if (strcmp (gInternalTypesTable[Index].mTypeName, "EFI_HII_REF") == 0) {\r
@@ -837,24 +907,28 @@ CVfrVarDataTypeDB::InternalTypesListInit (
         pQuestionIdField->mOffset     = 0;\r
         pQuestionIdField->mNext       = pFormIdField;\r
         pQuestionIdField->mArrayNum   = 0;\r
+        pQuestionIdField->mIsBitField = FALSE;\r
 \r
         strcpy (pFormIdField->mFieldName, "FormId");\r
         GetDataType ((CHAR8 *)"UINT16", &pFormIdField->mFieldType);\r
         pFormIdField->mOffset   = 2;\r
         pFormIdField->mNext     = pFormSetGuidField;\r
         pFormIdField->mArrayNum = 0;\r
+        pFormIdField->mIsBitField = FALSE;\r
 \r
         strcpy (pFormSetGuidField->mFieldName, "FormSetGuid");\r
         GetDataType ((CHAR8 *)"EFI_GUID", &pFormSetGuidField->mFieldType);\r
         pFormSetGuidField->mOffset   = 4;\r
         pFormSetGuidField->mNext     = pDevicePathField;\r
         pFormSetGuidField->mArrayNum = 0;\r
+        pFormSetGuidField->mIsBitField = FALSE;\r
 \r
         strcpy (pDevicePathField->mFieldName, "DevicePath");\r
         GetDataType ((CHAR8 *)"EFI_STRING_ID", &pDevicePathField->mFieldType);\r
         pDevicePathField->mOffset   = 20;\r
         pDevicePathField->mNext     = NULL;\r
         pDevicePathField->mArrayNum = 0;\r
+        pDevicePathField->mIsBitField = FALSE;\r
 \r
         New->mMembers            = pQuestionIdField;\r
       } else {\r
@@ -978,6 +1052,7 @@ CVfrVarDataTypeDB::DeclareDataTypeBegin (
   pNewType->mTotalSize   = 0;\r
   pNewType->mMembers     = NULL;\r
   pNewType->mNext        = NULL;\r
+  pNewType->mHasBitField = FALSE;\r
 \r
   mNewDataType           = pNewType;\r
 }\r
@@ -1009,6 +1084,125 @@ CVfrVarDataTypeDB::SetNewTypeName (
   return VFR_RETURN_SUCCESS;\r
 }\r
 \r
+/**\r
+  Record the bit field info in the data type.\r
+\r
+  @param  FieldName     Point to the field name.\r
+  @param  TypeName      Point to the type name.\r
+  @param  Width         The bit width.\r
+  @param  FieldInUnion  The filed is in Union type or Structure type.\r
+\r
+**/\r
+EFI_VFR_RETURN_CODE\r
+CVfrVarDataTypeDB::DataTypeAddBitField (\r
+  IN CHAR8   *FieldName,\r
+  IN CHAR8   *TypeName,\r
+  IN UINT32   Width,\r
+  IN BOOLEAN FieldInUnion\r
+  )\r
+{\r
+  SVfrDataField       *pNewField  = NULL;\r
+  SVfrDataType        *pFieldType = NULL;\r
+  SVfrDataField       *pTmp;\r
+  UINT32              Align;\r
+  UINT32              MaxDataTypeSize;\r
+  BOOLEAN             UpdateTotalSize;\r
+\r
+  CHECK_ERROR_RETURN (GetDataType (TypeName, &pFieldType), VFR_RETURN_SUCCESS);\r
+\r
+  if (Width > MAX_BIT_WIDTH) {\r
+    return VFR_RETURN_BIT_WIDTH_ERROR;\r
+  }\r
+\r
+  if (Width > pFieldType->mTotalSize * 8) {\r
+    return VFR_RETURN_BIT_WIDTH_ERROR;\r
+  }\r
+\r
+  if (FieldName != NULL && strlen (FieldName) >= MAX_NAME_LEN) {\r
+   return VFR_RETURN_INVALID_PARAMETER;\r
+  }\r
+\r
+  if (Width == 0 && FieldName != NULL) {\r
+    return VFR_RETURN_INVALID_PARAMETER;\r
+  }\r
+\r
+  for (pTmp = mNewDataType->mMembers; pTmp != NULL; pTmp = pTmp->mNext) {\r
+    if (FieldName != NULL && strcmp (pTmp->mFieldName, FieldName) == 0) {\r
+      return VFR_RETURN_REDEFINED;\r
+    }\r
+  }\r
+\r
+  Align = MIN (mPackAlign, pFieldType->mAlign);\r
+  UpdateTotalSize = FALSE;\r
+\r
+  if ((pNewField = new SVfrDataField) == NULL) {\r
+    return VFR_RETURN_OUT_FOR_RESOURCES;\r
+  }\r
+\r
+  MaxDataTypeSize = mNewDataType->mTotalSize;\r
+  if (FieldName != NULL) {\r
+    strcpy (pNewField->mFieldName, FieldName);\r
+  }\r
+  pNewField->mFieldType    = pFieldType;\r
+  pNewField->mIsBitField   = TRUE;\r
+  pNewField->mBitWidth     = Width;\r
+  pNewField->mArrayNum     = 0;\r
+  pNewField->mBitOffset    = 0;\r
+  pNewField->mOffset       = 0;\r
+\r
+  if (mNewDataType->mMembers == NULL) {\r
+    mNewDataType->mMembers = pNewField;\r
+    pNewField->mNext       = NULL;\r
+  } else {\r
+    for (pTmp = mNewDataType->mMembers; pTmp->mNext != NULL; pTmp = pTmp->mNext)\r
+      ;\r
+    pTmp->mNext            = pNewField;\r
+    pNewField->mNext       = NULL;\r
+  }\r
+\r
+  if (FieldInUnion) {\r
+    pNewField->mOffset = 0;\r
+    if (MaxDataTypeSize < pNewField->mFieldType->mTotalSize) {\r
+      mNewDataType->mTotalSize = pNewField->mFieldType->mTotalSize;\r
+    }\r
+  } else {\r
+    //\r
+    // Check whether the bit fileds can be contained within one FieldType.\r
+    //\r
+    if (pTmp != NULL && pTmp->mIsBitField && strcmp (pTmp->mFieldType->mTypeName, pNewField->mFieldType->mTypeName) == 0 &&\r
+       (pTmp->mBitOffset - pTmp->mOffset * 8) + pTmp->mBitWidth + pNewField->mBitWidth <= pNewField->mFieldType->mTotalSize * 8) {\r
+      pNewField->mBitOffset = pTmp->mBitOffset + pTmp->mBitWidth;\r
+      pNewField->mOffset = pTmp->mOffset;\r
+      //\r
+      // If BitWidth=0,used to force alignment at the next word boundary.\r
+      // So make this bit field occupy the remaing bit width of current field type.\r
+      //\r
+      if (pNewField->mBitWidth == 0) {\r
+        pNewField->mBitWidth = pNewField->mFieldType->mTotalSize * 8 - (pNewField->mBitOffset - pTmp->mOffset * 8);\r
+      }\r
+    } else {\r
+      //\r
+      // The bit filed start a new memory\r
+      //\r
+      pNewField->mBitOffset = mNewDataType->mTotalSize * 8;\r
+      UpdateTotalSize = TRUE;\r
+    }\r
+  }\r
+\r
+  if (UpdateTotalSize){\r
+    if ((mNewDataType->mTotalSize % Align) == 0) {\r
+      pNewField->mOffset     = mNewDataType->mTotalSize;\r
+    } else {\r
+      pNewField->mOffset     = mNewDataType->mTotalSize + ALIGN_STUFF(mNewDataType->mTotalSize, Align);\r
+    }\r
+    mNewDataType->mTotalSize = pNewField->mOffset + (pNewField->mFieldType->mTotalSize);\r
+  }\r
+\r
+  mNewDataType->mAlign     = MIN (mPackAlign, MAX (pFieldType->mAlign, mNewDataType->mAlign));\r
+  mNewDataType->mHasBitField = TRUE;\r
+  return VFR_RETURN_SUCCESS;\r
+}\r
+\r
 EFI_VFR_RETURN_CODE\r
 CVfrVarDataTypeDB::DataTypeAddField (\r
   IN CHAR8   *FieldName,\r
@@ -1044,6 +1238,7 @@ CVfrVarDataTypeDB::DataTypeAddField (
   strcpy (pNewField->mFieldName, FieldName);\r
   pNewField->mFieldType    = pFieldType;\r
   pNewField->mArrayNum     = ArrayNum;\r
+  pNewField->mIsBitField   = FALSE;\r
   if ((mNewDataType->mTotalSize % Align) == 0) {\r
     pNewField->mOffset     = mNewDataType->mTotalSize;\r
   } else {\r
@@ -1183,21 +1378,26 @@ CVfrVarDataTypeDB::GetDataFieldInfo (
   IN  CHAR8     *VarStr,\r
   OUT UINT16   &Offset,\r
   OUT UINT8    &Type,\r
-  OUT UINT32   &Size\r
+  OUT UINT32   &Size,\r
+  OUT BOOLEAN  &BitField\r
   )\r
 {\r
   CHAR8               TName[MAX_NAME_LEN], FName[MAX_NAME_LEN];\r
   UINT32              ArrayIdx, Tmp;\r
   SVfrDataType        *pType  = NULL;\r
   SVfrDataField       *pField = NULL;\r
+  CHAR8               *VarStrName;\r
 \r
   Offset = 0;\r
   Type   = EFI_IFR_TYPE_OTHER;\r
   Size   = 0;\r
+  VarStrName = VarStr;\r
 \r
   CHECK_ERROR_RETURN (ExtractStructTypeName (VarStr, TName), VFR_RETURN_SUCCESS);\r
   CHECK_ERROR_RETURN (GetDataType (TName, &pType), VFR_RETURN_SUCCESS);\r
 \r
+  BitField = IsThisBitField (VarStrName);\r
+\r
   //\r
   // if it is not struct data type\r
   //\r
@@ -1205,13 +1405,17 @@ CVfrVarDataTypeDB::GetDataFieldInfo (
   Size  = pType->mTotalSize;\r
 \r
   while (*VarStr != '\0') {\r
-       CHECK_ERROR_RETURN(ExtractFieldNameAndArrary(VarStr, FName, ArrayIdx), VFR_RETURN_SUCCESS);\r
+    CHECK_ERROR_RETURN(ExtractFieldNameAndArrary(VarStr, FName, ArrayIdx), VFR_RETURN_SUCCESS);\r
     CHECK_ERROR_RETURN(GetTypeField (FName, pType, pField), VFR_RETURN_SUCCESS);\r
     pType  = pField->mFieldType;\r
-    CHECK_ERROR_RETURN(GetFieldOffset (pField, ArrayIdx, Tmp), VFR_RETURN_SUCCESS);\r
-    Offset = (UINT16) (Offset + Tmp);\r
+    CHECK_ERROR_RETURN(GetFieldOffset (pField, ArrayIdx, Tmp, pField->mIsBitField), VFR_RETURN_SUCCESS);\r
+    if (BitField && !pField->mIsBitField) {\r
+      Offset = (UINT16) (Offset + Tmp * 8);\r
+    } else {\r
+      Offset = (UINT16) (Offset + Tmp);\r
+    }\r
     Type   = GetFieldWidth (pField);\r
-    Size   = GetFieldSize (pField, ArrayIdx);\r
+    Size   = GetFieldSize (pField, ArrayIdx, BitField);\r
   }\r
   return VFR_RETURN_SUCCESS;\r
 }\r
@@ -1360,6 +1564,7 @@ SVfrVarStorageNode::SVfrVarStorageNode (
   IN CHAR8                 *StoreName,\r
   IN EFI_VARSTORE_ID       VarStoreId,\r
   IN SVfrDataType          *DataType,\r
+  IN BOOLEAN               BitsVarstore,\r
   IN BOOLEAN               Flag\r
   )\r
 {\r
@@ -1376,7 +1581,11 @@ SVfrVarStorageNode::SVfrVarStorageNode (
   }\r
   mNext                    = NULL;\r
   mVarStoreId              = VarStoreId;\r
-  mVarStoreType            = EFI_VFR_VARSTORE_BUFFER;\r
+  if (BitsVarstore) {\r
+    mVarStoreType            = EFI_VFR_VARSTORE_BUFFER_BITS;\r
+  } else {\r
+    mVarStoreType            = EFI_VFR_VARSTORE_BUFFER;\r
+  }\r
   mStorageInfo.mDataType   = DataType;\r
   mAssignedFlag            = Flag;\r
 }\r
@@ -1648,6 +1857,7 @@ CVfrDataStorage::DeclareBufferVarStore (
   IN CVfrVarDataTypeDB *DataTypeDB,\r
   IN CHAR8             *TypeName,\r
   IN EFI_VARSTORE_ID   VarStoreId,\r
+  IN BOOLEAN           IsBitVarStore,\r
   IN BOOLEAN           Flag\r
   )\r
 {\r
@@ -1674,7 +1884,7 @@ CVfrDataStorage::DeclareBufferVarStore (
     MarkVarStoreIdUsed (VarStoreId);\r
   }\r
 \r
-  if ((pNew = new SVfrVarStorageNode (Guid, StoreName, VarStoreId, pDataType, Flag)) == NULL) {\r
+  if ((pNew = new SVfrVarStorageNode (Guid, StoreName, VarStoreId, pDataType, IsBitVarStore, Flag)) == NULL) {\r
     return VFR_RETURN_OUT_FOR_RESOURCES;\r
   }\r
 \r
@@ -2388,6 +2598,7 @@ EFI_VARSTORE_INFO::EFI_VARSTORE_INFO (
   mInfo.mVarOffset = EFI_VAROFFSET_INVALID;\r
   mVarType         = EFI_IFR_TYPE_OTHER;\r
   mVarTotalSize    = 0;\r
+  mIsBitVar        = FALSE;\r
 }\r
 \r
 EFI_VARSTORE_INFO::EFI_VARSTORE_INFO (\r
@@ -2399,6 +2610,7 @@ EFI_VARSTORE_INFO::EFI_VARSTORE_INFO (
   mInfo.mVarOffset = Info.mInfo.mVarOffset;\r
   mVarType         = Info.mVarType;\r
   mVarTotalSize    = Info.mVarTotalSize;\r
+  mIsBitVar        = Info.mIsBitVar;\r
 }\r
 \r
 EFI_VARSTORE_INFO&\r
@@ -2412,6 +2624,7 @@ EFI_VARSTORE_INFO::operator= (
     mInfo.mVarOffset = Info.mInfo.mVarOffset;\r
     mVarType         = Info.mVarType;\r
     mVarTotalSize    = Info.mVarTotalSize;\r
+    mIsBitVar        = Info.mIsBitVar;\r
   }\r
 \r
   return *this;\r
@@ -2426,7 +2639,8 @@ EFI_VARSTORE_INFO::operator == (
          (mInfo.mVarName == Info->mInfo.mVarName) &&\r
       (mInfo.mVarOffset == Info->mInfo.mVarOffset) &&\r
       (mVarType == Info->mVarType) &&\r
-      (mVarTotalSize == Info->mVarTotalSize)) {\r
+      (mVarTotalSize == Info->mVarTotalSize) &&\r
+      (mIsBitVar == Info->mIsBitVar)) {\r
     return TRUE;\r
   }\r
 \r
index 0c67d73a7f67b1c6104544926f22ff9fc4965ae0..70af9119e6597a7098bb4bf609330ceb4ba6f472 100644 (file)
@@ -22,7 +22,9 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include "VfrError.h"\r
 \r
 extern BOOLEAN  VfrCompatibleMode;\r
+static EFI_GUID gEdkiiIfrBitVarGuid = EDKII_IFR_BIT_VARSTORE_GUID;\r
 \r
+#define MAX_BIT_WIDTH                      32\r
 #define MAX_NAME_LEN                       64\r
 #define MAX_STRING_LEN                     0x100\r
 #define DEFAULT_ALIGN                      1\r
@@ -116,6 +118,9 @@ struct SVfrDataField {
   SVfrDataType              *mFieldType;\r
   UINT32                    mOffset;\r
   UINT32                    mArrayNum;\r
+  BOOLEAN                   mIsBitField;\r
+  UINT8                     mBitWidth;\r
+  UINT32                    mBitOffset;\r
   SVfrDataField             *mNext;\r
 };\r
 \r
@@ -124,6 +129,7 @@ struct SVfrDataType {
   UINT8                     mType;\r
   UINT32                    mAlign;\r
   UINT32                    mTotalSize;\r
+  BOOLEAN                   mHasBitField;\r
   SVfrDataField             *mMembers;\r
   SVfrDataType              *mNext;\r
 };\r
@@ -195,9 +201,9 @@ private:
 \r
   EFI_VFR_RETURN_CODE ExtractStructTypeName (IN CHAR8 *&, OUT CHAR8 *);\r
   EFI_VFR_RETURN_CODE GetTypeField (IN CONST CHAR8 *, IN SVfrDataType *, IN SVfrDataField *&);\r
-  EFI_VFR_RETURN_CODE GetFieldOffset (IN SVfrDataField *, IN UINT32, OUT UINT32 &);\r
+  EFI_VFR_RETURN_CODE GetFieldOffset (IN SVfrDataField *, IN UINT32, OUT UINT32 &, IN BOOLEAN);\r
   UINT8               GetFieldWidth (IN SVfrDataField *);\r
-  UINT32              GetFieldSize (IN SVfrDataField *, IN UINT32);\r
+  UINT32              GetFieldSize (IN SVfrDataField *, IN UINT32, IN BOOLEAN);\r
 \r
 public:\r
   CVfrVarDataTypeDB (VOID);\r
@@ -206,15 +212,18 @@ public:
   VOID                DeclareDataTypeBegin (VOID);\r
   EFI_VFR_RETURN_CODE SetNewTypeName (IN CHAR8 *);\r
   EFI_VFR_RETURN_CODE DataTypeAddField (IN CHAR8 *, IN CHAR8 *, IN UINT32, IN BOOLEAN);\r
+  EFI_VFR_RETURN_CODE DataTypeAddBitField (IN CHAR8 *, IN CHAR8 *, IN UINT32, IN BOOLEAN);\r
   VOID                DeclareDataTypeEnd (VOID);\r
 \r
   EFI_VFR_RETURN_CODE GetDataType (IN CHAR8 *, OUT SVfrDataType **);\r
   EFI_VFR_RETURN_CODE GetDataTypeSize (IN CHAR8 *, OUT UINT32 *);\r
   EFI_VFR_RETURN_CODE GetDataTypeSize (IN UINT8, OUT UINT32 *);\r
-  EFI_VFR_RETURN_CODE GetDataFieldInfo (IN CHAR8 *, OUT UINT16 &, OUT UINT8 &, OUT UINT32 &);\r
+  EFI_VFR_RETURN_CODE GetDataFieldInfo (IN CHAR8 *, OUT UINT16 &, OUT UINT8 &, OUT UINT32 &, OUT BOOLEAN &);\r
 \r
   EFI_VFR_RETURN_CODE GetUserDefinedTypeNameList (OUT CHAR8 ***, OUT UINT32 *);\r
   EFI_VFR_RETURN_CODE ExtractFieldNameAndArrary (IN CHAR8 *&, OUT CHAR8 *, OUT UINT32 &);\r
+  BOOLEAN             DataTypeHasBitField (IN  CHAR8 *);\r
+  BOOLEAN             IsThisBitField (IN  CHAR8 *);\r
 \r
   BOOLEAN             IsTypeNameDefined (IN CHAR8 *);\r
 \r
@@ -238,7 +247,8 @@ typedef enum {
   EFI_VFR_VARSTORE_INVALID,\r
   EFI_VFR_VARSTORE_BUFFER,\r
   EFI_VFR_VARSTORE_EFI,\r
-  EFI_VFR_VARSTORE_NAME\r
+  EFI_VFR_VARSTORE_NAME,\r
+  EFI_VFR_VARSTORE_BUFFER_BITS\r
 } EFI_VFR_VARSTORE_TYPE;\r
 \r
 struct SVfrVarStorageNode {\r
@@ -268,7 +278,7 @@ struct SVfrVarStorageNode {
 \r
 public:\r
   SVfrVarStorageNode (IN EFI_GUID *, IN CHAR8 *, IN EFI_VARSTORE_ID, IN EFI_STRING_ID, IN UINT32, IN BOOLEAN Flag = TRUE);\r
-  SVfrVarStorageNode (IN EFI_GUID *, IN CHAR8 *, IN EFI_VARSTORE_ID, IN SVfrDataType *, IN BOOLEAN Flag = TRUE);\r
+  SVfrVarStorageNode (IN EFI_GUID *, IN CHAR8 *, IN EFI_VARSTORE_ID, IN SVfrDataType *,IN BOOLEAN, IN BOOLEAN Flag = TRUE);\r
   SVfrVarStorageNode (IN CHAR8 *, IN EFI_VARSTORE_ID);\r
   ~SVfrVarStorageNode (VOID);\r
 \r
@@ -285,6 +295,7 @@ struct EFI_VARSTORE_INFO {
   } mInfo;\r
   UINT8                     mVarType;\r
   UINT32                    mVarTotalSize;\r
+  BOOLEAN                   mIsBitVar;\r
 \r
   EFI_VARSTORE_INFO (VOID);\r
   EFI_VARSTORE_INFO (IN EFI_VARSTORE_INFO &);\r
@@ -343,7 +354,7 @@ public:
 \r
   EFI_VFR_RETURN_CODE DeclareEfiVarStore (IN CHAR8 *, IN EFI_GUID *, IN EFI_STRING_ID, IN UINT32, IN BOOLEAN Flag = TRUE);\r
 \r
-  EFI_VFR_RETURN_CODE DeclareBufferVarStore (IN CHAR8 *, IN EFI_GUID *, IN CVfrVarDataTypeDB *, IN CHAR8 *, IN EFI_VARSTORE_ID, IN BOOLEAN Flag = TRUE);\r
+  EFI_VFR_RETURN_CODE DeclareBufferVarStore (IN CHAR8 *, IN EFI_GUID *, IN CVfrVarDataTypeDB *, IN CHAR8 *, IN EFI_VARSTORE_ID, IN BOOLEAN, IN BOOLEAN Flag = TRUE);\r
 \r
   EFI_VFR_RETURN_CODE GetVarStoreId (IN CHAR8 *, OUT EFI_VARSTORE_ID *, IN EFI_GUID *VarGuid = NULL);\r
   EFI_VFR_VARSTORE_TYPE GetVarStoreType (IN EFI_VARSTORE_ID);\r