]> git.proxmox.com Git - mirror_edk2.git/blobdiff - ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c
add comments to function declarations and definitions and updated to match coding...
[mirror_edk2.git] / ShellPkg / Library / UefiShellDebug1CommandsLib / SmbiosView / PrintInfo.c
index 91835106d56015a4c261dce1558d5a6497e1f2c1..3ae1eeaf7c40a0dcc789c4989af3044e3603f8fa 100644 (file)
@@ -1,7 +1,7 @@
 /** @file\r
   Module for clarifying the content of the smbios structure element information.\r
 \r
-  Copyright (c) 2005 - 2010, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2005 - 2011, 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
@@ -28,7 +28,7 @@
 //////////////////////////////////////////////////////////\r
 //  Macros of print structure element, simplify coding.\r
 //\r
-#define PrintPendingString(pStruct, type, element) \\r
+#define PRINT_PENDING_STRING(pStruct, type, element) \\r
   do { \\r
     CHAR8 StringBuf[64]; \\r
     SetMem (StringBuf, sizeof (StringBuf), 0x00); \\r
@@ -37,7 +37,7 @@
     ShellPrintEx(-1,-1,L": %a\n", StringBuf); \\r
   } while (0);\r
 \r
-#define PrintSmbiosString(pStruct, stringnumber, element) \\r
+#define PRINT_SMBIOS_STRING(pStruct, stringnumber, element) \\r
   do { \\r
     CHAR8 StringBuf[64]; \\r
     SetMem (StringBuf, sizeof (StringBuf), 0x00); \\r
     ShellPrintEx(-1,-1,L": %a\n", StringBuf); \\r
   } while (0);\r
 \r
-#define PrintStructValue(pStruct, type, element) \\r
+#define PRINT_STRUCT_VALUE(pStruct, type, element) \\r
   do { \\r
     ShellPrintEx(-1,-1,L"%a",#element); \\r
     ShellPrintEx(-1,-1,L": %d\n", (pStruct->type->element)); \\r
   } while (0);\r
 \r
-#define PrintStructValueH(pStruct, type, element) \\r
+#define PRINT_STRUCT_VALUE_H(pStruct, type, element) \\r
   do { \\r
     ShellPrintEx(-1,-1,L"%a",#element); \\r
     ShellPrintEx(-1,-1,L": 0x%x\n", (pStruct->type->element)); \\r
   } while (0);\r
 \r
-#define PrintBitField(pStruct, type, element, size) \\r
+#define PRINT_BIT_FIELD(pStruct, type, element, size) \\r
   do { \\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_DUMP), gShellDebug1HiiHandle); \\r
     ShellPrintEx(-1,-1,L"%a",#element); \\r
@@ -66,7 +66,7 @@
     DumpHex (0, 0, size, &(pStruct->type->element)); \\r
   } while (0);\r
 \r
-#define PrintSmbiosBitField(pStruct, startaddress, element, size) \\r
+#define PRINT_SMBIOS_BIT_FIELD(pStruct, startaddress, element, size) \\r
   do { \\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_DUMP), gShellDebug1HiiHandle); \\r
     ShellPrintEx(-1,-1,L"%a",#element); \\r
 //\r
 /////////////////////////////////////////\r
 //\r
+\r
+/**\r
+  Copy Length of Src buffer to Dest buffer,\r
+  add a NULL termination to Dest buffer.\r
+\r
+  @param[in,out] Dest  Destination buffer head.\r
+  @param[in] Src       Source buffer head.\r
+  @param[in] Length    Length of buffer to be copied.\r
+**/\r
 VOID\r
 MemToString (\r
   IN OUT VOID   *Dest,\r
   IN VOID       *Src,\r
   IN UINTN      Length\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Copy Length of Src buffer to Dest buffer,\r
-  add a NULL termination to Dest buffer.\r
-\r
-Arguments:\r
-  Dest      - Destination buffer head\r
-  Src       - Source buffer head\r
-  Length    - Length of buffer to be copied\r
-\r
-Returns:\r
-  None.\r
-\r
-**/\r
 {\r
   UINT8 *SrcBuffer;\r
   UINT8 *DestBuffer;\r
@@ -120,23 +114,18 @@ Returns:
 //\r
 // Functions below is to show the information\r
 //\r
+\r
+/**\r
+  Print the info of EPS(Entry Point Structure).\r
+\r
+  @param[in] SmbiosTable    Pointer to the SMBIOS table entry point.\r
+  @param[in] Option         Display option.\r
+**/\r
 VOID\r
 SmbiosPrintEPSInfo (\r
   IN  SMBIOS_STRUCTURE_TABLE  *SmbiosTable,\r
   IN  UINT8                   Option\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Print the info of EPS(Entry Point Structure)\r
-\r
-Arguments:\r
-  SmbiosTable    - Pointer to the SMBIOS table entry point\r
-  Option         - Display option\r
-\r
-Returns: None\r
-\r
-**/\r
 {\r
   UINT8 Anchor[5];\r
   UINT8 InAnchor[6];\r
@@ -199,33 +188,28 @@ Returns: None
   Print (L"\n");\r
 }\r
 \r
+/**\r
+  This function print the content of the structure pointed by Struct.\r
+\r
+  @param[in] Struct       Point to the structure to be printed.\r
+  @param[in] Option       Print option of information detail.\r
+\r
+  @retval EFI_SUCCESS               Successfully Printing this function.\r
+  @retval EFI_INVALID_PARAMETER     Invalid Structure.\r
+  @retval EFI_UNSUPPORTED           Unsupported.\r
+**/\r
 EFI_STATUS\r
 SmbiosPrintStructure (\r
-  IN  SMBIOS_STRUCTURE_POINTER  *pStruct,\r
+  IN  SMBIOS_STRUCTURE_POINTER  *Struct,\r
   IN  UINT8                     Option\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  This function print the content of the structure pointed by pStruct\r
-\r
-Arguments:\r
-  pStruct     - point to the structure to be printed\r
-  Option      - print option of information detail\r
-\r
-Returns:\r
-  EFI_SUCCESS             - Successfully Printing this function\r
-  EFI_INVALID_PARAMETER   - Invalid Structure\r
-  EFI_UNSUPPORTED         - Unsupported\r
-\r
-**/\r
 {\r
   UINT8 Index;\r
   UINT8 *Buffer;\r
 \r
-  Buffer = (UINT8 *) (UINTN) (pStruct->Raw);\r
+  Buffer = (UINT8 *) (UINTN) (Struct->Raw);\r
 \r
-  if (pStruct == NULL) {\r
+  if (Struct == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -235,35 +219,35 @@ Returns:
   //\r
   // Display structure header\r
   //\r
-  DisplayStructureTypeInfo (pStruct->Hdr->Type, SHOW_DETAIL);\r
-  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_FORMAT_PART_LEN), gShellDebug1HiiHandle, pStruct->Hdr->Length);\r
-  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_STRUCT_HANDLE), gShellDebug1HiiHandle, pStruct->Hdr->Handle);\r
+  DisplayStructureTypeInfo (Struct->Hdr->Type, SHOW_DETAIL);\r
+  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_FORMAT_PART_LEN), gShellDebug1HiiHandle, Struct->Hdr->Length);\r
+  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_STRUCT_HANDLE), gShellDebug1HiiHandle, Struct->Hdr->Handle);\r
 \r
   if (Option == SHOW_OUTLINE) {\r
     return EFI_SUCCESS;\r
   }\r
 \r
-  switch (pStruct->Hdr->Type) {\r
+  switch (Struct->Hdr->Type) {\r
   //\r
   //\r
   //\r
   case 0:\r
-    PrintPendingString (pStruct, Type0, Vendor);\r
-    PrintPendingString (pStruct, Type0, BiosVersion);\r
-    PrintStructValue (pStruct, Type0, BiosSegment);\r
-    PrintPendingString (pStruct, Type0, BiosReleaseDate);\r
-    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIOS_SIZE), gShellDebug1HiiHandle, 64 * (pStruct->Type0->BiosSize + 1));\r
+    PRINT_PENDING_STRING (Struct, Type0, Vendor);\r
+    PRINT_PENDING_STRING (Struct, Type0, BiosVersion);\r
+    PRINT_STRUCT_VALUE (Struct, Type0, BiosSegment);\r
+    PRINT_PENDING_STRING (Struct, Type0, BiosReleaseDate);\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIOS_SIZE), gShellDebug1HiiHandle, 64 * (Struct->Type0->BiosSize + 1));\r
 \r
     if (Option < SHOW_DETAIL) {\r
-      PrintStructValueH (pStruct, Type0, BiosCharacteristics);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type0, BiosCharacteristics);\r
     } else {\r
-      DisplayBiosCharacteristics (pStruct->Type0->BiosCharacteristics, Option);\r
+      DisplayBiosCharacteristics (Struct->Type0->BiosCharacteristics, Option);\r
 \r
       //\r
       // The length of above format part is 0x12 bytes,\r
       // Ext bytes are following, size = 'len-0x12'.\r
       // If len-0x12 > 0, then\r
-      //    there are extension bytes (byte1, byte2, byte3...)\r
+      //    there are extension bytes (Byte1, byte2, byte3...)\r
       // And byte3 not stated in spec, so dump all extension bytes(1, 2, 3..)\r
       //\r
       if ((Buffer[1] - (CHAR8) 0x12) > 0) {\r
@@ -275,8 +259,8 @@ Returns:
       }\r
 \r
       if ((Buffer[1] - (CHAR8) 0x12) > 2) {\r
-        PrintBitField (\r
-          pStruct,\r
+        PRINT_BIT_FIELD (\r
+          Struct,\r
           Type0,\r
           BiosCharacteristics,\r
           Buffer[1] - (CHAR8) 0x12\r
@@ -289,77 +273,77 @@ Returns:
   // System Information (Type 1)\r
   //\r
   case 1:\r
-    PrintPendingString (pStruct, Type1, Manufacturer);\r
-    PrintPendingString (pStruct, Type1, ProductName);\r
-    PrintPendingString (pStruct, Type1, Version);\r
-    PrintPendingString (pStruct, Type1, SerialNumber);\r
-    PrintBitField (pStruct, Type1, Uuid, 16);\r
-    DisplaySystemWakeupType (pStruct->Type1->WakeUpType, Option);\r
+    PRINT_PENDING_STRING (Struct, Type1, Manufacturer);\r
+    PRINT_PENDING_STRING (Struct, Type1, ProductName);\r
+    PRINT_PENDING_STRING (Struct, Type1, Version);\r
+    PRINT_PENDING_STRING (Struct, Type1, SerialNumber);\r
+    PRINT_BIT_FIELD (Struct, Type1, Uuid, 16);\r
+    DisplaySystemWakeupType (Struct->Type1->WakeUpType, Option);\r
     break;\r
 \r
   case 2:\r
-    PrintPendingString (pStruct, Type2, Manufacturer);\r
-    PrintPendingString (pStruct, Type2, ProductName);\r
-    PrintPendingString (pStruct, Type2, Version);\r
-    PrintPendingString (pStruct, Type2, SerialNumber);\r
+    PRINT_PENDING_STRING (Struct, Type2, Manufacturer);\r
+    PRINT_PENDING_STRING (Struct, Type2, ProductName);\r
+    PRINT_PENDING_STRING (Struct, Type2, Version);\r
+    PRINT_PENDING_STRING (Struct, Type2, SerialNumber);\r
     break;\r
 \r
   //\r
   // System Enclosure (Type 3)\r
   //\r
   case 3:\r
-    PrintPendingString (pStruct, Type3, Manufacturer);\r
-    PrintStructValue (pStruct, Type3, Type);\r
-    DisplaySystemEnclosureType (pStruct->Type3->Type, Option);\r
-    PrintPendingString (pStruct, Type3, Version);\r
-    PrintPendingString (pStruct, Type3, SerialNumber);\r
-    PrintPendingString (pStruct, Type3, AssetTag);\r
+    PRINT_PENDING_STRING (Struct, Type3, Manufacturer);\r
+    PRINT_STRUCT_VALUE (Struct, Type3, Type);\r
+    DisplaySystemEnclosureType (Struct->Type3->Type, Option);\r
+    PRINT_PENDING_STRING (Struct, Type3, Version);\r
+    PRINT_PENDING_STRING (Struct, Type3, SerialNumber);\r
+    PRINT_PENDING_STRING (Struct, Type3, AssetTag);\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BOOTUP_STATE), gShellDebug1HiiHandle);\r
-    DisplaySystemEnclosureStatus (pStruct->Type3->BootupState, Option);\r
+    DisplaySystemEnclosureStatus (Struct->Type3->BootupState, Option);\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_POWER_SUPPLY_STATE), gShellDebug1HiiHandle);\r
-    DisplaySystemEnclosureStatus (pStruct->Type3->PowerSupplyState, Option);\r
+    DisplaySystemEnclosureStatus (Struct->Type3->PowerSupplyState, Option);\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_THERMAL_STATE), gShellDebug1HiiHandle);\r
-    DisplaySystemEnclosureStatus (pStruct->Type3->ThermalState, Option);\r
+    DisplaySystemEnclosureStatus (Struct->Type3->ThermalState, Option);\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_SECURITY_STATUS), gShellDebug1HiiHandle);\r
-    DisplaySESecurityStatus (pStruct->Type3->SecurityStatus, Option);\r
-    PrintBitField (pStruct, Type3, OemDefined, 4);\r
+    DisplaySESecurityStatus (Struct->Type3->SecurityStatus, Option);\r
+    PRINT_BIT_FIELD (Struct, Type3, OemDefined, 4);\r
     break;\r
 \r
   //\r
   // Processor Information (Type 4)\r
   //\r
   case 4:\r
