Reviewed the code comments in the Include/Protocol directory for typos, grammar issue...

[mirror_edk2.git] / MdePkg / Include / Protocol / DeviceIo.h

diff --git a/MdePkg/Include/Protocol/DeviceIo.h b/MdePkg/Include/Protocol/DeviceIo.h
index cab447fc9851d4c9342b9c65cd2307d9fe5eb260..ea3949cada4abfaa33af35281ca72803cab69bd7 100644 (file)
--- a/MdePkg/Include/Protocol/DeviceIo.h
+++ b/MdePkg/Include/Protocol/DeviceIo.h
@@ -35,11 +35,18 @@ typedef struct _EFI_DEVICE_IO_PROTOCOL EFI_DEVICE_IO_PROTOCOL;
 /// \r
 typedef EFI_DEVICE_IO_PROTOCOL  EFI_DEVICE_IO_INTERFACE;\r
 \r
+///\r
+/// Device IO Access Width \r
+///\r
 typedef enum {\r
   IO_UINT8  = 0,\r
   IO_UINT16 = 1,\r
   IO_UINT32 = 2,\r
   IO_UINT64 = 3,\r
+  //\r
+  // Below enumerations are added in "Extensible Firmware Interface Specification, \r
+  // Version 1.10, Specification Update, Version 001".\r
+  //\r
   MMIO_COPY_UINT8  = 4,\r
   MMIO_COPY_UINT16 = 5,\r
   MMIO_COPY_UINT32 = 6,\r
@@ -115,7 +122,7 @@ typedef enum {
   ///\r
   /// Provides both read and write access to system memory\r
   /// by both the processor and a bus master. The buffer is\r
-  /// coherent from both the processor¡¯s and the bus master¡¯s\r
+  /// coherent from both the processor's and the bus master's\r
   /// point of view.\r
   ///\r
   EfiBusMasterCommonBuffer\r
@@ -127,7 +134,7 @@ typedef enum {
   @param  This                  A pointer to the EFI_DEVICE_IO_INTERFACE instance.\r
   @param  Operation             Indicates if the bus master is going to read or write to system memory.\r
   @param  HostAddress           The system memory address to map to the device.\r
-  @param  NumberOfBytes         On input the number of bytes to map.\r
+  @param  NumberOfBytes         On input, the number of bytes to map.\r
   @param  DeviceAddress         The resulting map address for the bus master device to use to access the\r
                                 hosts HostAddress.\r
   @param  Mapping               A resulting value to pass to Unmap().\r