]> git.proxmox.com Git - mirror_edk2.git/blobdiff - ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
ShellPkg: Clean up source files
[mirror_edk2.git] / ShellPkg / Library / UefiShellDebug1CommandsLib / SmbiosView / QueryTable.c
index 7b7e5928bd204923a96cbf3740e773bcffcc67a7..b56cd61d07fadea98559d1fad46a337e449d5027 100644 (file)
@@ -2,7 +2,8 @@
   Build a table, each item is (Key, Info) pair.\r
   And give a interface of query a string out of a table.\r
 \r
-  Copyright (c) 2005 - 2011, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2005 - 2018, Intel Corporation. All rights reserved.<BR>\r
+  (C) Copyright 2016-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
@@ -13,7 +14,7 @@
 \r
 **/\r
 \r
-#include "../UefiShellDebug1CommandsLib.h"\r
+#include "UefiShellDebug1CommandsLib.h"\r
 #include "QueryTable.h"\r
 #include "PrintInfo.h"\r
 \r
@@ -52,10 +53,88 @@ TABLE_ITEM  SystemWakeupTypeTable[] = {
   }\r
 };\r
 \r
+TABLE_ITEM  BaseBoardFeatureFlagsTable[] = {\r
+  {\r
+    0,\r
+    L" Hosting board"\r
+  },\r
+  {\r
+    1,\r
+    L" Requires at least one daughter board or auxiliary card"\r
+  },\r
+  {\r
+    2,\r
+    L" Removable"\r
+  },\r
+  {\r
+    3,\r
+    L" Replaceable"\r
+  },\r
+  {\r
+    4,\r
+    L" Hot swappable"\r
+  }\r
+};\r
+\r
+TABLE_ITEM  BaseBoardBoardTypeTable[] = {\r
+  {\r
+    0x01,\r
+    L" Unknown"\r
+  },\r
+  {\r
+    0x02,\r
+    L" Other"\r
+  },\r
+  {\r
+    0x03,\r
+    L" Server Blade"\r
+  },\r
+  {\r
+    0x04,\r
+    L" Connectivity Switch"\r
+  },\r
+  {\r
+    0x05,\r
+    L" System Management Module"\r
+  },\r
+  {\r
+    0x06,\r
+    L" Processor Module"\r
+  },\r
+  {\r
+    0x07,\r
+    L" I/O Module"\r
+  },\r
+  {\r
+    0x08,\r
+    L" Memory Module"\r
+  },\r
+  {\r
+    0x09,\r
+    L" Daughter board"\r
+  },\r
+  {\r
+    0x0A,\r
+    L" Motherboard"\r
+  },\r
+  {\r
+    0x0B,\r
+    L" Processor/Memory Module"\r
+  },\r
+  {\r
+    0x0C,\r
+    L" Processor/IO Module"\r
+  },\r
+  {\r
+    0x0D,\r
+    L" Interconnect Board"\r
+  }\r
+};\r
+\r
 TABLE_ITEM  SystemEnclosureTypeTable[] = {\r
   {\r
     0x01,\r
-    L"  None"\r
+    L"  Other"\r
   },\r
   {\r
     0x02,\r
@@ -87,7 +166,7 @@ TABLE_ITEM  SystemEnclosureTypeTable[] = {
   },\r
   {\r
     0x09,\r
-    L"  LapTop"\r
+    L"  Laptop"\r
   },\r
   {\r
     0x0A,\r
@@ -169,6 +248,34 @@ TABLE_ITEM  SystemEnclosureTypeTable[] = {
     0x1D,\r
     L"  Blade Enclosure"\r
   },\r
+  {\r
+    0x1E,\r
+    L"  Tablet"\r
+  },\r
+  {\r
+    0x1F,\r
+    L"  Convertible"\r
+  },\r
+  {\r
+    0x20,\r
+    L"  Detachable"\r
+  },\r
+  {\r
+    0x21,\r
+    L"  IoT Gateway"\r
+  },\r
+  {\r
+    0x22,\r
+    L"  Embedded PC"\r
+  },\r
+  {\r
+    0x23,\r
+    L"  Mini PC"\r
+  },\r
+  {\r
+    0x24,\r
+    L"  Stick PC"\r
+  },\r
 };\r
 \r
 TABLE_ITEM  SystemEnclosureStatusTable[] = {\r
@@ -348,9 +455,165 @@ TABLE_ITEM  ProcessorUpgradeTable[] = {
   {\r
     0x19,\r
     L"Socket LGA1366"\r
-  }\\r
+  },\r
+  {\r
+    0x1A,\r
+    L"Socket G34"\r
+  },\r
+  {\r
+    0x1B,\r
+    L"Socket AM3"\r
+  },\r
+  {\r
+    0x1C,\r
+    L"Socket C32"\r
+  },\r
+  {\r
+    0x1D,\r
+    L"Socket LGA1156"\r
+  },\r
+  {\r
+    0x1E,\r
+    L"Socket LGA1567"\r
+  },\r
+  {\r
+    0x1F,\r
+    L"Socket PGA988A"\r
+  },\r
+  {\r
+    0x20,\r
+    L"Socket BGA1288"\r
+  },\r
+  {\r
+    0x21,\r
+    L"Socket rPGA988B"\r
+  },\r
+  {\r
+    0x22,\r
+    L"Socket BGA1023"\r
+  },\r
+  {\r
+    0x23,\r
+    L"Socket BGA1224"\r
+  },\r
+  {\r
+    0x24,\r
+    L"Socket LGA1155"\r
+  },\r
+  {\r
+    0x25,\r
+    L"Socket LGA1356"\r
+  },\r
+  {\r
+    0x26,\r
+    L"Socket LGA2011"\r
+  },\r
+  {\r
+    0x27,\r
+    L"Socket FS1"\r
+  },\r
+  {\r
+    0x28,\r
+    L"Socket FS2"\r
+  },\r
+  {\r
+    0x29,\r
+    L"Socket FM1"\r
+  },\r
+  {\r
+    0x2A,\r
+    L"Socket FM2"\r
+  },\r
+  {\r
+    0x2B,\r
+    L"Socket LGA2011-3"\r
+  },\r
+  {\r
+    0x2C,\r
+    L"Socket LGA1356-3"\r
+  },\r
+  {\r
+    0x2D,\r
+    L"Socket LGA1150"\r
+  },\r
+  {\r
+    0x2E,\r
+    L"Socket BGA1168"\r
+  },\r
+  {\r
+    0x2F,\r
+    L"Socket BGA1234"\r
+  },\r
+  {\r
+    0x30,\r
+    L"Socket BGA1364"\r
+  },\r
+  {\r
+    0x31,\r
+    L"Socket AM4"\r
+  },\r
+  {\r
+    0x32,\r
+    L"Socket LGA1151"\r
+  },\r
+  {\r
+    0x33,\r
+    L"Socket BGA1356"\r
+  },\r
+  {\r
+    0x34,\r
+    L"Socket BGA1440"\r
+  },\r
+  {\r
+    0x35,\r
+    L"Socket BGA1515"\r
+  },\r
+  {\r
+    0x36,\r
+    L"Socket LGA3647-1"\r
+  },\r
+  {\r
+    0x37,\r
+    L"Socket SP3"\r
+  },\r
+  {\r
+    0x38,\r
+    L"Socket SP3r2"\r
+  }\r
+};\r
+\r
+TABLE_ITEM  ProcessorCharacteristicsTable[] = {\r
+  {\r
+    1,\r
+    L" Unknown"\r
+  },\r
+  {\r
+    2,\r
+    L" 64-bit Capable"\r
+  },\r
+  {\r
+    3,\r
+    L" Multi-Core"\r
+  },\r
+  {\r
+    4,\r
+    L" Hardware Thread"\r
+  },\r
+  {\r
+    5,\r
+    L" Execute Protection"\r
+  },\r
+  {\r
+    6,\r
+    L" Enhanced Virtualization"\r
+  },\r
+  {\r
+    7,\r
+    L" Power/Performance Control"\r
+  }\r
 };\r
 \r
+\r
 TABLE_ITEM  McErrorDetectMethodTable[] = {\r
   {\r
     0x01,\r
@@ -599,10 +862,6 @@ TABLE_ITEM  CacheErrCorrectingTypeTable[] = {
   {\r
     0x06,\r
     L"Multi-bit ECC"\r
-  },\r
-  {\r
-    0x07,\r
-    L"Sixteen Way Interleave"\r
   }\r
 };\r
 \r
@@ -681,6 +940,10 @@ TABLE_ITEM  CacheAssociativityTable[] = {
   {\r
     0x0D,\r
     L"64-way Set-Associative"\r
+  },\r
+  {\r
+    0x0E,\r
+    L"20-way Set-Associative"\r
   }\r
 };\r
 \r
@@ -1075,6 +1338,74 @@ TABLE_ITEM  SystemSlotTypeTable[] = {
     0x12,\r
     L"PCI-X"\r
   },\r
+  {\r
+    0x13,\r
+    L"AGP 8X"\r
+  },\r
+  {\r
+    0x14,\r
+    L"M.2 Socket 1-DP (Mechanical Key A)"\r
+  },\r
+  {\r
+    0x15,\r
+    L"M.2 Socket 1-SD (Mechanical Key E)"\r
+  },\r
+  {\r
+    0x16,\r
+    L"M.2 Socket 2 (Mechanical Key B)"\r
+  },\r
+  {\r
+    0x17,\r
+    L"M.2 Socket 3 (Mechanical Key M)"\r
+  },\r
+  {\r
+    0x18,\r
+    L"MXM Type I"\r
+  },\r
+  {\r
+    0x19,\r
+    L"MXM Type II"\r
+  },\r
+  {\r
+    0x1A,\r
+    L"MXM Type III (standard connector)"\r
+  },\r
+  {\r
+    0x1B,\r
+    L"MXM Type III (HE connector)"\r
+  },\r
+  {\r
+    0x1C,\r
+    L"MXM Type IV"\r
+  },\r
+  {\r
+    0x1D,\r
+    L"MXM 3.0 Type A"\r
+  },\r
+  {\r
+    0x1E,\r
+    L"MXM 3.0 Type B"\r
+  },\r
+  {\r
+    0x1F,\r
+    L"PCI Express Gen 2 SFF-8639"\r
+  },\r
+  {\r
+    0x20,\r
+    L"PCI Express Gen 3 SFF-8639"\r
+  },\r
+  {\r
+    0x21,\r
+    L"PCI Express Mini 52-pin (CEM spec. 2.0) with bottom-side keep-outs"\r
+  },\r
+  {\r
+    0x22,\r
+    L"PCI Express Mini 52-pin (CEM spec. 2.0) without bottom-side keep-outs"\r
+  },\r
+  {\r
+    0x23,\r
+    L"PCI Express Mini 76-pin (CEM spec. 2.0) Corresponds to Display-Mini card"\r
+  },\r
   {\r
     0xA0,\r
     L"PC-98/C20 "\r
@@ -1121,7 +1452,7 @@ TABLE_ITEM  SystemSlotTypeTable[] = {
   },\r
   {\r
     0xAB,\r
-    L"PCI Express Gen 26"\r
+    L"PCI Express Gen 2"\r
   },\r
   {\r
     0xAC,\r
@@ -1142,6 +1473,30 @@ TABLE_ITEM  SystemSlotTypeTable[] = {
   {\r
     0xB0,\r
     L"PCI Express Gen 2 X16"\r
+  },\r
+  {\r
+    0xB1,\r
+    L"PCI Express Gen 3"\r
+  },\r
+  {\r
+    0xB2,\r
+    L"PCI Express Gen 3 X1"\r
+  },\r
+  {\r
+    0xB3,\r
+    L"PCI Express Gen 3 X2"\r
+  },\r
+  {\r
+    0xB4,\r
+    L"PCI Express Gen 3 X4"\r
+  },\r
+  {\r
+    0xB5,\r
+    L"PCI Express Gen 3 X8"\r
+  },\r
+  {\r
+    0xB6,\r
+    L"PCI Express Gen 3 X16"\r
   }\r
 };\r
 \r
@@ -1174,16 +1529,44 @@ TABLE_ITEM  SystemSlotDataBusWidthTable[] = {
     0x07,\r
     L" 128 bit"\r
   },\r
-};\r
-\r
-TABLE_ITEM  SystemSlotCurrentUsageTable[] = {\r
   {\r
-    0x01,\r
-    L" Other"\r
+    0x08,\r
+    L" 1x or x1"\r
   },\r
   {\r
-    0x02,\r
-    L" Unknown"\r
+    0x09,\r
+    L" 2x or x2"\r
+  },\r
+  {\r
+    0x0A,\r
+    L" 4x or x4"\r
+  },\r
+  {\r
+    0x0B,\r
+    L" 8x or x8"\r
+  },\r
+  {\r
+    0x0C,\r
+    L" 12x or x12"\r
+  },\r
+  {\r
+    0x0D,\r
+    L" 16x or x16"\r
+  },\r
+  {\r
+    0x0E,\r
+    L" 32x or x32"\r
+  }\r
+};\r
+\r
+TABLE_ITEM  SystemSlotCurrentUsageTable[] = {\r
+  {\r
+    0x01,\r
+    L" Other"\r
+  },\r
+  {\r
+    0x02,\r
+    L" Unknown"\r
   },\r
   {\r
     0x03,\r
@@ -2065,6 +2448,26 @@ TABLE_ITEM  MemoryDeviceTypeTable[] = {
   {\r
     0x19,\r
     L"  FBD2"\r
+  },\r
+  {\r
+    0x1A,\r
+    L"  DDR4"\r
+  },\r
+  {\r
+    0x1B,\r
+    L"  LPDDR"\r
+  },\r
+  {\r
+    0x1C,\r
+    L"  LPDDR2"\r
+  },\r
+  {\r
+    0x1D,\r
+    L"  LPDDR3"\r
+  },\r
+  {\r
+    0x1E,\r
+    L"  LPDDR4"\r
   }\r
 };\r
 \r
@@ -2117,6 +2520,14 @@ TABLE_ITEM  MemoryDeviceTypeDetailTable[] = {
     12,\r
     L" Non-volatile"\r
   },\r
+  {\r
+    13,\r
+    L" Registered(Buffered)"\r
+  },\r
+  {\r
+    14,\r
+    L" Unbuffered(Unregistered)"\r
+  }\r
 };\r
 \r
 TABLE_ITEM  MemoryErrorTypeTable[] = {\r
@@ -2472,11 +2883,11 @@ TABLE_ITEM  CoolingDeviceTypeTable[] = {
     L" Integrated Refrigeration "\r
   },\r
   {\r
-    0x0A,\r
+    0x10,\r
     L" Active Cooling "\r
   },\r
   {\r
-    0x0B,\r
+    0x11,\r
     L" Passive Cooling "\r
   },\r
 };\r
@@ -2749,6 +3160,22 @@ TABLE_ITEM  IPMIDIBMCInterfaceTypeTable[] = {
   },\r
 };\r
 \r
+TABLE_ITEM  MCHostInterfaceTypeTable[] = {\r
+  {\r
+    0x3F00,\r
+    L" MCTP Host Interface "\r
+  },\r
+  {\r
+    0x40,\r
+    L" Network Host Interface "\r
+  },\r
+  {\r
+    0xF0,\r
+    L" OEM defined "\r
+  },\r
+};\r
+\r
+\r
 TABLE_ITEM  StructureTypeInfoTable[] = {\r
   {\r
     0,\r
@@ -2910,6 +3337,22 @@ TABLE_ITEM  StructureTypeInfoTable[] = {
     39,\r
     L" System Power Supply"\r
   },\r
+  {\r
+    40,\r
+    L" Additional Information"\r
+  },\r
+  {\r
+    41,\r
+    L" Onboard Devices Extended Information"\r
+  },\r
+  {\r
+    42,\r
+    L" Management Controller Host Interface"\r
+  },\r
+  {\r
+    43,\r
+    L" TPM Device"\r
+  },\r
   {\r
     0x7E,\r
     L" Inactive"\r
@@ -2936,11 +3379,11 @@ TABLE_ITEM  StructureTypeInfoTable[] = {
     Then all the Key Value between Low and High gets the same string\r
     L"Unused".\r
 \r
-  @param[in] Table    The begin address of table.\r
-  @param[in] Number   The number of table items.\r
-  @param[in] Key      The query Key.\r
-  @param[in,out] Info Input as empty buffer; output as data buffer.\r
-  @param[in] InfoLen  The max number of characters for Info.\r
+  @param[in] Table     The begin address of table.\r
+  @param[in] Number    The number of table items.\r
+  @param[in] Key       The query Key.\r
+  @param[in, out] Info Input as empty buffer; output as data buffer.\r
+  @param[in] InfoLen   The max number of characters for Info.\r
 \r
   @return the found Key and Info is valid.\r
   @retval QUERY_TABLE_UNFOUND and Info should be NULL.\r
@@ -2964,49 +3407,36 @@ QueryTable (
   for (Index = 0; Index < Number; Index++) {\r
     High  = (UINT8) (Table[Index].Key >> 8);\r
     Low   = (UINT8) (Table[Index].Key & 0x00FF);\r
+\r
     //\r
     // Check if Key is in the range\r
+    // or if Key == Value in the table\r
     //\r
-    if (High > Low && Key >= Low && Key <= High) {\r
-      StrnCpy (Info, Table[Index].Info, InfoLen-1);\r
-      StrCat (Info, L"\n");\r
-      return Key;\r
-    }\r
-    //\r
-    // Check if Key == Value in the table\r
-    //\r
-    if (Table[Index].Key == Key) {\r
-      StrnCpy (Info, Table[Index].Info, InfoLen-1);\r
-      StrCat (Info, L"\n");\r
+    if ((High > Low && Key >= Low && Key <= High)\r
+      || (Table[Index].Key == Key)) {\r
+      StrnCpyS (Info, InfoLen, Table[Index].Info, InfoLen - 1);\r
+      StrnCatS (Info, InfoLen, L"\n", InfoLen - 1 - StrLen(Info));\r
       return Key;\r
     }\r
   }\r
 \r
-  StrnCpy (Info, L"Undefined Value\n", InfoLen);\r
+  StrCpyS (Info, InfoLen, L"Undefined Value\n");\r
   return QUERY_TABLE_UNFOUND;\r
 }\r
 \r
+/**\r
+  Given a table of bit info and a Key, return the responding info to the Key.\r
+\r
+  @param[in] Table     Point to a table which maintains a map of 'bit' to 'message'.\r
+  @param[in] Number    Number of table items.\r
+  @param[in] Bits      The Key of query the bit map information.\r
+**/\r
 VOID\r
 PrintBitsInfo (\r
   IN  TABLE_ITEM    *Table,\r
   IN  UINTN         Number,\r
   IN  UINT32        Bits\r
   )\r
-/*++\r
-\r
-  Routine Description:\r
-      Given a table of bit info and a Key,\r
-      return the responding info to the Key.\r
-\r
-  Arguments:\r
-    Table     - Point to a table which maintains a map of 'bit' to 'message'\r
-    Number    - Number of table items.\r
-    Bits      - The Key of query the bit map information.\r
-\r
-  Returns:\r
-    None\r
-\r
-**/\r
 {\r
   //\r
   // Get certain bit of 'Value':\r
@@ -3019,19 +3449,24 @@ PrintBitsInfo (
 \r
   UINTN   Index;\r
   UINT32  Value;\r
-  BOOLEAN NoInfo;\r
+  BOOLEAN FirstInfo;\r
 \r
-  NoInfo  = TRUE;\r
+  FirstInfo = TRUE;\r
   Value   = Bits;\r
   //\r
   // query the table and print information\r
   //\r
   for (Index = 0; Index < Number; Index++) {\r
     if (BIT (Value, Table[Index].Key) != 0) {\r
+      if (!FirstInfo) {\r
+        //\r
+        // If it is not first info, print the separator first.\r
+        //\r
+        Print (L" | ");\r
+      }\r
       Print (Table[Index].Info);\r
-      Print (L" | ");\r
 \r
-      NoInfo = FALSE;\r
+      FirstInfo = FALSE;\r
       //\r
       // clear the bit, for reserved bits test\r
       //\r
@@ -3039,7 +3474,10 @@ PrintBitsInfo (
     }\r
   }\r
 \r
-  if (NoInfo) {\r
+  //\r
+  // There is no any info if FirstInfo is still TRUE.\r
+  //\r
+  if (FirstInfo) {\r
     ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_NO_INFO), gShellDebug1HiiHandle);\r
   }\r
 \r
@@ -3077,28 +3515,67 @@ PrintBitsInfo (
     PrintBitsInfo (Table, Num, (UINT32) bits); \\r
   } while (0);\r
 \r
-//\r
-////////////////////////////////////////////////////////////////////\r
-//\r
-// System Information (Type 1)\r
-//\r
+/**\r
+  Display System Information (Type 1) Type.\r
+\r
+  @param[in] Type           The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplaySystemWakeupType (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_SYSTEM_WAKEUP_TYPE), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Type, Option);\r
   PRINT_TABLE_ITEM (SystemWakeupTypeTable, Type);\r
 }\r
-//\r
-// System Enclosure (Type 3)\r
-//\r
+\r
+/**\r
+  Display Base Board (Type 2) Feature Flags.\r
+\r
+  @param[in] FeatureFlags   The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
+VOID\r
+DisplayBaseBoardFeatureFlags (\r
+  IN UINT8 FeatureFlags,\r
+  IN UINT8 Option\r
+  )\r
+{\r
+  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_BASE_BOARD_FEATURE_FLAGS), gShellDebug1HiiHandle);\r
+  PRINT_INFO_OPTION (FeatureFlags, Option);\r
+  PRINT_BITS_INFO (BaseBoardFeatureFlagsTable, FeatureFlags);\r
+}\r
+\r
+/**\r
+  Display Base Board (Type 2) Board Type.\r
+\r
+  @param[in] Type           The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
+VOID\r
+DisplayBaseBoardBoardType(\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
+  )\r
+{\r
+  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_BASE_BOARD_BOARD_TYPE), gShellDebug1HiiHandle);\r
+  PRINT_INFO_OPTION (Type, Option);\r
+  PRINT_TABLE_ITEM (BaseBoardBoardTypeTable, Type);\r
+}\r
+\r
+/**\r
+  Display System Enclosure (Type 3) Enclosure Type.\r
+\r
+  @param[in] Type           The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplaySystemEnclosureType (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_SYSTEM_CHASSIS_TYPE), gShellDebug1HiiHandle);\r
@@ -3113,10 +3590,16 @@ DisplaySystemEnclosureType (
   }\r
 }\r
 \r
+/**\r
+  Display System Enclosure (Type 3) Enclosure Status.\r
+\r
+  @param[in] Status         The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplaySystemEnclosureStatus (\r
-  UINT8 Status,\r
-  UINT8 Option\r
+  IN UINT8 Status,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_SYSTEM_CHASSIS_STATUS), gShellDebug1HiiHandle);\r
@@ -3124,23 +3607,33 @@ DisplaySystemEnclosureStatus (
   PRINT_TABLE_ITEM (SystemEnclosureStatusTable, Status);\r
 }\r
 \r
+/**\r
+  Display System Enclosure (Type 3) Security Status.\r
+\r
+  @param[in] Status         The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplaySESecurityStatus (\r
-  UINT8 Status,\r
-  UINT8 Option\r
+  IN UINT8 Status,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_SYSTEM_CHASSIS_SECURITY), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Status, Option);\r
   PRINT_TABLE_ITEM (SESecurityStatusTable, Status);\r
 }\r
-//\r
-// Processor Information (Type 4)\r
-//\r
+\r
+/**\r
+  Display Processor Information (Type 4) Type.\r
+\r
+  @param[in] Type           The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplayProcessorType (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_PROC_TYPE), gShellDebug1HiiHandle);\r
@@ -3148,23 +3641,50 @@ DisplayProcessorType (
   PRINT_TABLE_ITEM (ProcessorTypeTable, Type);\r
 }\r
 \r
+/**\r
+  Display Processor Information (Type 4) Upgrade.\r
+\r
+  @param[in] Upgrade        The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplayProcessorUpgrade (\r
-  UINT8 Upgrade,\r
-  UINT8 Option\r
+  IN UINT8 Upgrade,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_PROC_UPDATE), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Upgrade, Option);\r
   PRINT_TABLE_ITEM (ProcessorUpgradeTable, Upgrade);\r
 }\r
-//\r
-// Memory Controller Information (Type 5)\r
-//\r
+\r
+/**\r
+  Display Processor Information (Type 4) Characteristics.\r
+\r
+  @param[in] Type           The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
+VOID\r
+DisplayProcessorCharacteristics (\r
+  IN UINT16 Type,\r
+  IN UINT8 Option\r
+  )\r
+{\r
+  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_PROC_CHARACTERISTICS), gShellDebug1HiiHandle);\r
+  PRINT_INFO_OPTION (Type, Option);\r
+  PRINT_BITS_INFO (ProcessorCharacteristicsTable, Type);\r
+}\r
+\r
+/**\r
+  Display Memory Controller Information (Type 5) method.\r
+\r
+  @param[in] Method         The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplayMcErrorDetectMethod (\r
-  UINT8 Method,\r
-  UINT8 Option\r
+  IN UINT8 Method,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_DETECTMETHOD), gShellDebug1HiiHandle);\r
@@ -3172,10 +3692,16 @@ DisplayMcErrorDetectMethod (
   PRINT_TABLE_ITEM (McErrorDetectMethodTable, Method);\r
 }\r
 \r
+/**\r
+  Display Memory Controller Information (Type 5) Capability.\r
+\r
+  @param[in] Capability     The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplayMcErrorCorrectCapability (\r
-  UINT8 Capability,\r
-  UINT8 Option\r
+  IN UINT8 Capability,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_CORRECT_CAPABILITY), gShellDebug1HiiHandle);\r
@@ -3183,10 +3709,16 @@ DisplayMcErrorCorrectCapability (
   PRINT_BITS_INFO (McErrorCorrectCapabilityTable, Capability);\r
 }\r
 \r
+/**\r
+  Display Memory Controller Information (Type 5) Support.\r
+\r
+  @param[in] Support        The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplayMcInterleaveSupport (\r
-  UINT8 Support,\r
-  UINT8 Option\r
+  IN UINT8 Support,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_INTERLEAVE_SUPPORT), gShellDebug1HiiHandle);\r
@@ -3194,10 +3726,16 @@ DisplayMcInterleaveSupport (
   PRINT_TABLE_ITEM (McInterleaveSupportTable, Support);\r
 }\r
 \r
+/**\r
+  Display Memory Controller Information (Type 5) speeds.\r
+\r
+  @param[in] Speed          The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplayMcMemorySpeeds (\r
-  UINT16  Speed,\r
-  UINT8   Option\r
+  IN UINT16  Speed,\r
+  IN UINT8   Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_MEMORY_SPEED), gShellDebug1HiiHandle);\r
@@ -3205,23 +3743,33 @@ DisplayMcMemorySpeeds (
   PRINT_BITS_INFO (McMemorySpeedsTable, Speed);\r
 }\r
 \r
+/**\r
+  Display Memory Controller Information (Type 5) voltage.\r
+\r
+  @param[in] Voltage        The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplayMemoryModuleVoltage (\r
-  UINT8 Voltage,\r
-  UINT8 Option\r
+  IN UINT8 Voltage,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_REQUIRED_VOLTAGES), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Voltage, Option);\r
   PRINT_BITS_INFO (MemoryModuleVoltageTable, Voltage);\r
 }\r
-//\r
-// Memory Module Information (Type 6)\r
-//\r
+\r
+/**\r
+  Display Memory Module Information (Type 6) type.\r
+\r
+  @param[in] Type           The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplayMmMemoryType (\r
-  UINT16  Type,\r
-  UINT8   Option\r
+  IN UINT16  Type,\r
+  IN UINT8   Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_MODULE_TYPE), gShellDebug1HiiHandle);\r
@@ -3229,23 +3777,33 @@ DisplayMmMemoryType (
   PRINT_BITS_INFO (MmMemoryTypeTable, Type);\r
 }\r
 \r
+/**\r
+  Display Memory Module Information (Type 6) status.\r
+\r
+  @param[in] Status         The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplayMmErrorStatus (\r
-  UINT8 Status,\r
-  UINT8 Option\r
+  IN UINT8 Status,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_MODULE_ERROR_STATUS), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Status, Option);\r
   PRINT_BITS_INFO (MmErrorStatusTable, Status);\r
 }\r
-//\r
-// Cache Information (Type 7)\r
-//\r
+\r
+/**\r
+  Display Cache Information (Type 7) SRAM Type.\r
+\r
+  @param[in] Type           The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplayCacheSRAMType (\r
-  UINT16  Type,\r
-  UINT8   Option\r
+  IN UINT16  Type,\r
+  IN UINT8   Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_CACHE_SRAM_TYPE), gShellDebug1HiiHandle);\r
@@ -3253,10 +3811,16 @@ DisplayCacheSRAMType (
   PRINT_BITS_INFO (CacheSRAMTypeTable, (UINT8) Type);\r
 }\r
 \r
+/**\r
+  Display Cache Information (Type 7) correcting Type.\r
+\r
+  @param[in] Type           The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplayCacheErrCorrectingType (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_CACHE_ERROR_CORRECTING), gShellDebug1HiiHandle);\r
@@ -3264,10 +3828,16 @@ DisplayCacheErrCorrectingType (
   PRINT_TABLE_ITEM (CacheErrCorrectingTypeTable, Type);\r
 }\r
 \r
+/**\r
+  Display Cache Information (Type 7) Type.\r
+\r
+  @param[in] Type           The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplayCacheSystemCacheType (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_CACHE_SYSTEM_TYPE), gShellDebug1HiiHandle);\r
@@ -3275,23 +3845,33 @@ DisplayCacheSystemCacheType (
   PRINT_TABLE_ITEM (CacheSystemCacheTypeTable, Type);\r
 }\r
 \r
+/**\r
+  Display Cache Information (Type 7) Associativity.\r
+\r
+  @param[in] Associativity  The key of the structure.\r
+  @param[in] Option         The optional information.\r
+**/\r
 VOID\r
 DisplayCacheAssociativity (\r
-  UINT8 Associativity,\r
-  UINT8 Option\r
+  IN UINT8 Associativity,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_CACHE_ASSOCIATIVITY), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Associativity, Option);\r
   PRINT_TABLE_ITEM (CacheAssociativityTable, Associativity);\r
 }\r
-//\r
-// Port Connector Information (Type 8)\r
-//\r
+\r
+/**\r
+  Display Port Connector Information  (Type 8) type.\r
+\r
+  @param[in] Type       The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplayPortConnectorType (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_PORT_CONNECTOR_TYPE), gShellDebug1HiiHandle);\r
@@ -3299,23 +3879,33 @@ DisplayPortConnectorType (
   PRINT_TABLE_ITEM (PortConnectorTypeTable, Type);\r
 }\r
 \r
+/**\r
+  Display Port Connector Information  (Type 8) port type.\r
+\r
+  @param[in] Type       The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplayPortType (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_PORT_TYPE), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Type, Option);\r
   PRINT_TABLE_ITEM (PortTypeTable, Type);\r
 }\r
-//\r
-// System Slots (Type 9)\r
-//\r
+\r
+/**\r
+  Display System Slots (Type 9) slot type.\r
+\r
+  @param[in] Type       The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplaySystemSlotType (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_SYSTEM_SLOT_TYPE), gShellDebug1HiiHandle);\r
@@ -3323,10 +3913,16 @@ DisplaySystemSlotType (
   PRINT_TABLE_ITEM (SystemSlotTypeTable, Type);\r
 }\r
 \r
+/**\r
+  Display System Slots (Type 9) data bus width.\r
+\r
+  @param[in] Width      The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplaySystemSlotDataBusWidth (\r
-  UINT8 Width,\r
-  UINT8 Option\r
+  IN UINT8 Width,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_SYSTEM_SLOT_DATA), gShellDebug1HiiHandle);\r
@@ -3334,10 +3930,16 @@ DisplaySystemSlotDataBusWidth (
   PRINT_TABLE_ITEM (SystemSlotDataBusWidthTable, Width);\r
 }\r
 \r
+/**\r
+  Display System Slots (Type 9) usage information.\r
+\r
+  @param[in] Usage      The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplaySystemSlotCurrentUsage (\r
-  UINT8 Usage,\r
-  UINT8 Option\r
+  IN UINT8 Usage,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_SYSTEM_SLOT_CURRENT_USAGE), gShellDebug1HiiHandle);\r
@@ -3345,10 +3947,16 @@ DisplaySystemSlotCurrentUsage (
   PRINT_TABLE_ITEM (SystemSlotCurrentUsageTable, Usage);\r
 }\r
 \r
+/**\r
+  Display System Slots (Type 9) slot length.\r
+\r
+  @param[in] Length     The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplaySystemSlotLength (\r
-  UINT8 Length,\r
-  UINT8 Option\r
+  IN UINT8 Length,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_SYSTEM_SLOT_LENGTH), gShellDebug1HiiHandle);\r
@@ -3356,10 +3964,16 @@ DisplaySystemSlotLength (
   PRINT_TABLE_ITEM (SystemSlotLengthTable, Length);\r
 }\r
 \r
+/**\r
+  Display System Slots (Type 9) characteristics.\r
+\r
+  @param[in] Chara1     The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplaySlotCharacteristics1 (\r
-  UINT8 Chara1,\r
-  UINT8 Option\r
+  IN UINT8 Chara1,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_SLOT_CHARACTERISTICS), gShellDebug1HiiHandle);\r
@@ -3367,36 +3981,50 @@ DisplaySlotCharacteristics1 (
   PRINT_BITS_INFO (SlotCharacteristics1Table, Chara1);\r
 }\r
 \r
+/**\r
+  Display System Slots (Type 9) characteristics.\r
+\r
+  @param[in] Chara2     The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplaySlotCharacteristics2 (\r
-  UINT8 Chara2,\r
-  UINT8 Option\r
+  IN UINT8 Chara2,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_SLOT_CHARACTERISTICS_2), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Chara2, Option);\r
   PRINT_BITS_INFO (SlotCharacteristics2Table, Chara2);\r
 }\r
-//\r
-// On Board Devices Information (Type 10)\r
-//\r
+\r
+/**\r
+  Display On Board Devices Information (Type 10) types.\r
+\r
+  @param[in] Type       The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplayOnboardDeviceTypes (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_ONBOARD_DEVICE_TYPE), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Type, Option);\r
   PRINT_TABLE_ITEM (OnboardDeviceTypesTable, Type);\r
 }\r
-//\r
-// System Event Log (Type 15)\r
-//\r
+\r
+/**\r
+  Display System Event Log (Type 15) types.\r
+\r
+  @param[in] Type       The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplaySELTypes (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_SYSTEM_EVENT_LOG_TYPE), gShellDebug1HiiHandle);\r
@@ -3404,10 +4032,16 @@ DisplaySELTypes (
   PRINT_TABLE_ITEM (SELTypesTable, Type);\r
 }\r
 \r
+/**\r
+  Display System Event Log (Type 15) format type.\r
+\r
+  @param[in] Type       The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplaySELVarDataFormatType (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_EVENT_LOG_VAR_DATA_FORMAT), gShellDebug1HiiHandle);\r
@@ -3415,10 +4049,16 @@ DisplaySELVarDataFormatType (
   PRINT_TABLE_ITEM (SELVarDataFormatTypeTable, Type);\r
 }\r
 \r
+/**\r
+  Display System Event Log (Type 15) dw1.\r
+\r
+  @param[in] Key        The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplayPostResultsBitmapDw1 (\r
-  UINT32  Key,\r
-  UINT8   Option\r
+  IN UINT32  Key,\r
+  IN UINT8   Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_POST_RESULTS_BITMAP), gShellDebug1HiiHandle);\r
@@ -3426,10 +4066,16 @@ DisplayPostResultsBitmapDw1 (
   PRINT_BITS_INFO (PostResultsBitmapDw1Table, Key);\r
 }\r
 \r
+/**\r
+  Display System Event Log (Type 15) dw2.\r
+\r
+  @param[in] Key        The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplayPostResultsBitmapDw2 (\r
-  UINT32  Key,\r
-  UINT8   Option\r
+  IN UINT32  Key,\r
+  IN UINT8   Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_POST_RESULTS_SECOND_DWORD), gShellDebug1HiiHandle);\r
@@ -3437,10 +4083,16 @@ DisplayPostResultsBitmapDw2 (
   PRINT_BITS_INFO (PostResultsBitmapDw2Table, Key);\r
 }\r
 \r
+/**\r
+  Display System Event Log (Type 15) type.\r
+\r
+  @param[in] SMType     The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplaySELSysManagementTypes (\r
-  UINT32  SMType,\r
-  UINT8   Option\r
+  IN UINT32  SMType,\r
+  IN UINT8   Option\r
   )\r
 {\r
   UINT8       Temp;\r
@@ -3467,13 +4119,17 @@ DisplaySELSysManagementTypes (
     PRINT_TABLE_ITEM (SELSysManagementTypesTable, Temp);\r
   }\r
 }\r
-//\r
-// Physical Memory Array (Type 16)\r
-//\r
+\r
+/**\r
+  Display Physical Memory Array (Type 16) Location.\r
+\r
+  @param[in] Location   The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplayPMALocation (\r
-  UINT8 Location,\r
-  UINT8 Option\r
+  IN UINT8 Location,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_PHYS_MEM_ARRAY_LOCATION), gShellDebug1HiiHandle);\r
@@ -3481,34 +4137,50 @@ DisplayPMALocation (
   PRINT_TABLE_ITEM (PMALocationTable, Location);\r
 }\r
 \r
+/**\r
+  Display Physical Memory Array (Type 16) Use.\r
+\r
+  @param[in] Use        The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplayPMAUse (\r
-  UINT8 Use,\r
-  UINT8 Option\r
+  IN UINT8 Use,\r
+  IN UINT8 Option\r
   )\r
 {\r
-  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_PHYS_MEM_ARRAY_LOCATION), gShellDebug1HiiHandle);\r
+  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_PHYS_MEM_ARRAY_USE), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Use, Option);\r
   PRINT_TABLE_ITEM (PMAUseTable, Use);\r
 }\r
 \r
+/**\r
+  Display Physical Memory Array (Type 16) Types.\r
+\r
+  @param[in] Type       The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplayPMAErrorCorrectionTypes (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_PHYS_MEM_ARRAY_ERROR), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Type, Option);\r
   PRINT_TABLE_ITEM (PMAErrorCorrectionTypesTable, Type);\r
 }\r
-//\r
-// Memory Device (Type 17)\r
-//\r
+\r
+/**\r
+  Display Memory Device (Type 17) form factor.\r
+\r
+  @param[in] FormFactor The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplayMemoryDeviceFormFactor (\r
-  UINT8 FormFactor,\r
-  UINT8 Option\r
+  IN UINT8 FormFactor,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_DEVICE_FORM_FACTOR), gShellDebug1HiiHandle);\r
@@ -3516,10 +4188,16 @@ DisplayMemoryDeviceFormFactor (
   PRINT_TABLE_ITEM (MemoryDeviceFormFactorTable, FormFactor);\r
 }\r
 \r
+/**\r
+  Display Memory Device (Type 17) type.\r
+\r
+  @param[in] Type     The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayMemoryDeviceType (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_DEVICE_TYPE), gShellDebug1HiiHandle);\r
@@ -3527,23 +4205,33 @@ DisplayMemoryDeviceType (
   PRINT_TABLE_ITEM (MemoryDeviceTypeTable, Type);\r
 }\r
 \r
+/**\r
+  Display Memory Device (Type 17) details.\r
+\r
+  @param[in] Para     The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayMemoryDeviceTypeDetail (\r
-  UINT16  para,\r
-  UINT8   Option\r
+  IN UINT16  Para,\r
+  IN UINT8   Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_DEVICE_TYPE_DETAIL), gShellDebug1HiiHandle);\r
-  PRINT_INFO_OPTION (para, Option);\r
-  PRINT_BITS_INFO (MemoryDeviceTypeDetailTable, para);\r
+  PRINT_INFO_OPTION (Para, Option);\r
+  PRINT_BITS_INFO (MemoryDeviceTypeDetailTable, Para);\r
 }\r
-//\r
-// 32-bit Memory Error Information (Type 18)\r
-//\r
+\r
+/**\r
+  Display 32-bit Memory Error Information (Type 18) type.\r
+\r
+  @param[in] ErrorType  The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplayMemoryErrorType (\r
-  UINT8 ErrorType,\r
-  UINT8 Option\r
+  IN UINT8 ErrorType,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_ERROR_INFO), gShellDebug1HiiHandle);\r
@@ -3551,10 +4239,16 @@ DisplayMemoryErrorType (
   PRINT_TABLE_ITEM (MemoryErrorTypeTable, ErrorType);\r
 }\r
 \r
+/**\r
+  Display 32-bit Memory Error Information (Type 18) error granularity.\r
+\r
+  @param[in] Granularity  The key of the structure.\r
+  @param[in] Option       The optional information.\r
+**/\r
 VOID\r
 DisplayMemoryErrorGranularity (\r
-  UINT8 Granularity,\r
-  UINT8 Option\r
+  IN UINT8 Granularity,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_ERROR_GRANULARITY), gShellDebug1HiiHandle);\r
@@ -3562,23 +4256,33 @@ DisplayMemoryErrorGranularity (
   PRINT_TABLE_ITEM (MemoryErrorGranularityTable, Granularity);\r
 }\r
 \r
+/**\r
+  Display 32-bit Memory Error Information (Type 18) error information.\r
+\r
+  @param[in] Operation  The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplayMemoryErrorOperation (\r
-  UINT8 Operation,\r
-  UINT8 Option\r
+  IN UINT8 Operation,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_ERROR_OP), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Operation, Option);\r
   PRINT_TABLE_ITEM (MemoryErrorOperationTable, Operation);\r
 }\r
-//\r
-// Built-in Pointing Device (Type 21)\r
-//\r
+\r
+/**\r
+  Display Built-in Pointing Device (Type 21) type information.\r
+\r
+  @param[in] Type     The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayPointingDeviceType (\r
-  UINT8 Type,\r
-  UINT8 Option\r
+  IN UINT8 Type,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_POINTING_DEVICE_TYPE), gShellDebug1HiiHandle);\r
@@ -3586,36 +4290,50 @@ DisplayPointingDeviceType (
   PRINT_TABLE_ITEM (PointingDeviceTypeTable, Type);\r
 }\r
 \r
+/**\r
+  Display Built-in Pointing Device (Type 21) information.\r
+\r
+  @param[in] Interface  The key of the structure.\r
+  @param[in] Option     The optional information.\r
+**/\r
 VOID\r
 DisplayPointingDeviceInterface (\r
-  UINT8   Interface,\r
-  UINT8   Option\r
+  IN UINT8   Interface,\r
+  IN UINT8   Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_POINTING_DEVICE_INTERFACE), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Interface, Option);\r
   PRINT_TABLE_ITEM (PointingDeviceInterfaceTable, Interface);\r
 }\r
-//\r
-// Portable Battery (Type 22)\r
-//\r
+\r
+/**\r
+  Display Portable Battery  (Type 22) information.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayPBDeviceChemistry (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_PORTABLE_BATT_DEV_CHEM), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Key, Option);\r
   PRINT_TABLE_ITEM (PBDeviceChemistryTable, Key);\r
 }\r
-//\r
-// Voltage Probe (Type 26)\r
-//\r
+\r
+/**\r
+  Display Voltage Probe (Type 26) location information.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayVPLocation (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   UINT8       Loc;\r
@@ -3626,10 +4344,16 @@ DisplayVPLocation (
   PRINT_TABLE_ITEM (VPLocationTable, Loc);\r
 }\r
 \r
+/**\r
+  Display Voltage Probe (Type 26) status ype information.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayVPStatus (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   UINT8       Status;\r
@@ -3639,13 +4363,17 @@ DisplayVPStatus (
   PRINT_INFO_OPTION (Status, Option);\r
   PRINT_TABLE_ITEM (VPStatusTable, Status);\r
 }\r
-//\r
-// Voltage Probe (Type 27)\r
-//\r
+\r
+/**\r
+  Display Cooling (Type 27) status information.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayCoolingDeviceStatus (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   UINT8       Status;\r
@@ -3656,10 +4384,16 @@ DisplayCoolingDeviceStatus (
   PRINT_TABLE_ITEM (CoolingDeviceStatusTable, Status);\r
 }\r
 \r
+/**\r
+  Display Cooling (Type 27) type information.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayCoolingDeviceType (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   UINT8       Type;\r
@@ -3669,13 +4403,17 @@ DisplayCoolingDeviceType (
   PRINT_INFO_OPTION (Type, Option);\r
   PRINT_TABLE_ITEM (CoolingDeviceTypeTable, Type);\r
 }\r
-//\r
-// Temperature Probe  (Type 28)\r
-//\r
+\r
+/**\r
+  Display Temperature Probe (Type 28) status information.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayTemperatureProbeStatus (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   UINT8       Status;\r
@@ -3686,10 +4424,16 @@ DisplayTemperatureProbeStatus (
   PRINT_TABLE_ITEM (TemperatureProbeStatusTable, Status);\r
 }\r
 \r
+/**\r
+  Display Temperature Probe  (Type 28) location information.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayTemperatureProbeLoc (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   UINT8       Loc;\r
@@ -3699,13 +4443,17 @@ DisplayTemperatureProbeLoc (
   PRINT_INFO_OPTION (Loc, Option);\r
   PRINT_TABLE_ITEM (TemperatureProbeLocTable, Loc);\r
 }\r
-//\r
-// Electrical Current Probe (Type 29)\r
-//\r
+\r
+/**\r
+  Display Electrical Current Probe (Type 29)  status information.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayECPStatus (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   UINT8       Status;\r
@@ -3716,10 +4464,16 @@ DisplayECPStatus (
   PRINT_TABLE_ITEM (ECPStatusTable, Status);\r
 }\r
 \r
+/**\r
+  Display Type 29 information.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayECPLoc (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   UINT8       Loc;\r
@@ -3729,13 +4483,17 @@ DisplayECPLoc (
   PRINT_INFO_OPTION (Loc, Option);\r
   PRINT_TABLE_ITEM (ECPLocTable, Loc);\r
 }\r
-//\r
-// Management Device (Type 34)\r
-//\r
+\r
+/**\r
+  Display Management Device (Type 34) Type.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayMDType (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MANAGEMENT_DEV_TYPE), gShellDebug1HiiHandle);\r
@@ -3743,36 +4501,50 @@ DisplayMDType (
   PRINT_TABLE_ITEM (MDTypeTable, Key);\r
 }\r
 \r
+/**\r
+  Display Management Device (Type 34) Address Type.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayMDAddressType (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MANAGEMENT_DEV_ADDR_TYPE), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Key, Option);\r
   PRINT_TABLE_ITEM (MDAddressTypeTable, Key);\r
 }\r
-//\r
-// Memory Channel (Type 37)\r
-//\r
+\r
+/**\r
+  Display Memory Channel (Type 37) information.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayMemoryChannelType (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MEM_CHANNEL_TYPE), gShellDebug1HiiHandle);\r
   PRINT_INFO_OPTION (Key, Option);\r
   PRINT_TABLE_ITEM (MemoryChannelTypeTable, Key);\r
 }\r
-//\r
-// IPMI Device Information (Type 38)\r
-//\r
+\r
+/**\r
+  Display IPMI Device Information (Type 38) information.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayIPMIDIBMCInterfaceType (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_BMC_INTERFACE_TYPE), gShellDebug1HiiHandle);\r
@@ -3780,10 +4552,33 @@ DisplayIPMIDIBMCInterfaceType (
   PRINT_TABLE_ITEM (IPMIDIBMCInterfaceTypeTable, Key);\r
 }\r
 \r
+/**\r
+  Display Management Controller Host Interface (Type 42) information.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
+VOID\r
+DisplayMCHostInterfaceType (\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
+  )\r
+{\r
+  ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN (STR_SMBIOSVIEW_QUERYTABLE_MC_HOST_INTERFACE_TYPE), gShellDebug1HiiHandle);\r
+  PRINT_INFO_OPTION (Key, Option);\r
+  PRINT_TABLE_ITEM (MCHostInterfaceTypeTable, Key);\r
+}\r
+\r
+/**\r
+  Display the structure type information.\r
+\r
+  @param[in] Key      The key of the structure.\r
+  @param[in] Option   The optional information.\r
+**/\r
 VOID\r
 DisplayStructureTypeInfo (\r
-  UINT8 Key,\r
-  UINT8 Option\r
+  IN UINT8 Key,\r
+  IN UINT8 Option\r
   )\r
 {\r
   //\r