-    PrintStructValue (pStruct, Type4, Socket);\r
-    DisplayProcessorType (pStruct->Type4->ProcessorType, Option);\r
+    PRINT_STRUCT_VALUE (Struct, Type4, Socket);\r
+    DisplayProcessorType (Struct->Type4->ProcessorType, Option);\r
     if ((SmbiosMajorVersion > 0x2 || (SmbiosMajorVersion == 0x2 && SmbiosMinorVersion >= 0x6)) &&\r
-        (pStruct->Type4->ProcessorFamily == 0xFE)) {\r
+        (Struct->Type4->ProcessorFamily == 0xFE)) {\r
       //\r
       // Get family from ProcessorFamily2 field\r
       //\r
-      DisplayProcessorFamily2 (pStruct->Type4->ProcessorFamily2, Option);\r
+      DisplayProcessorFamily2 (Struct->Type4->ProcessorFamily2, Option);\r
     } else {\r
-      DisplayProcessorFamily (pStruct->Type4->ProcessorFamily, Option);\r
+      DisplayProcessorFamily (Struct->Type4->ProcessorFamily, Option);\r
     }\r
-    PrintPendingString (pStruct, Type4, ProcessorManufacture);\r
-    PrintBitField (pStruct, Type4, ProcessorId, 8);\r
-    PrintPendingString (pStruct, Type4, ProcessorVersion);\r
-    DisplayProcessorVoltage (pStruct->Type4->Voltage, Option);\r
-    PrintStructValue (pStruct, Type4, ExternalClock);\r
-    PrintStructValue (pStruct, Type4, MaxSpeed);\r
-    PrintStructValue (pStruct, Type4, CurrentSpeed);\r
-    DisplayProcessorStatus (pStruct->Type4->Status, Option);\r
-    DisplayProcessorUpgrade (pStruct->Type4->ProcessorUpgrade, Option);\r
-    PrintStructValueH (pStruct, Type4, L1CacheHandle);\r
-    PrintStructValueH (pStruct, Type4, L2CacheHandle);\r
-    PrintStructValueH (pStruct, Type4, L3CacheHandle);\r
-    PrintPendingString (pStruct, Type4, SerialNumber);\r
-    PrintPendingString (pStruct, Type4, AssetTag);\r
-    PrintPendingString (pStruct, Type4, PartNumber);\r
+    PRINT_PENDING_STRING (Struct, Type4, ProcessorManufacture);\r
+    PRINT_BIT_FIELD (Struct, Type4, ProcessorId, 8);\r
+    PRINT_PENDING_STRING (Struct, Type4, ProcessorVersion);\r
+    DisplayProcessorVoltage (Struct->Type4->Voltage, Option);\r
+    PRINT_STRUCT_VALUE (Struct, Type4, ExternalClock);\r
+    PRINT_STRUCT_VALUE (Struct, Type4, MaxSpeed);\r
+    PRINT_STRUCT_VALUE (Struct, Type4, CurrentSpeed);\r
+    DisplayProcessorStatus (Struct->Type4->Status, Option);\r
+    DisplayProcessorUpgrade (Struct->Type4->ProcessorUpgrade, Option);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type4, L1CacheHandle);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type4, L2CacheHandle);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type4, L3CacheHandle);\r
+    PRINT_PENDING_STRING (Struct, Type4, SerialNumber);\r
+    PRINT_PENDING_STRING (Struct, Type4, AssetTag);\r
+    PRINT_PENDING_STRING (Struct, Type4, PartNumber);\r
     if (SmbiosMajorVersion > 0x2 || (SmbiosMajorVersion == 0x2 && SmbiosMinorVersion >= 0x5)) {\r
-      PrintStructValue (pStruct, Type4, CoreCount);\r
-      PrintStructValue (pStruct, Type4, EnabledCoreCount);\r
-      PrintStructValue (pStruct, Type4, ThreadCount);\r
-      PrintStructValueH (pStruct, Type4, ProcessorCharacteristics);\r
+      PRINT_STRUCT_VALUE (Struct, Type4, CoreCount);\r
+      PRINT_STRUCT_VALUE (Struct, Type4, EnabledCoreCount);\r
+      PRINT_STRUCT_VALUE (Struct, Type4, ThreadCount);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type4, ProcessorCharacteristics);\r
     }\r
     break;\r
 \r
@@ -369,19 +353,19 @@ Returns:
   case 5:\r
     {\r
       UINT8 SlotNum;\r
-      SlotNum = pStruct->Type5->AssociatedMemorySlotNum;\r
+      SlotNum = Struct->Type5->AssociatedMemorySlotNum;\r
 \r
-      DisplayMcErrorDetectMethod (pStruct->Type5->ErrDetectMethod, Option);\r
-      DisplayMcErrorCorrectCapability (pStruct->Type5->ErrCorrectCapability, Option);\r
+      DisplayMcErrorDetectMethod (Struct->Type5->ErrDetectMethod, Option);\r
+      DisplayMcErrorCorrectCapability (Struct->Type5->ErrCorrectCapability, Option);\r
       ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_SUPOPRT), gShellDebug1HiiHandle);\r
-      DisplayMcInterleaveSupport (pStruct->Type5->SupportInterleave, Option);\r
+      DisplayMcInterleaveSupport (Struct->Type5->SupportInterleave, Option);\r
       ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_CURRENT), gShellDebug1HiiHandle);\r
-      DisplayMcInterleaveSupport (pStruct->Type5->CurrentInterleave, Option);\r
-      DisplayMaxMemoryModuleSize (pStruct->Type5->MaxMemoryModuleSize, SlotNum, Option);\r
-      DisplayMcMemorySpeeds (pStruct->Type5->SupportSpeed, Option);\r
-      DisplayMmMemoryType (pStruct->Type5->SupportMemoryType, Option);\r
-      DisplayMemoryModuleVoltage (pStruct->Type5->MemoryModuleVoltage, Option);\r
-      PrintStructValue (pStruct, Type5, AssociatedMemorySlotNum);\r
+      DisplayMcInterleaveSupport (Struct->Type5->CurrentInterleave, Option);\r
+      DisplayMaxMemoryModuleSize (Struct->Type5->MaxMemoryModuleSize, SlotNum, Option);\r
+      DisplayMcMemorySpeeds (Struct->Type5->SupportSpeed, Option);\r
+      DisplayMmMemoryType (Struct->Type5->SupportMemoryType, Option);\r
+      DisplayMemoryModuleVoltage (Struct->Type5->MemoryModuleVoltage, Option);\r
+      PRINT_STRUCT_VALUE (Struct, Type5, AssociatedMemorySlotNum);\r
       //\r
       // According to SMBIOS Specification, offset 0x0F\r
       //\r
@@ -394,67 +378,67 @@ Returns:
   // Memory Module Information (Type 6)\r
   //\r
   case 6:\r
