]> git.proxmox.com Git - mirror_edk2.git/blobdiff - PcAtChipsetPkg/PciHostBridgeDxe/PciRootBridgeIo.c
Add generic HPET Timer DXE Driver and support libraries
[mirror_edk2.git] / PcAtChipsetPkg / PciHostBridgeDxe / PciRootBridgeIo.c
index 36f83f62e00c926f7d916d7e06ee6ed5cd0944ba..512a5049c117fe46337dd06a1d7f171578fa47ba 100644 (file)
@@ -1,14 +1,14 @@
 /** @file\r
   PCI Root Bridge Io Protocol implementation\r
 \r
-  Copyright (c) 2008 - 2009, Intel Corporation<BR> All rights\r
-  reserved. This program and the accompanying materials are\r
-  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
-  http://opensource.org/licenses/bsd-license.php\r
-  \r
-  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
+Copyright (c) 2008 - 2010, Intel Corporation. All rights reserved.<BR>\r
+This program and the accompanying materials are\r
+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
+http://opensource.org/licenses/bsd-license.php\r
+\r
+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
 **/ \r
 \r
@@ -33,6 +33,35 @@ RESOURCE_CONFIGURATION Configuration = {
 // Protocol Member Function Prototypes\r
 //\r
 \r
+/**\r
+   Polls an address in memory mapped I/O space until an exit condition is met, or \r
+   a timeout occurs. \r
+\r
+   This function provides a standard way to poll a PCI memory location. A PCI memory read\r
+   operation is performed at the PCI memory address specified by Address for the width specified\r
+   by Width. The result of this PCI memory read operation is stored in Result. This PCI memory\r
+   read operation is repeated until either a timeout of Delay 100 ns units has expired, or (Result &\r
+   Mask) is equal to Value.\r
+\r
+   @param[in]   This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width     Signifies the width of the memory operations.\r
+   @param[in]   Address   The base address of the memory operations. The caller is\r
+                          responsible for aligning Address if required.\r
+   @param[in]   Mask      Mask used for the polling criteria. Bytes above Width in Mask\r
+                          are ignored. The bits in the bytes below Width which are zero in\r
+                          Mask are ignored when polling the memory address.\r
+   @param[in]   Value     The comparison value used for the polling exit criteria.\r
+   @param[in]   Delay     The number of 100 ns units to poll. Note that timer available may\r
+                          be of poorer granularity.\r
+   @param[out]  Result    Pointer to the last value read from the memory location.\r
+   \r
+   @retval EFI_SUCCESS            The last data returned from the access matched the poll exit criteria.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid.\r
+   @retval EFI_INVALID_PARAMETER  Result is NULL.\r
+   @retval EFI_TIMEOUT            Delay expired before a match occurred.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoPollMem ( \r
@@ -45,6 +74,35 @@ RootBridgeIoPollMem (
   OUT UINT64                                 *Result\r
   );\r
   \r
+/**\r
+   Reads from the I/O space of a PCI Root Bridge. Returns when either the polling exit criteria is\r
+   satisfied or after a defined duration.\r
+\r
+   This function provides a standard way to poll a PCI I/O location. A PCI I/O read operation is\r
+   performed at the PCI I/O address specified by Address for the width specified by Width.\r
+   The result of this PCI I/O read operation is stored in Result. This PCI I/O read operation is\r
+   repeated until either a timeout of Delay 100 ns units has expired, or (Result & Mask) is equal\r
+   to Value.\r
+\r
+   @param[in] This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in] Width     Signifies the width of the I/O operations.\r
+   @param[in] Address   The base address of the I/O operations. The caller is responsible\r
+                        for aligning Address if required.\r
+   @param[in] Mask      Mask used for the polling criteria. Bytes above Width in Mask\r
+                        are ignored. The bits in the bytes below Width which are zero in\r
+                        Mask are ignored when polling the I/O address.\r
+   @param[in] Value     The comparison value used for the polling exit criteria.\r
+   @param[in] Delay     The number of 100 ns units to poll. Note that timer available may\r
+                        be of poorer granularity.\r
+   @param[out] Result   Pointer to the last value read from the memory location.\r
+   \r
+   @retval EFI_SUCCESS            The last data returned from the access matched the poll exit criteria.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid.\r
+   @retval EFI_INVALID_PARAMETER  Result is NULL.\r
+   @retval EFI_TIMEOUT            Delay expired before a match occurred.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoPollIo ( \r
@@ -57,6 +115,29 @@ RootBridgeIoPollIo (
   OUT UINT64                                 *Result\r
   );\r
   \r
+/**\r
+   Enables a PCI driver to access PCI controller registers in the PCI root bridge memory space.\r
+\r
+   The Mem.Read(), and Mem.Write() functions enable a driver to access PCI controller\r
+   registers in the PCI root bridge memory space.\r
+   The memory operations are carried out exactly as requested. The caller is responsible for satisfying\r
+   any alignment and memory width restrictions that a PCI Root Bridge on a platform might require.\r
+\r
+   @param[in]   This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width     Signifies the width of the memory operation.\r
+   @param[in]   Address   The base address of the memory operation. The caller is\r
+                          responsible for aligning the Address if required.\r
+   @param[in]   Count     The number of memory operations to perform. Bytes moved is\r
+                          Width size * Count, starting at Address.\r
+   @param[out]  Buffer    For read operations, the destination buffer to store the results. For\r
+                          write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS            The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoMemRead (\r
@@ -64,9 +145,31 @@ RootBridgeIoMemRead (
   IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
   IN     UINT64                                 Address,\r
   IN     UINTN                                  Count,\r
-  IN OUT VOID                                   *Buffer\r
+  OUT    VOID                                   *Buffer\r
   );\r
 \r
+/**\r
+   Enables a PCI driver to access PCI controller registers in the PCI root bridge memory space.\r
+\r
+   The Mem.Read(), and Mem.Write() functions enable a driver to access PCI controller\r
+   registers in the PCI root bridge memory space.\r
+   The memory operations are carried out exactly as requested. The caller is responsible for satisfying\r
+   any alignment and memory width restrictions that a PCI Root Bridge on a platform might require.\r
+\r
+   @param[in]   This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width     Signifies the width of the memory operation.\r
+   @param[in]   Address   The base address of the memory operation. The caller is\r
+                          responsible for aligning the Address if required.\r
+   @param[in]   Count     The number of memory operations to perform. Bytes moved is\r
+                          Width size * Count, starting at Address.\r
+   @param[in]   Buffer    For read operations, the destination buffer to store the results. For\r
+                          write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS            The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoMemWrite (\r
@@ -74,9 +177,27 @@ RootBridgeIoMemWrite (
   IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
   IN     UINT64                                 Address,\r
   IN     UINTN                                  Count,\r
-  IN OUT VOID                                   *Buffer\r
+  IN     VOID                                   *Buffer\r
   );\r
 \r
+/**\r
+   Enables a PCI driver to access PCI controller registers in the PCI root bridge I/O space.\r
+\r
+   @param[in]   This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width       Signifies the width of the memory operations.\r
+   @param[in]   UserAddress The base address of the I/O operation. The caller is responsible for\r
+                            aligning the Address if required.\r
+   @param[in]   Count       The number of I/O operations to perform. Bytes moved is Width\r
+                            size * Count, starting at Address.\r
+   @param[out]  UserBuffer  For read operations, the destination buffer to store the results. For\r
+                            write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS              The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER    Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER    Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES     The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoIoRead (\r
@@ -84,9 +205,27 @@ RootBridgeIoIoRead (
   IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
   IN     UINT64                                 UserAddress,\r
   IN     UINTN                                  Count,\r
-  IN OUT VOID                                   *UserBuffer\r
+  OUT    VOID                                   *UserBuffer\r
   );\r
 \r
+/**\r
+   Enables a PCI driver to access PCI controller registers in the PCI root bridge I/O space.\r
+\r
+   @param[in]   This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width       Signifies the width of the memory operations.\r
+   @param[in]   UserAddress The base address of the I/O operation. The caller is responsible for\r
+                            aligning the Address if required.\r
+   @param[in]   Count       The number of I/O operations to perform. Bytes moved is Width\r
+                            size * Count, starting at Address.\r
+   @param[in]   UserBuffer  For read operations, the destination buffer to store the results. For\r
+                            write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS              The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER    Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER    Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES     The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoIoWrite (\r
@@ -94,9 +233,33 @@ RootBridgeIoIoWrite (
   IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
   IN     UINT64                                 UserAddress,\r
   IN     UINTN                                  Count,\r
-  IN OUT VOID                                   *UserBuffer\r
+  IN     VOID                                   *UserBuffer\r
   );\r
 \r
+/**\r
+   Enables a PCI driver to copy one region of PCI root bridge memory space to another region of PCI\r
+   root bridge memory space.\r
+\r
+   The CopyMem() function enables a PCI driver to copy one region of PCI root bridge memory\r
+   space to another region of PCI root bridge memory space. This is especially useful for video scroll\r
+   operation on a memory mapped video buffer.\r
+   The memory operations are carried out exactly as requested. The caller is responsible for satisfying\r
+   any alignment and memory width restrictions that a PCI root bridge on a platform might require.\r
+\r
+   @param[in] This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL instance.\r
+   @param[in] Width       Signifies the width of the memory operations.\r
+   @param[in] DestAddress The destination address of the memory operation. The caller is\r
+                          responsible for aligning the DestAddress if required.\r
+   @param[in] SrcAddress  The source address of the memory operation. The caller is\r
+                          responsible for aligning the SrcAddress if required.\r
+   @param[in] Count       The number of memory operations to perform. Bytes moved is\r
+                          Width size * Count, starting at DestAddress and SrcAddress.\r
+   \r
+   @retval  EFI_SUCCESS             The data was copied from one memory region to another memory region.\r
+   @retval  EFI_INVALID_PARAMETER   Width is invalid for this PCI root bridge.\r
+   @retval  EFI_OUT_OF_RESOURCES    The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoCopyMem (\r
@@ -107,6 +270,29 @@ RootBridgeIoCopyMem (
   IN     UINTN                                  Count\r
   );\r
 \r
+/**\r
+   Enables a PCI driver to access PCI controller registers in a PCI root bridge's configuration space.\r
+\r
+   The Pci.Read() and Pci.Write() functions enable a driver to access PCI configuration\r
+   registers for a PCI controller.\r
+   The PCI Configuration operations are carried out exactly as requested. The caller is responsible for\r
+   any alignment and PCI configuration width issues that a PCI Root Bridge on a platform might\r
+   require.\r
+\r
+   @param[in]   This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width     Signifies the width of the memory operations.\r
+   @param[in]   Address   The address within the PCI configuration space for the PCI controller.\r
+   @param[in]   Count     The number of PCI configuration operations to perform. Bytes\r
+                          moved is Width size * Count, starting at Address.\r
+   @param[out]  Buffer    For read operations, the destination buffer to store the results. For\r
+                          write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS            The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoPciRead (\r
@@ -114,9 +300,32 @@ RootBridgeIoPciRead (
   IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
   IN     UINT64                                 Address,\r
   IN     UINTN                                  Count,\r
-  IN OUT VOID                                   *Buffer\r
+  OUT    VOID                                   *Buffer\r
   );\r
 \r
+/**\r
+   Enables a PCI driver to access PCI controller registers in a PCI root bridge's configuration space.\r
+\r
+   The Pci.Read() and Pci.Write() functions enable a driver to access PCI configuration\r
+   registers for a PCI controller.\r
+   The PCI Configuration operations are carried out exactly as requested. The caller is responsible for\r
+   any alignment and PCI configuration width issues that a PCI Root Bridge on a platform might\r
+   require.\r
+\r
+   @param[in]   This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width     Signifies the width of the memory operations.\r
+   @param[in]   Address   The address within the PCI configuration space for the PCI controller.\r
+   @param[in]   Count     The number of PCI configuration operations to perform. Bytes\r
+                          moved is Width size * Count, starting at Address.\r
+   @param[in]   Buffer    For read operations, the destination buffer to store the results. For\r
+                          write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS            The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoPciWrite (\r
@@ -124,9 +333,35 @@ RootBridgeIoPciWrite (
   IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
   IN     UINT64                                 Address,\r
   IN     UINTN                                  Count,\r
-  IN OUT VOID                                   *Buffer\r
+  IN     VOID                                   *Buffer\r
   );\r
 \r
+/**\r
+   Provides the PCI controller-specific addresses required to access system memory from a\r
+   DMA bus master.\r
+\r
+   The Map() function provides the PCI controller specific addresses needed to access system\r
+   memory. This function is used to map system memory for PCI bus master DMA accesses.\r
+\r
+   @param[in]       This            A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]       Operation       Indicates if the bus master is going to read or write to system memory.\r
+   @param[in]       HostAddress     The system memory address to map to the PCI controller.\r
+   @param[in, out]  NumberOfBytes   On input the number of bytes to map. On output the number of bytes that were mapped.\r
+   @param[out]      DeviceAddress   The resulting map address for the bus master PCI controller to use\r
+                                    to access the system memory's HostAddress.\r
+   @param[out]      Mapping         The value to pass to Unmap() when the bus master DMA operation is complete.\r
+   \r
+   @retval EFI_SUCCESS            The range was mapped for the returned NumberOfBytes.\r
+   @retval EFI_INVALID_PARAMETER  Operation is invalid.\r
+   @retval EFI_INVALID_PARAMETER  HostAddress is NULL.\r
+   @retval EFI_INVALID_PARAMETER  NumberOfBytes is NULL.\r
+   @retval EFI_INVALID_PARAMETER  DeviceAddress is NULL.\r
+   @retval EFI_INVALID_PARAMETER  Mapping is NULL.\r
+   @retval EFI_UNSUPPORTED        The HostAddress cannot be mapped as a common buffer.\r
+   @retval EFI_DEVICE_ERROR       The system hardware could not map the requested address.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoMap (\r
@@ -138,6 +373,22 @@ RootBridgeIoMap (
   OUT    VOID                                       **Mapping\r
   );\r
 \r
+/**\r
+   Completes the Map() operation and releases any corresponding resources.\r
+\r
+   The Unmap() function completes the Map() operation and releases any corresponding resources.\r
+   If the operation was an EfiPciOperationBusMasterWrite or\r
+   EfiPciOperationBusMasterWrite64, the data is committed to the target system memory.\r
+   Any resources used for the mapping are freed.  \r
+\r
+   @param[in] This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in] Mapping   The mapping value returned from Map().\r
+   \r
+   @retval EFI_SUCCESS            The range was unmapped.\r
+   @retval EFI_INVALID_PARAMETER  Mapping is not a value that was returned by Map().\r
+   @retval EFI_DEVICE_ERROR       The data was not committed to the target system memory.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoUnmap (\r
@@ -145,6 +396,27 @@ RootBridgeIoUnmap (
   IN  VOID                             *Mapping\r
   );\r
 \r
+/**\r
+   Allocates pages that are suitable for an EfiPciOperationBusMasterCommonBuffer or\r
+   EfiPciOperationBusMasterCommonBuffer64 mapping.\r
+  \r
+   @param This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param Type        This parameter is not used and must be ignored.\r
+   @param MemoryType  The type of memory to allocate, EfiBootServicesData or EfiRuntimeServicesData.\r
+   @param Pages       The number of pages to allocate.\r
+   @param HostAddress A pointer to store the base system memory address of the allocated range.\r
+   @param Attributes  The requested bit mask of attributes for the allocated range. Only\r
+                      the attributes EFI_PCI_ATTRIBUTE_MEMORY_WRITE_COMBINE, EFI_PCI_ATTRIBUTE_MEMORY_CACHED, \r
+                      and EFI_PCI_ATTRIBUTE_DUAL_ADDRESS_CYCLE may be used with this function.\r
+   \r
+   @retval EFI_SUCCESS            The requested memory pages were allocated.\r
+   @retval EFI_INVALID_PARAMETER  MemoryType is invalid.\r
+   @retval EFI_INVALID_PARAMETER  HostAddress is NULL.\r
+   @retval EFI_UNSUPPORTED        Attributes is unsupported. The only legal attribute bits are\r
+                                  MEMORY_WRITE_COMBINE, MEMORY_CACHED, and DUAL_ADDRESS_CYCLE.\r
+   @retval EFI_OUT_OF_RESOURCES   The memory pages could not be allocated.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoAllocateBuffer (\r
@@ -156,6 +428,20 @@ RootBridgeIoAllocateBuffer (
   IN  UINT64                           Attributes\r
   );\r
 \r
+/**\r
+   Frees memory that was allocated with AllocateBuffer().\r
+\r
+   The FreeBuffer() function frees memory that was allocated with AllocateBuffer().\r
+\r
+   @param This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param Pages       The number of pages to free.\r
+   @param HostAddress The base system memory address of the allocated range.\r
+   \r
+   @retval EFI_SUCCESS            The requested memory pages were freed.\r
+   @retval EFI_INVALID_PARAMETER  The memory range specified by HostAddress and Pages\r
+                                  was not allocated with AllocateBuffer().\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoFreeBuffer (\r
@@ -164,12 +450,51 @@ RootBridgeIoFreeBuffer (
   OUT VOID                             *HostAddress\r
   );\r
 \r
+/**\r
+   Flushes all PCI posted write transactions from a PCI host bridge to system memory.\r
+\r
+   The Flush() function flushes any PCI posted write transactions from a PCI host bridge to system\r
+   memory. Posted write transactions are generated by PCI bus masters when they perform write\r
+   transactions to target addresses in system memory.\r
+   This function does not flush posted write transactions from any PCI bridges. A PCI controller\r
+   specific action must be taken to guarantee that the posted write transactions have been flushed from\r
+   the PCI controller and from all the PCI bridges into the PCI host bridge. This is typically done with\r
+   a PCI read transaction from the PCI controller prior to calling Flush().\r
+\r
+   @param This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   \r
+   @retval EFI_SUCCESS        The PCI posted write transactions were flushed from the PCI host\r
+                              bridge to system memory.\r
+   @retval EFI_DEVICE_ERROR   The PCI posted write transactions were not flushed from the PCI\r
+                              host bridge due to a hardware error.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoFlush (\r
   IN  EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL  *This\r
   );\r
 \r
+/**\r
+   Gets the attributes that a PCI root bridge supports setting with SetAttributes(), and the\r
+   attributes that a PCI root bridge is currently using.  \r
+\r
+   The GetAttributes() function returns the mask of attributes that this PCI root bridge supports\r
+   and the mask of attributes that the PCI root bridge is currently using.\r
+\r
+   @param This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param Supported   A pointer to the mask of attributes that this PCI root bridge\r
+                      supports setting with SetAttributes().\r
+   @param Attributes  A pointer to the mask of attributes that this PCI root bridge is\r
+                      currently using.\r
+   \r
+   @retval  EFI_SUCCESS           If Supports is not NULL, then the attributes that the PCI root\r
+                                  bridge supports is returned in Supports. If Attributes is\r
+                                  not NULL, then the attributes that the PCI root bridge is currently\r
+                                  using is returned in Attributes.\r
+   @retval  EFI_INVALID_PARAMETER Both Supports and Attributes are NULL.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoGetAttributes (\r
@@ -178,6 +503,36 @@ RootBridgeIoGetAttributes (
   OUT UINT64                           *Attributes\r
   );\r
 \r
+/**\r
+   Sets attributes for a resource range on a PCI root bridge.\r
+\r
+   The SetAttributes() function sets the attributes specified in Attributes for the PCI root\r
+   bridge on the resource range specified by ResourceBase and ResourceLength. Since the\r
+   granularity of setting these attributes may vary from resource type to resource type, and from\r
+   platform to platform, the actual resource range and the one passed in by the caller may differ. As a\r
+   result, this function may set the attributes specified by Attributes on a larger resource range\r
+   than the caller requested. The actual range is returned in ResourceBase and\r
+   ResourceLength. The caller is responsible for verifying that the actual range for which the\r
+   attributes were set is acceptable.\r
+\r
+   @param[in]       This            A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]       Attributes      The mask of attributes to set. If the attribute bit\r
+                                    MEMORY_WRITE_COMBINE, MEMORY_CACHED, or\r
+                                    MEMORY_DISABLE is set, then the resource range is specified by\r
+                                    ResourceBase and ResourceLength. If\r
+                                    MEMORY_WRITE_COMBINE, MEMORY_CACHED, and\r
+                                    MEMORY_DISABLE are not set, then ResourceBase and\r
+                                    ResourceLength are ignored, and may be NULL.\r
+   @param[in, out]  ResourceBase    A pointer to the base address of the resource range to be modified\r
+                                    by the attributes specified by Attributes.\r
+   @param[in, out]  ResourceLength  A pointer to the length of the resource range to be modified by the\r
+                                    attributes specified by Attributes.\r
+   \r
+   @retval  EFI_SUCCESS     The current configuration of this PCI root bridge was returned in Resources.\r
+   @retval  EFI_UNSUPPORTED The current configuration of this PCI root bridge could not be retrieved.\r
+   @retval  EFI_INVALID_PARAMETER Invalid pointer of EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoSetAttributes (\r
@@ -187,6 +542,29 @@ RootBridgeIoSetAttributes (
   IN OUT UINT64                           *ResourceLength \r
   ); \r
 \r
+/**\r
+   Retrieves the current resource settings of this PCI root bridge in the form of a set of ACPI 2.0\r
+   resource descriptors.\r
+\r
+   There are only two resource descriptor types from the ACPI Specification that may be used to\r
+   describe the current resources allocated to a PCI root bridge. These are the QWORD Address\r
+   Space Descriptor (ACPI 2.0 Section 6.4.3.5.1), and the End Tag (ACPI 2.0 Section 6.4.2.8). The\r
+   QWORD Address Space Descriptor can describe memory, I/O, and bus number ranges for dynamic\r
+   or fixed resources. The configuration of a PCI root bridge is described with one or more QWORD\r
+   Address Space Descriptors followed by an End Tag.\r
+\r
+   @param[in]   This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[out]  Resources   A pointer to the ACPI 2.0 resource descriptors that describe the\r
+                            current configuration of this PCI root bridge. The storage for the\r
+                            ACPI 2.0 resource descriptors is allocated by this function. The\r
+                            caller must treat the return buffer as read-only data, and the buffer\r
+                            must not be freed by the caller.\r
+   \r
+   @retval  EFI_SUCCESS     The current configuration of this PCI root bridge was returned in Resources.\r
+   @retval  EFI_UNSUPPORTED The current configuration of this PCI root bridge could not be retrieved.\r
+   @retval  EFI_INVALID_PARAMETER Invalid pointer of EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoConfiguration (\r
@@ -195,24 +573,58 @@ RootBridgeIoConfiguration (
   );\r
 \r
 //\r
-// Sub Function Prototypes\r
+// Memory Controller Pci Root Bridge Io Module Variables\r
 //\r
-EFI_STATUS\r
-RootBridgeIoPciRW (\r
-  IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL        *This,\r
-  IN     BOOLEAN                                Write,\r
-  IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
-  IN     UINT64                                 UserAddress,\r
-  IN     UINTN                                  Count,\r
-  IN OUT VOID                                   *UserBuffer\r
-  );\r
+EFI_METRONOME_ARCH_PROTOCOL *mMetronome;\r
 \r
 //\r
-// Memory Controller Pci Root Bridge Io Module Variables\r
+// Lookup table for increment values based on transfer widths\r
 //\r
-EFI_METRONOME_ARCH_PROTOCOL *mMetronome;\r
-EFI_CPU_IO2_PROTOCOL *mCpuIo;\r
+UINT8 mInStride[] = {\r
+  1, // EfiPciWidthUint8\r
+  2, // EfiPciWidthUint16\r
+  4, // EfiPciWidthUint32\r
+  8, // EfiPciWidthUint64\r
+  0, // EfiPciWidthFifoUint8\r
+  0, // EfiPciWidthFifoUint16\r
+  0, // EfiPciWidthFifoUint32\r
+  0, // EfiPciWidthFifoUint64\r
+  1, // EfiPciWidthFillUint8\r
+  2, // EfiPciWidthFillUint16\r
+  4, // EfiPciWidthFillUint32\r
+  8  // EfiPciWidthFillUint64\r
+};\r
 \r
+//\r
+// Lookup table for increment values based on transfer widths\r
+//\r
+UINT8 mOutStride[] = {\r
+  1, // EfiPciWidthUint8\r
+  2, // EfiPciWidthUint16\r
+  4, // EfiPciWidthUint32\r
+  8, // EfiPciWidthUint64\r
+  1, // EfiPciWidthFifoUint8\r
+  2, // EfiPciWidthFifoUint16\r
+  4, // EfiPciWidthFifoUint32\r
+  8, // EfiPciWidthFifoUint64\r
+  0, // EfiPciWidthFillUint8\r
+  0, // EfiPciWidthFillUint16\r
+  0, // EfiPciWidthFillUint32\r
+  0  // EfiPciWidthFillUint64\r
+};\r
+\r
+/**\r
+\r
+  Construct the Pci Root Bridge Io protocol\r
+\r
+  @param Protocol         Point to protocol instance\r
+  @param HostBridgeHandle Handle of host bridge\r
+  @param Attri            Attribute of host bridge\r
+  @param ResAppeture      ResourceAppeture for host bridge\r
+\r
+  @retval EFI_SUCCESS Success to initialize the Pci Root Bridge.\r
+\r
+**/\r
 EFI_STATUS\r
 RootBridgeConstructor (\r
   IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL    *Protocol,\r
@@ -220,21 +632,6 @@ RootBridgeConstructor (
   IN UINT64                             Attri,\r
   IN PCI_ROOT_BRIDGE_RESOURCE_APPETURE  *ResAppeture\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-    Construct the Pci Root Bridge Io protocol\r
-\r
-Arguments:\r
-\r
-    Protocol - protocol to initialize\r
-    \r
-Returns:\r
-\r
-    None\r
-\r
---*/\r
 {\r
   EFI_STATUS                        Status;\r
   PCI_ROOT_BRIDGE_INSTANCE          *PrivateData;\r
@@ -272,15 +669,16 @@ Returns:
     PrivateData->ResAllocNode[Index].Status    = ResNone;\r
   }\r
   \r
-\r
-  EfiInitializeLock (&PrivateData->PciLock, TPL_HIGH_LEVEL);\r
   PrivateData->PciAddress = 0xCF8;\r
   PrivateData->PciData    = 0xCFC;\r
 \r
   PrivateData->RootBridgeAttrib = Attri;\r
   \r
-  PrivateData->Attributes  = 0;\r
-  PrivateData->Supports    = 0;\r
+  PrivateData->Supports    = EFI_PCI_ATTRIBUTE_IDE_PRIMARY_IO | EFI_PCI_ATTRIBUTE_IDE_SECONDARY_IO | \\r
+                             EFI_PCI_ATTRIBUTE_ISA_IO_16 | EFI_PCI_ATTRIBUTE_ISA_MOTHERBOARD_IO | \\r
+                             EFI_PCI_ATTRIBUTE_VGA_MEMORY | \\r
+                             EFI_PCI_ATTRIBUTE_VGA_IO_16  | EFI_PCI_ATTRIBUTE_VGA_PALETTE_IO_16;\r
+  PrivateData->Attributes  = PrivateData->Supports;\r
 \r
   Protocol->ParentHandle   = HostBridgeHandle;\r
   \r
@@ -313,15 +711,462 @@ Returns:
 \r
   Protocol->SegmentNumber  = 0;\r
 \r
-  Status = gBS->LocateProtocol (&gEfiCpuIo2ProtocolGuid, NULL, (VOID **)&mCpuIo);\r
-  ASSERT_EFI_ERROR (Status);\r
-\r
   Status = gBS->LocateProtocol (&gEfiMetronomeArchProtocolGuid, NULL, (VOID **)&mMetronome);\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+  Check parameters for IO,MMIO,PCI read/write services of PCI Root Bridge IO.\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
+  @param[in] This           A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+  @param[in] OperationType  I/O operation type: IO/MMIO/PCI.\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
+RootBridgeIoCheckParameter (\r
+  IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL        *This,\r
+  IN OPERATION_TYPE                         OperationType,\r
+  IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
+  IN UINT64                                 Address,\r
+  IN UINTN                                  Count,\r
+  IN VOID                                   *Buffer\r
+  )\r
+{\r
+  PCI_ROOT_BRIDGE_INSTANCE                     *PrivateData;\r
+  EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_PCI_ADDRESS  *PciRbAddr;\r
+  UINT64                                       MaxCount;\r
+  UINT64                                       Base;\r
+  UINT64                                       Limit;\r
+\r
+  //\r
+  // Check to see if Buffer is NULL\r
+  //\r
+  if (Buffer == NULL) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
+  //\r
+  // Check to see if Width is in the valid range\r
+  //\r
+  if (Width < EfiPciWidthUint8 || Width >= EfiPciWidthMaximum) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
+  //\r
+  // For FIFO type, the target address won't increase during the access,\r
+  // so treat Count as 1\r
+  //\r
+  if (Width >= EfiPciWidthFifoUint8 && Width <= EfiPciWidthFifoUint64) {\r
+    Count = 1;\r
+  }\r
+\r
+  //\r
+  // Check to see if Width is in the valid range for I/O Port operations\r
+  //\r
+  Width = (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH) (Width & 0x03);\r
+  if ((OperationType != MemOperation) && (Width == EfiPciWidthUint64)) {\r
+    ASSERT (FALSE);\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
+  //\r
+  // Check to see if Address is aligned\r
+  //\r
+  if ((Address & (UINT64)(mInStride[Width] - 1)) != 0) {\r
+    return EFI_UNSUPPORTED;\r
+  }\r
+\r
+  PrivateData = DRIVER_INSTANCE_FROM_PCI_ROOT_BRIDGE_IO_THIS (This);\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
+  // Address + Size * Count.  If the following condition is met, then the transfer\r
+  // is not supported.\r
+  //\r
+  //    Address + Size * Count > Limit + 1\r
+  //\r
+  // Since Limit can be the maximum integer value supported by the CPU and Count \r
+  // can also be the maximum integer value supported by the CPU, this range\r
+  // check must be adjusted to avoid all oveflow conditions.\r
+  //   \r
+  // The following form of the range check is equivalent but assumes that \r
+  // Limit is of the form (2^n - 1).\r
+  //\r
+  if (OperationType == IoOperation) {\r
+    Base = PrivateData->IoBase;\r
+    Limit = PrivateData->IoLimit;\r
+  } else if (OperationType == MemOperation) {\r
+    Base = PrivateData->MemBase;\r
+    Limit = PrivateData->MemLimit;\r
+  } else {\r
+    PciRbAddr = (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_PCI_ADDRESS*) &Address;\r
+    if (PciRbAddr->Bus < PrivateData->BusBase || PciRbAddr->Bus > PrivateData->BusLimit) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+\r
+    if (PciRbAddr->Device > MAX_PCI_DEVICE_NUMBER || PciRbAddr->Function > MAX_PCI_FUNCTION_NUMBER) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+\r
+    if (PciRbAddr->ExtendedRegister != 0) {\r
+      Address = PciRbAddr->ExtendedRegister;\r
+    } else {\r
+      Address = PciRbAddr->Register;\r
+    }\r
+    Base = 0;\r
+    Limit = MAX_PCI_REG_ADDRESS;\r
+  }\r
+\r
+  if (Address < Base) {\r
+      return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
+  if (Count == 0) {\r
+    if (Address > Limit) {\r
+      return EFI_UNSUPPORTED;\r
+    }\r
+  } else {  \r
+    MaxCount = RShiftU64 (Limit, Width);\r
+    if (MaxCount < (Count - 1)) {\r
+      return EFI_UNSUPPORTED;\r
+    }\r
+    if (Address > LShiftU64 (MaxCount - Count + 1, Width)) {\r
+      return EFI_UNSUPPORTED;\r
+    }\r
+  }\r
+\r
+  return EFI_SUCCESS;\r
+}\r
+\r
+/**\r
+   Internal help function for read and write memory space.\r
+\r
+   @param[in]   This          A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Write         Switch value for Read or Write.\r
+   @param[in]   Width         Signifies the width of the memory operations.\r
+   @param[in]   UserAddress   The address within the PCI configuration space for the PCI controller.\r
+   @param[in]   Count         The number of PCI configuration operations to perform. Bytes\r
+                              moved is Width size * Count, starting at Address.\r
+   @param[in, out] UserBuffer For read operations, the destination buffer to store the results. For\r
+                              write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS            The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
+EFI_STATUS\r
+RootBridgeIoMemRW (\r
+  IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL        *This,\r
+  IN     BOOLEAN                                Write,\r
+  IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
+  IN     UINT64                                 Address,\r
+  IN     UINTN                                  Count,\r
+  IN OUT VOID                                   *Buffer\r
+  )\r
+{\r
+  EFI_STATUS                             Status;\r
+  UINT8                                  InStride;\r
+  UINT8                                  OutStride;\r
+  EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  OperationWidth;\r
+  UINT8                                  *Uint8Buffer;\r
+\r
+  Status = RootBridgeIoCheckParameter (This, MemOperation, Width, Address, Count, Buffer);\r
+  if (EFI_ERROR (Status)) {\r
+    return Status;\r
+  }\r
+\r
+  InStride = mInStride[Width];\r
+  OutStride = mOutStride[Width];\r
+  OperationWidth = (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH) (Width & 0x03);\r
+  for (Uint8Buffer = Buffer; Count > 0; Address += InStride, Uint8Buffer += OutStride, Count--) {\r
+    if (Write) {\r
+      switch (OperationWidth) {\r
+        case EfiPciWidthUint8:\r
+          MmioWrite8 ((UINTN)Address, *Uint8Buffer);\r
+          break;\r
+        case EfiPciWidthUint16:\r
+          MmioWrite16 ((UINTN)Address, *((UINT16 *)Uint8Buffer));\r
+          break;\r
+        case EfiPciWidthUint32:\r
+          MmioWrite32 ((UINTN)Address, *((UINT32 *)Uint8Buffer));\r
+          break;\r
+        case EfiPciWidthUint64:\r
+          MmioWrite64 ((UINTN)Address, *((UINT64 *)Uint8Buffer));\r
+          break;\r
+        default:\r
+          //\r
+          // The RootBridgeIoCheckParameter call above will ensure that this\r
+          // path is not taken.\r
+          //\r
+          ASSERT (FALSE);\r
+          break;\r
+      }\r
+    } else {\r
+      switch (OperationWidth) {\r
+        case EfiPciWidthUint8:\r
+          *Uint8Buffer = MmioRead8 ((UINTN)Address);\r
+          break;\r
+        case EfiPciWidthUint16:\r
+          *((UINT16 *)Uint8Buffer) = MmioRead16 ((UINTN)Address);\r
+          break;\r
+        case EfiPciWidthUint32:\r
+          *((UINT32 *)Uint8Buffer) = MmioRead32 ((UINTN)Address);\r
+          break;\r
+        case EfiPciWidthUint64:\r
+          *((UINT64 *)Uint8Buffer) = MmioRead64 ((UINTN)Address);\r
+          break;\r
+        default:\r
+          //\r
+          // The RootBridgeIoCheckParameter call above will ensure that this\r
+          // path is not taken.\r
+          //\r
+          ASSERT (FALSE);\r
+          break;\r
+      }\r
+    }\r
+  }\r
+  return EFI_SUCCESS;  \r
+}\r
+\r
+/**\r
+   Internal help function for read and write IO space.\r
+\r
+   @param[in]   This          A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Write         Switch value for Read or Write.\r
+   @param[in]   Width         Signifies the width of the memory operations.\r
+   @param[in]   UserAddress   The address within the PCI configuration space for the PCI controller.\r
+   @param[in]   Count         The number of PCI configuration operations to perform. Bytes\r
+                              moved is Width size * Count, starting at Address.\r
+   @param[in, out] UserBuffer For read operations, the destination buffer to store the results. For\r
+                              write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS            The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
+EFI_STATUS\r
+RootBridgeIoIoRW (\r
+  IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL        *This,\r
+  IN     BOOLEAN                                Write,\r
+  IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
+  IN     UINT64                                 Address,\r
+  IN     UINTN                                  Count,\r
+  IN OUT VOID                                   *Buffer\r
+  )\r
+{\r
+  EFI_STATUS                             Status;\r
+  UINT8                                  InStride;\r
+  UINT8                                  OutStride;\r
+  EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  OperationWidth;\r
+  UINT8                                  *Uint8Buffer;\r
+\r
+  Status = RootBridgeIoCheckParameter (This, IoOperation, Width, Address, Count, Buffer);\r
+  if (EFI_ERROR (Status)) {\r
+    return Status;\r
+  }\r
+\r
+  InStride = mInStride[Width];\r
+  OutStride = mOutStride[Width];\r
+  OperationWidth = (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH) (Width & 0x03);\r
+  for (Uint8Buffer = Buffer; Count > 0; Address += InStride, Uint8Buffer += OutStride, Count--) {\r
+    if (Write) {\r
+      switch (OperationWidth) {\r
+        case EfiPciWidthUint8:\r
+          IoWrite8 ((UINTN)Address, *Uint8Buffer);\r
+          break;\r
+        case EfiPciWidthUint16:\r
+          IoWrite16 ((UINTN)Address, *((UINT16 *)Uint8Buffer));\r
+          break;\r
+        case EfiPciWidthUint32:\r
+          IoWrite32 ((UINTN)Address, *((UINT32 *)Uint8Buffer));\r
+          break;\r
+        default:\r
+          //\r
+          // The RootBridgeIoCheckParameter call above will ensure that this\r
+          // path is not taken.\r
+          //\r
+          ASSERT (FALSE);\r
+          break;\r
+      }\r
+    } else {\r
+      switch (OperationWidth) {\r
+        case EfiPciWidthUint8:\r
+          *Uint8Buffer = IoRead8 ((UINTN)Address);\r
+          break;\r
+        case EfiPciWidthUint16:\r
+          *((UINT16 *)Uint8Buffer) = IoRead16 ((UINTN)Address);\r
+          break;\r
+        case EfiPciWidthUint32:\r
+          *((UINT32 *)Uint8Buffer) = IoRead32 ((UINTN)Address);\r
+          break;\r
+        default:\r
+          //\r
+          // The RootBridgeIoCheckParameter call above will ensure that this\r
+          // path is not taken.\r
+          //\r
+          ASSERT (FALSE);\r
+          break;\r
+      }\r
+    }\r
+  }\r
+  return EFI_SUCCESS;\r
+}\r
+\r
+/**\r
+   Internal help function for read and write PCI configuration space.\r
+\r
+   @param[in]   This          A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Write         Switch value for Read or Write.\r
+   @param[in]   Width         Signifies the width of the memory operations.\r
+   @param[in]   UserAddress   The address within the PCI configuration space for the PCI controller.\r
+   @param[in]   Count         The number of PCI configuration operations to perform. Bytes\r
+                              moved is Width size * Count, starting at Address.\r
+   @param[in, out] UserBuffer For read operations, the destination buffer to store the results. For\r
+                              write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS            The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
+EFI_STATUS\r
+RootBridgeIoPciRW (\r
+  IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL        *This,\r
+  IN BOOLEAN                                Write,\r
+  IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
+  IN UINT64                                 Address,\r
+  IN UINTN                                  Count,\r
+  IN OUT VOID                               *Buffer\r
+  )\r
+{\r
+  EFI_STATUS                                   Status;\r
+  UINT8                                        InStride;\r
+  UINT8                                        OutStride;\r
+  EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH        OperationWidth;\r
+  UINT8                                        *Uint8Buffer;\r
+  EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_PCI_ADDRESS  *PciRbAddr;\r
+  UINTN                                        PcieRegAddr;\r
+\r
+  Status = RootBridgeIoCheckParameter (This, PciOperation, Width, Address, Count, Buffer);\r
+  if (EFI_ERROR (Status)) {\r
+    return Status;\r
+  }\r
+\r
+  PciRbAddr = (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_PCI_ADDRESS*) &Address;\r
+\r
+  PcieRegAddr = (UINTN) PCI_LIB_ADDRESS (\r
+                          PciRbAddr->Bus,\r
+                          PciRbAddr->Device,\r
+                          PciRbAddr->Function,\r
+                          (PciRbAddr->ExtendedRegister != 0) ? \\r
+                            PciRbAddr->ExtendedRegister :\r
+                            PciRbAddr->Register\r
+                          );\r
+\r
+  InStride = mInStride[Width];\r
+  OutStride = mOutStride[Width];\r
+  OperationWidth = (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH) (Width & 0x03);\r
+  for (Uint8Buffer = Buffer; Count > 0; PcieRegAddr += InStride, Uint8Buffer += OutStride, Count--) {\r
+    if (Write) {\r
+      switch (OperationWidth) {\r
+        case EfiPciWidthUint8:\r
+          PciWrite8 (PcieRegAddr, *Uint8Buffer);\r
+          break;\r
+        case EfiPciWidthUint16:\r
+          PciWrite16 (PcieRegAddr, *((UINT16 *)Uint8Buffer));\r
+          break;\r
+        case EfiPciWidthUint32:\r
+          PciWrite32 (PcieRegAddr, *((UINT32 *)Uint8Buffer));\r
+          break;\r
+        default:\r
+          //\r
+          // The RootBridgeIoCheckParameter call above will ensure that this\r
+          // path is not taken.\r
+          //\r
+          ASSERT (FALSE);\r
+          break;\r
+      }\r
+    } else {\r
+      switch (OperationWidth) {\r
+        case EfiPciWidthUint8:\r
+          *Uint8Buffer = PciRead8 (PcieRegAddr);\r
+          break;\r
+        case EfiPciWidthUint16:\r
+          *((UINT16 *)Uint8Buffer) = PciRead16 (PcieRegAddr);\r
+          break;\r
+        case EfiPciWidthUint32:\r
+          *((UINT32 *)Uint8Buffer) = PciRead32 (PcieRegAddr);\r
+          break;\r
+        default:\r
+          //\r
+          // The RootBridgeIoCheckParameter call above will ensure that this\r
+          // path is not taken.\r
+          //\r
+          ASSERT (FALSE);\r
+          break;\r
+      }\r
+    }\r
+  }\r
+\r
+  return EFI_SUCCESS;\r
+}\r
+\r
+/**\r
+   Polls an address in memory mapped I/O space until an exit condition is met, or \r
+   a timeout occurs. \r
+\r
+   This function provides a standard way to poll a PCI memory location. A PCI memory read\r
+   operation is performed at the PCI memory address specified by Address for the width specified\r
+   by Width. The result of this PCI memory read operation is stored in Result. This PCI memory\r
+   read operation is repeated until either a timeout of Delay 100 ns units has expired, or (Result &\r
+   Mask) is equal to Value.\r
+\r
+   @param[in]   This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width     Signifies the width of the memory operations.\r
+   @param[in]   Address   The base address of the memory operations. The caller is\r
+                          responsible for aligning Address if required.\r
+   @param[in]   Mask      Mask used for the polling criteria. Bytes above Width in Mask\r
+                          are ignored. The bits in the bytes below Width which are zero in\r
+                          Mask are ignored when polling the memory address.\r
+   @param[in]   Value     The comparison value used for the polling exit criteria.\r
+   @param[in]   Delay     The number of 100 ns units to poll. Note that timer available may\r
+                          be of poorer granularity.\r
+   @param[out]  Result    Pointer to the last value read from the memory location.\r
+   \r
+   @retval EFI_SUCCESS            The last data returned from the access matched the poll exit criteria.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid.\r
+   @retval EFI_INVALID_PARAMETER  Result is NULL.\r
+   @retval EFI_TIMEOUT            Delay expired before a match occurred.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoPollMem ( \r
@@ -333,16 +1178,6 @@ RootBridgeIoPollMem (
   IN  UINT64                                 Delay,\r
   OUT UINT64                                 *Result\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Memory Poll\r
-  \r
-Arguments:\r
-    \r
-Returns:\r
-\r
---*/  \r
 {\r
   EFI_STATUS  Status;\r
   UINT64      NumberOfTicks;\r
@@ -387,7 +1222,7 @@ Returns:
     }\r
     NumberOfTicks += 1;\r
   \r
-    while (NumberOfTicks) {\r
+    while (NumberOfTicks != 0) {\r
 \r
       mMetronome->WaitForTick (mMetronome, 1);\r
     \r
@@ -406,6 +1241,35 @@ Returns:
   return EFI_TIMEOUT;\r
 }\r
   \r
+/**\r
+   Reads from the I/O space of a PCI Root Bridge. Returns when either the polling exit criteria is\r
+   satisfied or after a defined duration.\r
+\r
+   This function provides a standard way to poll a PCI I/O location. A PCI I/O read operation is\r
+   performed at the PCI I/O address specified by Address for the width specified by Width.\r
+   The result of this PCI I/O read operation is stored in Result. This PCI I/O read operation is\r
+   repeated until either a timeout of Delay 100 ns units has expired, or (Result & Mask) is equal\r
+   to Value.\r
+\r
+   @param[in] This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in] Width     Signifies the width of the I/O operations.\r
+   @param[in] Address   The base address of the I/O operations. The caller is responsible\r
+                        for aligning Address if required.\r
+   @param[in] Mask      Mask used for the polling criteria. Bytes above Width in Mask\r
+                        are ignored. The bits in the bytes below Width which are zero in\r
+                        Mask are ignored when polling the I/O address.\r
+   @param[in] Value     The comparison value used for the polling exit criteria.\r
+   @param[in] Delay     The number of 100 ns units to poll. Note that timer available may\r
+                        be of poorer granularity.\r
+   @param[out] Result   Pointer to the last value read from the memory location.\r
+   \r
+   @retval EFI_SUCCESS            The last data returned from the access matched the poll exit criteria.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid.\r
+   @retval EFI_INVALID_PARAMETER  Result is NULL.\r
+   @retval EFI_TIMEOUT            Delay expired before a match occurred.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoPollIo ( \r
@@ -417,16 +1281,6 @@ RootBridgeIoPollIo (
   IN  UINT64                                 Delay,\r
   OUT UINT64                                 *Result\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Io Poll\r
-  \r
-Arguments:\r
-    \r
-Returns:\r
-\r
---*/  \r
 {\r
   EFI_STATUS  Status;\r
   UINT64      NumberOfTicks;\r
@@ -469,7 +1323,7 @@ Returns:
     }\r
     NumberOfTicks += 1;\r
   \r
-    while (NumberOfTicks) {\r
+    while (NumberOfTicks != 0) {\r
 \r
       mMetronome->WaitForTick (mMetronome, 1);\r
     \r
@@ -488,6 +1342,29 @@ Returns:
   return EFI_TIMEOUT;\r
 }\r
 \r
+/**\r
+   Enables a PCI driver to access PCI controller registers in the PCI root bridge memory space.\r
+\r
+   The Mem.Read(), and Mem.Write() functions enable a driver to access PCI controller\r
+   registers in the PCI root bridge memory space.\r
+   The memory operations are carried out exactly as requested. The caller is responsible for satisfying\r
+   any alignment and memory width restrictions that a PCI Root Bridge on a platform might require.\r
+\r
+   @param[in]   This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width     Signifies the width of the memory operation.\r
+   @param[in]   Address   The base address of the memory operation. The caller is\r
+                          responsible for aligning the Address if required.\r
+   @param[in]   Count     The number of memory operations to perform. Bytes moved is\r
+                          Width size * Count, starting at Address.\r
+   @param[out]  Buffer    For read operations, the destination buffer to store the results. For\r
+                          write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS            The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoMemRead (\r
@@ -495,57 +1372,34 @@ RootBridgeIoMemRead (
   IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
   IN     UINT64                                 Address,\r
   IN     UINTN                                  Count,\r
-  IN OUT VOID                                   *Buffer\r
+  OUT    VOID                                   *Buffer\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Memory read\r
-  \r
-Arguments:\r
-    \r
-Returns:\r
-\r
---*/  \r
 {\r
-  PCI_ROOT_BRIDGE_INSTANCE                 *PrivateData;\r
-  EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH    OldWidth;\r
-  UINTN                                    OldCount;\r
-  \r
-  if (Buffer == NULL) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  if (Width < 0 || Width >= EfiPciWidthMaximum) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  PrivateData = DRIVER_INSTANCE_FROM_PCI_ROOT_BRIDGE_IO_THIS(This);\r
-\r
-  //\r
-  // Check memory access limit\r
-  //\r
-  if (Address < PrivateData->MemBase) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  OldWidth = Width;\r
-  OldCount = Count;\r
-\r
-  if (Width >= EfiPciWidthFifoUint8 && Width <= EfiPciWidthFifoUint64) {\r
-    Count = 1;\r
-  }\r
-\r
-  Width = (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH)(Width & 0x03);\r
-\r
-  if (Address + (((UINTN)1 << Width) * Count) - 1 > PrivateData->MemLimit) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  return mCpuIo->Mem.Read (mCpuIo, (EFI_CPU_IO_PROTOCOL_WIDTH) OldWidth, \r
-                       Address, OldCount, Buffer);\r
+  return RootBridgeIoMemRW (This, FALSE, Width, Address, Count, Buffer);\r
 }\r
 \r
+/**\r
+   Enables a PCI driver to access PCI controller registers in the PCI root bridge memory space.\r
+\r
+   The Mem.Read(), and Mem.Write() functions enable a driver to access PCI controller\r
+   registers in the PCI root bridge memory space.\r
+   The memory operations are carried out exactly as requested. The caller is responsible for satisfying\r
+   any alignment and memory width restrictions that a PCI Root Bridge on a platform might require.\r
+\r
+   @param[in]   This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width     Signifies the width of the memory operation.\r
+   @param[in]   Address   The base address of the memory operation. The caller is\r
+                          responsible for aligning the Address if required.\r
+   @param[in]   Count     The number of memory operations to perform. Bytes moved is\r
+                          Width size * Count, starting at Address.\r
+   @param[in]   Buffer    For read operations, the destination buffer to store the results. For\r
+                          write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS            The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoMemWrite (\r
@@ -553,56 +1407,30 @@ RootBridgeIoMemWrite (
   IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
   IN     UINT64                                 Address,\r
   IN     UINTN                                  Count,\r
-  IN OUT VOID                                   *Buffer\r
+  IN     VOID                                   *Buffer\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Memory write\r
-  \r
-Arguments:\r
-    \r
-Returns:\r
-\r
---*/  \r
 {\r
-  PCI_ROOT_BRIDGE_INSTANCE                    *PrivateData;\r
-  EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH       OldWidth;\r
-  UINTN                                       OldCount;\r
-\r
-  if (Buffer == NULL) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  if (Width < 0 || Width >= EfiPciWidthMaximum) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  PrivateData = DRIVER_INSTANCE_FROM_PCI_ROOT_BRIDGE_IO_THIS(This);\r
-\r
-  //\r
-  // Check memory access limit\r
-  //\r
-  if (Address < PrivateData->MemBase) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  OldWidth = Width;\r
-  OldCount = Count;\r
-  if (Width >= EfiPciWidthFifoUint8 && Width <= EfiPciWidthFifoUint64) {\r
-    Count = 1;\r
-  }\r
-\r
-  Width = (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH)(Width & 0x03);\r
-\r
-  if (Address + (((UINTN)1 << Width) * Count) - 1 > PrivateData->MemLimit) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  return mCpuIo->Mem.Write (mCpuIo, (EFI_CPU_IO_PROTOCOL_WIDTH) OldWidth, \r
-                       Address, OldCount, Buffer);\r
+  return RootBridgeIoMemRW (This, TRUE, Width, Address, Count, Buffer);  \r
 }\r
 \r
+/**\r
+   Enables a PCI driver to access PCI controller registers in the PCI root bridge I/O space.\r
+\r
+   @param[in]   This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width       Signifies the width of the memory operations.\r
+   @param[in]   Address     The base address of the I/O operation. The caller is responsible for\r
+                            aligning the Address if required.\r
+   @param[in]   Count       The number of I/O operations to perform. Bytes moved is Width\r
+                            size * Count, starting at Address.\r
+   @param[out]  Buffer      For read operations, the destination buffer to store the results. For\r
+                            write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS              The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER    Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER    Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES     The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoIoRead (\r
@@ -610,67 +1438,30 @@ RootBridgeIoIoRead (
   IN     EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
   IN     UINT64                                 Address,\r
   IN     UINTN                                  Count,\r
-  IN OUT VOID                                   *Buffer\r
+  OUT    VOID                                   *Buffer\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Io read\r
-  \r
-Arguments:\r
-    \r
-Returns:\r
-\r
---*/  \r
 {\r
-  \r
-  \r
-  UINTN                                    AlignMask;\r
-  PCI_ROOT_BRIDGE_INSTANCE                 *PrivateData;\r
-  EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH    OldWidth;\r
-  UINTN                                    OldCount;\r
-\r
-  if (Buffer == NULL) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-  \r
-  if (Width < 0 || Width >= EfiPciWidthMaximum) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-  \r
-  PrivateData = DRIVER_INSTANCE_FROM_PCI_ROOT_BRIDGE_IO_THIS(This);\r
-\r
-  //AlignMask = (1 << Width) - 1;\r
-  AlignMask = (1 << (Width & 0x03)) - 1;\r
-\r
-  //\r
-  // check Io access limit\r
-  //\r
-  if (Address < PrivateData->IoBase) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  OldWidth = Width;\r
-  OldCount = Count;\r
-  if (Width >= EfiPciWidthFifoUint8 && Width <= EfiPciWidthFifoUint64) {\r
-    Count = 1;\r
-  }\r
-\r
-  Width = (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH)(Width & 0x03);\r
-  \r
-  if (Address + (((UINTN)1 << Width) * Count) - 1 >= PrivateData->IoLimit) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  if (Address & AlignMask) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  return mCpuIo->Io.Read (mCpuIo, (EFI_CPU_IO_PROTOCOL_WIDTH) OldWidth, \r
-                      Address, OldCount, Buffer);\r
-\r
+  return RootBridgeIoIoRW (This, FALSE, Width, Address, Count, Buffer);  \r
 }\r
 \r
+/**\r
+   Enables a PCI driver to access PCI controller registers in the PCI root bridge I/O space.\r
+\r
+   @param[in]   This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width       Signifies the width of the memory operations.\r
+   @param[in]   Address     The base address of the I/O operation. The caller is responsible for\r
+                            aligning the Address if required.\r
+   @param[in]   Count       The number of I/O operations to perform. Bytes moved is Width\r
+                            size * Count, starting at Address.\r
+   @param[in]   Buffer       For read operations, the destination buffer to store the results. For\r
+                            write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS              The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER    Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER    Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES     The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoIoWrite (\r
@@ -678,84 +1469,45 @@ RootBridgeIoIoWrite (
   IN       EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH   Width,\r
   IN       UINT64                                  Address,\r
   IN       UINTN                                   Count,\r
-  IN OUT   VOID                                    *Buffer\r
+  IN       VOID                                    *Buffer\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Io write\r
-  \r
-Arguments:\r
-    \r
-Returns:\r
-\r
---*/  \r
 {\r
-  UINTN                                         AlignMask;\r
-  PCI_ROOT_BRIDGE_INSTANCE                      *PrivateData;\r
-  EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH         OldWidth;\r
-  UINTN                                         OldCount;\r
-\r
-  if (Buffer == NULL) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  if (Width < 0 || Width >= EfiPciWidthMaximum) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  PrivateData = DRIVER_INSTANCE_FROM_PCI_ROOT_BRIDGE_IO_THIS(This);\r
-\r
-  //AlignMask = (1 << Width) - 1;\r
-  AlignMask = (1 << (Width & 0x03)) - 1;\r
-\r
-  //\r
-  // Check Io access limit\r
-  //\r
-  if (Address < PrivateData->IoBase) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  OldWidth = Width;\r
-  OldCount = Count;\r
-  if (Width >= EfiPciWidthFifoUint8 && Width <= EfiPciWidthFifoUint64) {\r
-    Count = 1;\r
-  }\r
-\r
-  Width = (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH)(Width & 0x03);\r
-  \r
-  if (Address + (((UINTN)1 << Width) * Count) - 1 >= PrivateData->IoLimit) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  if (Address & AlignMask) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  return mCpuIo->Io.Write (mCpuIo, (EFI_CPU_IO_PROTOCOL_WIDTH) OldWidth, \r
-                      Address, OldCount, Buffer);\r
-\r
+  return RootBridgeIoIoRW (This, TRUE, Width, Address, Count, Buffer);  \r
 }\r
 \r
+/**\r
+   Enables a PCI driver to copy one region of PCI root bridge memory space to another region of PCI\r
+   root bridge memory space.\r
+\r
+   The CopyMem() function enables a PCI driver to copy one region of PCI root bridge memory\r
+   space to another region of PCI root bridge memory space. This is especially useful for video scroll\r
+   operation on a memory mapped video buffer.\r
+   The memory operations are carried out exactly as requested. The caller is responsible for satisfying\r
+   any alignment and memory width restrictions that a PCI root bridge on a platform might require.\r
+\r
+   @param[in] This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL instance.\r
+   @param[in] Width       Signifies the width of the memory operations.\r
+   @param[in] DestAddress The destination address of the memory operation. The caller is\r
+                          responsible for aligning the DestAddress if required.\r
+   @param[in] SrcAddress  The source address of the memory operation. The caller is\r
+                          responsible for aligning the SrcAddress if required.\r
+   @param[in] Count       The number of memory operations to perform. Bytes moved is\r
+                          Width size * Count, starting at DestAddress and SrcAddress.\r
+   \r
+   @retval  EFI_SUCCESS             The data was copied from one memory region to another memory region.\r
+   @retval  EFI_INVALID_PARAMETER   Width is invalid for this PCI root bridge.\r
+   @retval  EFI_OUT_OF_RESOURCES    The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoCopyMem (\r
-  IN struct _EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL      *This,\r
+  IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL              *This,\r
   IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH        Width,\r
   IN UINT64                                       DestAddress,\r
   IN UINT64                                       SrcAddress,\r
   IN UINTN                                        Count\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Memory copy\r
-  \r
-Arguments:\r
-    \r
-Returns:\r
-\r
---*/\r
 {\r
   EFI_STATUS  Status;\r
   BOOLEAN     Direction;\r
@@ -812,6 +1564,29 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+   Enables a PCI driver to access PCI controller registers in a PCI root bridge's configuration space.\r
+\r
+   The Pci.Read() and Pci.Write() functions enable a driver to access PCI configuration\r
+   registers for a PCI controller.\r
+   The PCI Configuration operations are carried out exactly as requested. The caller is responsible for\r
+   any alignment and PCI configuration width issues that a PCI Root Bridge on a platform might\r
+   require.\r
+\r
+   @param[in]   This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width     Signifies the width of the memory operations.\r
+   @param[in]   Address   The address within the PCI configuration space for the PCI controller.\r
+   @param[in]   Count     The number of PCI configuration operations to perform. Bytes\r
+                          moved is Width size * Count, starting at Address.\r
+   @param[out]  Buffer    For read operations, the destination buffer to store the results. For\r
+                          write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS            The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoPciRead (\r
@@ -819,33 +1594,35 @@ RootBridgeIoPciRead (
   IN       EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
   IN       UINT64                                 Address,\r
   IN       UINTN                                  Count,\r
-  IN OUT   VOID                                   *Buffer\r
+  OUT      VOID                                   *Buffer\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Pci read\r
-  \r
-Arguments:\r
-    \r
-Returns:\r
-\r
---*/  \r
 {\r
-  \r
-  if (Buffer == NULL) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  if (Width < 0 || Width >= EfiPciWidthMaximum) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-  //\r
-  // Read Pci configuration space\r
-  //\r
   return RootBridgeIoPciRW (This, FALSE, Width, Address, Count, Buffer);\r
 }\r
 \r
+/**\r
+   Enables a PCI driver to access PCI controller registers in a PCI root bridge's configuration space.\r
+\r
+   The Pci.Read() and Pci.Write() functions enable a driver to access PCI configuration\r
+   registers for a PCI controller.\r
+   The PCI Configuration operations are carried out exactly as requested. The caller is responsible for\r
+   any alignment and PCI configuration width issues that a PCI Root Bridge on a platform might\r
+   require.\r
+\r
+   @param[in]   This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]   Width     Signifies the width of the memory operations.\r
+   @param[in]   Address   The address within the PCI configuration space for the PCI controller.\r
+   @param[in]   Count     The number of PCI configuration operations to perform. Bytes\r
+                          moved is Width size * Count, starting at Address.\r
+   @param[in]   Buffer    For read operations, the destination buffer to store the results. For\r
+                          write operations, the source buffer to write data from.\r
+   \r
+   @retval EFI_SUCCESS            The data was read from or written to the PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Width is invalid for this PCI root bridge.\r
+   @retval EFI_INVALID_PARAMETER  Buffer is NULL.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoPciWrite (\r
@@ -853,33 +1630,38 @@ RootBridgeIoPciWrite (
   IN       EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
   IN       UINT64                                 Address,\r
   IN       UINTN                                  Count,\r
-  IN OUT   VOID                                   *Buffer\r
+  IN       VOID                                   *Buffer\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Pci write\r
-  \r
-Arguments:\r
-    \r
-Returns:\r
-\r
---*/  \r
 {\r
-  \r
-  if (Buffer == NULL) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  if (Width < 0 || Width >= EfiPciWidthMaximum) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-  //\r
-  // Write Pci configuration space\r
-  //\r
   return RootBridgeIoPciRW (This, TRUE, Width, Address, Count, Buffer);\r
 }\r
 \r
+/**\r
+   Provides the PCI controller-specific addresses required to access system memory from a\r
+   DMA bus master.\r
+\r
+   The Map() function provides the PCI controller specific addresses needed to access system\r
+   memory. This function is used to map system memory for PCI bus master DMA accesses.\r
+\r
+   @param[in]       This            A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]       Operation       Indicates if the bus master is going to read or write to system memory.\r
+   @param[in]       HostAddress     The system memory address to map to the PCI controller.\r
+   @param[in, out]  NumberOfBytes   On input the number of bytes to map. On output the number of bytes that were mapped.\r
+   @param[out]      DeviceAddress   The resulting map address for the bus master PCI controller to use\r
+                                    to access the system memory's HostAddress.\r
+   @param[out]      Mapping         The value to pass to Unmap() when the bus master DMA operation is complete.\r
+   \r
+   @retval EFI_SUCCESS            The range was mapped for the returned NumberOfBytes.\r
+   @retval EFI_INVALID_PARAMETER  Operation is invalid.\r
+   @retval EFI_INVALID_PARAMETER  HostAddress is NULL.\r
+   @retval EFI_INVALID_PARAMETER  NumberOfBytes is NULL.\r
+   @retval EFI_INVALID_PARAMETER  DeviceAddress is NULL.\r
+   @retval EFI_INVALID_PARAMETER  Mapping is NULL.\r
+   @retval EFI_UNSUPPORTED        The HostAddress cannot be mapped as a common buffer.\r
+   @retval EFI_DEVICE_ERROR       The system hardware could not map the requested address.\r
+   @retval EFI_OUT_OF_RESOURCES   The request could not be completed due to a lack of resources.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoMap (\r
@@ -890,7 +1672,6 @@ RootBridgeIoMap (
   OUT    EFI_PHYSICAL_ADDRESS                       *DeviceAddress,\r
   OUT    VOID                                       **Mapping\r
   )\r
-\r
 {\r
   EFI_STATUS            Status;\r
   EFI_PHYSICAL_ADDRESS  PhysicalAddress;\r
@@ -999,13 +1780,28 @@ RootBridgeIoMap (
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+   Completes the Map() operation and releases any corresponding resources.\r
+\r
+   The Unmap() function completes the Map() operation and releases any corresponding resources.\r
+   If the operation was an EfiPciOperationBusMasterWrite or\r
+   EfiPciOperationBusMasterWrite64, the data is committed to the target system memory.\r
+   Any resources used for the mapping are freed.  \r
+\r
+   @param[in] This      A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in] Mapping   The mapping value returned from Map().\r
+   \r
+   @retval EFI_SUCCESS            The range was unmapped.\r
+   @retval EFI_INVALID_PARAMETER  Mapping is not a value that was returned by Map().\r
+   @retval EFI_DEVICE_ERROR       The data was not committed to the target system memory.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoUnmap (\r
   IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL  *This,\r
   IN VOID                             *Mapping\r
   )\r
-\r
 {\r
   MAP_INFO    *MapInfo;\r
 \r
@@ -1041,6 +1837,27 @@ RootBridgeIoUnmap (
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+   Allocates pages that are suitable for an EfiPciOperationBusMasterCommonBuffer or\r
+   EfiPciOperationBusMasterCommonBuffer64 mapping.\r
+  \r
+   @param This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param Type        This parameter is not used and must be ignored.\r
+   @param MemoryType  The type of memory to allocate, EfiBootServicesData or EfiRuntimeServicesData.\r
+   @param Pages       The number of pages to allocate.\r
+   @param HostAddress A pointer to store the base system memory address of the allocated range.\r
+   @param Attributes  The requested bit mask of attributes for the allocated range. Only\r
+                      the attributes EFI_PCI_ATTRIBUTE_MEMORY_WRITE_COMBINE, EFI_PCI_ATTRIBUTE_MEMORY_CACHED, \r
+                      and EFI_PCI_ATTRIBUTE_DUAL_ADDRESS_CYCLE may be used with this function.\r
+   \r
+   @retval EFI_SUCCESS            The requested memory pages were allocated.\r
+   @retval EFI_INVALID_PARAMETER  MemoryType is invalid.\r
+   @retval EFI_INVALID_PARAMETER  HostAddress is NULL.\r
+   @retval EFI_UNSUPPORTED        Attributes is unsupported. The only legal attribute bits are\r
+                                  MEMORY_WRITE_COMBINE, MEMORY_CACHED, and DUAL_ADDRESS_CYCLE.\r
+   @retval EFI_OUT_OF_RESOURCES   The memory pages could not be allocated.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoAllocateBuffer (\r
@@ -1051,7 +1868,6 @@ RootBridgeIoAllocateBuffer (
   OUT VOID                             **HostAddress,\r
   IN  UINT64                           Attributes\r
   )\r
-\r
 {\r
   EFI_STATUS            Status;\r
   EFI_PHYSICAL_ADDRESS  PhysicalAddress;\r
@@ -1059,7 +1875,7 @@ RootBridgeIoAllocateBuffer (
   //\r
   // Validate Attributes\r
   //\r
-  if (Attributes & EFI_PCI_ATTRIBUTE_INVALID_FOR_ALLOCATE_BUFFER) {\r
+  if ((Attributes & EFI_PCI_ATTRIBUTE_INVALID_FOR_ALLOCATE_BUFFER) != 0) {\r
     return EFI_UNSUPPORTED;\r
   }\r
 \r
@@ -1092,6 +1908,20 @@ RootBridgeIoAllocateBuffer (
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+   Frees memory that was allocated with AllocateBuffer().\r
+\r
+   The FreeBuffer() function frees memory that was allocated with AllocateBuffer().\r
+\r
+   @param This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param Pages       The number of pages to free.\r
+   @param HostAddress The base system memory address of the allocated range.\r
+   \r
+   @retval EFI_SUCCESS            The requested memory pages were freed.\r
+   @retval EFI_INVALID_PARAMETER  The memory range specified by HostAddress and Pages\r
+                                  was not allocated with AllocateBuffer().\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoFreeBuffer (\r
@@ -1099,25 +1929,34 @@ RootBridgeIoFreeBuffer (
   IN  UINTN                            Pages,\r
   OUT VOID                             *HostAddress\r
   )\r
-\r
 {\r
   return gBS->FreePages ((EFI_PHYSICAL_ADDRESS) (UINTN) HostAddress, Pages);\r
 }\r
 \r
+/**\r
+   Flushes all PCI posted write transactions from a PCI host bridge to system memory.\r
+\r
+   The Flush() function flushes any PCI posted write transactions from a PCI host bridge to system\r
+   memory. Posted write transactions are generated by PCI bus masters when they perform write\r
+   transactions to target addresses in system memory.\r
+   This function does not flush posted write transactions from any PCI bridges. A PCI controller\r
+   specific action must be taken to guarantee that the posted write transactions have been flushed from\r
+   the PCI controller and from all the PCI bridges into the PCI host bridge. This is typically done with\r
+   a PCI read transaction from the PCI controller prior to calling Flush().\r
+\r
+   @param This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   \r
+   @retval EFI_SUCCESS        The PCI posted write transactions were flushed from the PCI host\r
+                              bridge to system memory.\r
+   @retval EFI_DEVICE_ERROR   The PCI posted write transactions were not flushed from the PCI\r
+                              host bridge due to a hardware error.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoFlush (\r
   IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL           *This\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-Arguments:\r
-    \r
-Returns:\r
-\r
---*/\r
 {\r
   //\r
   // not supported yet\r
@@ -1125,6 +1964,26 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+   Gets the attributes that a PCI root bridge supports setting with SetAttributes(), and the\r
+   attributes that a PCI root bridge is currently using.  \r
+\r
+   The GetAttributes() function returns the mask of attributes that this PCI root bridge supports\r
+   and the mask of attributes that the PCI root bridge is currently using.\r
+\r
+   @param This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param Supported   A pointer to the mask of attributes that this PCI root bridge\r
+                      supports setting with SetAttributes().\r
+   @param Attributes  A pointer to the mask of attributes that this PCI root bridge is\r
+                      currently using.\r
+   \r
+   @retval  EFI_SUCCESS           If Supports is not NULL, then the attributes that the PCI root\r
+                                  bridge supports is returned in Supports. If Attributes is\r
+                                  not NULL, then the attributes that the PCI root bridge is currently\r
+                                  using is returned in Attributes.\r
+   @retval  EFI_INVALID_PARAMETER Both Supports and Attributes are NULL.\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoGetAttributes (\r
@@ -1132,15 +1991,6 @@ RootBridgeIoGetAttributes (
   OUT UINT64                           *Supported,\r
   OUT UINT64                           *Attributes\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-Arguments:\r
-    \r
-Returns:\r
-\r
---*/\r
 {\r
   PCI_ROOT_BRIDGE_INSTANCE *PrivateData;\r
 \r
@@ -1153,17 +2003,47 @@ Returns:
   //\r
   // Set the return value for Supported and Attributes\r
   //\r
-  if (Supported) {\r
+  if (Supported != NULL) {\r
     *Supported  = PrivateData->Supports; \r
   }\r
 \r
-  if (Attributes) {\r
+  if (Attributes != NULL) {\r
     *Attributes = PrivateData->Attributes;\r
   }\r
   \r
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+   Sets attributes for a resource range on a PCI root bridge.\r
+\r
+   The SetAttributes() function sets the attributes specified in Attributes for the PCI root\r
+   bridge on the resource range specified by ResourceBase and ResourceLength. Since the\r
+   granularity of setting these attributes may vary from resource type to resource type, and from\r
+   platform to platform, the actual resource range and the one passed in by the caller may differ. As a\r
+   result, this function may set the attributes specified by Attributes on a larger resource range\r
+   than the caller requested. The actual range is returned in ResourceBase and\r
+   ResourceLength. The caller is responsible for verifying that the actual range for which the\r
+   attributes were set is acceptable.\r
+\r
+   @param[in]       This            A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[in]       Attributes      The mask of attributes to set. If the attribute bit\r
+                                    MEMORY_WRITE_COMBINE, MEMORY_CACHED, or\r
+                                    MEMORY_DISABLE is set, then the resource range is specified by\r
+                                    ResourceBase and ResourceLength. If\r
+                                    MEMORY_WRITE_COMBINE, MEMORY_CACHED, and\r
+                                    MEMORY_DISABLE are not set, then ResourceBase and\r
+                                    ResourceLength are ignored, and may be NULL.\r
+   @param[in, out]  ResourceBase    A pointer to the base address of the resource range to be modified\r
+                                    by the attributes specified by Attributes.\r
+   @param[in, out]  ResourceLength  A pointer to the length of the resource range to be modified by the\r
+                                    attributes specified by Attributes.\r
+   \r
+   @retval  EFI_SUCCESS     The current configuration of this PCI root bridge was returned in Resources.\r
+   @retval  EFI_UNSUPPORTED The current configuration of this PCI root bridge could not be retrieved.\r
+   @retval  EFI_INVALID_PARAMETER Invalid pointer of EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoSetAttributes (\r
@@ -1172,21 +2052,12 @@ RootBridgeIoSetAttributes (
   IN OUT UINT64                           *ResourceBase,\r
   IN OUT UINT64                           *ResourceLength \r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-Arguments:\r
-    \r
-Returns:\r
-\r
---*/\r
 {\r
   PCI_ROOT_BRIDGE_INSTANCE            *PrivateData;\r
   \r
   PrivateData = DRIVER_INSTANCE_FROM_PCI_ROOT_BRIDGE_IO_THIS(This);\r
   \r
-  if (Attributes) {\r
+  if (Attributes != 0) {\r
     if ((Attributes & (~(PrivateData->Supports))) != 0) {\r
       return EFI_UNSUPPORTED;\r
     }\r
@@ -1205,21 +2076,35 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+   Retrieves the current resource settings of this PCI root bridge in the form of a set of ACPI 2.0\r
+   resource descriptors.\r
+\r
+   There are only two resource descriptor types from the ACPI Specification that may be used to\r
+   describe the current resources allocated to a PCI root bridge. These are the QWORD Address\r
+   Space Descriptor (ACPI 2.0 Section 6.4.3.5.1), and the End Tag (ACPI 2.0 Section 6.4.2.8). The\r
+   QWORD Address Space Descriptor can describe memory, I/O, and bus number ranges for dynamic\r
+   or fixed resources. The configuration of a PCI root bridge is described with one or more QWORD\r
+   Address Space Descriptors followed by an End Tag.\r
+\r
+   @param[in]   This        A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.\r
+   @param[out]  Resources   A pointer to the ACPI 2.0 resource descriptors that describe the\r
+                            current configuration of this PCI root bridge. The storage for the\r
+                            ACPI 2.0 resource descriptors is allocated by this function. The\r
+                            caller must treat the return buffer as read-only data, and the buffer\r
+                            must not be freed by the caller.\r
+   \r
+   @retval  EFI_SUCCESS     The current configuration of this PCI root bridge was returned in Resources.\r
+   @retval  EFI_UNSUPPORTED The current configuration of this PCI root bridge could not be retrieved.\r
+   @retval  EFI_INVALID_PARAMETER Invalid pointer of EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 RootBridgeIoConfiguration (\r
   IN  EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL     *This,\r
   OUT VOID                                **Resources\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-Arguments:\r
-    \r
-Returns:\r
-\r
---*/\r
 {\r
   PCI_ROOT_BRIDGE_INSTANCE              *PrivateData;\r
   UINTN                                 Index;\r
@@ -1238,93 +2123,3 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
-//\r
-// Internal function\r
-//\r
-EFI_STATUS\r
-RootBridgeIoPciRW (\r
-  IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL        *This,\r
-  IN BOOLEAN                                Write,\r
-  IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH  Width,\r
-  IN UINT64                                 UserAddress,\r
-  IN UINTN                                  Count,\r
-  IN OUT VOID                               *UserBuffer\r
-  )\r
-{\r
-  PCI_CONFIG_ACCESS_CF8             Pci;\r
-  PCI_CONFIG_ACCESS_CF8             PciAligned;\r
-  UINT32                            InStride;\r
-  UINT32                            OutStride;\r
-  UINTN                             PciData;\r
-  UINTN                             PciDataStride;\r
-  PCI_ROOT_BRIDGE_INSTANCE     *PrivateData;\r
-  EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_PCI_ADDRESS  PciAddress;\r
-\r
-  if (Width < 0 || Width >= EfiPciWidthMaximum) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-  \r
-  if ((Width & 0x03) >= EfiPciWidthUint64) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-  \r
-  PrivateData = DRIVER_INSTANCE_FROM_PCI_ROOT_BRIDGE_IO_THIS(This);\r
-\r
-  InStride    = 1 << (Width & 0x03);\r
-  OutStride   = InStride;\r
-  if (Width >= EfiCpuIoWidthFifoUint8 && Width <= EfiCpuIoWidthFifoUint64) {\r
-    InStride = 0;\r
-  }\r
-\r
-  if (Width >= EfiCpuIoWidthFillUint8 && Width <= EfiCpuIoWidthFillUint64) {\r
-    OutStride = 0;\r
-  }\r
-\r
-  CopyMem (&PciAddress, &UserAddress, sizeof(UINT64));\r
-\r
-  if (PciAddress.ExtendedRegister > 0xFF) {\r
-    return EFI_UNSUPPORTED;\r
-  }\r
-\r
-  if (PciAddress.ExtendedRegister != 0) {\r
-    Pci.Bits.Reg = PciAddress.ExtendedRegister & 0xFF;\r
-  } else {\r
-    Pci.Bits.Reg = PciAddress.Register;\r
-  }\r
-\r
-  Pci.Bits.Func     = PciAddress.Function;\r
-  Pci.Bits.Dev      = PciAddress.Device;\r
-  Pci.Bits.Bus      = PciAddress.Bus;\r
-  Pci.Bits.Reserved = 0;\r
-  Pci.Bits.Enable   = 1;\r
-\r
-  //\r
-  // PCI Config access are all 32-bit alligned, but by accessing the\r
-  //  CONFIG_DATA_REGISTER (0xcfc) with different widths more cycle types\r
-  //  are possible on PCI.\r
-  //\r
-  // To read a byte of PCI config space you load 0xcf8 and \r
-  //  read 0xcfc, 0xcfd, 0xcfe, 0xcff\r
-  //\r
-  PciDataStride = Pci.Bits.Reg & 0x03;\r
-\r
-  while (Count) {\r
-    CopyMem (&PciAligned, &Pci, sizeof (PciAligned));\r
-    PciAligned.Bits.Reg &= 0xfc;\r
-    PciData = (UINTN)PrivateData->PciData + PciDataStride;\r
-    EfiAcquireLock(&PrivateData->PciLock);\r
-    This->Io.Write (This, EfiPciWidthUint32, PrivateData->PciAddress, 1, &PciAligned);\r
-    if (Write) {\r
-      This->Io.Write (This, Width, PciData, 1, UserBuffer);\r
-    } else {\r
-      This->Io.Read (This, Width, PciData, 1, UserBuffer);\r
-    }\r
-    EfiReleaseLock(&PrivateData->PciLock);\r
-    UserBuffer = ((UINT8 *)UserBuffer) + OutStride;\r
-    PciDataStride = (PciDataStride + InStride) % 4;\r
-    Pci.Bits.Reg += InStride;\r
-    Count -= 1;\r
-  }\r
-  \r
-  return EFI_SUCCESS;\r
-}\r