]> git.proxmox.com Git - mirror_edk2.git/blobdiff - ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c
ShellPkg: Update header file including style
[mirror_edk2.git] / ShellPkg / Library / UefiShellDebug1CommandsLib / SmbiosView / PrintInfo.c
index 91835106d56015a4c261dce1558d5a6497e1f2c1..7a75554d7952464aee784a5a038bc289f21d512a 100644 (file)
@@ -1,7 +1,9 @@
 /** @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 - 2017, Intel Corporation. All rights reserved.<BR>\r
+  (C) Copyright 2014 Hewlett-Packard Development Company, L.P.<BR>  \r
+  (C) Copyright 2015-2017 Hewlett Packard Enterprise Development LP<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
@@ -12,7 +14,7 @@
 \r
 **/\r
 \r
-#include "../UefiShellDebug1CommandsLib.h"\r
+#include "UefiShellDebug1CommandsLib.h"\r
 #include "PrintInfo.h"\r
 #include "LibSmbiosView.h"\r
 #include "QueryTable.h"\r
 //\r
 #define BIT(value, bit) ((value) & ((UINT64) 1) << (bit))\r
 \r
+//\r
+// Check if above or equal to version\r
+//\r
+#define AE_SMBIOS_VERSION(MajorVersion, MinorVersion) \\r
+  (SmbiosMajorVersion > (MajorVersion) || (SmbiosMajorVersion == (MajorVersion) && SmbiosMinorVersion >= (MinorVersion)))\r
+\r
 //\r
 //////////////////////////////////////////////////////////\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