-    PrintPendingString (pStruct, Type6, SocketDesignation);\r
-    DisplayMmBankConnections (pStruct->Type6->BankConnections, Option);\r
-    PrintStructValue (pStruct, Type6, CurrentSpeed);\r
-    DisplayMmMemoryType (pStruct->Type6->CurrentMemoryType, Option);\r
+    PRINT_PENDING_STRING (Struct, Type6, SocketDesignation);\r
+    DisplayMmBankConnections (Struct->Type6->BankConnections, Option);\r
+    PRINT_STRUCT_VALUE (Struct, Type6, CurrentSpeed);\r
+    DisplayMmMemoryType (Struct->Type6->CurrentMemoryType, Option);\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_INSTALLED), gShellDebug1HiiHandle);\r
-    DisplayMmMemorySize (pStruct->Type6->InstalledSize, Option);\r
+    DisplayMmMemorySize (Struct->Type6->InstalledSize, Option);\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ENABLED), gShellDebug1HiiHandle);\r
-    DisplayMmMemorySize (pStruct->Type6->EnabledSize, Option);\r
-    DisplayMmErrorStatus (pStruct->Type6->ErrorStatus, Option);\r
+    DisplayMmMemorySize (Struct->Type6->EnabledSize, Option);\r
+    DisplayMmErrorStatus (Struct->Type6->ErrorStatus, Option);\r
     break;\r
 \r
   //\r
   // Cache Information (Type 7)\r
   //\r
   case 7:\r
-    PrintPendingString (pStruct, Type7, SocketDesignation);\r
-    PrintStructValueH (pStruct, Type7, CacheConfiguration);\r
-    PrintStructValueH (pStruct, Type7, MaximumCacheSize);\r
-    PrintStructValueH (pStruct, Type7, InstalledSize);\r
-    PrintStructValueH (pStruct, Type7, SupportedSRAMType);\r
-    PrintStructValueH (pStruct, Type7, CurrentSRAMType);\r
-    DisplayCacheSRAMType (pStruct->Type7->CurrentSRAMType, Option);\r
-    PrintStructValueH (pStruct, Type7, CacheSpeed);\r
-    DisplayCacheErrCorrectingType (pStruct->Type7->ErrorCorrectionType, Option);\r
-    DisplayCacheSystemCacheType (pStruct->Type7->SystemCacheType, Option);\r
-    DisplayCacheAssociativity (pStruct->Type7->Associativity, Option);\r
+    PRINT_PENDING_STRING (Struct, Type7, SocketDesignation);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type7, CacheConfiguration);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type7, MaximumCacheSize);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type7, InstalledSize);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type7, SupportedSRAMType);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type7, CurrentSRAMType);\r
+    DisplayCacheSRAMType (Struct->Type7->CurrentSRAMType, Option);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type7, CacheSpeed);\r
+    DisplayCacheErrCorrectingType (Struct->Type7->ErrorCorrectionType, Option);\r
+    DisplayCacheSystemCacheType (Struct->Type7->SystemCacheType, Option);\r
+    DisplayCacheAssociativity (Struct->Type7->Associativity, Option);\r
     break;\r
 \r
   //\r
   // Port Connector Information  (Type 8)\r
   //\r
   case 8:\r
-    PrintPendingString (pStruct, Type8, InternalReferenceDesignator);\r
+    PRINT_PENDING_STRING (Struct, Type8, InternalReferenceDesignator);\r
     Print (L"Internal ");\r
-    DisplayPortConnectorType (pStruct->Type8->InternalConnectorType, Option);\r
-    PrintPendingString (pStruct, Type8, ExternalReferenceDesignator);\r
+    DisplayPortConnectorType (Struct->Type8->InternalConnectorType, Option);\r
+    PRINT_PENDING_STRING (Struct, Type8, ExternalReferenceDesignator);\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_EXTERNAL), gShellDebug1HiiHandle);\r
-    DisplayPortConnectorType (pStruct->Type8->ExternalConnectorType, Option);\r
-    DisplayPortType (pStruct->Type8->PortType, Option);\r
+    DisplayPortConnectorType (Struct->Type8->ExternalConnectorType, Option);\r
+    DisplayPortType (Struct->Type8->PortType, Option);\r
     break;\r
 \r
   //\r
   // System Slots (Type 9)\r
   //\r
   case 9:\r
-    PrintPendingString (pStruct, Type9, SlotDesignation);\r
-    DisplaySystemSlotType (pStruct->Type9->SlotType, Option);\r
-    DisplaySystemSlotDataBusWidth (pStruct->Type9->SlotDataBusWidth, Option);\r
-    DisplaySystemSlotCurrentUsage (pStruct->Type9->CurrentUsage, Option);\r
-    DisplaySystemSlotLength (pStruct->Type9->SlotLength, Option);\r
+    PRINT_PENDING_STRING (Struct, Type9, SlotDesignation);\r
+    DisplaySystemSlotType (Struct->Type9->SlotType, Option);\r
+    DisplaySystemSlotDataBusWidth (Struct->Type9->SlotDataBusWidth, Option);\r
+    DisplaySystemSlotCurrentUsage (Struct->Type9->CurrentUsage, Option);\r
+    DisplaySystemSlotLength (Struct->Type9->SlotLength, Option);\r
     DisplaySystemSlotId (\r
-      pStruct->Type9->SlotID,\r
-      pStruct->Type9->SlotType,\r
+      Struct->Type9->SlotID,\r
+      Struct->Type9->SlotType,\r
       Option\r
      );\r
-    DisplaySlotCharacteristics1 (pStruct->Type9->SlotCharacteristics1, Option);\r
-    DisplaySlotCharacteristics2 (pStruct->Type9->SlotCharacteristics2, Option);\r
+    DisplaySlotCharacteristics1 (Struct->Type9->SlotCharacteristics1, Option);\r
+    DisplaySlotCharacteristics2 (Struct->Type9->SlotCharacteristics2, Option);\r
     if (SmbiosMajorVersion > 0x2 || (SmbiosMajorVersion == 0x2 && SmbiosMinorVersion >= 0x6)) {\r
-      PrintStructValueH (pStruct, Type9, SegmentGroupNum);\r
-      PrintStructValueH (pStruct, Type9, BusNum);\r
-      PrintStructValueH (pStruct, Type9, DevFuncNum);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type9, SegmentGroupNum);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type9, BusNum);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type9, DevFuncNum);\r
     }\r
     break;\r
 \r
@@ -464,40 +448,40 @@ Returns:
   case 10:\r
     {\r
       UINTN NumOfDevice;\r
-      NumOfDevice = (pStruct->Type10->Hdr.Length - sizeof (SMBIOS_HEADER)) / (2 * sizeof (UINT8));\r
+      NumOfDevice = (Struct->Type10->Hdr.Length - sizeof (SMBIOS_HEADER)) / (2 * sizeof (UINT8));\r
       for (Index = 0; Index < NumOfDevice; Index++) {\r
-        DisplayOnboardDeviceTypes (pStruct->Type10->Device[Index].DeviceType, Option);\r
+        DisplayOnboardDeviceTypes (Struct->Type10->Device[Index].DeviceType, Option);\r
         ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_DESC_STRING), gShellDebug1HiiHandle);\r
-        ShellPrintEx(-1,-1,L"%a",LibGetSmbiosString (pStruct, pStruct->Type10->Device[Index].DescriptionString));\r
+        ShellPrintEx(-1,-1,L"%a",LibGetSmbiosString (Struct, Struct->Type10->Device[Index].DescriptionString));\r
       }\r
     }\r
     break;\r
 \r
   case 11:\r
-    PrintStructValue (pStruct, Type11, StringCount);\r
-    for (Index = 1; Index <= pStruct->Type11->StringCount; Index++) {\r
-      ShellPrintEx(-1,-1,L"%a\n", LibGetSmbiosString (pStruct, Index));\r
+    PRINT_STRUCT_VALUE (Struct, Type11, StringCount);\r
+    for (Index = 1; Index <= Struct->Type11->StringCount; Index++) {\r
+      ShellPrintEx(-1,-1,L"%a\n", LibGetSmbiosString (Struct, Index));\r
     }\r
     break;\r
 \r
   case 12:\r
-    PrintStructValue (pStruct, Type12, StringCount);\r
-    for (Index = 1; Index <= pStruct->Type12->StringCount; Index++) {\r
-      ShellPrintEx(-1,-1,L"%a\n", LibGetSmbiosString (pStruct, Index));\r
+    PRINT_STRUCT_VALUE (Struct, Type12, StringCount);\r
+    for (Index = 1; Index <= Struct->Type12->StringCount; Index++) {\r
+      ShellPrintEx(-1,-1,L"%a\n", LibGetSmbiosString (Struct, Index));\r
     }\r
     break;\r
 \r
   case 13:\r
-    PrintStructValue (pStruct, Type13, InstallableLanguages);\r
-    PrintStructValue (pStruct, Type13, Flags);\r
-    PrintBitField (pStruct, Type13, reserved, 15);\r
-    PrintPendingString (pStruct, Type13, CurrentLanguages);\r
+    PRINT_STRUCT_VALUE (Struct, Type13, InstallableLanguages);\r
+    PRINT_STRUCT_VALUE (Struct, Type13, Flags);\r
+    PRINT_BIT_FIELD (Struct, Type13, Reserved, 15);\r
+    PRINT_PENDING_STRING (Struct, Type13, CurrentLanguages);\r
     break;\r
 \r
   case 14:\r
-    PrintPendingString (pStruct, Type14, GroupName);\r
-    PrintStructValue (pStruct, Type14, ItemType);\r
-    PrintStructValue (pStruct, Type14, ItemHandle);\r
+    PRINT_PENDING_STRING (Struct, Type14, GroupName);\r
+    PRINT_STRUCT_VALUE (Struct, Type14, ItemType);\r
+    PRINT_STRUCT_VALUE (Struct, Type14, ItemHandle);\r
     break;\r
 \r
   //\r
@@ -509,20 +493,20 @@ Returns:
       UINT8         Count;\r
       UINT8         *AccessMethodAddress;\r
 \r
-      PrintStructValueH (pStruct, Type15, LogAreaLength);\r
-      PrintStructValueH (pStruct, Type15, LogHeaderStartOffset);\r
-      PrintStructValueH (pStruct, Type15, LogDataStartOffset);\r
-      DisplaySELAccessMethod (pStruct->Type15->AccessMethod, Option);\r
-      PrintStructValueH (pStruct, Type15, AccessMethodAddress);\r
-      DisplaySELLogStatus (pStruct->Type15->LogStatus, Option);\r
-      PrintStructValueH (pStruct, Type15, LogChangeToken);\r
-      DisplaySysEventLogHeaderFormat (pStruct->Type15->LogHeaderFormat, Option);\r
-      PrintStructValueH (pStruct, Type15, NumberOfSupportedLogTypeDescriptors);\r
-      PrintStructValueH (pStruct, Type15, LengthOfLogTypeDescriptor);\r
-\r
-      Count = pStruct->Type15->NumberOfSupportedLogTypeDescriptors;\r
+      PRINT_STRUCT_VALUE_H (Struct, Type15, LogAreaLength);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type15, LogHeaderStartOffset);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type15, LogDataStartOffset);\r
+      DisplaySELAccessMethod (Struct->Type15->AccessMethod, Option);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type15, AccessMethodAddress);\r
+      DisplaySELLogStatus (Struct->Type15->LogStatus, Option);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type15, LogChangeToken);\r
+      DisplaySysEventLogHeaderFormat (Struct->Type15->LogHeaderFormat, Option);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type15, NumberOfSupportedLogTypeDescriptors);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type15, LengthOfLogTypeDescriptor);\r
+\r
+      Count = Struct->Type15->NumberOfSupportedLogTypeDescriptors;\r
       if (Count > 0) {\r
-        Ptr = pStruct->Type15->EventLogTypeDescriptors;\r
+        Ptr = Struct->Type15->EventLogTypeDescriptors;\r
 \r
         //\r
         // Display all Event Log type descriptors supported by system\r
@@ -534,9 +518,9 @@ Returns:
         }\r
 \r
         if (Option >= SHOW_DETAIL) {\r
-          switch (pStruct->Type15->AccessMethod) {\r
+          switch (Struct->Type15->AccessMethod) {\r
           case 03:\r
-            AccessMethodAddress = (UINT8 *) (UINTN) (pStruct->Type15->AccessMethodAddress);\r
+            AccessMethodAddress = (UINT8 *) (UINTN) (Struct->Type15->AccessMethodAddress);\r
             break;\r
 \r
           case 00:\r
@@ -547,7 +531,7 @@ Returns:
             ShellPrintHiiEx(-1,-1,NULL,\r
               STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ACCESS_METHOD_NOT_SUPOPRTED),\r
               gShellDebug1HiiHandle,\r
-              pStruct->Type15->AccessMethod\r
+              Struct->Type15->AccessMethod\r
              );\r
             return EFI_UNSUPPORTED;\r
           }\r
@@ -558,8 +542,8 @@ Returns:
           // of the event-log's header, from the Access Method Address\r
           //\r
           DisplaySysEventLogHeader (\r
-            pStruct->Type15->LogHeaderFormat,\r
-            AccessMethodAddress + pStruct->Type15->LogHeaderStartOffset\r
+            Struct->Type15->LogHeaderFormat,\r
+            AccessMethodAddress + Struct->Type15->LogHeaderStartOffset\r
            );\r
 \r
           //\r
@@ -569,11 +553,11 @@ Returns:
           // of the event-log's first data byte, from the Access Method Address(0x14)\r
           //\r
           DisplaySysEventLogData (\r
-            AccessMethodAddress + pStruct->Type15->LogDataStartOffset,\r
+            AccessMethodAddress + Struct->Type15->LogDataStartOffset,\r
             (UINT16)\r
             (\r
-            pStruct->Type15->LogAreaLength -\r
-            (pStruct->Type15->LogDataStartOffset - pStruct->Type15->LogDataStartOffset)\r
+            Struct->Type15->LogAreaLength -\r
+            (Struct->Type15->LogDataStartOffset - Struct->Type15->LogDataStartOffset)\r
            )\r
            );\r
         }\r
