]> git.proxmox.com Git - mirror_edk2.git/commitdiff
Clean up file header comments to match CPU I/O Protocol .h file.
authormdkinney <mdkinney@6f19259b-4bc3-4df7-8a09-765794883524>
Thu, 14 Jan 2010 22:12:32 +0000 (22:12 +0000)
committermdkinney <mdkinney@6f19259b-4bc3-4df7-8a09-765794883524>
Thu, 14 Jan 2010 22:12:32 +0000 (22:12 +0000)
Add check to make sure the Address parameter passed in is properly aligned for the requested operation.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@9760 6f19259b-4bc3-4df7-8a09-765794883524

IntelFrameworkModulePkg/Universal/CpuIoDxe/CpuIo.c

index ed1a6df3fc88ca74a5e8dd8d88561dac34c73544..d76ce04227dc4b17972a3b26ddd027f447739a2d 100644 (file)
@@ -124,18 +124,26 @@ UINT8 mOutStride[] = {
 /**\r
   Check parameters to a CPU I/O Protocol service request.\r
 \r
-  @param  Width                 Signifies the width of the I/O or Memory operation.\r
-  @param  Address               The base address of the I/O or Memory operation.\r
-  @param  Count                 The number of I/O or Memory operations to perform.\r
-                                The number of bytes moved is Width size * Count, starting at Address.\r
-  @param  Buffer                For read operations, the destination buffer to store the results.\r
-                                For write operations, the source buffer from which to write data.\r
-  @param  MmioOperation         TRUE for an MMIO operation, FALSE for I/O Port operation.\r
-\r
-  @retval EFI_SUCCESS           The parameters for this request pass the checks.\r
-  @retval EFI_INVALID_PARAMETER Buffer is NULL or Width is not valid.\r
-  @retval EFI_UNSUPPORTED       The address range specified by Address, Width, and Count exceeds Limit.\r
-                                The Buffer is not aligned for the given Width.\r
+  The I/O operations are carried out exactly as requested. The caller is responsible \r
+  for satisfying any alignment and I/O width restrictions that a PI System on a \r
+  platform might require. For example on some platforms, width requests of \r
+  EfiCpuIoWidthUint64 do not work. Misaligned buffers, on the other hand, will \r
+  be handled by the driver.\r
+  \r
+  @param[in] MmioOperation  TRUE for an MMIO operation, FALSE for I/O Port operation.\r
+  @param[in] Width          Signifies the width of the I/O or Memory operation.\r
+  @param[in] Address        The base address of the I/O operation. \r
+  @param[in] Count          The number of I/O operations to perform. The number of  \r
+                            bytes moved is Width size * Count, starting at Address.\r
+  @param[in] Buffer         For read operations, the destination buffer to store the results.\r
+                            For write operations, the source buffer from which to write data.\r
+\r
+  @retval EFI_SUCCESS            The parameters for this request pass the checks.\r
+  @retval EFI_INVALID_PARAMETER  Width is invalid for this PI system.\r
+  @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+  @retval EFI_UNSUPPORTED        The Buffer is not aligned for the given Width.\r
+  @retval EFI_UNSUPPORTED        The address range specified by Address, Width, \r
+                                 and Count is not valid for this PI system.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -149,7 +157,7 @@ CpuIoCheckParameter (
 {\r
   UINT64  MaxCount;\r
   UINT64  Limit;\r
-  \r
+\r
   //\r
   // Check to see if Buffer is NULL\r
   //\r
@@ -180,6 +188,13 @@ CpuIoCheckParameter (
     return EFI_INVALID_PARAMETER;\r
   }\r
   \r
+  //\r
+  // Check to see if Address is alligned\r
+  //\r
+  if ((Address & (UINT64)(mInStride[Width] - 1)) != 0) {\r
+    return EFI_UNSUPPORTED;\r
+  }\r
+\r
   //\r
   // Check to see if any address associated with this transfer exceeds the maximum \r
   // allowed address.  The maximum address implied by the parameters passed in is\r
@@ -209,7 +224,7 @@ CpuIoCheckParameter (
       return EFI_UNSUPPORTED;\r
     }\r
   }\r
-  \r
+\r
   //\r
   // Check to see if Buffer is alligned\r
   //\r
@@ -223,18 +238,40 @@ CpuIoCheckParameter (
 /**\r
   Reads memory-mapped registers.\r
 \r
-  @param  This                  A pointer to the EFI_CPU_IO_PROTOCOL instance.\r
-  @param  Width                 Signifies the width of the I/O or Memory operation.\r
-  @param  Address               The base address of the I/O or Memoryoperation.\r
-  @param  Count                 The number of I/O or Memory operations to perform.\r
-                                The number of bytes moved is Width size * Count, starting at Address.\r
-  @param  Buffer                For read operations, the destination buffer to store the results.\r
-                                For write operations, the source buffer from which to write data.\r
-\r
-  @retval EFI_SUCCESS           The data was read from or written to the EFI system.\r
-  @retval EFI_INVALID_PARAMETER Width is invalid for this EFI system.Or Buffer is NULL.\r
-  @retval EFI_UNSUPPORTED       The Buffer is not aligned for the given Width.\r
-                                Or,The address range specified by Address, Width, and Count is not valid for this EFI system.\r
+  The I/O operations are carried out exactly as requested. The caller is responsible \r
+  for satisfying any alignment and I/O width restrictions that a PI System on a \r
+  platform might require. For example on some platforms, width requests of \r
+  EfiCpuIoWidthUint64 do not work. Misaligned buffers, on the other hand, will \r
+  be handled by the driver.\r
+  \r
+  If Width is EfiCpuIoWidthUint8, EfiCpuIoWidthUint16, EfiCpuIoWidthUint32, \r
+  or EfiCpuIoWidthUint64, then both Address and Buffer are incremented for \r
+  each of the Count operations that is performed.\r
+  \r
+  If Width is EfiCpuIoWidthFifoUint8, EfiCpuIoWidthFifoUint16, \r
+  EfiCpuIoWidthFifoUint32, or EfiCpuIoWidthFifoUint64, then only Buffer is \r
+  incremented for each of the Count operations that is performed. The read or \r
+  write operation is performed Count times on the same Address.\r
+  \r
+  If Width is EfiCpuIoWidthFillUint8, EfiCpuIoWidthFillUint16, \r
+  EfiCpuIoWidthFillUint32, or EfiCpuIoWidthFillUint64, then only Address is \r
+  incremented for each of the Count operations that is performed. The read or \r
+  write operation is performed Count times from the first element of Buffer.\r
+  \r
+  @param[in]  This     A pointer to the EFI_CPU_IO_PROTOCOL instance.\r
+  @param[in]  Width    Signifies the width of the I/O or Memory operation.\r
+  @param[in]  Address  The base address of the I/O operation. \r
+  @param[in]  Count    The number of I/O operations to perform. The number of \r
+                       bytes moved is Width size * Count, starting at Address.\r
+  @param[out] Buffer   For read operations, the destination buffer to store the results.\r
+                       For write operations, the source buffer from which to write data.\r
+\r
+  @retval EFI_SUCCESS            The data was read from or written to the PI system.\r
+  @retval EFI_INVALID_PARAMETER  Width is invalid for this PI system.\r
+  @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+  @retval EFI_UNSUPPORTED        The Buffer is not aligned for the given Width.\r
+  @retval EFI_UNSUPPORTED        The address range specified by Address, Width, \r
+                                 and Count is not valid for this PI system.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -281,18 +318,40 @@ CpuMemoryServiceRead (
 /**\r
   Writes memory-mapped registers.\r
 \r
-  @param  This                  A pointer to the EFI_CPU_IO_PROTOCOL instance.\r
-  @param  Width                 Signifies the width of the I/O or Memory operation.\r
-  @param  Address               The base address of the I/O or Memoryoperation.\r
-  @param  Count                 The number of I/O or Memory operations to perform.\r
-                                The number of bytes moved is Width size * Count, starting at Address.\r
-  @param  Buffer                For read operations, the destination buffer to store the results.\r
-                                For write operations, the source buffer from which to write data.\r
-\r
-  @retval EFI_SUCCESS           The data was read from or written to the EFI system.\r
-  @retval EFI_INVALID_PARAMETER Width is invalid for this EFI system.Or Buffer is NULL.\r
-  @retval EFI_UNSUPPORTED       The Buffer is not aligned for the given Width.\r
-                                Or,The address range specified by Address, Width, and Count is not valid for this EFI system.\r
+  The I/O operations are carried out exactly as requested. The caller is responsible \r
+  for satisfying any alignment and I/O width restrictions that a PI System on a \r
+  platform might require. For example on some platforms, width requests of \r
+  EfiCpuIoWidthUint64 do not work. Misaligned buffers, on the other hand, will \r
+  be handled by the driver.\r
+  \r
+  If Width is EfiCpuIoWidthUint8, EfiCpuIoWidthUint16, EfiCpuIoWidthUint32, \r
+  or EfiCpuIoWidthUint64, then both Address and Buffer are incremented for \r
+  each of the Count operations that is performed.\r
+  \r
+  If Width is EfiCpuIoWidthFifoUint8, EfiCpuIoWidthFifoUint16, \r
+  EfiCpuIoWidthFifoUint32, or EfiCpuIoWidthFifoUint64, then only Buffer is \r
+  incremented for each of the Count operations that is performed. The read or \r
+  write operation is performed Count times on the same Address.\r
+  \r
+  If Width is EfiCpuIoWidthFillUint8, EfiCpuIoWidthFillUint16, \r
+  EfiCpuIoWidthFillUint32, or EfiCpuIoWidthFillUint64, then only Address is \r
+  incremented for each of the Count operations that is performed. The read or \r
+  write operation is performed Count times from the first element of Buffer.\r
+  \r
+  @param[in]  This     A pointer to the EFI_CPU_IO_PROTOCOL instance.\r
+  @param[in]  Width    Signifies the width of the I/O or Memory operation.\r
+  @param[in]  Address  The base address of the I/O operation. \r
+  @param[in]  Count    The number of I/O operations to perform. The number of \r
+                       bytes moved is Width size * Count, starting at Address.\r
+  @param[in]  Buffer   For read operations, the destination buffer to store the results.\r
+                       For write operations, the source buffer from which to write data.\r
+\r
+  @retval EFI_SUCCESS            The data was read from or written to the PI system.\r
+  @retval EFI_INVALID_PARAMETER  Width is invalid for this PI system.\r
+  @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+  @retval EFI_UNSUPPORTED        The Buffer is not aligned for the given Width.\r
+  @retval EFI_UNSUPPORTED        The address range specified by Address, Width, \r
+                                 and Count is not valid for this PI system.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -339,18 +398,40 @@ CpuMemoryServiceWrite (
 /**\r
   Reads I/O registers.\r
 \r
-  @param  This                  A pointer to the EFI_CPU_IO_PROTOCOL instance.\r
-  @param  Width                 Signifies the width of the I/O or Memory operation.\r
-  @param  Address               The base address of the I/O or Memoryoperation.\r
-  @param  Count                 The number of I/O or Memory operations to perform.\r
-                                The number of bytes moved is Width size * Count, starting at Address.\r
-  @param  Buffer                For read operations, the destination buffer to store the results.\r
-                                For write operations, the source buffer from which to write data.\r
-\r
-  @retval EFI_SUCCESS           The data was read from or written to the EFI system.\r
-  @retval EFI_INVALID_PARAMETER Width is invalid for this EFI system.Or Buffer is NULL.\r
-  @retval EFI_UNSUPPORTED       The Buffer is not aligned for the given Width.\r
-                                Or,The address range specified by Address, Width, and Count is not valid for this EFI system.\r
+  The I/O operations are carried out exactly as requested. The caller is responsible \r
+  for satisfying any alignment and I/O width restrictions that a PI System on a \r
+  platform might require. For example on some platforms, width requests of \r
+  EfiCpuIoWidthUint64 do not work. Misaligned buffers, on the other hand, will \r
+  be handled by the driver.\r
+  \r
+  If Width is EfiCpuIoWidthUint8, EfiCpuIoWidthUint16, EfiCpuIoWidthUint32, \r
+  or EfiCpuIoWidthUint64, then both Address and Buffer are incremented for \r
+  each of the Count operations that is performed.\r
+  \r
+  If Width is EfiCpuIoWidthFifoUint8, EfiCpuIoWidthFifoUint16, \r
+  EfiCpuIoWidthFifoUint32, or EfiCpuIoWidthFifoUint64, then only Buffer is \r
+  incremented for each of the Count operations that is performed. The read or \r
+  write operation is performed Count times on the same Address.\r
+  \r
+  If Width is EfiCpuIoWidthFillUint8, EfiCpuIoWidthFillUint16, \r
+  EfiCpuIoWidthFillUint32, or EfiCpuIoWidthFillUint64, then only Address is \r
+  incremented for each of the Count operations that is performed. The read or \r
+  write operation is performed Count times from the first element of Buffer.\r
+  \r
+  @param[in]  This     A pointer to the EFI_CPU_IO_PROTOCOL instance.\r
+  @param[in]  Width    Signifies the width of the I/O or Memory operation.\r
+  @param[in]  Address  The base address of the I/O operation. \r
+  @param[in]  Count    The number of I/O operations to perform. The number of \r
+                       bytes moved is Width size * Count, starting at Address.\r
+  @param[out] Buffer   For read operations, the destination buffer to store the results.\r
+                       For write operations, the source buffer from which to write data.\r
+\r
+  @retval EFI_SUCCESS            The data was read from or written to the PI system.\r
+  @retval EFI_INVALID_PARAMETER  Width is invalid for this PI system.\r
+  @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+  @retval EFI_UNSUPPORTED        The Buffer is not aligned for the given Width.\r
+  @retval EFI_UNSUPPORTED        The address range specified by Address, Width, \r
+                                 and Count is not valid for this PI system.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -396,19 +477,41 @@ CpuIoServiceRead (
 /**\r
   Write I/O registers.\r
 \r
-  @param  This                  A pointer to the EFI_CPU_IO_PROTOCOL instance.\r
-  @param  Width                 Signifies the width of the I/O or Memory operation.\r
-  @param  Address               The base address of the I/O or Memoryoperation.\r
-  @param  Count                 The number of I/O or Memory operations to perform.\r
-                                The number of bytes moved is Width size * Count, starting at Address.\r
-  @param  Buffer                For read operations, the destination buffer to store the results.\r
-                                For write operations, the source buffer from which to write data.\r
-\r
-  @retval EFI_SUCCESS           The data was read from or written to the EFI system.\r
-  @retval EFI_INVALID_PARAMETER Width is invalid for this EFI system.Or Buffer is NULL.\r
-  @retval EFI_UNSUPPORTED       The Buffer is not aligned for the given Width.\r
-                                Or,The address range specified by Address, Width, and Count is not valid for this EFI system.\r
-\r
+  The I/O operations are carried out exactly as requested. The caller is responsible \r
+  for satisfying any alignment and I/O width restrictions that a PI System on a \r
+  platform might require. For example on some platforms, width requests of \r
+  EfiCpuIoWidthUint64 do not work. Misaligned buffers, on the other hand, will \r
+  be handled by the driver.\r
+  \r
+  If Width is EfiCpuIoWidthUint8, EfiCpuIoWidthUint16, EfiCpuIoWidthUint32, \r
+  or EfiCpuIoWidthUint64, then both Address and Buffer are incremented for \r
+  each of the Count operations that is performed.\r
+  \r
+  If Width is EfiCpuIoWidthFifoUint8, EfiCpuIoWidthFifoUint16, \r
+  EfiCpuIoWidthFifoUint32, or EfiCpuIoWidthFifoUint64, then only Buffer is \r
+  incremented for each of the Count operations that is performed. The read or \r
+  write operation is performed Count times on the same Address.\r
+  \r
+  If Width is EfiCpuIoWidthFillUint8, EfiCpuIoWidthFillUint16, \r
+  EfiCpuIoWidthFillUint32, or EfiCpuIoWidthFillUint64, then only Address is \r
+  incremented for each of the Count operations that is performed. The read or \r
+  write operation is performed Count times from the first element of Buffer.\r
+  \r
+  @param[in]  This     A pointer to the EFI_CPU_IO_PROTOCOL instance.\r
+  @param[in]  Width    Signifies the width of the I/O or Memory operation.\r
+  @param[in]  Address  The base address of the I/O operation. \r
+  @param[in]  Count    The number of I/O operations to perform. The number of \r
+                       bytes moved is Width size * Count, starting at Address.\r
+  @param[in]  Buffer   For read operations, the destination buffer to store the results.\r
+                       For write operations, the source buffer from which to write data.\r
+\r
+  @retval EFI_SUCCESS            The data was read from or written to the PI system.\r
+  @retval EFI_INVALID_PARAMETER  Width is invalid for this PI system.\r
+  @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+  @retval EFI_UNSUPPORTED        The Buffer is not aligned for the given Width.\r
+  @retval EFI_UNSUPPORTED        The address range specified by Address, Width, \r
+                                 and Count is not valid for this PI system.\r
+                                 \r
 **/\r
 EFI_STATUS\r
 EFIAPI\r