-    SmbiosGetPendingString ((pStruct), (pStruct->type->element), StringBuf); \\r
+    CHAR8 *StringBuf; \\r
+    StringBuf = LibGetSmbiosString ((pStruct), (pStruct->type->element)); \\r
     ShellPrintEx(-1,-1,L"%a",#element); \\r
-    ShellPrintEx(-1,-1,L": %a\n", StringBuf); \\r
+    ShellPrintEx(-1,-1,L": %a\n", (StringBuf != NULL) ? 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
-    SmbiosGetPendingString ((pStruct), (stringnumber), StringBuf); \\r
+    CHAR8 *StringBuf; \\r
+    StringBuf = LibGetSmbiosString ((pStruct), (stringnumber)); \\r
     ShellPrintEx(-1,-1,L"%a",#element); \\r
-    ShellPrintEx(-1,-1,L": %a\n", StringBuf); \\r
+    ShellPrintEx(-1,-1,L": %a\n", (StringBuf != NULL) ? 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_STRUCT_VALUE_LH(pStruct, type, element) \\r
+  do { \\r
+    ShellPrintEx(-1,-1,L"%a",#element); \\r
+    ShellPrintEx(-1,-1,L": 0x%lx\n", (pStruct->type->element)); \\r
+  } while (0);\r
+\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 +78,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 +126,18 @@ Returns:
 //\r
 // Functions below is to show the information\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
+  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_TABLE_ENTRY_POINT  *SmbiosTable,\r
+  IN  UINT8                     Option\r
+  )\r
 {\r
   UINT8 Anchor[5];\r
   UINT8 InAnchor[6];\r
@@ -199,89 +200,151 @@ Returns: None
   Print (L"\n");\r
 }\r
 \r
-EFI_STATUS\r
-SmbiosPrintStructure (\r
-  IN  SMBIOS_STRUCTURE_POINTER  *pStruct,\r
-  IN  UINT8                     Option\r
+/**\r
+  Print the info of 64-bit 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
+Smbios64BitPrintEPSInfo (\r
+  IN  SMBIOS_TABLE_3_0_ENTRY_POINT  *SmbiosTable,\r
+  IN  UINT8                         Option\r
   )\r
-/*++\r
+{\r
+  UINT8 Anchor[5];\r
+\r
+  if (SmbiosTable == NULL) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_SMBIOSTABLE_NULL), gShellDebug1HiiHandle);\r
+    return ;\r
+  }\r
+\r
+  if (Option == SHOW_NONE) {\r
+    return ;\r
+  }\r
+\r
+  if (Option >= SHOW_NORMAL) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_64_BIT_ENTRY_POINT_SIGN), gShellDebug1HiiHandle);\r
+\r
+    MemToString (Anchor, SmbiosTable->AnchorString, 5);\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ANCHOR_STR), gShellDebug1HiiHandle, Anchor);\r
+\r
+    ShellPrintHiiEx(-1,-1,NULL,\r
+      STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_EPS_CHECKSUM),\r
+      gShellDebug1HiiHandle,\r
+      SmbiosTable->EntryPointStructureChecksum\r
+     );\r
+\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ENTRY_POINT_LEN), gShellDebug1HiiHandle, SmbiosTable->EntryPointLength);\r
+\r
+    ShellPrintHiiEx(-1,-1,NULL,\r
+      STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_VERSION),\r
+      gShellDebug1HiiHandle,\r
+      SmbiosTable->MajorVersion,\r
+      SmbiosTable->MinorVersion\r
+     );\r
+\r
+    ShellPrintHiiEx(-1,-1,NULL,\r
+      STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_DOCREV),\r
+      gShellDebug1HiiHandle,\r
+      SmbiosTable->DocRev\r
+     );\r
 \r
-Routine Description:\r
-  This function print the content of the structure pointed by pStruct\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_TABLE_MAX_SIZE), gShellDebug1HiiHandle, SmbiosTable->TableMaximumSize);\r
 \r
-Arguments:\r
-  pStruct     - point to the structure to be printed\r
-  Option      - print option of information detail\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_TABLE_ADDR), gShellDebug1HiiHandle, SmbiosTable->TableAddress);\r
 \r
-Returns:\r
-  EFI_SUCCESS             - Successfully Printing this function\r
-  EFI_INVALID_PARAMETER   - Invalid Structure\r
-  EFI_UNSUPPORTED         - Unsupported\r
+  }\r
+  //\r
+  // If SHOW_ALL, also print followings.\r
+  //\r
+  if (Option >= SHOW_DETAIL) {\r
+    ShellPrintHiiEx(-1,-1,NULL,\r
+      STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ENTRY_POINT_REVISION),\r
+      gShellDebug1HiiHandle,\r
+      SmbiosTable->EntryPointRevision\r
+     );\r
+  }\r
+\r
+  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  *Struct,\r
+  IN  UINT8                     Option\r
+  )\r
 {\r
   UINT8 Index;\r
   UINT8 *Buffer;\r
 \r
-  Buffer = (UINT8 *) (UINTN) (pStruct->Raw);\r
-\r
-  if (pStruct == NULL) {\r
+  if (Struct == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
   if (Option == SHOW_NONE) {\r
     return EFI_SUCCESS;\r
   }\r
+\r
+  Buffer = (UINT8 *) (UINTN) (Struct->Raw);\r
+\r
   //\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
-  //\r
+  switch (Struct->Hdr->Type) {\r
   //\r
+  // BIOS Information (Type 0)\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
-\r
-    if (Option < SHOW_DETAIL) {\r
-      PrintStructValueH (pStruct, Type0, BiosCharacteristics);\r
-    } else {\r
-      DisplayBiosCharacteristics (pStruct->Type0->BiosCharacteristics, Option);\r
+    PRINT_PENDING_STRING (Struct, Type0, Vendor);\r
+    PRINT_PENDING_STRING (Struct, Type0, BiosVersion);\r
+    PRINT_STRUCT_VALUE_H (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
-      //\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
-      // And byte3 not stated in spec, so dump all extension bytes(1, 2, 3..)\r
-      //\r
-      if ((Buffer[1] - (CHAR8) 0x12) > 0) {\r
-        DisplayBiosCharacteristicsExt1 (Buffer[0x12], Option);\r
-      }\r
+    DisplayBiosCharacteristics (ReadUnaligned64 ((UINT64 *) (UINTN) &(Struct->Type0->BiosCharacteristics)), Option);\r
 \r
-      if ((Buffer[1] - (CHAR8) 0x12) > 1) {\r
-        DisplayBiosCharacteristicsExt2 (Buffer[0x13], Option);\r
-      }\r
+    if (Struct->Hdr->Length > 0x12) {\r
+      DisplayBiosCharacteristicsExt1 (Struct->Type0->BIOSCharacteristicsExtensionBytes[0], Option);\r
+    }\r
+    if (Struct->Hdr->Length > 0x13) {\r
+      DisplayBiosCharacteristicsExt2 (Struct->Type0->BIOSCharacteristicsExtensionBytes[1], Option);\r
+    }\r
 \r
-      if ((Buffer[1] - (CHAR8) 0x12) > 2) {\r
-        PrintBitField (\r
-          pStruct,\r
-          Type0,\r
-          BiosCharacteristics,\r
-          Buffer[1] - (CHAR8) 0x12\r
-         );\r
-      }\r
+    if (AE_SMBIOS_VERSION (0x2, 0x4) && (Struct->Hdr->Length > 0x14)) {\r
+      PRINT_STRUCT_VALUE (Struct, Type0, SystemBiosMajorRelease);\r
+      PRINT_STRUCT_VALUE (Struct, Type0, SystemBiosMinorRelease);\r
+      PRINT_STRUCT_VALUE (Struct, Type0, EmbeddedControllerFirmwareMajorRelease);\r
+      PRINT_STRUCT_VALUE (Struct, Type0, EmbeddedControllerFirmwareMinorRelease);\r
+    }\r
+    if (AE_SMBIOS_VERSION (0x3, 0x1) && (Struct->Hdr->Length > 0x18)) {\r
+      ShellPrintHiiEx (\r
+        -1,\r
+        -1,\r
+        NULL,\r
+        STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_EXTENDED_BIOS_SIZE),\r
+        gShellDebug1HiiHandle,\r
+        Struct->Type0->ExtendedBiosSize.Size,\r
+        (Struct->Type0->ExtendedBiosSize.Unit == 0x0) ? L"MB": L"GB"\r
+        );\r
     }\r
     break;\r
 \r
@@ -289,77 +352,114 @@ 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
+    if (AE_SMBIOS_VERSION (0x2, 0x4) && (Struct->Hdr->Length > 0x19)) {\r
+      PRINT_PENDING_STRING (Struct, Type1, SKUNumber);\r
+      PRINT_PENDING_STRING (Struct, Type1, Family);\r
+    }\r
+\r
     break;\r
 \r
+  //\r
+  // Baseboard Information (Type 2)\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
+    if (Struct->Hdr->Length > 0x8) {\r
+      PRINT_PENDING_STRING (Struct, Type2, AssetTag);\r
+      DisplayBaseBoardFeatureFlags (*(UINT8 *) &Struct->Type2->FeatureFlag, Option);\r
+      PRINT_PENDING_STRING (Struct, Type2, LocationInChassis);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type2, ChassisHandle);\r
+      DisplayBaseBoardBoardType (Struct->Type2->BoardType, Option);\r
+    }\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
+    if (AE_SMBIOS_VERSION (0x2, 0x3)) {\r
+      if (Struct->Hdr->Length > 0xD) {\r
+        PRINT_BIT_FIELD (Struct, Type3, OemDefined, 4);\r
+      }\r
+      if (Struct->Hdr->Length > 0x11) {\r
+        PRINT_STRUCT_VALUE (Struct, Type3, Height);\r
+      }\r
+      if (Struct->Hdr->Length > 0x12) {\r
+        PRINT_STRUCT_VALUE (Struct, Type3, NumberofPowerCords);\r
+      }\r
+    }\r
+    if (AE_SMBIOS_VERSION (0x2, 0x7) && (Struct->Hdr->Length > 0x13)) {\r
+      if (Struct->Hdr->Length > (0x15 + (Struct->Type3->ContainedElementCount * Struct->Type3->ContainedElementRecordLength))) {\r
+        PRINT_SMBIOS_STRING (Struct, Buffer[0x15 + (Struct->Type3->ContainedElementCount * Struct->Type3->ContainedElementRecordLength)], SKUNumber);\r
+      }\r
+    }\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
-    if ((SmbiosMajorVersion > 0x2 || (SmbiosMajorVersion == 0x2 && SmbiosMinorVersion >= 0x6)) &&\r
-        (pStruct->Type4->ProcessorFamily == 0xFE)) {\r
+    PRINT_SMBIOS_STRING (Struct, Struct->Type4->Socket, SocketDesignation)\r
+    DisplayProcessorType (Struct->Type4->ProcessorType, Option);\r
+    if (AE_SMBIOS_VERSION (0x2, 0x6) && (Struct->Hdr->Length > 0x28) &&\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
+    PRINT_PENDING_STRING (Struct, Type4, ProcessorManufacture);\r
+    PRINT_BIT_FIELD (Struct, Type4, ProcessorId, 8);\r
+    PRINT_PENDING_STRING (Struct, Type4, ProcessorVersion);\r
+    DisplayProcessorVoltage (*(UINT8 *) &(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
+    if (AE_SMBIOS_VERSION (0x2, 0x3) && (Struct->Hdr->Length > 0x20)) {\r
+      PRINT_PENDING_STRING (Struct, Type4, SerialNumber);\r
+      PRINT_PENDING_STRING (Struct, Type4, AssetTag);\r
+      PRINT_PENDING_STRING (Struct, Type4, PartNumber);\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
-    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
+    if (AE_SMBIOS_VERSION (0x2, 0x5) && (Struct->Hdr->Length > 0x23)) {\r
+      PRINT_STRUCT_VALUE (Struct, Type4, CoreCount);\r
+      PRINT_STRUCT_VALUE (Struct, Type4, EnabledCoreCount);\r
+      PRINT_STRUCT_VALUE (Struct, Type4, ThreadCount);\r
+      DisplayProcessorCharacteristics (Struct->Type4->ProcessorCharacteristics, Option);\r
+    }\r
+    if ((SmbiosMajorVersion >= 0x3) && (Struct->Hdr->Length > 0x2A)) {\r
+      PRINT_STRUCT_VALUE (Struct, Type4, CoreCount2);\r
+      PRINT_STRUCT_VALUE (Struct, Type4, EnabledCoreCount2);\r
+      PRINT_STRUCT_VALUE (Struct, Type4, ThreadCount2);\r
     }\r
     break;\r
 \r
@@ -369,19 +469,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 (*(UINT8 *) &(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 (*(UINT16 *) &(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 +494,71 @@ 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 (*(UINT16 *) &(Struct->Type6->CurrentMemoryType), Option);\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_INSTALLED), gShellDebug1HiiHandle);\r
-    DisplayMmMemorySize (pStruct->Type6->InstalledSize, Option);\r
+    DisplayMmMemorySize (*(UINT8 *) &(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 (*(UINT8 *) &(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
+    DisplayCacheConfiguration (Struct->Type7->CacheConfiguration, Option);\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 (ReadUnaligned16 ((UINT16 *) (UINTN) &(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
+    if (AE_SMBIOS_VERSION (0x3, 0x1) && (Struct->Hdr->Length > 0x13)) {\r
+      PRINT_STRUCT_VALUE_H (Struct, Type7, MaximumCacheSize2);\r
+      PRINT_STRUCT_VALUE_H (Struct, Type7, InstalledSize2);\r
+    }\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
-    if (SmbiosMajorVersion > 0x2 || (SmbiosMajorVersion == 0x2 && SmbiosMinorVersion >= 0x6)) {\r
-      PrintStructValueH (pStruct, Type9, SegmentGroupNum);\r
-      PrintStructValueH (pStruct, Type9, BusNum);\r
-      PrintStructValueH (pStruct, Type9, DevFuncNum);\r
+    DisplaySlotCharacteristics1 (*(UINT8 *) &(Struct->Type9->SlotCharacteristics1), Option);\r
+    DisplaySlotCharacteristics2 (*(UINT8 *) &(Struct->Type9->SlotCharacteristics2), Option);\r
+    if (AE_SMBIOS_VERSION (0x2, 0x6) && (Struct->Hdr->Length > 0xD)) {\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 +568,59 @@ 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_STRUCTURE)) / (2 * sizeof (UINT8));\r
       for (Index = 0; Index < NumOfDevice; Index++) {\r
-        DisplayOnboardDeviceTypes (pStruct->Type10->Device[Index].DeviceType, Option);\r
+        ShellPrintEx(-1,-1,(((Struct->Type10->Device[Index].DeviceType) & 0x80) != 0) ? L"Device Enabled\n": L"Device Disabled\n");\r
+        DisplayOnboardDeviceTypes ((Struct->Type10->Device[Index].DeviceType) & 0x7F, 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\n",LibGetSmbiosString (Struct, Struct->Type10->Device[Index].DescriptionString));\r
       }\r
     }\r
     break;\r
 \r
+  //\r
+  // Oem Strings (Type 11)\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
+  //\r
+  // System Configuration Options (Type 12)\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
+  //\r
+  // BIOS Language Information (Type 13)\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
+  //\r
+  // Group Associations (Type 14)\r
+  //\r
   case 14:\r
-    PrintPendingString (pStruct, Type14, GroupName);\r
-    PrintStructValue (pStruct, Type14, ItemType);\r
-    PrintStructValue (pStruct, Type14, ItemHandle);\r
+    {\r
+      UINT8 NumOfItem;\r
+      NumOfItem = (Struct->Type14->Hdr.Length - 5) / 3;\r
+      PRINT_PENDING_STRING (Struct, Type14, GroupName);\r
+      for (Index = 0; Index < NumOfItem; Index++) {\r
+        ShellPrintEx(-1,-1,L"ItemType %d: %d\n", Index + 1, Struct->Type14->Group[Index].ItemType);\r
+        ShellPrintEx(-1,-1,L"ItemHandle %d: %d\n", Index + 1, Struct->Type14->Group[Index].ItemHandle);\r
+      }\r
+    }\r
     break;\r
 \r
   //\r
@@ -505,24 +628,24 @@ Returns:
   //\r
   case 15:\r
     {\r
-      EVENTLOGTYPE  *Ptr;\r
-      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
+      EVENT_LOG_TYPE  *Ptr;\r
+      UINT8           Count;\r
+      UINT8           *AccessMethodAddress;\r
+\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 +657,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 +670,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 +681,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,12 +692,12 @@ 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
-           )\r
+              Struct->Type15->LogAreaLength -\r
+              (Struct->Type15->LogDataStartOffset - Struct->Type15->LogHeaderStartOffset)\r
+            )\r
            );\r
         }\r
 \r
@@ -586,39 +709,51 @@ 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
+    if (AE_SMBIOS_VERSION (0x2, 0x7) && Struct->Hdr->Length > 0xF) {\r
+      PRINT_STRUCT_VALUE_LH (Struct, Type16, ExtendedMaximumCapacity);\r
+    }\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
-    if (SmbiosMajorVersion > 0x2 || (SmbiosMajorVersion == 0x2 && SmbiosMinorVersion >= 0x6)) {\r
-      PrintStructValueH (pStruct, Type17, Attributes);\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 (ReadUnaligned16 ((UINT16 *) (UINTN) &(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 (AE_SMBIOS_VERSION (0x2, 0x6) && (Struct->Hdr->Length > 0x1B)) {\r
+      PRINT_STRUCT_VALUE_H (Struct, Type17, Attributes);\r
+    }\r
+    if (AE_SMBIOS_VERSION (0x2, 0x7) && (Struct->Hdr->Length > 0x1C)) {\r
+      PRINT_STRUCT_VALUE (Struct, Type17, ExtendedSize);\r
+      PRINT_STRUCT_VALUE (Struct, Type17, ConfiguredMemoryClockSpeed);\r
+    }\r
+    if (AE_SMBIOS_VERSION (0x2, 0x8) && (Struct->Hdr->Length > 0x22)) {\r
+      PRINT_STRUCT_VALUE (Struct, Type17, MinimumVoltage);\r
+      PRINT_STRUCT_VALUE (Struct, Type17, MaximumVoltage);\r
+      PRINT_STRUCT_VALUE (Struct, Type17, ConfiguredVoltage);\r
     }\r
     break;\r
 \r
@@ -626,256 +761,306 @@ 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
+    if (AE_SMBIOS_VERSION (0x2, 0x7) && (Struct->Hdr->Length > 0xF)) {\r
+      PRINT_STRUCT_VALUE_LH (Struct, Type19, ExtendedStartingAddress);\r
+      PRINT_STRUCT_VALUE_LH (Struct, Type19, ExtendedEndingAddress);\r
+    }\r
     break;\r
 \r
   //\r
-  // Memory Device Mapped Address  (Type 20)\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
+    if (AE_SMBIOS_VERSION (0x2, 0x7) && (Struct->Hdr->Length > 0x13)) {\r
+      PRINT_STRUCT_VALUE_LH (Struct, Type19, ExtendedStartingAddress);\r
+      PRINT_STRUCT_VALUE_LH (Struct, Type19, ExtendedEndingAddress);\r
+    }\r
     break;\r
 \r
   //\r
-  // Built-in Pointing Device  (Type 21)\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
+  // 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
+  //\r
+  // System Reset (Type 23)\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
+  //\r
+  // Hardware Security (Type 24)\r
+  //\r
   case 24:\r
     DisplayHardwareSecuritySettings (\r
-      pStruct->Type24->HardwareSecuritySettings,\r
+      Struct->Type24->HardwareSecuritySettings,\r
       Option\r
      );\r
     break;\r
 \r
+  //\r
+  // System Power Controls (Type 25)\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
+  //\r
+  // Voltage Probe (Type 26)\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 (*(UINT8 *) &(Struct->Type26->LocationAndStatus), Option);\r
+    DisplayVPStatus (*(UINT8 *) &(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
+  //\r
+  // Cooling Device (Type 27)\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 (*(UINT8 *) &(Struct->Type27->DeviceTypeAndStatus), Option);\r
+    DisplayCoolingDeviceType (*(UINT8 *) &(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
+    if (AE_SMBIOS_VERSION (0x2, 0x7) && (Struct->Hdr->Length > 0xE)) {\r
+      PRINT_PENDING_STRING (Struct, Type27, Description);\r
+    }\r
     break;\r
 \r
+  //\r
+  // Temperature Probe (Type 28)\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 (*(UINT8 *) &(Struct->Type28->LocationAndStatus), Option);\r
+    DisplayTemperatureProbeLoc (*(UINT8 *) &(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
+  //\r
+  // Electrical Current Probe (Type 29)\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 (*(UINT8 *) &(Struct->Type29->LocationAndStatus), Option);\r
+    DisplayECPLoc (*(UINT8 *) &(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
+  //\r
+  // Out-of-Band Remote Access (Type 30)\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
+  //\r
+  // Boot Integrity Services (BIS) Entry Point (Type 31)\r
+  //\r
   case 31:\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_STRUCT_TYPE31), gShellDebug1HiiHandle);\r
     break;\r
 \r
+  //\r
+  // System Boot Information (Type 32)\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, Option);\r
     break;\r
 \r
+  //\r
+  // 64-Bit Memory Error Information (Type 33)\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_LH (Struct, Type33, MemoryArrayErrorAddress);\r
+    PRINT_STRUCT_VALUE_LH (Struct, Type33, DeviceErrorAddress);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type33, ErrorResolution);\r
     break;\r
 \r
   //\r
-  // Management Device  (Type 34)\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
+    DisplayMDAddressType (Struct->Type34->AddressType, Option);\r
     break;\r
 \r
+  //\r
+  // Management Device Component (Type 35)\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
+  //\r
+  // Management Device Threshold Data (Type 36)\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
-  // Memory Channel  (Type 37)\r
+  // Memory Channel (Type 37)\r
   //\r
   case 37:\r
     {\r
-      UINT8         Count;\r
-      MEMORYDEVICE  *Ptr;\r
-      DisplayMemoryChannelType (pStruct->Type37->ChannelType, Option);\r
-      PrintStructValueH (pStruct, Type37, MaximumChannelLoad);\r
-      PrintStructValueH (pStruct, Type37, MemoryDeviceCount);\r
-\r
-      Count = pStruct->Type37->MemoryDeviceCount;\r
-      Ptr   = pStruct->Type37->MemoryDevice;\r
+      UINT8          Count;\r
+      MEMORY_DEVICE  *Ptr;\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 = 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
-        ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_DEV_HANDLE), gShellDebug1HiiHandle, Ptr->DeviceHandle);\r
+        ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_DEV_LOAD), gShellDebug1HiiHandle, Ptr[Index].DeviceLoad);\r
+        ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_DEV_HANDLE), gShellDebug1HiiHandle, Ptr[Index].DeviceHandle);\r
       }\r
     }\r
     break;\r
 \r
   //\r
-  // IPMI Device Information  (Type 38)\r
+  // 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_LH (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
+      *(UINT16 *) &(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 +1073,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,18 +1094,46 @@ 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
+    ShellPrintEx(-1,-1,(((Struct->Type41->DeviceType) & 0x80) != 0) ? L"Device Enabled\n": L"Device Disabled\n");\r
+    DisplayOnboardDeviceTypes ((Struct->Type41->DeviceType) & 0x7F, Option);\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
+  //\r
+  // Management Controller Host Interface (Type 42)\r
+  //\r
+  case 42:\r
+    DisplayMCHostInterfaceType (Struct->Type42->InterfaceType, Option);\r
+    break;\r
+\r
+  //\r
+  // TPM Device (Type 43)\r
+  //\r
+  case 43:\r
+    PRINT_BIT_FIELD (Struct, Type43, VendorID, 4);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type43, MajorSpecVersion);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type43, MinorSpecVersion);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type43, FirmwareVersion1);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type43, FirmwareVersion2);\r
+    PRINT_PENDING_STRING (Struct, Type43, Description);\r
+    DisplayTpmDeviceCharacteristics (ReadUnaligned64 ((UINT64 *) (UINTN) &(Struct->Type43->Characteristics)), Option);\r
+    PRINT_STRUCT_VALUE_H (Struct, Type43, OemDefined);\r
     break;\r
 \r
+  //\r
+  // Inactive (Type 126)\r
+  //\r
   case 126:\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_INACTIVE_STRUCT), gShellDebug1HiiHandle);\r
     break;\r
 \r
+  //\r
+  // End-of-Table (Type 127)\r
+  //\r
   case 127:\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_THIS_STRUCT_END_TABLE), gShellDebug1HiiHandle);\r
     break;\r
@@ -933,10 +1146,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 +1165,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
@@ -1155,9 +1386,32 @@ DisplayBiosCharacteristicsExt2 (
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_FUNCTION_KEY_INIT), gShellDebug1HiiHandle);\r
   }\r
 \r
-  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BITS_RSVD_FOR_FUTURE), gShellDebug1HiiHandle);\r
+  if (AE_SMBIOS_VERSION (0x2, 0x4)) {\r
+    if (BIT (byte2, 2) != 0) {\r
+      ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ENABLE_TAR_CONT_DIST), gShellDebug1HiiHandle);\r
+    }\r
+    if (AE_SMBIOS_VERSION (0x2, 0x7)) {\r
+      if (BIT (byte2, 3) != 0) {\r
+        ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_UEFI_SPEC_SUPPORT), gShellDebug1HiiHandle);\r
+      }\r
+      if (BIT (byte2, 4) != 0) {\r
+        ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_VIRTUAL_MACHINE), gShellDebug1HiiHandle);\r
+      }\r
+      ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BITS_RSVD_FOR_FUTURE), gShellDebug1HiiHandle, 5);\r
+    } else {\r
+      ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BITS_RSVD_FOR_FUTURE), gShellDebug1HiiHandle, 3);\r
+    }\r
+  } else {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BITS_RSVD_FOR_FUTURE), gShellDebug1HiiHandle, 2);\r
+  }\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
@@ -1249,6 +1503,18 @@ DisplayProcessorFamily (
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_M1_FAMILY), gShellDebug1HiiHandle);\r
     break;\r
 \r
+  case 0x13:\r
+    Print (L"M2 Family\n");\r
+    break;\r
+\r
+  case 0x14:\r
+    Print (L"Intel Celeron M\n");\r
+    break;\r
+\r
+  case 0x15:\r
+    Print (L"Intel Pentium 4 HT\n");\r
+    break;\r
+\r
   case 0x18:\r
     Print (L"AMD Duron\n");\r
     break;\r
@@ -1257,6 +1523,30 @@ DisplayProcessorFamily (
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_K5_FAMILY), gShellDebug1HiiHandle);\r
     break;\r
 \r
+  case 0x1A:\r
+    Print (L"K6 Family\n");\r
+    break;\r
+\r
+  case 0x1B:\r
+    Print (L"K6-2\n");\r
+    break;\r
+\r
+  case 0x1C:\r
+    Print (L"K6-3\n");\r
+    break;\r
+\r
+  case 0x1D:\r
+    Print (L"AMD Althon Processor Family\n");\r
+    break;\r
+\r
+  case 0x1E:\r
+    Print (L"AMD 29000 Family\n");\r
+    break;\r
+\r
+  case 0x1F:\r
+    Print (L"K6-2+\n");\r
+    break;\r
+\r
   case 0x20:\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_POWER_PC_FAMILY), gShellDebug1HiiHandle);\r
     break;\r
@@ -1305,6 +1595,22 @@ DisplayProcessorFamily (
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_INTEL_ATOM), gShellDebug1HiiHandle);\r
     break;\r
 \r
+  case 0x2C:\r
+    Print (L"Intel(R) Core(TM) M processor\n");\r
+    break;\r
+\r
+  case 0x2D:\r
+    Print (L"Intel(R) Core(TM) m3 processor\n");\r
+    break;\r
+\r
+  case 0x2E:\r
+    Print (L"Intel(R) Core(TM) m5 processor\n");\r
+    break;\r
+\r
+  case 0x2F:\r
+    Print (L"Intel(R) Core(TM) m7 processor\n");\r
+    break;\r
+\r
   case 0x30:\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ALPHA_FAMILY_2), gShellDebug1HiiHandle);\r
     break;\r
@@ -1337,6 +1643,38 @@ DisplayProcessorFamily (
     Print (L"Alpha 21364\n");\r
     break;\r
 \r
+  case 0x38:\r
+    Print (L"AMD Turion II Ultra Dual-Core Mobile M Processor Family\n");\r
+    break;\r
+\r
+  case 0x39:\r
+    Print (L"AMD Turion II Dual-Core Mobile M Processor Family\n");\r
+    break;\r
+\r
+  case 0x3A:\r
+    Print (L"AMD Althon II Dual-Core M Processor Family\n");\r
+    break;\r
+\r
+  case 0x3B:\r
+    Print (L"AMD Opteron 6100 Series Processor\n");\r
+    break;\r
+\r
+  case 0x3C:\r
+    Print (L"AMD Opteron 4100 Series Processor\n");\r
+    break;\r
+\r
+  case 0x3D:\r
+    Print (L"AMD Opteron 6200 Series Processor\n");\r
+    break;\r
+\r
+  case 0x3E:\r
+    Print (L"AMD Opteron 4200 Series Processor\n");\r
+    break;\r
+\r
+  case 0x3F:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_AMD_FX_SERIES), gShellDebug1HiiHandle);\r
+    break;\r
+\r
   case 0x40:\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_MIPS_FAMILY), gShellDebug1HiiHandle);\r
     break;\r
@@ -1361,6 +1699,46 @@ DisplayProcessorFamily (
     Print (L"MIPS R10000\n");\r
     break;\r
 \r
+  case 0x46:\r
+    Print (L"AMD C-Series Processor\n");\r
+    break;\r
+\r
+  case 0x47:\r
+    Print (L"AMD E-Series Processor\n");\r
+    break;\r
+\r
+  case 0x48:\r
+    Print (L"AMD A-Series Processor\n");\r
+    break;\r
+\r
+  case 0x49:\r
+    Print (L"AMD G-Series Processor\n");\r
+    break;\r
+\r
+  case 0x4A:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_AMD_Z_SERIES), gShellDebug1HiiHandle);\r
+    break;\r
+\r
+  case 0x4B:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_AMD_R_SERIES), gShellDebug1HiiHandle);\r
+    break;\r
+\r
+  case 0x4C:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_AMD_OPTERON_4300_SERIES), gShellDebug1HiiHandle);\r
+    break;\r
+\r
+  case 0x4D:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_AMD_OPTERON_6300_SERIES), gShellDebug1HiiHandle);\r
+    break;\r
+\r
+  case 0x4E:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_AMD_OPTERON_3300_SERIES), gShellDebug1HiiHandle);\r
+    break;\r
+\r
+  case 0x4F:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_AMD_OPTERON_FIREPRO_SERIES), gShellDebug1HiiHandle);\r
+    break;\r
+\r
   case 0x50:\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_SPARC_FAMILY), gShellDebug1HiiHandle);\r
     break;\r
@@ -1417,8 +1795,32 @@ DisplayProcessorFamily (
     Print (L"68020\n");\r
     break;\r
 \r
-  case 0x65:\r
-    Print (L"68030\n");\r
+  case 0x65:\r
+    Print (L"68030\n");\r
+    break;\r
+\r
+  case 0x66:\r
+    Print (L"AMD Athlon(TM) X4 Quad-Core Processor Family\n");\r
+    break;\r
+\r
+  case 0x67:\r
+    Print (L"AMD Opteron(TM) X1000 Series Processor\n");\r
+    break;\r
+\r
+  case 0x68:\r
+    Print (L"AMD Opteron(TM) X2000 Series APU\n");\r
+    break;\r
+\r
+  case 0x69:\r
+    Print (L"AMD Opteron(TM) A-Series Processor\n");\r
+    break;\r
+\r
+  case 0x6A:\r
+    Print (L"AMD Opteron(TM) X3000 Series APU\n");\r
+    break;\r
+\r
+  case 0x6B:\r
+    Print (L"AMD Zen Processor Family\n");\r
     break;\r
 \r
   case 0x70:\r
@@ -1501,6 +1903,30 @@ DisplayProcessorFamily (
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_PA_RISC_FAMILY), gShellDebug1HiiHandle);\r
     break;\r
 \r
+  case 0x91:\r
+    Print (L"PA-RISC 8500\n");\r
+    break;\r
+\r
+  case 0x92:\r
+    Print (L"PA-RISC 8000\n");\r
+    break;\r
+\r
+  case 0x93:\r
+    Print (L"PA-RISC 7300LC\n");\r
+    break;\r
+\r
+  case 0x94:\r
+    Print (L"PA-RISC 7200\n");\r
+    break;\r
+\r
+  case 0x95:\r
+    Print (L"PA-RISC 7100LC\n");\r
+    break;\r
+\r
+  case 0x96:\r
+    Print (L"PA-RISC 7100\n");\r
+    break;\r
+\r
   case 0xA0:\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_V30_FAMILY), gShellDebug1HiiHandle);\r
     break;\r
@@ -1569,6 +1995,62 @@ DisplayProcessorFamily (
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_PENTIUM_III_XEON), gShellDebug1HiiHandle);\r
     break;\r
 \r
+  case 0xB1:\r
+    Print (L"Pentium III Processorwith Intel SpeedStep Technology\n");\r
+    break;\r
+\r
+  case 0xB2:\r
+    Print (L"Pentium 4 processor\n");\r
+    break;\r
+\r
+  case 0xB3:\r
+    Print (L"Intel Xeon Processor\n");\r
+    break;\r
+\r
+  case 0xB4:\r
+    Print (L"AS400 Family\n");\r
+    break;\r
+\r
+  case 0xB5:\r
+    Print (L"Intel Xeon processor MP\n");\r
+    break;\r
+\r
+  case 0xB6:\r
+    Print (L"AMD Althon XP Processor Family\n");\r
+    break;\r
+\r
+  case 0xB7:\r
+    Print (L"AMD Althon MP Promcessor Family\n");\r
+    break;\r
+\r
+  case 0xB8:\r
+    Print (L"Intel Itanium 2 processor\n");\r
+    break;\r
+\r
+  case 0xB9:\r
+    Print (L"Intel Penium M processor\n");\r
+    break;\r
+\r
+  case 0xBA:\r
+    Print (L"Intel Celeron D processor\n");\r
+    break;\r
+\r
+  case 0xBB:\r
+    Print (L"Intel Pentium D processor\n");\r
+    break;\r
+\r
+  case 0xBC:\r
+    Print (L"Intel Pentium Processor Extreme Edition\n");\r
+    break;\r
+\r
+  case 0xBD:\r
+    Print (L"Intel Core Solo Processor\n");\r
+    break;\r
+\r
+  case 0xBF:\r
+    Print (L"Intel Core 2 Duo Processor\n");\r
+    break;\r
+\r
   case 0xC0:\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_INTEL_CORE2_SOLO), gShellDebug1HiiHandle);\r
     break;\r
@@ -1618,7 +2100,15 @@ DisplayProcessorFamily (
     break;\r
 \r
   case 0xCC:\r
-    Print (L"zArchitectur\n");\r
+    Print (L"zArchitecture\n");\r
+    break;\r
+\r
+  case 0xCD:\r
+    Print (L"Intel Core i5 processor\n");\r
+    break;\r
+\r
+  case 0xCE:\r
+    Print (L"Intel Core i3 processor\n");\r
     break;\r
 \r
   case 0xD2:\r
@@ -1669,6 +2159,19 @@ DisplayProcessorFamily (
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_INTEL_XEON_7_SERIES_MULTI_CORE), gShellDebug1HiiHandle);\r
     break;\r
 \r
+  case 0xE0:\r
+    Print (L"Multi-Core Intel Xeon processor 3400 Series\n");\r
+    break;\r
+\r
+  case 0xE4:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_AMD_OPTERON_3000_SERIES), gShellDebug1HiiHandle);\r
+    break;\r
+\r
+  case 0xE5:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_AMD_SEMPRON_II), gShellDebug1HiiHandle);\r
+    break;\r
+\r
+\r
   case 0xE6:\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_AMD_EMBEDDED_OPTERON_QUAD_CORE), gShellDebug1HiiHandle);\r
     break;\r
@@ -1693,6 +2196,22 @@ DisplayProcessorFamily (
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_AMD_SEMPRON_SI), gShellDebug1HiiHandle);\r
     break;\r
 \r
+  case 0xEC:\r
+    Print (L"AMD Phenom II Processor Family\n");\r
+    break;\r
+\r
+  case 0xED:\r
+    Print (L"AMD Althon II Processor Family\n");\r
+    break;\r
+\r
+  case 0xEE:\r
+    Print (L"Six-Core AMD Opteron Processor Family\n");\r
+    break;\r
+\r
+  case 0xEF:\r
+    Print (L"AMD Sempron M Processor Family\n");\r
+    break;\r
+\r
   case 0xFA:\r
     Print (L"i860\n");\r
     break;\r
@@ -1702,31 +2221,23 @@ DisplayProcessorFamily (
     break;\r
 \r
   default:\r
-    //\r
-    // In order to reduce code quality notice of\r
-    // case & break not pair, so\r
-    // move multiple case into the else part and\r
-    // use if/else to check value.\r
-    //\r
-    if (Family >= 0x13 && Family <= 0x17) {\r
-      ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_RSVD_FOR_SPEC_M1), gShellDebug1HiiHandle);\r
-    } else if (Family >= 0x1A && Family <= 0x1F) {\r
-      ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_RSVD_FOR_SPEC_K5), gShellDebug1HiiHandle);\r
-    } else if (Family >= 0xB1 && Family <= 0xBF) {\r
-      ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_RSVD_FOR_SPEC_PENTIUM), gShellDebug1HiiHandle);\r
-    } else {\r
-      ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_UNDEFINED_PROC_FAMILY), gShellDebug1HiiHandle);\r
-    }\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_UNDEFINED_PROC_FAMILY), gShellDebug1HiiHandle);\r
   }\r
   //\r
   // end switch\r
   //\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
@@ -1743,6 +2254,14 @@ DisplayProcessorFamily2 (
   // Use switch to check\r
   //\r
   switch (Family2) {\r
+    case 0x100:\r
+      Print (L"ARMv7\n");\r
+      break;\r
+\r
+    case 0x101:\r
+      Print (L"ARMv8\n");\r
+      break;\r
+\r
     case 0x104:\r
       Print (L"SH-3\n");\r
       break;\r
@@ -1789,13 +2308,10 @@ DisplayProcessorFamily2 (
 \r
 }\r
 \r
-VOID\r
-DisplayProcessorVoltage (\r
-  UINT8 Voltage,\r
-  UINT8 Option\r
-  )\r
-/*++\r
-Routine Description:\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
@@ -1812,13 +2328,13 @@ Routine Description:
     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
+  @param[in] Option       The option.\r
 **/\r
+VOID\r
+DisplayProcessorVoltage (\r
+  IN UINT8 Voltage,\r
+  IN UINT8 Option\r
+  )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_PROC_INFO), gShellDebug1HiiHandle);\r
   //\r
@@ -1861,20 +2377,16 @@ Returns:
   }\r
 }\r
 \r
-VOID\r
-DisplayProcessorStatus (\r
-  UINT8 Status,\r
-  UINT8 Option\r
-  )\r
-/*++\r
-Routine Description:\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
+Bit 6 CPU Socket Populated\r
 1 - CPU Socket Populated\r
+  0 - CPU Socket Unpopulated\r
+Bits 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
@@ -1883,25 +2395,25 @@ Bit 6   CPU Socket Populated
   5-6h - Reserved\r
   7h - Other\r
 \r
-Arguments:\r
-  Status  - The status\r
-  Option  - The option\r
-\r
-Returns:\r
-\r
+  @param[in] Option   The option\r
 **/\r
+VOID\r
+DisplayProcessorStatus (\r
+  IN UINT8 Status,\r
+  IN UINT8 Option\r
+  )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_PROC_STATUS), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Status, Option);\r
 \r
   if (BIT (Status, 7) != 0) {\r
-    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ERROR_BIT7), gShellDebug1HiiHandle);\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIT7_NOT_ZERO), gShellDebug1HiiHandle);\r
   } else if (BIT (Status, 5) != 0) {\r
-    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ERROR_BIT5), gShellDebug1HiiHandle);\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIT5_NOT_ZERO), gShellDebug1HiiHandle);\r
   } else if (BIT (Status, 4) != 0) {\r
-    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ERROR_BIT4), gShellDebug1HiiHandle);\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIT4_NOT_ZERO), gShellDebug1HiiHandle);\r
   } else if (BIT (Status, 3) != 0) {\r
-    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ERROR_BIT3), gShellDebug1HiiHandle);\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIT3_NOT_ZERO), gShellDebug1HiiHandle);\r
   }\r
   //\r
   // Check BIT 6\r