@@ -586,39 +570,39 @@ Returns:
   // Physical Memory Array (Type 16)\r
   //\r
   case 16:\r
-    DisplayPMALocation (pStruct->Type16->Location, Option);\r
-    DisplayPMAUse (pStruct->Type16->Use, Option);\r
+    DisplayPMALocation (Struct->Type16->Location, Option);\r
+    DisplayPMAUse (Struct->Type16->Use, Option);\r
     DisplayPMAErrorCorrectionTypes (\r
-      pStruct->Type16->MemoryErrorCorrection,\r
+      Struct->Type16->MemoryErrorCorrection,\r
       Option\r
      );\r
-    PrintStructValueH (pStruct, Type16, MaximumCapacity);\r
-    PrintStructValueH (pStruct, Type16, MemoryErrorInformationHandle);\r
-    PrintStructValueH (pStruct, Type16, NumberOfMemoryDevices);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type16, MaximumCapacity);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type16, MemoryErrorInformationHandle);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type16, NumberOfMemoryDevices);\r
     break;\r
 \r
   //\r
   // Memory Device (Type 17)\r
   //\r
   case 17:\r
-    PrintStructValueH (pStruct, Type17, MemoryArrayHandle);\r
-    PrintStructValueH (pStruct, Type17, MemoryErrorInformationHandle);\r
-    PrintStructValue (pStruct, Type17, TotalWidth);\r
-    PrintStructValue (pStruct, Type17, DataWidth);\r
-    PrintStructValue (pStruct, Type17, Size);\r
-    DisplayMemoryDeviceFormFactor (pStruct->Type17->FormFactor, Option);\r
-    PrintStructValueH (pStruct, Type17, DeviceSet);\r
-    PrintPendingString (pStruct, Type17, DeviceLocator);\r
-    PrintPendingString (pStruct, Type17, BankLocator);\r
-    DisplayMemoryDeviceType (pStruct->Type17->MemoryType, Option);\r
-    DisplayMemoryDeviceTypeDetail (pStruct->Type17->TypeDetail, Option);\r
-    PrintStructValueH (pStruct, Type17, Speed);\r
-    PrintPendingString (pStruct, Type17, Manufacturer);\r
-    PrintPendingString (pStruct, Type17, SerialNumber);\r
-    PrintPendingString (pStruct, Type17, AssetTag);\r
-    PrintPendingString (pStruct, Type17, PartNumber);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type17, MemoryArrayHandle);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type17, MemoryErrorInformationHandle);\r
+    PRINT_STRUCT_VALUE (Struct, Type17, TotalWidth);\r
+    PRINT_STRUCT_VALUE (Struct, Type17, DataWidth);\r
+    PRINT_STRUCT_VALUE (Struct, Type17, Size);\r
+    DisplayMemoryDeviceFormFactor (Struct->Type17->FormFactor, Option);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type17, DeviceSet);\r
+    PRINT_PENDING_STRING (Struct, Type17, DeviceLocator);\r
+    PRINT_PENDING_STRING (Struct, Type17, BankLocator);\r
+    DisplayMemoryDeviceType (Struct->Type17->MemoryType, Option);\r
+    DisplayMemoryDeviceTypeDetail (Struct->Type17->TypeDetail, Option);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type17, Speed);\r
+    PRINT_PENDING_STRING (Struct, Type17, Manufacturer);\r
+    PRINT_PENDING_STRING (Struct, Type17, SerialNumber);\r
+    PRINT_PENDING_STRING (Struct, Type17, AssetTag);\r
+    PRINT_PENDING_STRING (Struct, Type17, PartNumber);\r
     if (SmbiosMajorVersion > 0x2 || (SmbiosMajorVersion == 0x2 && SmbiosMinorVersion >= 0x6)) {\r
-      PrintStructValueH (pStruct, Type17, Attributes);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type17, Attributes);\r
     }\r
     break;\r
 \r
@@ -626,154 +610,154 @@ Returns:
   // 32-bit Memory Error Information (Type 18)\r
   //\r
   case 18:\r
-    DisplayMemoryErrorType (pStruct->Type18->ErrorType, Option);\r
+    DisplayMemoryErrorType (Struct->Type18->ErrorType, Option);\r
     DisplayMemoryErrorGranularity (\r
-      pStruct->Type18->ErrorGranularity,\r
+      Struct->Type18->ErrorGranularity,\r
       Option\r
      );\r
-    DisplayMemoryErrorOperation (pStruct->Type18->ErrorOperation, Option);\r
-    PrintStructValueH (pStruct, Type18, VendorSyndrome);\r
-    PrintStructValueH (pStruct, Type18, MemoryArrayErrorAddress);\r
-    PrintStructValueH (pStruct, Type18, DeviceErrorAddress);\r
-    PrintStructValueH (pStruct, Type18, ErrorResolution);\r
+    DisplayMemoryErrorOperation (Struct->Type18->ErrorOperation, Option);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type18, VendorSyndrome);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type18, MemoryArrayErrorAddress);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type18, DeviceErrorAddress);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type18, ErrorResolution);\r
     break;\r
 \r
   //\r
   // Memory Array Mapped Address (Type 19)\r
   //\r
   case 19:\r
-    PrintStructValueH (pStruct, Type19, StartingAddress);\r
-    PrintStructValueH (pStruct, Type19, EndingAddress);\r
-    PrintStructValueH (pStruct, Type19, MemoryArrayHandle);\r
-    PrintStructValueH (pStruct, Type19, PartitionWidth);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type19, StartingAddress);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type19, EndingAddress);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type19, MemoryArrayHandle);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type19, PartitionWidth);\r
     break;\r
 \r
   //\r
   // Memory Device Mapped Address  (Type 20)\r
   //\r
   case 20:\r
-    PrintStructValueH (pStruct, Type20, StartingAddress);\r
-    PrintStructValueH (pStruct, Type20, EndingAddress);\r
-    PrintStructValueH (pStruct, Type20, MemoryDeviceHandle);\r
-    PrintStructValueH (pStruct, Type20, MemoryArrayMappedAddressHandle);\r
-    PrintStructValueH (pStruct, Type20, PartitionRowPosition);\r
-    PrintStructValueH (pStruct, Type20, InterleavePosition);\r
-    PrintStructValueH (pStruct, Type20, InterleavedDataDepth);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type20, StartingAddress);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type20, EndingAddress);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type20, MemoryDeviceHandle);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type20, MemoryArrayMappedAddressHandle);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type20, PartitionRowPosition);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type20, InterleavePosition);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type20, InterleavedDataDepth);\r
     break;\r
 \r
   //\r
   // Built-in Pointing Device  (Type 21)\r
   //\r
   case 21:\r
-    DisplayPointingDeviceType (pStruct->Type21->Type, Option);\r
-    DisplayPointingDeviceInterface (pStruct->Type21->Interface, Option);\r
-    PrintStructValue (pStruct, Type21, NumberOfButtons);\r
+    DisplayPointingDeviceType (Struct->Type21->Type, Option);\r
+    DisplayPointingDeviceInterface (Struct->Type21->Interface, Option);\r
+    PRINT_STRUCT_VALUE (Struct, Type21, NumberOfButtons);\r
     break;\r
 \r
   //\r
   // Portable Battery  (Type 22)\r
   //\r
   case 22:\r
