]> git.proxmox.com Git - mirror_edk2.git/blobdiff - IntelFrameworkPkg/Include/Ppi/Smbus.h
Committing changes to the comments, after review with engineers.
[mirror_edk2.git] / IntelFrameworkPkg / Include / Ppi / Smbus.h
index 3833937adbcf2c36ddf8b75d5efb599400a6b12e..8834bd2a7d6f1f02a6878f3e3d327c7b05e61df5 100644 (file)
@@ -1,5 +1,5 @@
 /** @file\r
-  This file declares Smbus PPI which provides the basic I/O interfaces that a PEIM \r
+  This file declares the Smbus PPI, which provides the basic I/O interfaces that a PEIM \r
   uses to access its SMBus controller and the slave devices attached to it.\r
 \r
   Copyright (c) 2007 - 2009, Intel Corporation\r
@@ -40,10 +40,10 @@ typedef struct _EFI_PEI_SMBUS_PPI EFI_PEI_SMBUS_PPI;
                                 controller to the SMBus slave device and the interpretation is\r
                                 SMBus slave device specific.\r
   @param[in]      Operation     Signifies which particular SMBus hardware protocol\r
-                                instance that it will use to execute the SMBus transactions.\r
+                                instance to use to execute the SMBus transactions.\r
   @param[in]      PecCheck      Defines if Packet Error Code (PEC) checking is required\r
                                 for this operation.\r
-  @param[in, out] Length        Signifies the number of bytes that this operation will do.\r
+  @param[in, out] Length        The number of bytes for this operation\r
   @param[in, out] Buffer        Contains the value of data to execute to the SMBus slave device.\r
 \r
   @retval EFI_SUCCESS           The last data that was returned from the access\r
@@ -54,7 +54,7 @@ typedef struct _EFI_PEI_SMBUS_PPI EFI_PEI_SMBUS_PPI;
   @retval EFI_OUT_OF_RESOURCES  The request could not be completed\r
                                 due to a lack of resources.\r
   @retval EFI_DEVICE_ERROR      The request was not completed because\r
-                                a failure reflected in the Host Status Register bit.\r
+                                a failure was recorded in the Host Status Register bit.\r
   @retval EFI_INVALID_PARAMETER Operation is not defined in EFI_SMBUS_OPERATION.\r
   @retval EFI_INVALID_PARAMETER Length/Buffer is NULL for operations except for EfiSmbusQuickRead and\r
                                 EfiSmbusQuickWrite. Length is outside the range of valid values.\r
@@ -76,15 +76,13 @@ EFI_STATUS
   );\r
 \r
 /**\r
-  CallBack function can be registered in EFI_PEI_SMBUS_PPI_NOTIFY.\r
-\r
-  This function is user-defined and will called when the SlaveAddress/Data pair happens.\r
+  This function is user-defined, and is called when the SlaveAddress/Data pair happens.\r
 \r
   @param[in]  PeiServices    A pointer to the system PEI Services Table.\r
   @param[in]  This           A pointer to the EFI_PEI_SMBUS_PPI instance.\r
   @param[in]  SlaveAddress   The SMBUS hardware address to which the SMBUS\r
                              device is preassigned or allocated.\r
-  @param[in]  Data           Data of the SMBus host notify command that\r
+  @param[in]  Data           Data of the SMBus host notify command, which denotes that\r
                              the caller wants to be called.\r
 \r
   @return Status Code returned by callback function.\r
@@ -100,14 +98,14 @@ EFI_STATUS
   );\r
 \r
 /**\r
-  The ArpDevice() function enumerates the entire bus or enumerates a specific\r
-  device that is identified by SmbusUdid.\r
+  The ArpDevice() function enumerates either the entire bus or a specific\r
+  device identified by SmbusUdid.\r
 \r
   @param[in]      PeiServices   A pointer to the system PEI Services Table.\r
   @param[in]      This          A pointer to the EFI_PEI_SMBUS_PPI instance.\r
   @param[in]      ArpAll        A Boolean expression that indicates if the host drivers need\r
                                 to enumerate all the devices or enumerate only the device that is identified\r
-                                by SmbusUdid. If ArpAll is TRUE, SmbusUdid and SlaveAddress are optional.\r
+                                by SmbusUdid. If ArpAll is TRUE, SmbusUdid and SlaveAddress are optional and ignored if entered.\r
                                 If ArpAll is FALSE, ArpDevice will enumerate SmbusUdid and the address\r
                                 will be at SlaveAddress.\r
   @param[in]      SmbusUdid     The targeted SMBus Unique Device Identifier (UDID).\r
@@ -122,9 +120,9 @@ EFI_STATUS
                                 due to a lack of resources.\r
   @retval EFI_TIMEOUT           The SMBus slave device did not respond.\r
   @retval EFI_DEVICE_ERROR      The request was not completed because the transaction failed.\r
-  @retval EFI_UNSUPPORTED       ArpDevice() are not implemented by this PEIM. \r
+  @retval EFI_UNSUPPORTED       ArpDevice() is not implemented by this PEIM. \r
                                 This return value is not defined in Framwork Specification.\r
-                                This return value had been intruduced in PI Specification.\r
+                                This return value was introduced in the PI Specification.\r
 \r
 **/\r
 typedef\r
@@ -150,7 +148,7 @@ EFI_STATUS
   @retval EFI_SUCCESS       The device map was returned correctly in the buffer.\r
   @retval EFI_UNSUPPORTED   GetArpMap() are not implemented by this PEIM. \r
                             This return value was not defined in Framwork Specification.\r
-                            This return value had been intruduced in PI Specification.\r
+                            This return value was introduced in the PI Specification.\r
 \r
 **/\r
 typedef\r
@@ -174,16 +172,16 @@ EFI_STATUS
   @param[in] PeiServices    A pointer to the system PEI Services Table.\r
   @param[in] This           A pointer to the EFI_PEI_SMBUS_PPI instance.\r
   @param[in] SlaveAddress   Address that the host controller detects as\r
-                            sending a message and calls all the registered functions.\r
+                            sending a message and triggers all the registered functions.\r
   @param[in] Data           Data that the host controller detects as sending a message\r
-                            and calls all the registered functions.\r
+                            and triggers all the registered functions.\r
   @param[in] NotifyFunction The function to call when the bus driver\r
                             detects the SlaveAddress and Data pair.\r
 \r
   @retval EFI_SUCCESS       NotifyFunction has been registered.\r
   @retval EFI_UNSUPPORTED   Notify() are not implemented by this PEIM. \r
                             This return value is not defined in Framwork Specification.\r
-                            This return value had been intruduced in PI Specification.\r
+                            This return value was introduced in the PI Specification.\r
 \r
 **/\r
 typedef\r