@@ -1944,11 +2456,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 +2484,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 +2515,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 +2557,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 +2568,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 +2605,138 @@ Returns:
   }\r
 }\r
 \r
+/**\r
+  Display Cache Configuration.\r
+\r
+  @param[in] CacheConfiguration   Cache Configuration.\r
+Bits 15:10 Reserved, must be 0\r
+Bits 9:8 Operational Mode\r
+  0h - Write Through\r
+  1h - Write Back\r
+  2h - Varies with Memory Address\r
+  3h - Unknown\r
+Bit 7 Enabled/Disabled\r
+  1 - Enabled\r
+  0 - Disabled\r
+Bits 6:5 Location\r
+  0h - Internal\r
+  1h - External\r
+  2h - Reserved\r
+  3h - Unknown\r
+Bit 4 Reserved, must be zero\r
+Bit 3 Cache Socketed\r
+  1 - Socketed\r
+  0 - Unsocketed\r
+Bits 2:0 Cache Level\r
+  1 through 8 (For example, an L1 cache would\r
+  use value 000b and an L3 cache would use 010b.)\r
+\r
+  @param[in] Option   The option\r
+**/\r
 VOID\r
-DisplaySystemSlotId (\r
-  UINT16  SlotId,\r
-  UINT8   SlotType,\r
-  UINT8   Option\r
+DisplayCacheConfiguration (\r
+  IN UINT16 CacheConfiguration,\r
+  IN UINT8 Option\r
   )\r
-/*++\r
-Routine Description:\r
+{\r
+  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_CACHE_CONFIGURATION), gShellDebug1HiiHandle);\r
+  PRINT_INFO_OPTION (CacheConfiguration, Option);\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
+  if (BIT (CacheConfiguration, 15) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIT15_NOT_ZERO), gShellDebug1HiiHandle);\r
+  } else if (BIT (CacheConfiguration, 14) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIT14_NOT_ZERO), gShellDebug1HiiHandle);\r
+  } else if (BIT (CacheConfiguration, 13) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIT13_NOT_ZERO), gShellDebug1HiiHandle);\r
+  } else if (BIT (CacheConfiguration, 12) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIT12_NOT_ZERO), gShellDebug1HiiHandle);\r
+  } else if (BIT (CacheConfiguration, 11) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIT11_NOT_ZERO), gShellDebug1HiiHandle);\r
+  } else if (BIT (CacheConfiguration, 10) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIT10_NOT_ZERO), gShellDebug1HiiHandle);\r
+  } else if (BIT (CacheConfiguration, 4) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BIT4_NOT_ZERO), gShellDebug1HiiHandle);\r
+  }\r
+\r
+  //\r
+  // Check BITs 9:8\r
+  //\r
+  switch ((CacheConfiguration & 0x300) >> 8) {\r
+  case 0:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_CACHE_WRITE_THROUGH), gShellDebug1HiiHandle);\r
+    break;\r
+\r
+  case 1:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_CACHE_WRITE_BACK), gShellDebug1HiiHandle);\r
+    break;\r
+\r
+  case 2:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_CACHE_VARIES_WITH_MEM_ADDR), gShellDebug1HiiHandle);\r
+    break;\r
+\r
+  case 3:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_UNKNOWN), gShellDebug1HiiHandle);\r
+    break;\r
+  }\r
+\r
+  //\r
+  // Check BIT 7\r
+  //\r
+  if (BIT (CacheConfiguration, 7) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ENABLED), gShellDebug1HiiHandle);\r
+  } else {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_DISABLED), gShellDebug1HiiHandle);\r
+  }\r
+\r
+  //\r
+  // Check BITs 6:5\r
+  //\r
+  switch ((CacheConfiguration & 0x60) >> 5) {\r
+  case 0:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_CACHE_INTERNAL), gShellDebug1HiiHandle);\r
+    break;\r
+\r
+  case 1:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_CACHE_EXTERNAL), gShellDebug1HiiHandle);\r
+    break;\r
+\r
+  case 2:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_RESERVED), gShellDebug1HiiHandle);\r
+    break;\r
+\r
+  case 3:\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_UNKNOWN), gShellDebug1HiiHandle);\r
+    break;\r
+  }\r
+\r
+  //\r
+  // Check BIT 3\r
+  //\r
+  if (BIT (CacheConfiguration, 3) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_CACHE_SOCKETED), gShellDebug1HiiHandle);\r
+  } else {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_CACHE_NOT_SOCKETED), gShellDebug1HiiHandle);\r
+  }\r
 \r
-Arguments:\r
 \r
-  SlotId   - The slot ID\r
-  SlotType - The slot type\r
-  Option   - The Option\r
+  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_CACHE_LEVEL), gShellDebug1HiiHandle, (CacheConfiguration & 0x07) + 1);\r
+}\r
 \r
-Returns:\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
+  @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
@@ -2159,7 +2796,7 @@ Returns:
     break;\r
 \r
   default:\r
-    if (SlotType >= 0x0E && SlotType <= 0x12) {\r
+    if ((SlotType >= 0x0E && SlotType <= 0x12) || (SlotType >= 0xA6 && SlotType <= 0xB6)){\r
       ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_VALUE_PRESENT), gShellDebug1HiiHandle, SlotId);\r
     } else {\r
       ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_UNDEFINED_SLOT_ID), gShellDebug1HiiHandle);\r
@@ -2167,10 +2804,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
@@ -2204,22 +2847,16 @@ DisplaySystemBootStatus (
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_RSVD_FUTURE_ASSIGNMENT), gShellDebug1HiiHandle);\r
   } else if (Parameter >= 128 && Parameter <= 191) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_VENDOR_OEM_SPECIFIC), gShellDebug1HiiHandle);\r
-  } else if (Parameter >= 192 && Parameter <= 255) {\r
+  } else if (Parameter >= 192) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_PRODUCT_SPEC_IMPLMENTATION), gShellDebug1HiiHandle);\r
   } else {\r
     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 +2864,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
@@ -2245,21 +2883,16 @@ Returns:
   // Print date\r
   //\r
   Day   = Date & 0x001F;\r
-  Month = (Date & 0x00E0) >> 5;\r
-  Year  = ((Date & 0xFF00) >> 8) + 1980;\r
+  Month = (Date & 0x01E0) >> 5;\r
+  Year  = ((Date & 0xFE00) >> 9) + 1980;\r
   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
-VOID\r
-DisplaySystemResetCapabilities (\r
-  UINT8 Reset,\r
-  UINT8 Option\r
-  )\r
-/*++\r
-Routine Description:\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
@@ -2278,13 +2911,14 @@ Identifies the system-reset capabilities for the system.
   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
+  @param[in] Reset   Reset\r
+  @param[in] Option  The option\r
 **/\r
+VOID\r
+DisplaySystemResetCapabilities (\r
+  IN UINT8 Reset,\r
+  IN UINT8 Option\r
+  )\r
 {\r
   UINTN Temp;\r
 \r
@@ -2358,16 +2992,11 @@ Returns:
     Print (L"The system reset is disabled by the user\n");\r
   }\r
 }\r