-    PrintPendingString (pStruct, Type22, Location);\r
-    PrintPendingString (pStruct, Type22, Manufacturer);\r
-    PrintPendingString (pStruct, Type22, ManufactureDate);\r
-    PrintPendingString (pStruct, Type22, SerialNumber);\r
-    PrintPendingString (pStruct, Type22, DeviceName);\r
+    PRINT_PENDING_STRING (Struct, Type22, Location);\r
+    PRINT_PENDING_STRING (Struct, Type22, Manufacturer);\r
+    PRINT_PENDING_STRING (Struct, Type22, ManufactureDate);\r
+    PRINT_PENDING_STRING (Struct, Type22, SerialNumber);\r
+    PRINT_PENDING_STRING (Struct, Type22, DeviceName);\r
     DisplayPBDeviceChemistry (\r
-      pStruct->Type22->DeviceChemistry,\r
+      Struct->Type22->DeviceChemistry,\r
       Option\r
      );\r
-    PrintStructValueH (pStruct, Type22, DeviceCapacity);\r
-    PrintStructValueH (pStruct, Type22, DesignVoltage);\r
-    PrintPendingString (pStruct, Type22, SBDSVersionNumber);\r
-    PrintStructValueH (pStruct, Type22, MaximumErrorInBatteryData);\r
-    PrintStructValueH (pStruct, Type22, SBDSSerialNumber);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type22, DeviceCapacity);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type22, DesignVoltage);\r
+    PRINT_PENDING_STRING (Struct, Type22, SBDSVersionNumber);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type22, MaximumErrorInBatteryData);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type22, SBDSSerialNumber);\r
     DisplaySBDSManufactureDate (\r
-      pStruct->Type22->SBDSManufactureDate,\r
+      Struct->Type22->SBDSManufactureDate,\r
       Option\r
      );\r
-    PrintPendingString (pStruct, Type22, SBDSDeviceChemistry);\r
-    PrintStructValueH (pStruct, Type22, DesignCapacityMultiplier);\r
-    PrintStructValueH (pStruct, Type22, OEMSpecific);\r
+    PRINT_PENDING_STRING (Struct, Type22, SBDSDeviceChemistry);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type22, DesignCapacityMultiplier);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type22, OEMSpecific);\r
     break;\r
 \r
   case 23:\r
     DisplaySystemResetCapabilities (\r
-      pStruct->Type23->Capabilities,\r
+      Struct->Type23->Capabilities,\r
       Option\r
      );\r
-    PrintStructValueH (pStruct, Type23, ResetCount);\r
-    PrintStructValueH (pStruct, Type23, ResetLimit);\r
-    PrintStructValueH (pStruct, Type23, TimerInterval);\r
-    PrintStructValueH (pStruct, Type23, Timeout);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type23, ResetCount);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type23, ResetLimit);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type23, TimerInterval);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type23, Timeout);\r
     break;\r
 \r
   case 24:\r
     DisplayHardwareSecuritySettings (\r
-      pStruct->Type24->HardwareSecuritySettings,\r
+      Struct->Type24->HardwareSecuritySettings,\r
       Option\r
      );\r
     break;\r
 \r
   case 25:\r
-    PrintStructValueH (pStruct, Type25, NextScheduledPowerOnMonth);\r
-    PrintStructValueH (pStruct, Type25, NextScheduledPowerOnDayOfMonth);\r
-    PrintStructValueH (pStruct, Type25, NextScheduledPowerOnHour);\r
-    PrintStructValueH (pStruct, Type25, NextScheduledPowerOnMinute);\r
-    PrintStructValueH (pStruct, Type25, NextScheduledPowerOnSecond);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type25, NextScheduledPowerOnMonth);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type25, NextScheduledPowerOnDayOfMonth);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type25, NextScheduledPowerOnHour);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type25, NextScheduledPowerOnMinute);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type25, NextScheduledPowerOnSecond);\r
     break;\r
 \r
   case 26:\r
-    PrintPendingString (pStruct, Type26, Description);\r
-    DisplayVPLocation (pStruct->Type26->LocationAndStatus, Option);\r
-    DisplayVPStatus (pStruct->Type26->LocationAndStatus, Option);\r
-    PrintStructValueH (pStruct, Type26, MaximumValue);\r
-    PrintStructValueH (pStruct, Type26, MinimumValue);\r
-    PrintStructValueH (pStruct, Type26, Resolution);\r
-    PrintStructValueH (pStruct, Type26, Tolerance);\r
-    PrintStructValueH (pStruct, Type26, Accuracy);\r
-    PrintStructValueH (pStruct, Type26, OEMDefined);\r
-    PrintStructValueH (pStruct, Type26, NominalValue);\r
+    PRINT_PENDING_STRING (Struct, Type26, Description);\r
+    DisplayVPLocation (Struct->Type26->LocationAndStatus, Option);\r
+    DisplayVPStatus (Struct->Type26->LocationAndStatus, Option);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type26, MaximumValue);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type26, MinimumValue);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type26, Resolution);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type26, Tolerance);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type26, Accuracy);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type26, OEMDefined);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type26, NominalValue);\r
     break;\r
 \r
   case 27:\r
-    PrintStructValueH (pStruct, Type27, TemperatureProbeHandle);\r
-    DisplayCoolingDeviceStatus (pStruct->Type27->DeviceTypeAndStatus, Option);\r
-    DisplayCoolingDeviceType (pStruct->Type27->DeviceTypeAndStatus, Option);\r
-    PrintStructValueH (pStruct, Type27, CoolingUnitGroup);\r
-    PrintStructValueH (pStruct, Type27, OEMDefined);\r
-    PrintStructValueH (pStruct, Type27, NominalSpeed);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type27, TemperatureProbeHandle);\r
+    DisplayCoolingDeviceStatus (Struct->Type27->DeviceTypeAndStatus, Option);\r
+    DisplayCoolingDeviceType (Struct->Type27->DeviceTypeAndStatus, Option);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type27, CoolingUnitGroup);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type27, OEMDefined);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type27, NominalSpeed);\r
     break;\r
 \r
   case 28:\r
-    PrintPendingString (pStruct, Type28, Description);\r
-    DisplayTemperatureProbeStatus (pStruct->Type28->LocationAndStatus, Option);\r
-    DisplayTemperatureProbeLoc (pStruct->Type28->LocationAndStatus, Option);\r
-    PrintStructValueH (pStruct, Type28, MaximumValue);\r
-    PrintStructValueH (pStruct, Type28, MinimumValue);\r
-    PrintStructValueH (pStruct, Type28, Resolution);\r
-    PrintStructValueH (pStruct, Type28, Tolerance);\r
-    PrintStructValueH (pStruct, Type28, Accuracy);\r
-    PrintStructValueH (pStruct, Type28, OEMDefined);\r
-    PrintStructValueH (pStruct, Type28, NominalValue);\r
+    PRINT_PENDING_STRING (Struct, Type28, Description);\r
+    DisplayTemperatureProbeStatus (Struct->Type28->LocationAndStatus, Option);\r
+    DisplayTemperatureProbeLoc (Struct->Type28->LocationAndStatus, Option);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type28, MaximumValue);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type28, MinimumValue);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type28, Resolution);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type28, Tolerance);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type28, Accuracy);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type28, OEMDefined);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type28, NominalValue);\r
     break;\r
 \r
   case 29:\r
-    PrintPendingString (pStruct, Type29, Description);\r
-    DisplayECPStatus (pStruct->Type29->LocationAndStatus, Option);\r
-    DisplayECPLoc (pStruct->Type29->LocationAndStatus, Option);\r
-    PrintStructValueH (pStruct, Type29, MaximumValue);\r
-    PrintStructValueH (pStruct, Type29, MinimumValue);\r
-    PrintStructValueH (pStruct, Type29, Resolution);\r
-    PrintStructValueH (pStruct, Type29, Tolerance);\r
-    PrintStructValueH (pStruct, Type29, Accuracy);\r
-    PrintStructValueH (pStruct, Type29, OEMDefined);\r
-    PrintStructValueH (pStruct, Type29, NominalValue);\r
+    PRINT_PENDING_STRING (Struct, Type29, Description);\r
+    DisplayECPStatus (Struct->Type29->LocationAndStatus, Option);\r
+    DisplayECPLoc (Struct->Type29->LocationAndStatus, Option);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type29, MaximumValue);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type29, MinimumValue);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type29, Resolution);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type29, Tolerance);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type29, Accuracy);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type29, OEMDefined);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type29, NominalValue);\r
     break;\r
 \r
   case 30:\r
-    PrintPendingString (pStruct, Type30, ManufacturerName);\r
-    DisplayOBRAConnections (pStruct->Type30->Connections, Option);\r
+    PRINT_PENDING_STRING (Struct, Type30, ManufacturerName);\r
+    DisplayOBRAConnections (Struct->Type30->Connections, Option);\r
     break;\r
 \r
   case 31:\r
@@ -781,47 +765,47 @@ Returns:
     break;\r
 \r
   case 32:\r
-    PrintBitField (pStruct, Type32, Reserved, 6);\r
-    DisplaySystemBootStatus (pStruct->Type32->BootStatus[0], Option);\r
+    PRINT_BIT_FIELD (Struct, Type32, Reserved, 6);\r
+    DisplaySystemBootStatus (Struct->Type32->BootStatus[0], Option);\r
     break;\r
 \r
   case 33:\r
-    DisplayMemoryErrorType (pStruct->Type33->ErrorType, Option);\r
+    DisplayMemoryErrorType (Struct->Type33->ErrorType, Option);\r
     DisplayMemoryErrorGranularity (\r
-      pStruct->Type33->ErrorGranularity,\r
+      Struct->Type33->ErrorGranularity,\r
       Option\r
      );\r
-    DisplayMemoryErrorOperation (pStruct->Type33->ErrorOperation, Option);\r
-    PrintStructValueH (pStruct, Type33, VendorSyndrome);\r
-    PrintStructValueH (pStruct, Type33, MemoryArrayErrorAddress);\r
-    PrintStructValueH (pStruct, Type33, DeviceErrorAddress);\r
-    PrintStructValueH (pStruct, Type33, ErrorResolution);\r
+    DisplayMemoryErrorOperation (Struct->Type33->ErrorOperation, Option);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type33, VendorSyndrome);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type33, MemoryArrayErrorAddress);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type33, DeviceErrorAddress);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type33, ErrorResolution);\r
     break;\r
 \r
   //\r
   // Management Device  (Type 34)\r
   //\r
   case 34:\r
-    PrintPendingString (pStruct, Type34, Description);\r
-    DisplayMDType (pStruct->Type34->Type, Option);\r
-    PrintStructValueH (pStruct, Type34, Address);\r
-    PrintStructValueH (pStruct, Type34, AddressType);\r
+    PRINT_PENDING_STRING (Struct, Type34, Description);\r
+    DisplayMDType (Struct->Type34->Type, Option);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type34, Address);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type34, AddressType);\r
     break;\r
 \r
   case 35:\r
