]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdePkg/Library/BaseSmbusLib/SmbusLib.c
*BaseSmbusLib: (new version)
[mirror_edk2.git] / MdePkg / Library / BaseSmbusLib / SmbusLib.c
index 5aad978836af3c5e3129e4a25cc5990672ebde03..29c09e730e7728b210040a56cdfb0f94b93b7433 100644 (file)
   THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
   WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 \r
-  Module Name:  SmbusLib.h\r
+  Module Name:  SmbusLib.c\r
 \r
 **/\r
 \r
-RETURN_STATUS\r
-EFIAPI\r
-BaseSmBusLibConstructor (\r
-  IN      VOID                      *Param1,\r
-  IN      VOID                      *Param2\r
-  )\r
-{\r
-  return RETURN_SUCCESS;\r
-}\r
+#include "SmbusLibRegisters.h"\r
 \r
-//\r
-// BUGBUG: use PCD to retrieve BUS, DEV, FUNC & OFFSET for SMBUS host BAR\r
-//\r
-#define SMBUS_HOST_BUS              0\r
-#define SMBUS_HOST_DEV              31\r
-#define SMBUS_HOST_FUNC             3\r
-#define SMBUS_HOST_SMB_BASE         0x20\r
-\r
-//\r
-// Offsets of registers for SMBUS controller\r
-//\r
-#define R_HST_STS                   0\r
-#define R_HST_CNT                   2\r
-#define R_HST_CMD                   3\r
-#define R_XMIT_SLVA                 4\r
-#define R_HST_D0                    5\r
-#define R_HST_D1                    6\r
-#define R_HOST_BLOCK_DB             7\r
-#define R_PEC                       8\r
-#define R_RCV_SLVA                  9\r
-#define R_SLV_DATA                  0x0a\r
-#define R_AUX_STS                   0x0c\r
-#define R_AUX_CTL                   0x0d\r
-#define R_SMLINK_PIN_CTL            0x0e\r
-#define R_SMBUS_PIN_CTL             0x0f\r
-#define R_SLV_STS                   0x10\r
-#define R_SLV_CMD                   0x11\r
-#define R_NOTIFY_DADDR              0x14\r
-#define R_NOTIFY_DLOW               0x16\r
-#define R_NOTIFY_DHIGH              0x17\r
+#define SMBUS_LIB_SLAVE_ADDRESS(SmBusAddress)      (((SmBusAddress) >> 1)  & 0x7f)\r
+#define SMBUS_LIB_COMMAND(SmBusAddress)            (((SmBusAddress) >> 8)  & 0xff)\r
+#define SMBUS_LIB_LENGTH(SmBusAddress)             (((SmBusAddress) >> 16) & 0x1f)\r
+#define SMBUS_LIB_PEC(SmBusAddress)     ((BOOLEAN) (((SmBusAddress) & SMBUS_LIB_PEC_BIT) != 0))\r
+#define SMBUS_LIB_RESEARVED(SmBusAddress)          ((SmBusAddress) & ~(((1 << 21) - 2) | SMBUS_LIB_PEC_BIT))\r
 \r
 //\r
-// Bits in HST_STS\r
+// Replaced by PCD\r
 //\r
-#define B_HST_STS_DS                0x80\r
-#define B_HST_STS_INUSE             0x40\r
-#define B_HST_STS_SMBALERT          0x20\r
-#define B_HST_STS_FAILED            0x10\r
-#define B_HST_STS_BUS_ERR           0x08\r
-#define B_HST_STS_DEV_ERR           0x04\r
-#define B_HST_STS_INTR              0x02\r
-#define B_HST_STS_BUSY              0x01\r
-#define B_HST_STS_ERR               ( B_HST_STS_BUS_ERR   | \\r
-                                      B_HST_STS_DEV_ERR   | \\r
-                                      B_HST_STS_FAILED )\r
-#define B_HST_STS_ALL               ( B_HST_STS_DS        | \\r
-                                      B_HST_STS_INUSE     | \\r
-                                      B_HST_STS_SMBALERT  | \\r
-                                      B_HST_STS_ERR       | \\r
-                                      B_HST_STS_INTR )\r
+#define ICH_SMBUS_BASE_ADDRESS               0xEFA0\r
 \r
-//\r
-// Bits in HST_CNT\r
-//\r
-#define B_HST_CNT_PEC               0x80\r
-#define B_HST_CNT_START             0x40\r
-#define B_HST_CNT_LAST_BYTE         0x20\r
-#define B_HST_CNT_SMB_CMD           0x1c\r
-#define B_HST_CNT_KILL              0x02\r
-#define B_HST_CNT_INTREN            0x01\r
+/**\r
+  Reads an 8-bit SMBUS register on ICH.\r
 \r
-//\r
-// SMBUS Protocols\r
-//\r
-#define B_SMB_CMD_QUICK             0\r
-#define B_SMB_CMD_BYTE              1\r
-#define B_SMB_CMD_BYTE_DATA         2\r
-#define B_SMB_CMD_WORD_DATA         3\r
-#define B_SMB_CMD_PROCESS_CALL      4\r
-#define B_SMB_CMD_BLOCK             5\r
-#define B_SMB_CMD_I2C               6\r
-#define B_SMB_CMD_BLOCK_PROCESS     7\r
+  This internal function reads an SMBUS register specified by Offset.\r
 \r
-//\r
-// Bits in AUX_CTL\r
-//\r
-#define B_AUX_CTL_E32B              0x02\r
-#define B_AUX_CTL_AAC               0x01\r
+  @param  Offset  The offset of SMBUS register.\r
 \r
-//\r
-// SMBUS Rd/Wr control\r
-//\r
-#define B_SMBUS_READ                1\r
-#define B_SMBUS_WRITE               0\r
+  @return The value read.\r
 \r
-static\r
-UINT16\r
-EFIAPI\r
-GetSmBusIOBaseAddress (\r
-  VOID\r
+**/\r
+UINT8\r
+InternalSmBusIoRead8 (\r
+  IN UINTN      Offset\r
   )\r
 {\r
-  UINT32                            SmbusBar;\r
-\r
-  SmbusBar = PciRead32 (\r
-               PCI_LIB_ADDRESS (\r
-                 SMBUS_HOST_BUS,\r
-                 SMBUS_HOST_DEV,\r
-                 SMBUS_HOST_FUNC,\r
-                 SMBUS_HOST_SMB_BASE\r
-                 )\r
-               );\r
-  ASSERT ((SmbusBar & 0xffff001f) == 1);\r
-  return (UINT16)(SmbusBar & ~1);\r
+  return IoRead8 (ICH_SMBUS_BASE_ADDRESS + Offset);\r
 }\r
 \r