-//\r
-// Hardware Security (Type 24)\r
-//\r
-VOID\r
-DisplayHardwareSecuritySettings (\r
-  UINT8 Settings,\r
-  UINT8 Option\r
-  )\r
-/*++\r
-Routine Description:\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
@@ -2391,13 +3020,14 @@ Bits 1:0    Front Panel Reset Status, one of:
   10b Not Implemented\r
   11b Unknown\r
 \r
-Arguments:\r
-  Settings    - The settings\r
-  Option      - the option\r
-\r
-Returns:\r
-\r
+  @param[in] Settings The device settings.\r
+  @param[in] Option   The device options.\r
 **/\r
+VOID\r
+DisplayHardwareSecuritySettings (\r
+  IN UINT8 Settings,\r
+  IN UINT8 Option\r
+  )\r
 {\r
   UINTN Temp;\r
 \r
@@ -2493,13 +3123,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 +3162,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
@@ -2552,7 +3190,7 @@ DisplaySPSCharacteristics (
   // Bits 13:10 - DMTF Power Supply Type\r
   //\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_TYPE), gShellDebug1HiiHandle);\r
-  Temp = (Characteristics & 0x1C00) << 10;\r
+  Temp = (Characteristics & 0x1C00) >> 10;\r
   switch (Temp) {\r
   case 1:\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_OTHER_SPACE), gShellDebug1HiiHandle);\r
@@ -2593,7 +3231,7 @@ DisplaySPSCharacteristics (
   // Bits 9:7 - Status\r
   //\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_STATUS_DASH), gShellDebug1HiiHandle);\r
-  Temp = (Characteristics & 0x380) << 7;\r
+  Temp = (Characteristics & 0x380) >> 7;\r
   switch (Temp) {\r
   case 1:\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_OTHER_SPACE), gShellDebug1HiiHandle);\r
@@ -2622,7 +3260,7 @@ DisplaySPSCharacteristics (
   // Bits 6:3 - DMTF Input Voltage Range Switching\r
   //\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_INPUT_VOLTAGE_RANGE), gShellDebug1HiiHandle);\r
-  Temp = (Characteristics & 0x78) << 3;\r
+  Temp = (Characteristics & 0x78) >> 3;\r
   switch (Temp) {\r
   case 1:\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_OTHER_SPACE), gShellDebug1HiiHandle);\r
@@ -2677,3 +3315,59 @@ DisplaySPSCharacteristics (
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_POWER_SUPPLY_NOT_REPLACE), gShellDebug1HiiHandle);\r
   }\r
 }\r