-    PrintPendingString (pStruct, Type35, Description);\r
-    PrintStructValueH (pStruct, Type35, ManagementDeviceHandle);\r
-    PrintStructValueH (pStruct, Type35, ComponentHandle);\r
-    PrintStructValueH (pStruct, Type35, ThresholdHandle);\r
+    PRINT_PENDING_STRING (Struct, Type35, Description);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type35, ManagementDeviceHandle);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type35, ComponentHandle);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type35, ThresholdHandle);\r
     break;\r
 \r
   case 36:\r
-    PrintStructValueH (pStruct, Type36, LowerThresholdNonCritical);\r
-    PrintStructValueH (pStruct, Type36, UpperThresholdNonCritical);\r
-    PrintStructValueH (pStruct, Type36, LowerThresholdCritical);\r
-    PrintStructValueH (pStruct, Type36, UpperThresholdCritical);\r
-    PrintStructValueH (pStruct, Type36, LowerThresholdNonRecoverable);\r
-    PrintStructValueH (pStruct, Type36, UpperThresholdNonRecoverable);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type36, LowerThresholdNonCritical);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type36, UpperThresholdNonCritical);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type36, LowerThresholdCritical);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type36, UpperThresholdCritical);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type36, LowerThresholdNonRecoverable);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type36, UpperThresholdNonRecoverable);\r
     break;\r
 \r
   //\r
@@ -831,12 +815,12 @@ Returns:
     {\r
       UINT8         Count;\r
       MEMORYDEVICE  *Ptr;\r
-      DisplayMemoryChannelType (pStruct->Type37->ChannelType, Option);\r
-      PrintStructValueH (pStruct, Type37, MaximumChannelLoad);\r
-      PrintStructValueH (pStruct, Type37, MemoryDeviceCount);\r
+      DisplayMemoryChannelType (Struct->Type37->ChannelType, Option);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type37, MaximumChannelLoad);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type37, MemoryDeviceCount);\r
 \r
-      Count = pStruct->Type37->MemoryDeviceCount;\r
-      Ptr   = pStruct->Type37->MemoryDevice;\r
+      Count = Struct->Type37->MemoryDeviceCount;\r
+      Ptr   = Struct->Type37->MemoryDevice;\r
       for (Index = 0; Index < Count; Index++) {\r
         ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_MEM_DEVICE), gShellDebug1HiiHandle, Index + 1);\r
         ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_DEV_LOAD), gShellDebug1HiiHandle, Ptr->DeviceLoad);\r
@@ -849,33 +833,33 @@ Returns:
   // IPMI Device Information  (Type 38)\r
   //\r
   case 38:\r
-    DisplayIPMIDIBMCInterfaceType (pStruct->Type38->InterfaceType, Option);\r
-    PrintStructValueH (pStruct, Type38, IPMISpecificationRevision);\r
-    PrintStructValueH (pStruct, Type38, I2CSlaveAddress);\r
-    PrintStructValueH (pStruct, Type38, NVStorageDeviceAddress);\r
-    PrintStructValueH (pStruct, Type38, BaseAddress);\r
+    DisplayIPMIDIBMCInterfaceType (Struct->Type38->InterfaceType, Option);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type38, IPMISpecificationRevision);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type38, I2CSlaveAddress);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type38, NVStorageDeviceAddress);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type38, BaseAddress);\r
     break;\r
 \r
   //\r
   // System Power Supply (Type 39)\r
   //\r
   case 39:\r
-    PrintStructValueH (pStruct, Type39, PowerUnitGroup);\r
-    PrintPendingString (pStruct, Type39, Location);\r
-    PrintPendingString (pStruct, Type39, DeviceName);\r
-    PrintPendingString (pStruct, Type39, Manufacturer);\r
-    PrintPendingString (pStruct, Type39, SerialNumber);\r
-    PrintPendingString (pStruct, Type39, AssetTagNumber);\r
-    PrintPendingString (pStruct, Type39, ModelPartNumber);\r
-    PrintPendingString (pStruct, Type39, RevisionLevel);\r
-    PrintStructValueH (pStruct, Type39, MaxPowerCapacity);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type39, PowerUnitGroup);\r
+    PRINT_PENDING_STRING (Struct, Type39, Location);\r
+    PRINT_PENDING_STRING (Struct, Type39, DeviceName);\r
+    PRINT_PENDING_STRING (Struct, Type39, Manufacturer);\r
+    PRINT_PENDING_STRING (Struct, Type39, SerialNumber);\r
+    PRINT_PENDING_STRING (Struct, Type39, AssetTagNumber);\r
+    PRINT_PENDING_STRING (Struct, Type39, ModelPartNumber);\r
+    PRINT_PENDING_STRING (Struct, Type39, RevisionLevel);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type39, MaxPowerCapacity);\r
     DisplaySPSCharacteristics (\r
-      pStruct->Type39->PowerSupplyCharacteristics,\r
+      Struct->Type39->PowerSupplyCharacteristics,\r
       Option\r
      );\r
-    PrintStructValueH (pStruct, Type39, InputVoltageProbeHandle);\r
-    PrintStructValueH (pStruct, Type39, CoolingDeviceHandle);\r
-    PrintStructValueH (pStruct, Type39, InputCurrentProbeHandle);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type39, InputVoltageProbeHandle);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type39, CoolingDeviceHandle);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type39, InputCurrentProbeHandle);\r
     break;\r
 \r
   //\r
@@ -888,18 +872,18 @@ Returns:
       ADDITIONAL_INFORMATION_ENTRY   *Entries;\r
 \r
       EntryLength     = 0;\r
-      Entries         = pStruct->Type40->AdditionalInfoEntries;\r
-      NumberOfEntries = pStruct->Type40->NumberOfAdditionalInformationEntries;\r
+      Entries         = Struct->Type40->AdditionalInfoEntries;\r
+      NumberOfEntries = Struct->Type40->NumberOfAdditionalInformationEntries;\r
 \r
-      PrintStructValueH (pStruct, Type40, NumberOfAdditionalInformationEntries);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type40, NumberOfAdditionalInformationEntries);\r
 \r
       for (Index = 0; Index < NumberOfEntries; Index++) {\r
         EntryLength = Entries->EntryLength;\r
         ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_SMBIOSVIEW_ENTRYLEN), gShellDebug1HiiHandle, EntryLength);\r
         ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_SMBIOSVIEW_REFERENCEDHANDLE), gShellDebug1HiiHandle, Entries->ReferencedHandle);\r
         ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_SMBIOSVIEW_REFERENCEDOFFSET), gShellDebug1HiiHandle, Entries->ReferencedOffset);\r
-        PrintSmbiosString (pStruct, Entries->EntryString, String);\r
-        PrintSmbiosBitField (pStruct, Entries->Value, Value, EntryLength - 5);\r
+        PRINT_SMBIOS_STRING (Struct, Entries->EntryString, String);\r
+        PRINT_SMBIOS_BIT_FIELD (Struct, Entries->Value, Value, EntryLength - 5);\r
         Entries = (ADDITIONAL_INFORMATION_ENTRY *) ((UINT8 *)Entries + EntryLength);\r
       }\r
     }\r
@@ -909,12 +893,12 @@ Returns:
   // Onboard Devices Extended Information (Type 41)\r
   //\r
   case 41:\r
-    PrintPendingString (pStruct, Type41, ReferenceDesignation);\r
-    PrintStructValueH (pStruct, Type41, DeviceType);\r
-    PrintStructValueH (pStruct, Type41, DeviceTypeInstance);\r
-    PrintStructValueH (pStruct, Type41, SegmentGroupNum);\r
-    PrintStructValueH (pStruct, Type41, BusNum);\r
-    PrintStructValueH (pStruct, Type41, DevFuncNum);\r
+    PRINT_PENDING_STRING (Struct, Type41, ReferenceDesignation);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type41, DeviceType);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type41, DeviceTypeInstance);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type41, SegmentGroupNum);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type41, BusNum);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type41, DevFuncNum);\r
     break;\r
 \r
   case 126:\r
@@ -933,10 +917,16 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+  Display BIOS Information (Type 0) information.\r
+\r
+  @param[in] Chara    The information bits.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayBiosCharacteristics (\r
-  UINT64  chara,\r
-  UINT8   Option\r
+  IN UINT64  Chara,\r
+  IN UINT8   Option\r
   )\r
 {\r
   //\r
@@ -946,199 +936,211 @@ DisplayBiosCharacteristics (
   //\r
   // print option\r
   //\r
-  PRINT_INFO_OPTION (chara, Option);\r
+  PRINT_INFO_OPTION (Chara, Option);\r
 \r
   //\r
   // Check all the bits and print information\r
   // This function does not use Table because table of bits\r
   //   are designed not to deal with UINT64\r
   //\r
-  if (BIT (chara, 0) != 0) {\r
+  if (BIT (Chara, 0) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_RESERVED_BIT), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 1) != 0) {\r
+  if (BIT (Chara, 1) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_RESERVED_BIT), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 2) != 0) {\r
+  if (BIT (Chara, 2) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_UNKNOWN_BIT), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 3) != 0) {\r
+  if (BIT (Chara, 3) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIOS_CHAR_NOT_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 4) != 0) {\r
+  if (BIT (Chara, 4) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ISA_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 5) != 0) {\r
+  if (BIT (Chara, 5) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_MSA_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 6) != 0) {\r
+  if (BIT (Chara, 6) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_EISA_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 7) != 0) {\r
+  if (BIT (Chara, 7) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_PCI_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 8) != 0) {\r
+  if (BIT (Chara, 8) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_PC_CARD_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 9) != 0) {\r
+  if (BIT (Chara, 9) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_PLUG_PLAY_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 10) != 0) {\r
+  if (BIT (Chara, 10) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_APM_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 11) != 0) {\r
+  if (BIT (Chara, 11) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIOS_UPGRADEABLE), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 12) != 0) {\r
+  if (BIT (Chara, 12) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIOS_SHADOWING), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 13) != 0) {\r
+  if (BIT (Chara, 13) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_VESA_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 14) != 0) {\r
+  if (BIT (Chara, 14) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ECSD_SUPPORT), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 15) != 0) {\r
+  if (BIT (Chara, 15) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BOOT_FORM_CD_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 16) != 0) {\r
+  if (BIT (Chara, 16) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_SELECTED_BOOT_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 17) != 0) {\r
+  if (BIT (Chara, 17) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIOS_ROM_SOCKETED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 18) != 0) {\r
+  if (BIT (Chara, 18) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BOOT_FROM_PC_CARD), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 19) != 0) {\r
+  if (BIT (Chara, 19) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_EDD_ENHANCED_DRIVER), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 20) != 0) {\r
+  if (BIT (Chara, 20) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_JAPANESE_FLOPPY_NEC), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 21) != 0) {\r
+  if (BIT (Chara, 21) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_JAPANESE_FLOPPY_TOSHIBA), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 22) != 0) {\r
+  if (BIT (Chara, 22) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_FLOPPY_SERVICES_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 23) != 0) {\r
+  if (BIT (Chara, 23) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ONE_POINT_TWO_MB), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 24) != 0) {\r
+  if (BIT (Chara, 24) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_720_KB), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 25) != 0) {\r
+  if (BIT (Chara, 25) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_TWO_POINT_EIGHT_EIGHT_MB), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 26) != 0) {\r
+  if (BIT (Chara, 26) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_PRINT_SCREEN_SUPPORT), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 27) != 0) {\r
+  if (BIT (Chara, 27) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_KEYBOARD_SERV_SUPPORT), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 28) != 0) {\r
+  if (BIT (Chara, 28) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_SERIAL_SERVICES_SUPPORT), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 29) != 0) {\r
+  if (BIT (Chara, 29) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_PRINTER_SERVICES_SUPPORT), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 30) != 0) {\r
+  if (BIT (Chara, 30) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_MONO_VIDEO_SUPPORT), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (chara, 31) != 0) {\r
+  if (BIT (Chara, 31) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_NEC_PC_98), gShellDebug1HiiHandle);\r
   }\r
   //\r