-static\r
-BOOLEAN\r
-EFIAPI\r
-SmBusAcquire (\r
-  IN      UINT16                    SmBusBase\r
-  )\r
-{\r
-  UINT8                             HstSts;\r
+/**\r
+  Writes an 8-bit SMBUS register on ICH.\r
 \r
-  HstSts = IoRead8 (SmBusBase + R_HST_STS);\r
-  if (HstSts & B_HST_STS_INUSE) {\r
-    return FALSE;\r
-  }\r
+  This internal function writes an SMBUS register specified by Offset.\r
 \r
-  //\r
-  // BUGBUG: Dead loop may occur here\r
-  //\r
-  while (HstSts & B_HST_STS_BUSY) {\r
-    ASSERT (HstSts & B_HST_STS_INUSE);\r
-    HstSts = IoRead8 (SmBusBase + R_HST_STS);\r
-  }\r
-  return TRUE;\r
-}\r
+  @param  Offset  The offset of SMBUS register.\r
+  @param  Value   The value to write to SMBUS register.\r
 \r
-static\r
-VOID\r
-EFIAPI\r
-SmBusStart (\r
-  IN      UINT16                    SmBusBase,\r
-  IN      UINT8                     SmBusProtocol,\r
-  IN      UINT8                     SlaveAddress\r
+  @return The value written the SMBUS register.\r
+\r
+**/\r
+UINT8\r
+InternalSmBusIoWrite8 (\r
+  IN UINTN      Offset,\r
+  IN UINT8      Value\r
   )\r
 {\r
-  IoWrite8 (SmBusBase + R_XMIT_SLVA, SlaveAddress);\r
-  IoWrite8 (\r
-    SmBusBase + R_HST_CNT,\r
-    IoBitFieldWrite8 (SmBusBase + R_HST_CNT, 2, 4, SmBusProtocol) |\r
-    B_HST_CNT_START\r
-    );\r
+  return IoWrite8 (ICH_SMBUS_BASE_ADDRESS + Offset, Value);\r
 }\r
 \r
-static\r
-UINT8\r
-EFIAPI\r
-SmBusWait (\r
-  IN      UINT16                    SmBusBase\r
+/**\r
+  Acquires the ownership of SMBUS.\r
+\r
+  This internal function reads the host state register.\r
+  If the SMBUS is not available, RETURN_TIMEOUT is returned;\r
+  Otherwise, it performs some basic initializations and returns\r
+  RETURN_SUCCESS. \r
+\r
+  @retval RETURN_SUCCESS    The SMBUS command was executed successfully.\r
+  @retval RETURN_TIMEOUT    A timeout occurred while executing the SMBUS command.\r
+\r
+**/\r
+RETURN_STATUS\r
+InternalSmBusAcquire (\r
+  VOID \r
   )\r
 {\r
-  UINT8                             HstSts;\r
+  UINT8   HostStatus;\r
 \r
-  while (((HstSts = IoRead8 (SmBusBase + R_HST_STS)) & B_HST_STS_INTR) == 0);\r
-  return HstSts;\r
+  HostStatus = InternalSmBusIoRead8 (SMBUS_R_HST_STS);\r
+  if ((HostStatus & SMBUS_B_INUSE_STS) != 0) {\r
+    return RETURN_TIMEOUT;\r
+  } else if ((HostStatus & SMBUS_B_HOST_BUSY) != 0) {\r
+    //\r
+    // Clear Status Register and exit\r
+    //\r
+    InternalSmBusIoWrite8 (SMBUS_R_HST_STS, SMBUS_B_HSTS_ALL);\r
+    return RETURN_TIMEOUT;\r
+  }\r
+  //\r
+  // Clear byte pointer of 32-byte buffer.\r
+  //\r
+  InternalSmBusIoRead8 (SMBUS_R_HST_CTL);\r
+  //\r
+  // Clear BYTE_DONE status\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_STS, SMBUS_B_BYTE_DONE_STS);\r
+  \r
+  return RETURN_SUCCESS;\r
 }\r
 \r
-static\r
-VOID\r
-EFIAPI\r
-SmBusCleanup (\r
-  IN      UINT16                    SmBusBase\r
+/**\r
+  Waits until the completion of SMBUS transaction.\r
+\r
+  This internal function waits until the transaction of SMBUS is over\r
+  by polling the INTR bit of Host status register.\r
+  If the SMBUS is not available, RETURN_TIMEOUT is returned;\r
+  Otherwise, it performs some basic initializations and returns\r
+  RETURN_SUCCESS. \r
+\r
+  @retval RETURN_SUCCESS      The SMBUS command was executed successfully.\r
+  @retval RETURN_CRC_ERROR    The checksum is not correct (PEC is incorrect).\r
+  @retval RETURN_DEVICE_ERROR The request was not completed because a failure reflected\r
+                              in the Host Status Register bit.  Device errors are\r
+                              a result of a transaction collision, illegal command field,\r
+                              unclaimed cycle (host initiated), or bus errors (collisions).\r
+\r
+**/\r
+RETURN_STATUS\r
+InternalSmBusWait (\r
+  VOID \r
   )\r
 {\r
-  IoWrite8 (SmBusBase + R_HST_STS, B_HST_STS_ALL);\r
+  UINT8   HostStatus;\r
+  UINT8   AuxiliaryStatus;\r
+  BOOLEAN First;\r
+  First = TRUE;\r
+\r
+  do {\r
+    //\r
+    // Poll INTR bit of host status register.\r
+    //\r
+    HostStatus = InternalSmBusIoRead8 (SMBUS_R_HST_STS);\r
+  } while ((HostStatus & (SMBUS_B_INTR | SMBUS_B_ERROR | SMBUS_B_BYTE_DONE_STS)) == 0);\r
+  \r
+  if ((HostStatus & SMBUS_B_ERROR) == 0) {\r
+    return RETURN_SUCCESS;\r
+  }\r
+  //\r
+  // Clear error bits of host status register\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_STS, SMBUS_B_ERROR);\r
+  //\r
+  // Read auxiliary status register to judge CRC error.\r
+  //\r
+  AuxiliaryStatus = InternalSmBusIoRead8 (SMBUS_R_AUX_STS);\r
+  if ((AuxiliaryStatus & SMBUS_B_CRCE) != 0) {\r
+    return RETURN_CRC_ERROR;\r
+  }\r
+\r
+  return RETURN_DEVICE_ERROR;\r
 }\r
 \r
-static\r
-RETURN_STATUS\r
+/**\r
+  Executes an SMBUS quick read/write command.\r
+\r
+  This internal function executes an SMBUS quick read/write command\r
+  on the SMBUS device specified by SmBusAddress.\r
+  Only the SMBUS slave address field of SmBusAddress is required.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+\r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+**/\r
+VOID\r
 EFIAPI\r
-SmBusQuick (\r
-  IN      UINT8                     SmBusAddress\r
+InternalSmBusQuick (\r
+  IN  UINTN                     SmBusAddress,\r
+  OUT RETURN_STATUS             *Status       OPTIONAL\r
   )\r
 {\r
-  RETURN_STATUS                     Status;\r
-  UINT16                            SmBusBase;\r
+  RETURN_STATUS   ReturnStatus;\r
 \r
-  SmBusBase = GetSmBusIOBaseAddress ();\r
-  if (!SmBusAcquire (SmBusBase)) {\r
-    return RETURN_TIMEOUT;\r
+  ReturnStatus = InternalSmBusAcquire ();\r
+  if (RETURN_ERROR (ReturnStatus)) {\r
+    goto Done;\r
   }\r
\r
+  //\r
+  // Set Command register\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_CMD, 0);\r
+  //\r
+  // Set Auxiliary Control register\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_AUX_CTL, 0);\r
+  //\r
+  // Set SMBus slave address for the device to send/receive from\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_XMIT_SLVA, (UINT8) SmBusAddress);\r
+  //\r
+  // Set Control Register (Initiate Operation, Interrupt disabled)\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_CTL, SMBUS_V_SMB_CMD_QUICK + SMBUS_B_START);\r
 \r
-  SmBusStart (SmBusAddress, B_SMB_CMD_QUICK, SmBusAddress);\r
-  if (SmBusWait (SmBusAddress) & B_HST_STS_ERR) {\r
-    Status = RETURN_DEVICE_ERROR;\r
-  } else {\r
-    Status = RETURN_SUCCESS;\r
-  }\r
+  //\r
+  // Wait for the end\r
+  //\r
+  ReturnStatus = InternalSmBusWait ();\r
 \r
-  SmBusCleanup (SmBusAddress);\r
-  return Status;\r
+  //\r
+  // Clear status register and exit\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_STS, SMBUS_B_HSTS_ALL);;\r
+\r
+Done:\r
+  if (Status != NULL) {\r
+    *Status = ReturnStatus;\r
+  }\r
 }\r
 \r
+/**\r
+  Executes an SMBUS quick read command.\r
+\r
+  Executes an SMBUS quick read command on the SMBUS device specified by SmBusAddress.\r
+  Only the SMBUS slave address field of SmBusAddress is required.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+  If PEC is set in SmBusAddress, then ASSERT().\r
+  If Command in SmBusAddress is not zero, then ASSERT().\r
+  If Length in SmBusAddress is not zero, then ASSERT().\r
+  If any reserved bits of SmBusAddress are set, then ASSERT().\r
+\r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+**/\r
 VOID\r
 EFIAPI\r
 SmBusQuickRead (\r
-  IN      UINTN                     SmBusAddress,\r
-  OUT     RETURN_STATUS             *Status\r
+  IN  UINTN                     SmBusAddress,\r
+  OUT RETURN_STATUS             *Status       OPTIONAL\r
   )\r
 {\r
-  RETURN_STATUS                     RetStatus;\r
+  ASSERT (!SMBUS_LIB_PEC (SmBusAddress));\r
+  ASSERT (SMBUS_LIB_COMMAND (SmBusAddress)   == 0);\r
+  ASSERT (SMBUS_LIB_LENGTH (SmBusAddress)    == 0);\r
+  ASSERT (SMBUS_LIB_RESEARVED (SmBusAddress) == 0);\r
 \r
-  ASSERT ((SmBusAddress & ~0xfe) == 0);\r
-  RetStatus = SmBusQuick ((UINT8)SmBusAddress | B_SMBUS_READ);\r
-  if (Status) {\r
-    *Status = RetStatus;\r
-  }\r
+  InternalSmBusQuick (SmBusAddress | SMBUS_B_READ, Status);\r
 }\r
 \r
-BOOLEAN\r
+/**\r
+  Executes an SMBUS quick write command.\r
+\r
+  Executes an SMBUS quick write command on the SMBUS device specified by SmBusAddress.\r
+  Only the SMBUS slave address field of SmBusAddress is required.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+  If PEC is set in SmBusAddress, then ASSERT().\r
+  If Command in SmBusAddress is not zero, then ASSERT().\r
+  If Length in SmBusAddress is not zero, then ASSERT().\r
+  If any reserved bits of SmBusAddress are set, then ASSERT().\r
+\r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+**/\r
+VOID\r
 EFIAPI\r
 SmBusQuickWrite (\r
-  IN      UINTN                     SmBusAddress,\r
-  OUT     RETURN_STATUS             *Status\r
+  IN  UINTN                     SmBusAddress,\r
+  OUT RETURN_STATUS             *Status       OPTIONAL\r
   )\r
 {\r
-  RETURN_STATUS                     RetStatus;\r
+  ASSERT (!SMBUS_LIB_PEC (SmBusAddress));\r
+  ASSERT (SMBUS_LIB_COMMAND (SmBusAddress)   == 0);\r
+  ASSERT (SMBUS_LIB_LENGTH (SmBusAddress)    == 0);\r
+  ASSERT (SMBUS_LIB_RESEARVED (SmBusAddress) == 0);\r
 \r
-  ASSERT ((SmBusAddress & ~0xfe) == 0);\r
-  RetStatus = SmBusQuick ((UINT8)SmBusAddress | B_SMBUS_WRITE);\r
-  if (Status) {\r
-    *Status = RetStatus;\r
-  }\r
-  return (BOOLEAN)!RETURN_ERROR (RetStatus);\r
+  InternalSmBusQuick (SmBusAddress | SMBUS_B_WRITE, Status);\r
 }\r
 \r
-static\r
+/**\r
+  Executes an SMBUS byte or word command.\r
+\r
+  This internal function executes an .\r
+  Only the SMBUS slave address field of SmBusAddress is required.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+\r
+  @param  HostControl     The value of Host Control Register to set.  \r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  Value           The byte/word write to the SMBUS.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+  @return The byte/word read from the SMBUS.\r
+\r
+**/\r
 UINT16\r
-EFIAPI\r
-SmBusByteWord (\r
-  IN      UINTN                     SmBusAddress,\r
-  IN      UINT16                    Value,\r
-  IN      UINT8                     SmBusProtocol,\r
-  OUT     RETURN_STATUS             *Status\r
+InternalSmBusByteWord (\r
+  IN  UINT8                     HostControl,\r
+  IN  UINTN                     SmBusAddress,\r
+  IN  UINT16                    Value,\r
+  OUT RETURN_STATUS             *Status\r
   )\r
 {\r
-  RETURN_STATUS                     RetStatus;\r
-  UINT16                            SmBusBase;\r
+  RETURN_STATUS                 ReturnStatus;\r
+  UINT8                         AuxiliaryControl;\r
 \r
-  if (Status == NULL) {\r
-    Status = &RetStatus;\r
+  ReturnStatus = InternalSmBusAcquire ();\r
+  if (RETURN_ERROR (ReturnStatus)) {\r
+    goto Done;\r
   }\r
 \r
-  SmBusBase = GetSmBusIOBaseAddress ();\r
-  if (!SmBusAcquire (SmBusBase)) {\r
-    *Status = RETURN_TIMEOUT;\r
-    return Value;\r
+  AuxiliaryControl = 0;\r
+  if (SMBUS_LIB_PEC (SmBusAddress)) {\r
+    AuxiliaryControl |= SMBUS_B_AAC;\r
+    HostControl      |= SMBUS_B_PEC_EN;\r
   }\r
\r
+  //\r
+  // Set commond register\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_CMD, (UINT8) SMBUS_LIB_COMMAND (SmBusAddress));\r
 \r
-  IoWrite8 (SmBusBase + R_HST_CMD, (UINT8)(SmBusAddress >> 8));\r
-  IoWrite8 (SmBusBase + R_HST_D0, (UINT8)Value);\r
-  IoWrite8 (SmBusBase + R_HST_D1, (UINT8)(Value >> 8));\r
-  if ((INTN)SmBusAddress < 0) {\r
-    IoOr8 (SmBusBase + R_HST_CNT, B_HST_CNT_PEC);\r
-    IoOr8 (SmBusBase + R_AUX_CTL, B_AUX_CTL_AAC);\r
-  } else {\r
-    IoAnd8 (SmBusBase + R_HST_CNT, (UINT8)~B_HST_CNT_PEC);\r
-    IoAnd8 (SmBusBase + R_AUX_CTL, (UINT8)~B_AUX_CTL_AAC);\r
-  }\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_D0, (UINT8) Value);\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_D1, (UINT8) (Value >> 8));\r
+\r
+  //\r
+  // Set Auxiliary Control Regiester.\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_AUX_CTL, AuxiliaryControl);\r
+  //\r
+  // Set SMBus slave address for the device to send/receive from.\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_XMIT_SLVA, (UINT8) SmBusAddress);\r
+  //\r
+  // Set Control Register (Initiate Operation, Interrupt disabled)\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_CTL, HostControl + SMBUS_B_START);\r
+\r
+  //\r
+  // Wait for the end\r
+  //\r
+  ReturnStatus = InternalSmBusWait ();\r
\r
+  Value  = InternalSmBusIoRead8 (SMBUS_R_HST_D1) << 8;\r
+  Value |= InternalSmBusIoRead8 (SMBUS_R_HST_D0);\r
 \r
-  SmBusStart (SmBusBase, SmBusProtocol, (UINT8)SmBusAddress);\r
+  //\r
+  // Clear status register and exit\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_STS, SMBUS_B_HSTS_ALL);;\r
 \r
-  if (SmBusWait (SmBusBase) & B_HST_STS_ERR) {\r
-    *Status = RETURN_DEVICE_ERROR;\r
-  } else {\r
-    *Status = RETURN_SUCCESS;\r
-    Value = IoRead8 (SmBusBase + R_HST_D0);\r
-    Value |= (UINT16)IoRead8 (SmBusBase + R_HST_D1) << 8;\r
+Done:\r
+  if (Status != NULL) {\r
+    *Status = ReturnStatus;\r
   }\r
 \r
-  SmBusCleanup (SmBusBase);\r
   return Value;\r
 }\r
 \r
+/**\r
+  Executes an SMBUS receive byte command.\r
+\r
+  Executes an SMBUS receive byte command on the SMBUS device specified by SmBusAddress.\r
+  Only the SMBUS slave address field of SmBusAddress is required.\r
+  The byte received from the SMBUS is returned.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+  If Command in SmBusAddress is not zero, then ASSERT().\r
+  If Length in SmBusAddress is not zero, then ASSERT().\r
+  If any reserved bits of SmBusAddress are set, then ASSERT().\r
+\r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+  @return The byte received from the SMBUS.\r
+\r
+**/\r
 UINT8\r
 EFIAPI\r
 SmBusReceiveByte (\r
-  IN      UINTN                     SmBusAddress,\r
-  OUT     RETURN_STATUS             *Status\r
+  IN  UINTN          SmBusAddress,\r
+  OUT RETURN_STATUS  *Status        OPTIONAL\r
   )\r
 {\r
-  ASSERT ((SmBusAddress & ~(0xfe | MAX_BIT)) == 0);\r
-  return (UINT8)SmBusByteWord (\r
-                  SmBusAddress | B_SMBUS_READ,\r
-                  0,\r
-                  B_SMB_CMD_BYTE,\r
-                  Status\r
-                  );\r
+  ASSERT (SMBUS_LIB_COMMAND (SmBusAddress)   == 0);\r
+  ASSERT (SMBUS_LIB_LENGTH (SmBusAddress)    == 0);\r
+  ASSERT (SMBUS_LIB_RESEARVED (SmBusAddress) == 0);\r
+\r
+  return (UINT8) InternalSmBusByteWord (\r
+                   SMBUS_V_SMB_CMD_BYTE,\r
+                   SmBusAddress | SMBUS_B_READ,\r
+                   0,\r
+                   Status\r
+                   );\r
 }\r
 \r
+/**\r
+  Executes an SMBUS send byte command.\r
+\r
+  Executes an SMBUS send byte command on the SMBUS device specified by SmBusAddress.\r
+  The byte specified by Value is sent.\r
+  Only the SMBUS slave address field of SmBusAddress is required.  Value is returned.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+  If Command in SmBusAddress is not zero, then ASSERT().\r
+  If Length in SmBusAddress is not zero, then ASSERT().\r
+  If any reserved bits of SmBusAddress are set, then ASSERT().\r
+\r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  Value           The 8-bit value to send.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+  @return The parameter of Value.\r
+\r
+**/\r
 UINT8\r
 EFIAPI\r
 SmBusSendByte (\r
-  IN      UINTN                     SmBusAddress,\r
-  IN      UINT8                     Value,\r
-  OUT     RETURN_STATUS             *Status\r
+  IN  UINTN          SmBusAddress,\r
+  IN  UINT8          Value,\r
+  OUT RETURN_STATUS  *Status        OPTIONAL\r
   )\r
 {\r
-  ASSERT ((SmBusAddress & ~(0xfe | MAX_BIT)) == 0);\r
-  return (UINT8)SmBusByteWord (\r
-                  SmBusAddress | B_SMBUS_WRITE,\r
-                  Value,\r
-                  B_SMB_CMD_BYTE,\r
-                  Status\r
-                  );\r
+  ASSERT (SMBUS_LIB_COMMAND (SmBusAddress)   == 0);\r
+  ASSERT (SMBUS_LIB_LENGTH (SmBusAddress)    == 0);\r
+  ASSERT (SMBUS_LIB_RESEARVED (SmBusAddress) == 0);\r
+\r
+  return (UINT8) InternalSmBusByteWord (\r
+                   SMBUS_V_SMB_CMD_BYTE,\r
+                   SmBusAddress | SMBUS_B_WRITE,\r
+                   Value,\r
+                   Status\r
+                   );\r
 }\r
 \r
+/**\r
+  Executes an SMBUS read data byte command.\r
+\r
+  Executes an SMBUS read data byte command on the SMBUS device specified by SmBusAddress.\r
+  Only the SMBUS slave address and SMBUS command fields of SmBusAddress are required.\r
+  The 8-bit value read from the SMBUS is returned.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+  If Length in SmBusAddress is not zero, then ASSERT().\r
+  If any reserved bits of SmBusAddress are set, then ASSERT().\r
+\r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+  @return The byte read from the SMBUS.\r
+\r
+**/\r
 UINT8\r
 EFIAPI\r
 SmBusReadDataByte (\r
-  IN      UINTN                     SmBusAddress,\r
-  OUT     RETURN_STATUS             *Status\r
+  IN  UINTN          SmBusAddress,\r
+  OUT RETURN_STATUS  *Status        OPTIONAL\r
   )\r
 {\r
-  ASSERT ((SmBusAddress & ~(0xfffe | MAX_BIT)) == 0);\r
-  return (UINT8)SmBusByteWord (\r
-                  SmBusAddress | B_SMBUS_READ,\r
-                  0,\r
-                  B_SMB_CMD_BYTE_DATA,\r
-                  Status\r
-                  );\r
+  ASSERT (SMBUS_LIB_LENGTH (SmBusAddress)    == 0);\r
+  ASSERT (SMBUS_LIB_RESEARVED (SmBusAddress) == 0);\r
+\r
+  return (UINT8) InternalSmBusByteWord (\r
+                   SMBUS_V_SMB_CMD_BYTE_DATA,\r
+                   SmBusAddress | SMBUS_B_READ,\r
+                   0,\r
+                   Status\r
+                   );\r
 }\r
 \r
+/**\r
+  Executes an SMBUS write data byte command.\r
+\r
+  Executes an SMBUS write data byte command on the SMBUS device specified by SmBusAddress.\r
+  The 8-bit value specified by Value is written.\r
+  Only the SMBUS slave address and SMBUS command fields of SmBusAddress are required.\r
+  Value is returned.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+  If Length in SmBusAddress is not zero, then ASSERT().\r
+  If any reserved bits of SmBusAddress are set, then ASSERT().\r
+\r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  Value           The 8-bit value to write.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+  @return The parameter of Value.\r
+\r
+**/\r
 UINT8\r
 EFIAPI\r
 SmBusWriteDataByte (\r
-  IN      UINTN                     SmBusAddress,\r
-  IN      UINT8                     Value,\r
-  OUT     RETURN_STATUS             *Status\r
+  IN  UINTN          SmBusAddress,\r
+  IN  UINT8          Value,\r
+  OUT RETURN_STATUS  *Status        OPTIONAL\r
   )\r
 {\r
-  ASSERT (((UINT32)SmBusAddress & ~(0xfffe | MAX_BIT)) == 0);\r
-  return (UINT8)SmBusByteWord (\r
-                  SmBusAddress | B_SMBUS_WRITE,\r
-                  Value,\r
-                  B_SMB_CMD_BYTE_DATA,\r
-                  Status\r
-                  );\r
+  ASSERT (SMBUS_LIB_LENGTH (SmBusAddress)    == 0);\r
+  ASSERT (SMBUS_LIB_RESEARVED (SmBusAddress) == 0);\r
+\r
+  return (UINT8) InternalSmBusByteWord (\r
+                   SMBUS_V_SMB_CMD_BYTE_DATA,\r
+                   SmBusAddress | SMBUS_B_WRITE,\r
+                   Value,\r
+                   Status\r
+                   );\r
 }\r
 \r
+/**\r
+  Executes an SMBUS read data word command.\r
+\r
+  Executes an SMBUS read data word command on the SMBUS device specified by SmBusAddress.\r
+  Only the SMBUS slave address and SMBUS command fields of SmBusAddress are required.\r
+  The 16-bit value read from the SMBUS is returned.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+  If Length in SmBusAddress is not zero, then ASSERT().\r
+  If any reserved bits of SmBusAddress are set, then ASSERT().\r
+  \r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+  @return The byte read from the SMBUS.\r
+\r
+**/\r
 UINT16\r
 EFIAPI\r
 SmBusReadDataWord (\r
-  IN      UINTN                     SmBusAddress,\r
-  OUT     RETURN_STATUS             *Status\r
+  IN  UINTN          SmBusAddress,\r
+  OUT RETURN_STATUS  *Status        OPTIONAL\r
   )\r
 {\r
-  ASSERT ((SmBusAddress & ~(0xfffe | MAX_BIT)) == 0);\r
-  return SmBusByteWord (\r
-           SmBusAddress | B_SMBUS_READ,\r
+  ASSERT (SMBUS_LIB_LENGTH (SmBusAddress)    == 0);\r
+  ASSERT (SMBUS_LIB_RESEARVED (SmBusAddress) == 0);\r
+\r
+  return InternalSmBusByteWord (\r
+           SMBUS_V_SMB_CMD_WORD_DATA,\r
+           SmBusAddress | SMBUS_B_READ,\r
            0,\r
-           B_SMB_CMD_WORD_DATA,\r
            Status\r
            );\r
 }\r
 \r
+/**\r
+  Executes an SMBUS write data word command.\r
+\r
+  Executes an SMBUS write data word command on the SMBUS device specified by SmBusAddress.\r
+  The 16-bit value specified by Value is written.\r
+  Only the SMBUS slave address and SMBUS command fields of SmBusAddress are required.\r
+  Value is returned.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+  If Length in SmBusAddress is not zero, then ASSERT().\r
+  If any reserved bits of SmBusAddress are set, then ASSERT().\r
+\r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  Value           The 16-bit value to write.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+  @return The parameter of Value.\r
+\r
+**/\r
 UINT16\r
 EFIAPI\r
 SmBusWriteDataWord (\r
-  IN      UINTN                     SmBusAddress,\r
-  IN      UINT16                    Value,\r
-  OUT     RETURN_STATUS             *Status\r
+  IN  UINTN          SmBusAddress,\r
+  IN  UINT16         Value,\r
+  OUT RETURN_STATUS  *Status        OPTIONAL\r
   )\r
 {\r
-  ASSERT ((SmBusAddress & ~(0xfffe | MAX_BIT)) == 0);\r
-  return SmBusByteWord (\r
-           SmBusAddress | B_SMBUS_WRITE,\r
+  ASSERT (SMBUS_LIB_LENGTH (SmBusAddress)    == 0);\r
+  ASSERT (SMBUS_LIB_RESEARVED (SmBusAddress) == 0);\r
+\r
+  return InternalSmBusByteWord (\r
+           SMBUS_V_SMB_CMD_WORD_DATA,\r
+           SmBusAddress | SMBUS_B_WRITE,\r
            Value,\r
-           B_SMB_CMD_WORD_DATA,\r
            Status\r
            );\r
 }\r
 \r
+/**\r
+  Executes an SMBUS process call command.\r
+\r
+  Executes an SMBUS process call command on the SMBUS device specified by SmBusAddress.\r
+  The 16-bit value specified by Value is written.\r
+  Only the SMBUS slave address and SMBUS command fields of SmBusAddress are required.\r
+  The 16-bit value returned by the process call command is returned.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+  If Length in SmBusAddress is not zero, then ASSERT().\r
+  If any reserved bits of SmBusAddress are set, then ASSERT().\r
+\r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  Value           The 16-bit value to write.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+  @return The 16-bit value returned by the process call command.\r
+\r
+**/\r
 UINT16\r
 EFIAPI\r
 SmBusProcessCall (\r
-  IN      UINTN                     SmBusAddress,\r
-  IN      UINT16                    Value,\r
-  OUT     RETURN_STATUS             *Status\r
+  IN  UINTN          SmBusAddress,\r
+  IN  UINT16         Value,\r
+  OUT RETURN_STATUS  *Status        OPTIONAL\r
   )\r
 {\r
-  ASSERT ((SmBusAddress & ~(0xfffe | MAX_BIT)) == 0);\r
-  return SmBusByteWord (\r
-           SmBusAddress | B_SMBUS_WRITE,\r
+  ASSERT (SMBUS_LIB_LENGTH (SmBusAddress)    == 0);\r
+  ASSERT (SMBUS_LIB_RESEARVED (SmBusAddress) == 0);\r
+\r
+  return InternalSmBusByteWord (\r
+           SMBUS_V_SMB_CMD_PROCESS_CALL,\r
+           SmBusAddress | SMBUS_B_WRITE,\r
            Value,\r
-           B_SMB_CMD_PROCESS_CALL,\r
            Status\r
            );\r
 }\r
 \r
-static\r
+/**\r
+  Executes an SMBUS block command.\r
+\r
+  Executes an SMBUS block read, block write and block write-block read command\r
+  on the SMBUS device specified by SmBusAddress.\r
+  Bytes are read from the SMBUS and stored in Buffer.\r
+  The number of bytes read is returned, and will never return a value larger than 32-bytes.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+  It is the caller¡¯s responsibility to make sure Buffer is large enough for the total number of bytes read.\r
+  SMBUS supports a maximum transfer size of 32 bytes, so Buffer does not need to be any larger than 32 bytes.\r
+\r
+  @param  HostControl     The value of Host Control Register to set.  \r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  OutBuffer       Pointer to the buffer of bytes to write to the SMBUS.\r
+  @param  InBuffer        Pointer to the buffer of bytes to read from the SMBUS.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+  @return The number of bytes read from the SMBUS.\r
+\r
+**/\r
 UINTN\r
-EFIAPI\r
-SmBusBlock (\r
-  IN      UINTN                     SmBusAddress,\r
-  IN      UINT8                     SmBusProtocol,\r
-  IN      VOID                      *InBuffer,\r
-  OUT     VOID                      *OutBuffer,\r
-  OUT     RETURN_STATUS             *Status\r
+InternalSmBusBlock (\r
+  IN  UINT8                     HostControl,\r
+  IN  UINTN                     SmBusAddress,\r
+  IN  UINT8                     *OutBuffer,\r
+  OUT UINT8                     *InBuffer,\r
+  OUT RETURN_STATUS             *Status\r
   )\r
 {\r
-  RETURN_STATUS                     RetStatus;\r
-  UINT16                            SmBusBase;\r
-  UINTN                             Index;\r
-  UINTN                             BytesCount;\r
+  RETURN_STATUS                 ReturnStatus;\r
+  UINTN                         Index;\r
+  UINTN                         BytesCount;\r
+  UINT8                         AuxiliaryControl;\r
 \r
-  BytesCount = (UINT8)(SmBusAddress >> 16);\r
-  ASSERT (BytesCount <= 32);\r
+  BytesCount = SMBUS_LIB_LENGTH (SmBusAddress) + 1;\r
 \r
-  if (Status == NULL) {\r
-    Status = &RetStatus;\r
+  ReturnStatus = InternalSmBusAcquire ();\r
+  if (RETURN_ERROR (ReturnStatus)) {\r
+    goto Done;\r
   }\r
-\r
-  SmBusBase = GetSmBusIOBaseAddress ();\r
-  if (!SmBusAcquire (SmBusBase)) {\r
-    *Status = RETURN_TIMEOUT;\r
-    return 0;\r
+  \r
+  AuxiliaryControl = SMBUS_B_E32B;\r
+  if (SMBUS_LIB_PEC (SmBusAddress)) {\r
+    AuxiliaryControl |= SMBUS_B_AAC;\r
+    HostControl      |= SMBUS_B_PEC_EN;\r
   }\r
 \r
-  IoWrite8 (SmBusBase + R_HST_CMD, (UINT8)(SmBusAddress >> 8));\r
-  IoWrite8 (SmBusBase + R_HST_D0, (UINT8)BytesCount);\r
-  if ((INTN)SmBusAddress < 0) {\r
-    IoOr8 (SmBusBase + R_HST_CNT, B_HST_CNT_PEC);\r
-    IoOr8 (SmBusBase + R_AUX_CTL, B_AUX_CTL_AAC);\r
-  } else {\r
-    IoAnd8 (SmBusBase + R_HST_CNT, (UINT8)~B_HST_CNT_PEC);\r
-    IoAnd8 (SmBusBase + R_AUX_CTL, (UINT8)~B_AUX_CTL_AAC);\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_CMD, (UINT8) SMBUS_LIB_COMMAND (SmBusAddress));\r
+\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_D0, (UINT8) BytesCount);\r
+\r
+  if (OutBuffer != NULL) {\r
+    for (Index = 0; Index < BytesCount; Index++) {\r
+      InternalSmBusIoWrite8 (SMBUS_R_HOST_BLOCK_DB, OutBuffer[Index]);\r
+    }\r
   }\r
+  //\r
+  // Set Auxiliary Control Regiester.\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_AUX_CTL, AuxiliaryControl);\r
+  //\r
+  // Set SMBus slave address for the device to send/receive from\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_XMIT_SLVA, (UINT8) SmBusAddress);\r
+  //\r
+  // Set Control Register (Initiate Operation, Interrupt disabled)\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_CTL, HostControl + SMBUS_B_START);\r
 \r
   //\r
-  // BUGBUG: E32B bit does not exist in ICH3 or earlier\r
+  // Wait for the end\r
   //\r
-  IoOr8 (SmBusBase + R_AUX_CTL, B_AUX_CTL_E32B);\r
-  ASSERT (IoRead8 (SmBusBase + R_AUX_CTL) & B_AUX_CTL_E32B);\r
-  for (Index = 0; InBuffer != NULL && Index < BytesCount; Index++) {\r
-    IoWrite8 (SmBusBase + R_HOST_BLOCK_DB, ((UINT8*)InBuffer)[Index]);\r
+  ReturnStatus = InternalSmBusWait ();\r
+  if (RETURN_ERROR (ReturnStatus)) {\r
+    goto Done;\r
   }\r
 \r
-  SmBusStart (SmBusBase, SmBusProtocol, (UINT8)SmBusAddress);\r
-\r
-  if (SmBusWait (SmBusBase) & B_HST_STS_ERR) {\r
-    *Status = RETURN_DEVICE_ERROR;\r
-  } else {\r
-    *Status = RETURN_SUCCESS;\r
-    BytesCount = IoRead8 (SmBusBase + R_HST_D0);\r
-    for (Index = 0; OutBuffer != NULL && Index < BytesCount; Index++) {\r
-      ((UINT8*)OutBuffer)[Index] = IoRead8 (SmBusBase + R_HOST_BLOCK_DB);\r
+  BytesCount = InternalSmBusIoRead8 (SMBUS_R_HST_D0);\r
+  if (InBuffer != NULL) {\r
+    for (Index = 0; Index < BytesCount; Index++) {\r
+      InBuffer[Index] = InternalSmBusIoRead8 (SMBUS_R_HOST_BLOCK_DB);\r
     }\r
   }\r
 \r
-  SmBusCleanup (SmBusBase);\r
+  //\r
+  // Clear status register and exit\r
+  //\r
+  InternalSmBusIoWrite8 (SMBUS_R_HST_STS, SMBUS_B_HSTS_ALL);\r
+\r
+Done:\r
+  if (Status != NULL) {\r
+    *Status = ReturnStatus;\r
+  }\r
+\r
   return BytesCount;\r
 }\r
 \r
+/**\r
+  Executes an SMBUS read block command.\r
+\r
+  Executes an SMBUS read block command on the SMBUS device specified by SmBusAddress.\r
+  Only the SMBUS slave address and SMBUS command fields of SmBusAddress are required.\r
+  Bytes are read from the SMBUS and stored in Buffer.\r
+  The number of bytes read is returned, and will never return a value larger than 32-bytes.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+  It is the caller¡¯s responsibility to make sure Buffer is large enough for the total number of bytes read.\r
+  SMBUS supports a maximum transfer size of 32 bytes, so Buffer does not need to be any larger than 32 bytes.\r
+  If Length in SmBusAddress is not zero, then ASSERT().\r
+  If Buffer is NULL, then ASSERT().\r
+  If any reserved bits of SmBusAddress are set, then ASSERT().\r
+\r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  Buffer          Pointer to the buffer to store the bytes read from the SMBUS.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+  @return The number of bytes read.\r
+\r
+**/\r
 UINTN\r
 EFIAPI\r
 SmBusReadBlock (\r
-  IN      UINTN                     SmBusAddress,\r
-  OUT     VOID                      *Buffer,\r
-  OUT     RETURN_STATUS             *Status\r
+  IN  UINTN          SmBusAddress,\r
+  OUT VOID           *Buffer,\r
+  OUT RETURN_STATUS  *Status        OPTIONAL\r
   )\r
 {\r
-  ASSERT ((SmBusAddress & ~(0xfffffe | MAX_BIT)) == 0);\r
-  return SmBusBlock (\r
-           SmBusAddress | B_SMBUS_READ,\r
-           B_SMB_CMD_BLOCK,\r
+  ASSERT (Buffer != NULL);\r
+  ASSERT (SMBUS_LIB_LENGTH (SmBusAddress) == 0);\r
+  ASSERT (SMBUS_LIB_RESEARVED (SmBusAddress) == 0);\r
+\r
+  return InternalSmBusBlock (\r
+           SMBUS_V_SMB_CMD_BLOCK,\r
+           SmBusAddress | SMBUS_B_READ,\r
            NULL,\r
            Buffer,\r
            Status\r
            );\r
 }\r
 \r
+/**\r
+  Executes an SMBUS write block command.\r
+\r
+  Executes an SMBUS write block command on the SMBUS device specified by SmBusAddress.\r
+  The SMBUS slave address, SMBUS command, and SMBUS length fields of SmBusAddress are required.\r
+  Bytes are written to the SMBUS from Buffer.\r
+  The number of bytes written is returned, and will never return a value larger than 32-bytes.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.  \r
+  If Length in SmBusAddress is zero or greater than 32, then ASSERT().\r
+  If Buffer is NULL, then ASSERT().\r
+  If any reserved bits of SmBusAddress are set, then ASSERT().\r
+\r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  Buffer          Pointer to the buffer to store the bytes read from the SMBUS.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+  @return The number of bytes written.\r
+\r
+**/\r
 UINTN\r
 EFIAPI\r
 SmBusWriteBlock (\r
-  IN      UINTN                     SmBusAddress,\r
-  OUT     VOID                      *Buffer,\r
-  OUT     RETURN_STATUS             *Status\r
+  IN  UINTN          SmBusAddress,\r
+  OUT VOID           *Buffer,\r
+  OUT RETURN_STATUS  *Status        OPTIONAL\r
   )\r
 {\r
-  ASSERT ((SmBusAddress & ~(0xfffffe | MAX_BIT)) == 0);\r
-  return SmBusBlock (\r
-           SmBusAddress | B_SMBUS_WRITE,\r
-           B_SMB_CMD_BLOCK,\r
+  ASSERT (Buffer != NULL);\r
+  ASSERT (SMBUS_LIB_RESEARVED (SmBusAddress) == 0);\r
+\r
+  return InternalSmBusBlock (\r
+           SMBUS_V_SMB_CMD_BLOCK,\r
+           SmBusAddress | SMBUS_B_WRITE,\r
            Buffer,\r
            NULL,\r
            Status\r
            );\r
 }\r
 \r
+/**\r
+  Executes an SMBUS block process call command.\r
+\r
+  Executes an SMBUS block process call command on the SMBUS device specified by SmBusAddress.\r
+  The SMBUS slave address, SMBUS command, and SMBUS length fields of SmBusAddress are required.\r
+  Bytes are written to the SMBUS from OutBuffer.  Bytes are then read from the SMBUS into InBuffer.\r
+  If Status is not NULL, then the status of the executed command is returned in Status.\r
+  It is the caller¡¯s responsibility to make sure InBuffer is large enough for the total number of bytes read.\r
+  SMBUS supports a maximum transfer size of 32 bytes, so Buffer does not need to be any larger than 32 bytes.\r
+  If OutBuffer is NULL, then ASSERT().\r
+  If InBuffer is NULL, then ASSERT().\r
+  If any reserved bits of SmBusAddress are set, then ASSERT().\r
+\r
+  @param  SmBusAddress    Address that encodes the SMBUS Slave Address,\r
+                          SMBUS Command, SMBUS Data Length, and PEC.\r
+  @param  OutBuffer       Pointer to the buffer of bytes to write to the SMBUS.\r
+  @param  InBuffer        Pointer to the buffer of bytes to read from the SMBUS.\r
+  @param  Status          Return status for the executed command.\r
+                          This is an optional parameter and may be NULL.\r
+\r
+  @return The number of bytes written.\r
+\r
+**/\r
 UINTN\r
 EFIAPI\r
 SmBusBlockProcessCall (\r
-  IN      UINTN                     SmBusAddress,\r
-  IN      VOID                      *OutBuffer,\r
-  OUT     VOID                      *InBuffer,\r
-  OUT     RETURN_STATUS             *Status\r
+  IN  UINTN          SmBusAddress,\r
+  IN  VOID           *OutBuffer,\r
+  OUT VOID           *InBuffer,\r
+  OUT RETURN_STATUS  *Status        OPTIONAL\r
   )\r
 {\r
-  ASSERT ((SmBusAddress & ~(0xfffffe | MAX_BIT)) == 0);\r
-  return SmBusBlock (\r
-           SmBusAddress | B_SMBUS_WRITE,\r
-           B_SMB_CMD_BLOCK_PROCESS,\r
+  ASSERT (InBuffer  != NULL);\r
+  ASSERT (OutBuffer != NULL);\r
+  ASSERT (SMBUS_LIB_RESEARVED (SmBusAddress) == 0);\r
+\r
+  return InternalSmBusBlock (\r
+           SMBUS_V_SMB_CMD_BLOCK_PROCESS,\r
+           SmBusAddress | SMBUS_B_WRITE,\r
            OutBuffer,\r
            InBuffer,\r
            Status\r
            );\r
 }\r
-\r
-RETURN_STATUS\r
-EFIAPI\r
-SmBusArpAll (\r
-  IN      UINTN                     SmBusAddress\r
-  );\r
-\r
-RETURN_STATUS\r
-EFIAPI\r
-SmBusArpDevice (\r
-  IN      UINTN                     SmBusAddress,\r
-  IN      CONST GUID                *Uuid\r
-  );\r
-\r
-RETURN_STATUS\r
-EFIAPI\r
-SmBusGetUuid (\r
-  IN      UINTN                     SmBusAddress,\r
-  OUT     GUID                      *Uuid\r
-  );\r