+\r
+/**\r
+  Display TPM Device (Type 43) Characteristics.\r
+\r
+  @param[in] Chara    The information bits.\r
+  @param[in] Option   The optional information.\r
+**/\r
+VOID\r
+DisplayTpmDeviceCharacteristics (\r
+  IN UINT64  Chara,\r
+  IN UINT8   Option\r
+  )\r
+{\r
+  //\r
+  // Print header\r
+  //\r
+  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_TPM_DEVICE_CHAR), gShellDebug1HiiHandle);\r
+  //\r
+  // print option\r
+  //\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
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_RESERVED_BIT), gShellDebug1HiiHandle);\r
+  }\r
+\r
+  if (BIT (Chara, 1) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_RESERVED_BIT), gShellDebug1HiiHandle);\r
+  }\r
+  if (BIT (Chara, 2) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_TPM_DEVICE_CHAR_NOT_SUPPORTED), gShellDebug1HiiHandle);\r
+  }\r
+\r
+  if (BIT (Chara, 3) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_TPM_DEVICE_CONFIG_FWU), gShellDebug1HiiHandle);\r
+  }\r
+\r
+  if (BIT (Chara, 4) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_TPM_DEVICE_CONFIG_PLAT_SW), gShellDebug1HiiHandle);\r
+  }\r
+\r
+  if (BIT (Chara, 5) != 0) {\r
+    ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_TPM_DEVICE_CONFIG_OEM), gShellDebug1HiiHandle);\r
+  }\r
+\r
+  //\r
+  // Just print the Reserved\r
+  //\r
+  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_BITS_06_63), gShellDebug1HiiHandle);\r
+\r
+}\r