-  // Just print the reserved\r
+  // Just print the Reserved\r
   //\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BITS_32_47), gShellDebug1HiiHandle);\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BITS_48_64), gShellDebug1HiiHandle);\r
 }\r
 \r
+/**\r
+  Display Bios Characteristice extensions1 information.\r
+\r
+  @param[in] Byte1    The information.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayBiosCharacteristicsExt1 (\r
-  UINT8 byte1,\r
-  UINT8 Option\r
+  IN UINT8 Byte1,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIOS_CHAR_EXTENSION), gShellDebug1HiiHandle);\r
   //\r
   // Print option\r
   //\r
-  PRINT_INFO_OPTION (byte1, Option);\r
+  PRINT_INFO_OPTION (Byte1, Option);\r
 \r
   //\r
   // check bit and print\r
   //\r
-  if (BIT (byte1, 0) != 0) {\r
+  if (BIT (Byte1, 0) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ACPI_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (byte1, 1) != 0) {\r
+  if (BIT (Byte1, 1) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_USB_LEGACY_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (byte1, 2) != 0) {\r
+  if (BIT (Byte1, 2) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_AGP_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (byte1, 3) != 0) {\r
+  if (BIT (Byte1, 3) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_I2O_BOOT_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (byte1, 4) != 0) {\r
+  if (BIT (Byte1, 4) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_LS_120_BOOT_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (byte1, 5) != 0) {\r
+  if (BIT (Byte1, 5) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ATAPI_ZIP_DRIVE), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (byte1, 6) != 0) {\r
+  if (BIT (Byte1, 6) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_1394_BOOT_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 \r
-  if (BIT (byte1, 7) != 0) {\r
+  if (BIT (Byte1, 7) != 0) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_SMART_BATTERY_SUPPORTED), gShellDebug1HiiHandle);\r
   }\r
 }\r
 \r
+/**\r
+  Display Bios Characteristice extensions2 information.\r
+\r
+  @param[in] byte2    The information.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayBiosCharacteristicsExt2 (\r
-  UINT8 byte2,\r
-  UINT8 Option\r
+  IN UINT8 byte2,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIOS_CHAR_EXTENSION_2), gShellDebug1HiiHandle);\r
@@ -1158,6 +1160,12 @@ DisplayBiosCharacteristicsExt2 (
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BITS_RSVD_FOR_FUTURE), gShellDebug1HiiHandle);\r
 }\r
 \r
+/**\r
+  Display Processor Information (Type 4) information.\r
+\r
+  @param[in] Family       The family value.\r
+  @param[in] Option       The option value.\r
+**/\r
 VOID\r
 DisplayProcessorFamily (\r
   UINT8 Family,\r
@@ -1723,10 +1731,16 @@ DisplayProcessorFamily (
   //\r
 }\r
 \r
+/**\r
+  Display processor family information.\r
+\r
+  @param[in] Family2      The family value.\r
+  @param[in] Option       The option value.\r
+**/\r
 VOID\r
 DisplayProcessorFamily2 (\r
-  UINT16 Family2,\r
-  UINT8  Option\r
+  IN UINT16 Family2,\r
+  IN UINT8  Option\r
   )\r
 {\r
   //\r
@@ -1789,36 +1803,33 @@ DisplayProcessorFamily2 (
 \r
 }\r
 \r
+/**\r
+  Display processor voltage information.\r
+\r
+  @param[in] Voltage      The Voltage.\r
+                      Bit 7 Set to 0, indicating 'legacy' mode for processor voltage\r
+                      Bits 6:4  Reserved, must be zero\r
+                      Bits 3:0  Voltage Capability.\r
+                                A Set bit indicates that the voltage is supported.\r
+                        Bit 0 - 5V\r
+                        Bit 1 - 3.3V\r
+                        Bit 2 - 2.9V\r
+                        Bit 3 - Reserved, must be zero.\r
+\r
+                      Note:\r
+                        Setting of multiple bits indicates the socket is configurable\r
+                        If bit 7 is set to 1, the remaining seven bits of the field are set to\r
+                        contain the processor's current voltage times 10.\r
+                        For example, the field value for a processor voltage of 1.8 volts would be\r
+                        92h = 80h + (1.8 * 10) = 80h + 18 = 80h +12h.\r
+\r
+  @param[in] Option       The option.\r
+**/\r
 VOID\r
 DisplayProcessorVoltage (\r
-  UINT8 Voltage,\r
-  UINT8 Option\r
+  IN UINT8 Voltage,\r
+  IN UINT8 Option\r
   )\r
-/*++\r
-Routine Description:\r
-  Bit 7 Set to 0, indicating 'legacy' mode for processor voltage\r
-  Bits 6:4  Reserved, must be zero\r
-  Bits 3:0  Voltage Capability.\r
-            A Set bit indicates that the voltage is supported.\r
-    Bit 0 - 5V\r
-    Bit 1 - 3.3V\r
-    Bit 2 - 2.9V\r
-    Bit 3 - Reserved, must be zero.\r
-\r
-  Note:\r
-    Setting of multiple bits indicates the socket is configurable\r
-    If bit 7 is set to 1, the remaining seven bits of the field are set to\r
-    contain the processor's current voltage times 10.\r
-    For example, the field value for a processor voltage of 1.8 volts would be\r
-    92h = 80h + (1.8 * 10) = 80h + 18 = 80h +12h.\r
-\r
-Arguments:\r
-  Voltage  - The Voltage\r
-  Option   - The option\r
-\r
-Returns:\r
-\r
-**/\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_PROC_INFO), gShellDebug1HiiHandle);\r
   //\r
@@ -1861,35 +1872,31 @@ Returns:
   }\r
 }\r
 \r
+/**\r
+  Display processor information.\r
+\r
+  @param[in] Status   The status.\r
+                        Bit 7 Reserved, must be 0\r
+                        Bit 6   CPU Socket Populated\r
+                         1 - CPU Socket Populated\r
+                         0 - CPU Socket UnpopulatedBits\r
+                         5:3  Reserved, must be zero\r
+                         Bits 2:0 CPU Status\r
+                          0h - Unknown\r
+                          1h - CPU Enabled\r
+                          2h - CPU Disabled by User via BIOS Setup\r
+                          3h - CPU Disabled By BIOS (POST Error)\r
+                          4h - CPU is Idle, waiting to be enabled.\r
+                          5-6h - Reserved\r
+                          7h - Other\r
+\r
+  @param[in] Option   The option\r
+**/\r
 VOID\r
 DisplayProcessorStatus (\r
-  UINT8 Status,\r
-  UINT8 Option\r
+  IN UINT8 Status,\r
+  IN UINT8 Option\r
   )\r
-/*++\r
-Routine Description:\r
-\r
-Bit 7 Reserved, must be 0\r
-Bit 6   CPU Socket Populated\r
- 1 - CPU Socket Populated\r
- 0 - CPU Socket UnpopulatedBits\r
- 5:3  Reserved, must be zero\r
- Bits 2:0 CPU Status\r
-  0h - Unknown\r
-  1h - CPU Enabled\r
-  2h - CPU Disabled by User via BIOS Setup\r
-  3h - CPU Disabled By BIOS (POST Error)\r
-  4h - CPU is Idle, waiting to be enabled.\r
-  5-6h - Reserved\r
-  7h - Other\r
-\r
-Arguments:\r
-  Status  - The status\r
-  Option  - The option\r
-\r
-Returns:\r
-\r
-**/\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_PROC_STATUS), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Status, Option);\r
@@ -1944,11 +1951,18 @@ Returns:
   }\r
 }\r
 \r
+/**\r
+  Display information about Memory Controller Information (Type 5).\r
+\r
+  @param[in] Size     Memory size.\r
+  @param[in] SlotNum  Which slot is this about.\r
+  @param[in] Option   Option for the level of detail output required.\r
+**/\r
 VOID\r
 DisplayMaxMemoryModuleSize (\r
-  UINT8 Size,\r
-  UINT8 SlotNum,\r
-  UINT8 Option\r
+  IN UINT8 Size,\r
+  IN UINT8 SlotNum,\r
+  IN UINT8 Option\r
   )\r
 {\r
   UINTN MaxSize;\r
@@ -1965,11 +1979,18 @@ DisplayMaxMemoryModuleSize (
   }\r
 }\r
 \r
+/**\r
+  Display information about memory configuration handles.\r
+\r
+  @param[in] Handles  The buffer of handles to output info on.\r
+  @param[in] SlotNum  The number of handles in the above buffer.\r
+  @param[in] Option   Option for the level of detail output required.\r
+**/\r
 VOID\r
 DisplayMemoryModuleConfigHandles (\r
-  UINT16 *Handles,\r
-  UINT8  SlotNum,\r
-  UINT8  Option\r
+  IN UINT16 *Handles,\r
+  IN UINT8  SlotNum,\r
+  IN UINT8  Option\r
   )\r
 {\r
   UINT8 Index;\r
@@ -1989,13 +2010,17 @@ DisplayMemoryModuleConfigHandles (
     }\r
   }\r
 }\r
-//\r
-// Memory Module Information (Type 6)\r
-//\r
+\r
+/**\r
+  Display Memory Module Information (Type 6).\r
+\r
+  @param[in] BankConnections\r
+  @param[in] Option\r
+**/\r
 VOID\r
 DisplayMmBankConnections (\r
-  UINT8 BankConnections,\r
-  UINT8 Option\r
+  IN UINT8 BankConnections,\r
+  IN UINT8 Option\r
   )\r
 {\r
   UINT8 High;\r
@@ -2027,13 +2052,9 @@ DisplayMmBankConnections (
   }\r
 }\r
 \r
-VOID\r
-DisplayMmMemorySize (\r
-  UINT8 Size,\r
-  UINT8 Option\r
-  )\r
-/*++\r
-Routine Description:\r
+/**\r
+  Display memory informcation.\r
+\r
   Bits 0:6  Size (n),\r
       where 2**n is the size in MB with three special-case values:\r
       7Dh Not determinable (Installed Size only)\r
@@ -2042,13 +2063,14 @@ Routine Description:
   Bit  7  Defines whether the memory module has a single- (0)\r
           or double-bank (1) connection.\r
 \r
-Arguments:\r
-  Size   - The size\r
-  Option - The option\r
-\r
-Returns:\r
-\r
+  @param[in] Size   - The size\r
+  @param[in] Option - The option\r
 **/\r
+VOID\r
+DisplayMmMemorySize (\r
+  IN UINT8 Size,\r
+  IN UINT8 Option\r
+  )\r
 {\r
   UINT8 Value;\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_MEMORY_SIZE), gShellDebug1HiiHandle);\r
@@ -2078,28 +2100,21 @@ Returns:
   }\r
 }\r
 \r
-VOID\r
-DisplaySystemSlotId (\r
-  UINT16  SlotId,\r
-  UINT8   SlotType,\r
-  UINT8   Option\r
-  )\r
-/*++\r
-Routine Description:\r
-\r
+/**\r
   The Slot ID field of the System Slot structure provides a mechanism to\r
   correlate the physical attributes of the slot to its logical access method\r
   (which varies based on the Slot Type field).\r
 \r
-Arguments:\r
-\r
-  SlotId   - The slot ID\r
-  SlotType - The slot type\r
-  Option   - The Option\r
-\r
-Returns:\r
-\r
+  @param[in] SlotId   - The slot ID\r
+  @param[in] SlotType - The slot type\r
+  @param[in] Option   - The Option\r
 **/\r
+VOID\r
+DisplaySystemSlotId (\r
+  IN UINT16  SlotId,\r
+  IN UINT8   SlotType,\r
+  IN UINT8   Option\r
+  )\r
 {\r
   //\r
   // Display slot type first\r
@@ -2167,10 +2182,16 @@ Returns:
   }\r
 }\r
 \r
+/**\r
+  Display System Boot Information (Type 32) information.\r
+\r
+  @param[in] Parameter      The parameter.\r
+  @param[in] Option         The options.\r
+**/\r
 VOID\r
 DisplaySystemBootStatus (\r
-  UINT8 Parameter,\r
-  UINT8 Option\r
+  IN UINT8 Parameter,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_SYSTEM_BOOT_STATUS), gShellDebug1HiiHandle);\r
@@ -2210,16 +2231,10 @@ DisplaySystemBootStatus (
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ERROR_VALUE), gShellDebug1HiiHandle);\r
   }\r
 }\r
-//\r
-// Portable Battery (Type 22)\r
-//\r
-VOID\r
-DisplaySBDSManufactureDate (\r
-  UINT16  Date,\r
-  UINT8   Option\r
-  )\r
-/*++\r
-Routine Description:\r
+\r
+/**\r
+  Display Portable Battery (Type 22) information.\r
+\r
   The date the cell pack was manufactured, in packed format:\r
    Bits 15:9  Year, biased by 1980, in the range 0 to 127.\r
    Bits 8:5 Month, in the range 1 to 12.\r
@@ -2227,13 +2242,14 @@ Routine Description:
   For example, 01 February 2000 would be identified as\r
   0010 1000 0100 0001b (0x2841).\r
 \r
-Arguments:\r
-  Date   - The date\r
-  Option - The option\r
-\r
-Returns:\r
-\r
+  @param[in] Date     The date\r
+  @param[in] Option   The option\r
 **/\r
+VOID\r
+DisplaySBDSManufactureDate (\r
+  IN UINT16  Date,\r
+  IN UINT8   Option\r
+  )\r
 {\r
   UINTN Day;\r
   UINTN Month;\r
@@ -2250,41 +2266,37 @@ Returns:
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_MONTH_DAY_YEAR), gShellDebug1HiiHandle, Day, Month, Year);\r
 \r
 }\r
-//\r
-// System Reset  (Type 23)\r
-//\r
+\r
+/**\r
+  Display System Reset  (Type 23) information.\r
+\r
+  \r
+  Identifies the system-reset capabilities for the system.\r
+   Bits 7:6 Reserved for future assignment via this specification, set to 00b.\r
+   Bit 5  System contains a watchdog timer, either True (1) or False (0).\r
+   Bits 4:3 Boot Option on Limit.\r
+    Identifies the system action to be taken when the Reset Limit is reached, one of:\r
+    00b Reserved, do not use.\r
+    01b Operating system\r
+    10b System utilities\r
+    11b Do not rebootBits\r
+   2:1  Boot Option.  Indicates the action to be taken following a watchdog reset, one of:\r
+    00b Reserved, do not use.\r
+    01b Operating system\r
+    10b System utilities\r
+    11b Do not reboot\r
+   Bit 0  Status.\r
+    1b The system reset is enabled by the user\r
+    0b The system reset is not enabled by the user\r
+\r
+  @param[in] Reset   Reset\r
+  @param[in] Option  The option\r
+**/\r
 VOID\r
 DisplaySystemResetCapabilities (\r
-  UINT8 Reset,\r
-  UINT8 Option\r
+  IN UINT8 Reset,\r
+  IN UINT8 Option\r
   )\r
-/*++\r
-Routine Description:\r
-Identifies the system-reset capabilities for the system.\r
- Bits 7:6 Reserved for future assignment via this specification, set to 00b.\r
- Bit 5  System contains a watchdog timer, either True (1) or False (0).\r
- Bits 4:3 Boot Option on Limit.\r
-  Identifies the system action to be taken when the Reset Limit is reached, one of:\r
-  00b Reserved, do not use.\r
-  01b Operating system\r
-  10b System utilities\r
-  11b Do not rebootBits\r
- 2:1  Boot Option.  Indicates the action to be taken following a watchdog reset, one of:\r
-  00b Reserved, do not use.\r
-  01b Operating system\r
-  10b System utilities\r
-  11b Do not reboot\r
- Bit 0  Status.\r
-  1b The system reset is enabled by the user\r
-  0b The system reset is not enabled by the user\r
-\r
-Arguments:\r
-  Reset   - Reset\r
-  Option  - The option\r
-\r
-Returns:\r
-\r
-**/\r
 {\r
   UINTN Temp;\r
 \r
@@ -2358,46 +2370,42 @@ Returns:
     Print (L"The system reset is disabled by the user\n");\r
   }\r
 }\r
-//\r
-// Hardware Security (Type 24)\r
-//\r
+\r
+/**\r
+  Display Hardware Security (Type 24) information.\r
+\r
+    \r
+    Identifies the password and reset status for the system:\r
+\r
+    Bits 7:6    Power-on Password Status, one of:\r
+      00b Disabled\r
+      01b Enabled\r
+      10b Not Implemented\r
+      11b Unknown\r
+    Bits 5:4    Keyboard Password Status, one of:\r
+      00b Disabled\r
+      01b Enabled\r
+      10b Not Implemented\r
+      11b Unknown\r
+    Bits 3:2    Administrator Password Status, one  of:\r
+      00b Disabled\r
+      01b Enabled\r
+      10b Not Implemented\r
+      11b Unknown\r
+    Bits 1:0    Front Panel Reset Status, one of:\r
+      00b Disabled\r
+      01b Enabled\r
+      10b Not Implemented\r
+      11b Unknown\r
+\r
+  @param[in] Settings The device settings.\r
+  @param[in] Option   The device options.\r
+**/\r
 VOID\r
 DisplayHardwareSecuritySettings (\r
-  UINT8 Settings,\r
-  UINT8 Option\r
+  IN UINT8 Settings,\r
+  IN UINT8 Option\r
   )\r
-/*++\r
-Routine Description:\r
-Identifies the password and reset status for the system:\r
-\r
-Bits 7:6    Power-on Password Status, one of:\r
-  00b Disabled\r
-  01b Enabled\r
-  10b Not Implemented\r
-  11b Unknown\r
-Bits 5:4    Keyboard Password Status, one of:\r
-  00b Disabled\r
-  01b Enabled\r
-  10b Not Implemented\r
-  11b Unknown\r
-Bits 3:2    Administrator Password Status, one  of:\r
-  00b Disabled\r
-  01b Enabled\r
-  10b Not Implemented\r
-  11b Unknown\r
-Bits 1:0    Front Panel Reset Status, one of:\r
-  00b Disabled\r
-  01b Enabled\r
-  10b Not Implemented\r
-  11b Unknown\r
-\r
-Arguments:\r
-  Settings    - The settings\r
-  Option      - the option\r
-\r
-Returns:\r
-\r
-**/\r
 {\r
   UINTN Temp;\r
 \r
@@ -2493,13 +2501,17 @@ Returns:
     break;\r
   }\r
 }\r
-//\r
-// Out-of-Band Remote Access (Type 30)\r
-//\r
+\r
+/**\r
+  Display Out-of-Band Remote Access (Type 30) information.\r
+\r
+  @param[in] Connections        The device characteristics.\r
+  @param[in] Option             The device options.\r
+**/\r
 VOID\r
 DisplayOBRAConnections (\r
-  UINT8   Connections,\r
-  UINT8   Option\r
+  IN UINT8   Connections,\r
+  IN UINT8   Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_CONNECTIONS), gShellDebug1HiiHandle);\r
@@ -2528,13 +2540,17 @@ DisplayOBRAConnections (
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_INBOUND_CONN_DISABLED), gShellDebug1HiiHandle);\r
   }\r
 }\r
-//\r
-// System Power Supply (Type 39)\r
-//\r
+\r
+/**\r
+  Display System Power Supply (Type 39) information.\r
+\r
+  @param[in] Characteristics    The device characteristics.\r
+  @param[in] Option             The device options.\r
+**/\r
 VOID\r
 DisplaySPSCharacteristics (\r
-  UINT16  Characteristics,\r
-  UINT8   Option\r
+  IN UINT16  Characteristics,\r
+  IN UINT8   Option\r
   )\r
 {\r
   UINTN Temp;\r