From 98419ef4b972aade5ff5201ff6dd99561872e52c Mon Sep 17 00:00:00 2001 From: vanjeff Date: Thu, 12 Apr 2007 09:07:01 +0000 Subject: [PATCH] 1. Added EdkPciIncompatibleDeviceSupportLib in EdkModulePkg, this library is used for PciBus driver and includes 3 Incompatible device lists. 2. Used EdkPciIncompatibleDeviceSupportLib in PciBus driver. 3. Redirect all Pci Configuration access to new APIs added in PciLib.c for PciBus driver. 4. Added one PCD PcdIncompatibleDeviceSupportMask in EdkModulePkg.spd to provide mask of PCI devices incompatibility check. git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@2562 6f19259b-4bc3-4df7-8a09-765794883524 --- EdkModulePkg/Bus/Pci/PciBus/Dxe/PciBus.msa | 8 + EdkModulePkg/Bus/Pci/PciBus/Dxe/PciCommand.c | 82 +- .../Bus/Pci/PciBus/Dxe/PciDeviceSupport.c | 7 +- .../Bus/Pci/PciBus/Dxe/PciEnumerator.c | 94 +- .../Bus/Pci/PciBus/Dxe/PciEnumeratorSupport.c | 259 ++--- EdkModulePkg/Bus/Pci/PciBus/Dxe/PciLib.c | 1009 +++++++++++++++-- EdkModulePkg/Bus/Pci/PciBus/Dxe/PciLib.h | 126 +- .../Bus/Pci/PciBus/Dxe/PciOptionRomSupport.c | 32 +- .../Bus/Pci/PciBus/Dxe/PciPowerManagement.c | 32 +- .../Bus/Pci/PciBus/Dxe/PciResourceSupport.c | 80 +- EdkModulePkg/Bus/Pci/PciBus/Dxe/pcibus.c | 45 +- EdkModulePkg/Bus/Pci/PciBus/Dxe/pcibus.h | 19 +- EdkModulePkg/EdkModulePkg.fpd | 40 + EdkModulePkg/EdkModulePkg.spd | 30 +- .../Library/PciIncompatibleDeviceSupportLib.h | 134 +++ .../EdkPciIncompatibleDeviceSupportLib.c | 388 +++++++ .../EdkPciIncompatibleDeviceSupportLib.msa | 44 + .../IncompatiblePciDeviceList.h | 209 ++++ 18 files changed, 2226 insertions(+), 412 deletions(-) create mode 100644 EdkModulePkg/Include/Library/PciIncompatibleDeviceSupportLib.h create mode 100644 EdkModulePkg/Library/EdkPciIncompatibleDeviceSupportLib/EdkPciIncompatibleDeviceSupportLib.c create mode 100644 EdkModulePkg/Library/EdkPciIncompatibleDeviceSupportLib/EdkPciIncompatibleDeviceSupportLib.msa create mode 100644 EdkModulePkg/Library/EdkPciIncompatibleDeviceSupportLib/IncompatiblePciDeviceList.h diff --git a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciBus.msa b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciBus.msa index 712ae999da..f291218624 100644 --- a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciBus.msa +++ b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciBus.msa @@ -57,6 +57,9 @@ PcdLib + + PciIncompatibleDeviceSupportLib + pcibus.h @@ -174,5 +177,10 @@ TRUE If TRUE, the PCI bus driver will support hot plug device. If not hot plug device is supported, this feature flag can be set to FALSE to save size. + + PcdPciIncompatibleDeviceSupportMask + gEfiEdkModulePkgTokenSpaceGuid + The PCD masks for PCI incompatible devices support + diff --git a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciCommand.c b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciCommand.c index 6b472448d9..dd4b650612 100644 --- a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciCommand.c +++ b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciCommand.c @@ -1,18 +1,18 @@ /*++ -Copyright (c) 2006, Intel Corporation -All rights reserved. This program and the accompanying materials -are licensed and made available under the terms and conditions of the BSD License -which accompanies this distribution. The full text of the license may be found at -http://opensource.org/licenses/bsd-license.php - -THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, -WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. +Copyright (c) 2006 - 2007, Intel Corporation +All rights reserved. This program and the accompanying materials +are licensed and made available under the terms and conditions of the BSD License +which accompanies this distribution. The full text of the license may be found at +http://opensource.org/licenses/bsd-license.php + +THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, +WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. Module Name: PciCommand.c - + Abstract: PCI Bus Driver @@ -56,13 +56,13 @@ Returns: PciIo = &PciIoDevice->PciIo; if (Operation != EFI_SET_REGISTER) { - Status = PciIo->Pci.Read ( - PciIo, - EfiPciIoWidthUint16, - Offset, - 1, - &OldCommand - ); + Status = PciIoRead ( + PciIo, + EfiPciIoWidthUint16, + Offset, + 1, + &OldCommand + ); if (Operation == EFI_GET_REGISTER) { *PtrCommand = OldCommand; @@ -78,13 +78,13 @@ Returns: OldCommand = Command; } - return PciIo->Pci.Write ( + return PciIoWrite ( PciIo, EfiPciIoWidthUint16, Offset, 1, &OldCommand - ); + ); } BOOLEAN @@ -98,7 +98,7 @@ Routine Description: Arguments: Returns: - + None --*/ @@ -131,7 +131,7 @@ Arguments: NextRegBlock - A pointer to the next block. Returns: - + None --*/ @@ -157,22 +157,22 @@ Returns: CapabilityPtr = 0; if (IS_CARDBUS_BRIDGE (&PciIoDevice->Pci)) { - PciIoDevice->PciIo.Pci.Read ( - &PciIoDevice->PciIo, - EfiPciIoWidthUint8, - EFI_PCI_CARDBUS_BRIDGE_CAPABILITY_PTR, - 1, - &CapabilityPtr - ); + PciIoRead ( + &PciIoDevice->PciIo, + EfiPciIoWidthUint8, + EFI_PCI_CARDBUS_BRIDGE_CAPABILITY_PTR, + 1, + &CapabilityPtr + ); } else { - PciIoDevice->PciIo.Pci.Read ( - &PciIoDevice->PciIo, - EfiPciIoWidthUint8, - EFI_PCI_CAPABILITY_PTR, - 1, - &CapabilityPtr - ); + PciIoRead ( + &PciIoDevice->PciIo, + EfiPciIoWidthUint8, + EFI_PCI_CAPABILITY_PTR, + 1, + &CapabilityPtr + ); } } @@ -181,13 +181,13 @@ Returns: // Mask it to DWORD alignment per PCI spec // CapabilityPtr &= 0xFC; - PciIoDevice->PciIo.Pci.Read ( - &PciIoDevice->PciIo, - EfiPciIoWidthUint16, - CapabilityPtr, - 1, - &CapabilityEntry - ); + PciIoRead ( + &PciIoDevice->PciIo, + EfiPciIoWidthUint16, + CapabilityPtr, + 1, + &CapabilityEntry + ); CapabilityID = (UINT8) CapabilityEntry; diff --git a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciDeviceSupport.c b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciDeviceSupport.c index cbbc00b183..0e95dc1865 100644 --- a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciDeviceSupport.c +++ b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciDeviceSupport.c @@ -1,6 +1,6 @@ /*++ -Copyright (c) 2006, Intel Corporation +Copyright (c) 2006 - 2007, Intel Corporation All rights reserved. This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -335,14 +335,13 @@ Returns: if (!EFI_ERROR (Status)) { PciIoDevice->IsPciExp = TRUE; } - + // // Force Interrupt line to "Unknown" or "No Connection" // PciIo = &(PciIoDevice->PciIo); Data8 = PCI_INT_LINE_UNKNOWN; - - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint8, 0x3C, 1, &Data8); + PciIoWrite (PciIo, EfiPciIoWidthUint8, 0x3C, 1, &Data8); // // Process Platform OpRom diff --git a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciEnumerator.c b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciEnumerator.c index 384e86ff34..e7a05a2ec9 100644 --- a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciEnumerator.c +++ b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciEnumerator.c @@ -1,18 +1,18 @@ /*++ -Copyright (c) 2006, Intel Corporation -All rights reserved. This program and the accompanying materials -are licensed and made available under the terms and conditions of the BSD License -which accompanies this distribution. The full text of the license may be found at -http://opensource.org/licenses/bsd-license.php - -THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, -WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. +Copyright (c) 2006 - 2007, Intel Corporation +All rights reserved. This program and the accompanying materials +are licensed and made available under the terms and conditions of the BSD License +which accompanies this distribution. The full text of the license may be found at +http://opensource.org/licenses/bsd-license.php + +THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, +WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. Module Name: PciEnumerator.c - + Abstract: PCI Bus Driver @@ -34,7 +34,7 @@ PciEnumerator ( Routine Description: - This routine is used to enumerate entire pci bus system + This routine is used to enumerate entire pci bus system in a given platform Arguments: @@ -118,7 +118,7 @@ Returns: if (EFI_ERROR (Status)) { return Status; } - + // // Submit the resource request // @@ -127,7 +127,7 @@ Returns: if (EFI_ERROR (Status)) { return Status; } - + // // Process P2C // @@ -136,7 +136,7 @@ Returns: if (EFI_ERROR (Status)) { return Status; } - + // // Process attributes for devices on this host bridge // @@ -286,7 +286,7 @@ Returns: while (CurrentLink && CurrentLink != &Bridge->ChildList) { Temp = PCI_IO_DEVICE_FROM_LINK (CurrentLink); if (!IsListEmpty (&Temp->ChildList)) { - + // // Go further to process the option rom under this bridge // @@ -294,7 +294,7 @@ Returns: } if (Temp->RomSize != 0 && Temp->RomSize <= MaxLength) { - + // // Load and process the option rom // @@ -382,8 +382,9 @@ Returns: Address = EFI_PCI_ADDRESS (StartBusNumber, Device, Func, 0x18); - Status = PciRootBridgeIo->Pci.Write ( + Status = PciRootBridgeIoWrite ( PciRootBridgeIo, + &Pci, EfiPciWidthUint16, Address, 1, @@ -394,8 +395,9 @@ Returns: // Initialize SubBusNumber to SecondBus // Address = EFI_PCI_ADDRESS (StartBusNumber, Device, Func, 0x1A); - Status = PciRootBridgeIo->Pci.Write ( + Status = PciRootBridgeIoWrite ( PciRootBridgeIo, + &Pci, EfiPciWidthUint8, Address, 1, @@ -407,8 +409,9 @@ Returns: if (IS_PCI_BRIDGE (&Pci)) { Register8 = 0xFF; - Status = PciRootBridgeIo->Pci.Write ( + Status = PciRootBridgeIoWrite ( PciRootBridgeIo, + &Pci, EfiPciWidthUint8, Address, 1, @@ -432,8 +435,9 @@ Returns: Address = EFI_PCI_ADDRESS (StartBusNumber, Device, Func, 0x1A); - Status = PciRootBridgeIo->Pci.Write ( + Status = PciRootBridgeIoWrite ( PciRootBridgeIo, + &Pci, EfiPciWidthUint8, Address, 1, @@ -503,7 +507,7 @@ Returns: // Here is the point where PCI bus driver calls HOST bridge allocation protocol // Currently we hardcoded for ea815 // - + if (Attributes & EFI_PCI_HOST_BRIDGE_COMBINE_MEM_PMEM) { RootBridgeDev->Decodes |= EFI_BRIDGE_PMEM_MEM_COMBINE_SUPPORTED; } @@ -526,7 +530,7 @@ GetMaxOptionRomSize ( /*++ Routine Description: - + Get Max Option Rom size on this bridge Arguments: @@ -552,7 +556,7 @@ Returns: while (CurrentLink && CurrentLink != &Bridge->ChildList) { Temp = PCI_IO_DEVICE_FROM_LINK (CurrentLink); if (!IsListEmpty (&Temp->ChildList)) { - + // // Get max option rom size under this bridge // @@ -567,13 +571,13 @@ Returns: } } else { - + // // For devices get the rom size directly // TempOptionRomSize = Temp->RomSize; } - + // // Get the largest rom size on this bridge // @@ -594,7 +598,7 @@ PciHostBridgeDeviceAttribute ( /*++ Routine Description: - + Process attributes of devices on this host bridge Arguments: @@ -650,7 +654,7 @@ GetResourceAllocationStatus ( /*++ Routine Description: - + Get resource allocation status from the ACPI pointer Arguments: @@ -736,7 +740,7 @@ RejectPciDevice ( /*++ Routine Description: - + Remove a PCI device from device pool and mark its bar Arguments: @@ -789,7 +793,7 @@ Returns: // InitializeP2C (PciDevice); } - + // // Remove the device // @@ -817,7 +821,7 @@ IsRejectiveDevice ( /*++ Routine Description: - + Determine whethter a PCI device can be rejected Arguments: @@ -839,7 +843,7 @@ Returns: if (!Temp) { return FALSE; } - + // // PPB and RB should go ahead // @@ -853,7 +857,7 @@ Returns: if ((Temp->Parent) && (Temp->BusNumber == 0)) { return FALSE; } - + // // Skip VGA // @@ -872,7 +876,7 @@ GetLargerConsumerDevice ( /*++ Routine Description: - + Get the larger resource consumer Arguments: @@ -914,8 +918,8 @@ GetMaxResourceConsumerDevice ( /*++ Routine Description: - - Get the max resource consumer in the host resource pool + + Get the max resource consumer in the host resource pool Arguments: @@ -974,7 +978,7 @@ PciHostBridgeAdjustAllocation ( /*++ Routine Description: - + Adjust host bridge allocation so as to reduce resource requirement Arguments: @@ -1039,7 +1043,7 @@ Returns: // return EFI_ABORTED; } - + // // Hostbridge hasn't enough resource // @@ -1047,7 +1051,7 @@ Returns: if (!PciResNode) { continue; } - + // // Check if the device has been removed before // @@ -1056,13 +1060,13 @@ Returns: continue; } } - + // // Remove the device if it isn't in the array // Status = RejectPciDevice (PciResNode->PciDev); if (Status == EFI_SUCCESS) { - + // // Raise the EFI_IOB_EC_RESOURCE_CONFLICT status code // @@ -1399,7 +1403,7 @@ Returns: // Memory type aperture // case 0: - + // // Check to see the granularity // @@ -1471,7 +1475,7 @@ Returns: SubBusNumber = 0; StartBusNumber = 0; PciIo = &(BridgeDev->PciIo); - Status = PciIo->Pci.Read (PciIo, EfiPciIoWidthUint8, 0x19, 1, &StartBusNumber); + Status = PciIoRead (PciIo, EfiPciIoWidthUint8, 0x19, 1, &StartBusNumber); if (EFI_ERROR (Status)) { return Status; @@ -1896,7 +1900,7 @@ Returns: if (EFI_ERROR (Status)) { return EFI_UNSUPPORTED; } - + // // Get Root Brige Handle // @@ -1965,7 +1969,7 @@ PciHotPlugRequestNotify ( Routine Description: Hot plug request notify. - + Arguments: This - A pointer to the hot plug request protocol. @@ -2082,7 +2086,7 @@ SearchHostBridgeHandle ( /*++ Routine Description: - + Arguments: Returns: @@ -2130,7 +2134,7 @@ AddHostBridgeEnumerator ( /*++ Routine Description: - + Arguments: Returns: diff --git a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciEnumeratorSupport.c b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciEnumeratorSupport.c index 0403402ddd..52a3a7bee5 100644 --- a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciEnumeratorSupport.c +++ b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciEnumeratorSupport.c @@ -1,18 +1,18 @@ /*++ -Copyright (c) 2006 - 2007, Intel Corporation -All rights reserved. This program and the accompanying materials -are licensed and made available under the terms and conditions of the BSD License -which accompanies this distribution. The full text of the license may be found at -http://opensource.org/licenses/bsd-license.php - -THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, -WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. +Copyright (c) 2006 - 2007, Intel Corporation +All rights reserved. This program and the accompanying materials +are licensed and made available under the terms and conditions of the BSD License +which accompanies this distribution. The full text of the license may be found at +http://opensource.org/licenses/bsd-license.php + +THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, +WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. Module Name: PciEnumeratorSupport.c - + Abstract: PCI Bus Driver @@ -66,8 +66,9 @@ Returns: // // Read the Vendor Id register // - Status = PciRootBridgeIo->Pci.Read ( + Status = PciRootBridgeIoRead ( PciRootBridgeIo, + NULL, EfiPciWidthUint32, Address, 1, @@ -80,8 +81,9 @@ Returns: // Read the entire config header for the device // - Status = PciRootBridgeIo->Pci.Read ( + Status = PciRootBridgeIoRead ( PciRootBridgeIo, + NULL, EfiPciWidthUint32, Address, sizeof (PCI_TYPE00) / sizeof (UINT32), @@ -172,12 +174,12 @@ Returns: // PciIo = &(PciIoDevice->PciIo); - Status = PciIo->Pci.Read (PciIo, EfiPciIoWidthUint8, 0x19, 1, &SecBus); + Status = PciIoRead (PciIo, EfiPciIoWidthUint8, 0x19, 1, &SecBus); if (EFI_ERROR (Status)) { return Status; } - + // // Get resource padding for PPB // @@ -256,7 +258,7 @@ Returns: Func ); if ((PciIoDevice != NULL) && gFullEnumeration) { - InitializeP2C (PciIoDevice); + InitializeP2C (PciIoDevice); } } else { @@ -297,16 +299,18 @@ Returns: if (!PciIoDevice) { return EFI_OUT_OF_RESOURCES; } - + // // Update the bar information for this PCI device so as to support some specific device // - UpdatePciInfo (PciIoDevice); + if (PcdGet8 (PcdPciIncompatibleDeviceSupportMask) & PCI_INCOMPATIBLE_ACPI_RESOURCE_SUPPORT) { + UpdatePciInfo (PciIoDevice); + } if (PciIoDevice->DevicePath == NULL) { return EFI_OUT_OF_RESOURCES; } - + // // Detect this function has option rom // @@ -321,7 +325,7 @@ Returns: ResetPowerManagementFeature (PciIoDevice); } - + // // Insert it into a global tree for future reference // @@ -452,7 +456,7 @@ Returns: if (!PciIoDevice) { return NULL; } - + // // Create a device path for this PCI device and store it into its private data // @@ -486,10 +490,10 @@ Returns: // // Test whether it support 32 decode or not // - PciIo->Pci.Read (PciIo, EfiPciIoWidthUint8, 0x1C, 1, &Temp); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint8, 0x1C, 1, &gAllOne); - PciIo->Pci.Read (PciIo, EfiPciIoWidthUint8, 0x1C, 1, &Value); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint8, 0x1C, 1, &Temp); + PciIoRead (PciIo, EfiPciIoWidthUint8, 0x1C, 1, &Temp); + PciIoWrite (PciIo, EfiPciIoWidthUint8, 0x1C, 1, &gAllOne); + PciIoRead (PciIo, EfiPciIoWidthUint8, 0x1C, 1, &Value); + PciIoWrite (PciIo, EfiPciIoWidthUint8, 0x1C, 1, &Temp); if (Value) { if (Value & 0x01) { @@ -684,20 +688,20 @@ Returns: // Preserve the original value // - PciIo->Pci.Read (PciIo, EfiPciIoWidthUint32, (UINT8) Offset, 1, &OriginalValue); + PciIoRead (PciIo, EfiPciIoWidthUint32, (UINT8) Offset, 1, &OriginalValue); // // Raise TPL to high level to disable timer interrupt while the BAR is probed // OldTpl = gBS->RaiseTPL (EFI_TPL_HIGH_LEVEL); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, (UINT8) Offset, 1, &gAllOne); - PciIo->Pci.Read (PciIo, EfiPciIoWidthUint32, (UINT8) Offset, 1, &Value); + PciIoWrite (PciIo, EfiPciIoWidthUint32, (UINT8) Offset, 1, &gAllOne); + PciIoRead (PciIo, EfiPciIoWidthUint32, (UINT8) Offset, 1, &Value); // // Write back the original value // - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, (UINT8) Offset, 1, &OriginalValue); + PciIoWrite (PciIo, EfiPciIoWidthUint32, (UINT8) Offset, 1, &OriginalValue); // // Restore TPL to its original level @@ -771,7 +775,7 @@ Returns: gBS->RestoreTPL (OldTpl); if (IS_PCI_BRIDGE (&PciIoDevice->Pci) || IS_CARDBUS_BRIDGE (&PciIoDevice->Pci)) { - + // // Preserve the original value // @@ -813,7 +817,7 @@ PciSetDeviceAttribute ( /*++ Routine Description: - Set the supported or current attributes of a PCI device + Set the supported or current attributes of a PCI device Arguments: PciIoDevice - Structure pointer for PCI device. @@ -821,7 +825,7 @@ PciSetDeviceAttribute ( BridgeControl - Bridge control value for PPB or P2C. Option - Make a choice of EFI_SET_SUPPORTS or EFI_SET_ATTRIBUTES. - Returns: + Returns: --*/ @@ -829,15 +833,15 @@ PciSetDeviceAttribute ( Routine Description: - + Arguments: - - + + Returns: - + EFI_SUCCESS Always success - + --*/ { @@ -878,7 +882,7 @@ Returns: if (Option == EFI_SET_SUPPORTS) { - Attributes |= EFI_PCI_IO_ATTRIBUTE_MEMORY_WRITE_COMBINE | + Attributes |= EFI_PCI_IO_ATTRIBUTE_MEMORY_WRITE_COMBINE | EFI_PCI_IO_ATTRIBUTE_MEMORY_CACHED | EFI_PCI_IO_ATTRIBUTE_MEMORY_DISABLE | EFI_PCI_IO_ATTRIBUTE_EMBEDDED_DEVICE | @@ -929,7 +933,7 @@ GetFastBackToBackSupport ( /*++ Routine Description: - + Determine if the device can support Fast Back to Back attribute Arguments: @@ -953,11 +957,11 @@ Returns: // Read the status register // PciIo = &PciIoDevice->PciIo; - Status = PciIo->Pci.Read (PciIo, EfiPciIoWidthUint16, StatusIndex, 1, &StatusRegister); + Status = PciIoRead (PciIo, EfiPciIoWidthUint16, StatusIndex, 1, &StatusRegister); if (EFI_ERROR (Status)) { return EFI_UNSUPPORTED; } - + // // Check the Fast B2B bit // @@ -977,7 +981,7 @@ ProcessOptionRomLight ( /*++ Routine Description: - + Process the option ROM for all the children of the specified parent PCI device. It can only be used after the first full Option ROM process. @@ -1020,7 +1024,7 @@ DetermineDeviceAttribute ( /*++ Routine Description: - + Determine the related attributes of all devices under a Root Bridge Arguments: @@ -1061,7 +1065,7 @@ Returns: return Status; } } else { - + // // Set the attributes to be checked for common PCI devices and PPB or P2C // Since some devices only support part of them, it is better to set the @@ -1100,30 +1104,30 @@ Returns: /* if (IS_PCI_IDE(&PciIoDevice->Pci)) { - PciIo = &PciIoDevice->PciIo; + PciIo = &PciIoDevice->PciIo; - PciIo->Pci.Read ( - PciIo, - EfiPciIoWidthUint8, - 0x09, - 1, + PciIoRead ( + PciIo, + EfiPciIoWidthUint8, + 0x09, + 1, &IdePI ); - + // // Set native mode if it can be supported - // + // IdePI |= (((IdePI & 0x0F) >> 1) & 0x05); - PciIo->Pci.Write ( - PciIo, - EfiPciIoWidthUint8, - 0x09, - 1, + PciIoWrite ( + PciIo, + EfiPciIoWidthUint8, + 0x09, + 1, &IdePI - ); - - } + ); + + } */ } @@ -1135,7 +1139,7 @@ Returns: if (IS_CARDBUS_BRIDGE (&PciIoDevice->Pci)) { FastB2BSupport = FALSE; } - + // // For RootBridge, PPB , P2C, go recursively to traverse all its children // @@ -1216,45 +1220,28 @@ Returns: UINTN BarIndex; UINTN BarEndIndex; BOOLEAN SetFlag; + EFI_PCI_DEVICE_INFO PciDeviceInfo; VOID *Configuration; EFI_ACPI_ADDRESS_SPACE_DESCRIPTOR *Ptr; Configuration = NULL; - // - // It can only be supported after the Incompatible PCI Device - // Support Protocol has been installed - // - if (gEfiIncompatiblePciDeviceSupport == NULL) { - - Status = gBS->LocateProtocol ( - &gEfiIncompatiblePciDeviceSupportProtocolGuid, - NULL, - (VOID **) &gEfiIncompatiblePciDeviceSupport - ); - if (EFI_ERROR (Status)) { - return EFI_UNSUPPORTED; - } - } - // // Check whether the device belongs to incompatible devices or not // If it is , then get its special requirement in the ACPI table // - Status = gEfiIncompatiblePciDeviceSupport->CheckDevice ( - gEfiIncompatiblePciDeviceSupport, - PciIoDevice->Pci.Hdr.VendorId, - PciIoDevice->Pci.Hdr.DeviceId, - PciIoDevice->Pci.Hdr.RevisionID, - PciIoDevice->Pci.Device.SubsystemVendorID, - PciIoDevice->Pci.Device.SubsystemID, - &Configuration - ); + PciDeviceInfo.VendorID = PciIoDevice->Pci.Hdr.VendorId; + PciDeviceInfo.DeviceID = PciIoDevice->Pci.Hdr.DeviceId; + PciDeviceInfo.RevisionID = PciIoDevice->Pci.Hdr.RevisionID; + PciDeviceInfo.SubsystemVendorID = PciIoDevice->Pci.Device.SubsystemVendorID; + PciDeviceInfo.SubsystemID = PciIoDevice->Pci.Device.SubsystemID; + + Status = PciResourceUpdateCheck (&PciDeviceInfo, &Configuration); if (EFI_ERROR (Status)) { return Status; } - + // // Update PCI device information from the ACPI table // @@ -1289,7 +1276,7 @@ Returns: SetFlag = FALSE; switch (Ptr->ResType) { case ACPI_ADDRESS_SPACE_TYPE_MEM: - + // // Make sure the bar is memory type // @@ -1299,7 +1286,7 @@ Returns: break; case ACPI_ADDRESS_SPACE_TYPE_IO: - + // // Make sure the bar is IO type // @@ -1310,7 +1297,7 @@ Returns: } if (SetFlag) { - + // // Update the new alignment for the device // @@ -1367,9 +1354,9 @@ Returns: // // Check the validity of the parameter // - if (NewAlignment != PCI_BAR_EVEN_ALIGN && + if (NewAlignment != PCI_BAR_EVEN_ALIGN && NewAlignment != PCI_BAR_SQUAD_ALIGN && - NewAlignment != PCI_BAR_DQUAD_ALIGN ) { + NewAlignment != PCI_BAR_DQUAD_ALIGN ) { *Alignment = NewAlignment; return ; } @@ -1384,7 +1371,7 @@ Returns: OldAlignment = RShiftU64 (OldAlignment, 4); ShiftBit += 4; } - + // // Adjust the alignment to even, quad or double quad boundary // @@ -1401,7 +1388,7 @@ Returns: OldAlignment = OldAlignment + 8 - (OldAlignment & 0x07); } } - + // // Update the old value // @@ -1563,7 +1550,7 @@ Returns: for (Data = Value; Data != 0; Data >>= 1) { Index ++; } - Value |= ((UINT32)(-1) << Index); + Value |= ((UINT32)(-1) << Index); // // Calculate the size of 64bit bar @@ -1587,7 +1574,7 @@ Returns: break; } } - + // // Check the length again so as to keep compatible with some special bars // @@ -1596,7 +1583,7 @@ Returns: PciIoDevice->PciBar[BarIndex].BaseAddress = 0; PciIoDevice->PciBar[BarIndex].Alignment = 0; } - + // // Increment number of bar // @@ -1610,7 +1597,7 @@ InitializePciDevice ( /*++ Routine Description: - + This routine is used to initialize the bar of a PCI device It can be called typically when a device is going to be rejected @@ -1635,7 +1622,7 @@ Returns: // has not been alloacted // for (Offset = 0x10; Offset <= 0x24; Offset += sizeof (UINT32)) { - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, Offset, 1, &gAllOne); + PciIoWrite (PciIo, EfiPciIoWidthUint32, Offset, 1, &gAllOne); } return EFI_SUCCESS; @@ -1668,28 +1655,28 @@ Returns: // Io32, pMem32, pMem64 to quiescent state // Resource base all ones, Resource limit all zeros // - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint8, 0x1C, 1, &gAllOne); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint8, 0x1D, 1, &gAllZero); + PciIoWrite (PciIo, EfiPciIoWidthUint8, 0x1C, 1, &gAllOne); + PciIoWrite (PciIo, EfiPciIoWidthUint8, 0x1D, 1, &gAllZero); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint16, 0x20, 1, &gAllOne); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint16, 0x22, 1, &gAllZero); + PciIoWrite (PciIo, EfiPciIoWidthUint16, 0x20, 1, &gAllOne); + PciIoWrite (PciIo, EfiPciIoWidthUint16, 0x22, 1, &gAllZero); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint16, 0x24, 1, &gAllOne); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint16, 0x26, 1, &gAllZero); + PciIoWrite (PciIo, EfiPciIoWidthUint16, 0x24, 1, &gAllOne); + PciIoWrite (PciIo, EfiPciIoWidthUint16, 0x26, 1, &gAllZero); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, 0x28, 1, &gAllOne); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, 0x2C, 1, &gAllZero); + PciIoWrite (PciIo, EfiPciIoWidthUint32, 0x28, 1, &gAllOne); + PciIoWrite (PciIo, EfiPciIoWidthUint32, 0x2C, 1, &gAllZero); // // don't support use io32 as for now // - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint16, 0x30, 1, &gAllOne); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint16, 0x32, 1, &gAllZero); + PciIoWrite (PciIo, EfiPciIoWidthUint16, 0x30, 1, &gAllOne); + PciIoWrite (PciIo, EfiPciIoWidthUint16, 0x32, 1, &gAllZero); // // Force Interrupt line to zero for cards that come up randomly // - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint8, 0x3C, 1, &gAllZero); + PciIoWrite (PciIo, EfiPciIoWidthUint8, 0x3C, 1, &gAllZero); return EFI_SUCCESS; } @@ -1721,22 +1708,22 @@ Returns: // Io32, pMem32, pMem64 to quiescent state( // Resource base all ones, Resource limit all zeros // - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, 0x1c, 1, &gAllOne); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, 0x20, 1, &gAllZero); + PciIoWrite (PciIo, EfiPciIoWidthUint32, 0x1c, 1, &gAllOne); + PciIoWrite (PciIo, EfiPciIoWidthUint32, 0x20, 1, &gAllZero); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, 0x24, 1, &gAllOne); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, 0x28, 1, &gAllZero); + PciIoWrite (PciIo, EfiPciIoWidthUint32, 0x24, 1, &gAllOne); + PciIoWrite (PciIo, EfiPciIoWidthUint32, 0x28, 1, &gAllZero); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, 0x2c, 1, &gAllOne); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, 0x30, 1, &gAllZero); + PciIoWrite (PciIo, EfiPciIoWidthUint32, 0x2c, 1, &gAllOne); + PciIoWrite (PciIo, EfiPciIoWidthUint32, 0x30, 1, &gAllZero); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, 0x34, 1, &gAllOne); - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, 0x38, 1, &gAllZero); + PciIoWrite (PciIo, EfiPciIoWidthUint32, 0x34, 1, &gAllOne); + PciIoWrite (PciIo, EfiPciIoWidthUint32, 0x38, 1, &gAllZero); // // Force Interrupt line to zero for cards that come up randomly // - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint8, 0x3C, 1, &gAllZero); + PciIoWrite (PciIo, EfiPciIoWidthUint8, 0x3C, 1, &gAllZero); return EFI_SUCCESS; } @@ -1845,7 +1832,7 @@ PciEnumeratorLight ( Routine Description: - This routine is used to enumerate entire pci bus system + This routine is used to enumerate entire pci bus system in a given platform Arguments: @@ -1910,7 +1897,7 @@ Returns: Descriptors++; continue; } - + // // Record the root bridge io protocol // @@ -1922,7 +1909,7 @@ Returns: ); if (!EFI_ERROR (Status)) { - + // // Remove those PCI devices which are rejected when full enumeration // @@ -1975,9 +1962,9 @@ Arguments: MinBus - The min bus. MaxBus - The max bus. BusRange - The bus range. - + Returns: - + Status Code. --*/ @@ -2129,11 +2116,11 @@ Returns: } if (TestValue & 0x01) { - + // // IO Bar // - + Mask = 0xFFFFFFFC; TestValue = TestValue & Mask; if ((TestValue != 0) && (TestValue == (OldValue & Mask))) { @@ -2141,26 +2128,26 @@ Returns: } } else { - + // // Mem Bar // - + Mask = 0xFFFFFFF0; TestValue = TestValue & Mask; if ((TestValue & 0x07) == 0x04) { - + // // Mem64 or PMem64 // BarOffset += sizeof (UINT32); if ((TestValue != 0) && (TestValue == (OldValue & Mask))) { - + // // Test its high 32-Bit BAR // - + Status = BarExisted (PciIoDevice, BarOffset, &TestValue, &OldValue); if (TestValue == OldValue) { return TRUE; @@ -2168,7 +2155,7 @@ Returns: } } else { - + // // Mem32 or PMem32 // @@ -2207,7 +2194,7 @@ Returns: EFI_STATUS Status; PCI_TYPE00 Pci; UINT8 Device; - UINT32 Register; + UINT32 Register; UINT8 Func; UINT64 Address; EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL *PciRootBridgeIo; @@ -2231,9 +2218,10 @@ Returns: if (!EFI_ERROR (Status) && (IS_PCI_BRIDGE (&Pci))) { Register = 0; Address = EFI_PCI_ADDRESS (StartBusNumber, Device, Func, 0x18); - Status = PciRootBridgeIo->Pci.Read ( + Status = PciRootBridgeIoRead ( PciRootBridgeIo, - EfiPciWidthUint32, + &Pci, + EfiPciWidthUint32, Address, 1, &Register @@ -2242,9 +2230,10 @@ Returns: // Reset register 18h, 19h, 1Ah on PCI Bridge // Register &= 0xFF000000; - Status = PciRootBridgeIo->Pci.Write ( + Status = PciRootBridgeIoWrite ( PciRootBridgeIo, - EfiPciWidthUint32, + &Pci, + EfiPciWidthUint32, Address, 1, &Register diff --git a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciLib.c b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciLib.c index c9c46b3adf..fb6f21fe9d 100644 --- a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciLib.c +++ b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciLib.c @@ -1,22 +1,22 @@ /*++ -Copyright (c) 2006, Intel Corporation -All rights reserved. This program and the accompanying materials -are licensed and made available under the terms and conditions of the BSD License -which accompanies this distribution. The full text of the license may be found at -http://opensource.org/licenses/bsd-license.php - -THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, -WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. +Copyright (c) 2006 - 2007, Intel Corporation +All rights reserved. This program and the accompanying materials +are licensed and made available under the terms and conditions of the BSD License +which accompanies this distribution. The full text of the license may be found at +http://opensource.org/licenses/bsd-license.php + +THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, +WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. Module Name: - PciLib.c - + PciLib.c + Abstract: PCI Bus Driver Lib file - It abstracts some functions that can be different + It abstracts some functions that can be different between light PCI bus driver and full PCI bus driver Revision History @@ -175,7 +175,7 @@ Returns: if (!gFullEnumeration) { Address = 0; - PciIoDevice->PciIo.Pci.Read ( + PciIoRead ( &(PciIoDevice->PciIo), EfiPciIoWidthUint32, 0x1c, @@ -188,7 +188,7 @@ Returns: (PciIoDevice->PciBar)[P2C_MEM_1].BarType = PciBarTypeMem32; Address = 0; - PciIoDevice->PciIo.Pci.Read ( + PciIoRead ( &(PciIoDevice->PciIo), EfiPciIoWidthUint32, 0x20, @@ -200,7 +200,7 @@ Returns: (PciIoDevice->PciBar)[P2C_MEM_2].BarType = PciBarTypePMem32; Address = 0; - PciIoDevice->PciIo.Pci.Read ( + PciIoRead ( &(PciIoDevice->PciIo), EfiPciIoWidthUint32, 0x2c, @@ -212,7 +212,7 @@ Returns: (PciIoDevice->PciBar)[P2C_IO_1].BarType = PciBarTypeIo16; Address = 0; - PciIoDevice->PciIo.Pci.Read ( + PciIoRead ( &(PciIoDevice->PciIo), EfiPciIoWidthUint32, 0x34, @@ -276,19 +276,19 @@ Returns: // Skip rejection for all PPBs, while detect rejection for others // if (IsPciDeviceRejected (Temp)) { - + // // For P2C, remove all devices on it // - + if (!IsListEmpty (&Temp->ChildList)) { RemoveAllPciDeviceOnBridge (RootBridgeHandle, Temp); } - + // // Finally remove itself // - + LastLink = CurrentLink->BackLink; RemoveEntryList (CurrentLink); FreePciDevice (Temp); @@ -316,7 +316,7 @@ PciHostBridgeResourceAllocator ( return PciHostBridgeResourceAllocator_WithoutHotPlugDeviceSupport ( PciResAlloc ); - } + } } @@ -366,7 +366,7 @@ Returns: // // Initialize resource pool // - + InitializeResourcePool (&IoPool, PciBarTypeIo16); InitializeResourcePool (&Mem32Pool, PciBarTypeMem32); InitializeResourcePool (&PMem32Pool, PciBarTypePMem32); @@ -479,7 +479,7 @@ Returns: Mem32Bridge->Alignment = MaxOptionRomSize - 1; } } - + // // Based on the all the resource tree, contruct ACPI resource node to // submit the resource aperture to pci host bridge protocol @@ -516,8 +516,8 @@ Returns: // // Free acpi resource node // - if (AcpiConfig) { - gBS->FreePool (AcpiConfig); + if (AcpiConfig != NULL) { + FreePool (AcpiConfig); } if (EFI_ERROR (Status)) { @@ -575,7 +575,7 @@ Returns: if (RootBridgeDev == NULL) { return EFI_NOT_FOUND; } - + // // Get acpi resource node for all the resource types // @@ -671,7 +671,7 @@ Returns: ); if (AcpiConfig != NULL) { - gBS->FreePool (AcpiConfig); + FreePool (AcpiConfig); } } @@ -758,7 +758,7 @@ Returns: // // Initialize resource pool - // + // InitializeResourcePool (&IoPool, PciBarTypeIo16); InitializeResourcePool (&Mem32Pool, PciBarTypeMem32); InitializeResourcePool (&PMem32Pool, PciBarTypePMem32); @@ -871,8 +871,8 @@ Returns: Mem32Bridge->Alignment = MaxOptionRomSize - 1; } } - } - + } + // // Based on the all the resource tree, contruct ACPI resource node to // submit the resource aperture to pci host bridge protocol @@ -911,7 +911,7 @@ Returns: // Free acpi resource node // if (AcpiConfig != NULL) { - gBS->FreePool (AcpiConfig); + FreePool (AcpiConfig); } if (EFI_ERROR (Status)) { @@ -930,7 +930,7 @@ Returns: // // Notify pci bus driver starts to program the resource // - + Status = NotifyPhase (PciResAlloc, EfiPciHostBridgeAllocateResources); if (!EFI_ERROR (Status)) { @@ -939,11 +939,11 @@ Returns: // break; } - + // // If the resource allocation is unsuccessful, free resources on bridge // - + RootBridgeDev = NULL; RootBridgeHandle = 0; @@ -961,7 +961,7 @@ Returns: if (RootBridgeDev == NULL) { return EFI_NOT_FOUND; } - + // // Get host bridge handle for status report // @@ -994,7 +994,7 @@ Returns: &Mem64ResStatus, &PMem64ResStatus ); - gBS->FreePool (AcpiConfig); + FreePool (AcpiConfig); } } // @@ -1081,7 +1081,7 @@ Returns: if (RootBridgeDev == NULL) { return EFI_NOT_FOUND; } - + // // Get acpi resource node for all the resource types // @@ -1209,16 +1209,16 @@ PciScanBus ( { if (FeaturePcdGet (PcdPciBusHotplugDeviceSupport)) { return PciScanBus_WithHotPlugDeviceSupport ( - Bridge, - StartBusNumber, - SubBusNumber, + Bridge, + StartBusNumber, + SubBusNumber, PaddedBusRange ); } else { return PciScanBus_WithoutHotPlugDeviceSupport ( - Bridge, - StartBusNumber, - SubBusNumber, + Bridge, + StartBusNumber, + SubBusNumber, PaddedBusRange ); } @@ -1282,7 +1282,7 @@ Returns: Func ); - if (!EFI_ERROR (Status) && + if (!EFI_ERROR (Status) && (IS_PCI_BRIDGE (&Pci) || IS_CARDBUS_BRIDGE (&Pci))) { @@ -1310,8 +1310,9 @@ Returns: Address = EFI_PCI_ADDRESS (StartBusNumber, Device, Func, 0x18); - Status = PciRootBridgeIo->Pci.Write ( + Status = PciRootBridgeIoWrite ( PciRootBridgeIo, + &Pci, EfiPciWidthUint16, Address, 1, @@ -1322,8 +1323,9 @@ Returns: // Initialize SubBusNumber to SecondBus // Address = EFI_PCI_ADDRESS (StartBusNumber, Device, Func, 0x1A); - Status = PciRootBridgeIo->Pci.Write ( + Status = PciRootBridgeIoWrite ( PciRootBridgeIo, + &Pci, EfiPciWidthUint8, Address, 1, @@ -1339,8 +1341,9 @@ Returns: // Address = EFI_PCI_ADDRESS (StartBusNumber, Device, Func, 0x1A); Register = 0xFF; - Status = PciRootBridgeIo->Pci.Write ( + Status = PciRootBridgeIoWrite ( PciRootBridgeIo, + &Pci, EfiPciWidthUint8, Address, 1, @@ -1373,8 +1376,9 @@ Returns: Address = EFI_PCI_ADDRESS (StartBusNumber, Device, Func, 0x1A); - Status = PciRootBridgeIo->Pci.Write ( + Status = PciRootBridgeIoWrite ( PciRootBridgeIo, + &Pci, EfiPciWidthUint8, Address, 1, @@ -1476,7 +1480,7 @@ Returns: continue; } - + // // Get the PCI device information // @@ -1507,7 +1511,7 @@ Returns: EfiPciBeforeChildBusEnumeration ); } - + // // For Pci Hotplug controller devcie only // @@ -1530,14 +1534,14 @@ Returns: Event, &State ); - + PreprocessController ( PciDevice, PciDevice->BusNumber, PciDevice->DeviceNumber, PciDevice->FunctionNumber, EfiPciBeforeChildBusEnumeration - ); + ); continue; } } @@ -1552,7 +1556,7 @@ Returns: if (gPciHotPlugInit != NULL) { if (IsRootPciHotPlugBus (PciDevice->DevicePath, &HpIndex)) { - + // // If it is initialized, get the padded bus range // @@ -1593,8 +1597,9 @@ Returns: Register = (UINT16) ((SecondBus << 8) | (UINT16) StartBusNumber); Address = EFI_PCI_ADDRESS (StartBusNumber, Device, Func, 0x18); - Status = PciRootBridgeIo->Pci.Write ( + Status = PciRootBridgeIoWrite ( PciRootBridgeIo, + &Pci, EfiPciWidthUint16, Address, 1, @@ -1606,14 +1611,15 @@ Returns: // If it is PPB, resursively search down this bridge // if (IS_PCI_BRIDGE (&Pci)) { - + // // Initialize SubBusNumber to Maximum bus number // Register = 0xFF; Address = EFI_PCI_ADDRESS (StartBusNumber, Device, Func, 0x1A); - Status = PciRootBridgeIo->Pci.Write ( + Status = PciRootBridgeIoWrite ( PciRootBridgeIo, + &Pci, EfiPciWidthUint8, Address, 1, @@ -1661,8 +1667,9 @@ Returns: // Address = EFI_PCI_ADDRESS (StartBusNumber, Device, Func, 0x1A); - Status = PciRootBridgeIo->Pci.Write ( + Status = PciRootBridgeIoWrite ( PciRootBridgeIo, + &Pci, EfiPciWidthUint8, Address, 1, @@ -1691,7 +1698,7 @@ PciRootBridgeP2CProcess ( /*++ Routine Description: - + Process Option Rom on this host bridge Arguments: @@ -1719,7 +1726,7 @@ Returns: if (IS_CARDBUS_BRIDGE (&Temp->Pci)) { if (gPciHotPlugInit && Temp->Allocated) { - + // // Raise the EFI_IOB_PCI_HPC_INIT status code // @@ -1769,7 +1776,7 @@ PciHostBridgeP2CProcess ( /*++ Routine Description: - + Arguments: Returns: @@ -1821,7 +1828,7 @@ PciHostBridgeEnumerator ( Routine Description: - This function is used to enumerate the entire host bridge + This function is used to enumerate the entire host bridge in a given platform Arguments: @@ -1887,12 +1894,12 @@ Returns: if (FeaturePcdGet (PcdPciBusHotplugDeviceSupport)) { - // + // // Notify the bus allocation phase is finished for the first time - // + // NotifyPhase (PciResAlloc, EfiPciHostBridgeEndBusAllocation); - - + + if (gPciHotPlugInit != NULL) { // // Wait for all HPC initialized @@ -1907,7 +1914,7 @@ Returns: // Notify the bus allocation phase is about to start for the 2nd time // NotifyPhase (PciResAlloc, EfiPciHostBridgeBeginBusAllocation); - + RootBridgeHandle = NULL; while (PciResAlloc->GetNextRootBridge (PciResAlloc, &RootBridgeHandle) == EFI_SUCCESS) { @@ -1935,7 +1942,7 @@ Returns: return Status; } } - + // // Notify the bus allocation phase is to end // @@ -2018,3 +2025,873 @@ Returns: return EFI_SUCCESS; } + +/** + Read PCI device configuration register by specified address. + + This function check the incompatiblilites on PCI device. Return the register + value. + + @param PciRootBridgeIo A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL. + @param PciIo A pointer to EFI_PCI_PROTOCOL. + @param PciDeviceInfo A pointer to EFI_PCI_DEVICE_INFO. + @param Width Signifies the width of the memory operations. + @Param Address The address within the PCI configuration space for the PCI controller. + @param Buffer For read operations, the destination buffer to store the results. For + write operations, the source buffer to write data from. + + @retval EFI_SUCCESS The data was read from or written to the PCI root bridge. + @retval EFI_INVALID_PARAMETER Width is invalid for this PCI root bridge. + @retval EFI_INVALID_PARAMETER Buffer is NULL. + @retval EFI_OUT_OF_RESOURCES The request could not be completed due to a lack of resources. + +**/ +STATIC +EFI_STATUS +ReadConfigData ( + IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL *PciRootBridgeIo, OPTIONAL + IN EFI_PCI_IO_PROTOCOL *PciIo, OPTIONAL + IN EFI_PCI_DEVICE_INFO *PciDeviceInfo, + IN UINT64 Width, + IN UINT64 Address, + IN OUT VOID *Buffer + ) +{ + EFI_STATUS Status; + UINT64 AccessWidth; + EFI_PCI_REGISTER_ACCESS_DATA *PciRegisterAccessData; + UINT64 AccessAddress; + UINTN Stride; + UINT64 TempBuffer; + UINT8 *Pointer; + + ASSERT ((PciRootBridgeIo == NULL) ^ (PciIo == NULL)); + + if (PcdGet8 (PcdPciIncompatibleDeviceSupportMask) & PCI_INCOMPATIBLE_ACCESS_WIDTH_SUPPORT) { + // + // check access compatibility at first time + // + Status = PciRegisterAccessCheck (PciDeviceInfo, PCI_REGISTER_READ, Address & 0xff, Width, &PciRegisterAccessData); + + if (Status == EFI_SUCCESS) { + // + // there exist incompatibility on this operation + // + AccessWidth = Width; + + if (PciRegisterAccessData->Width != VALUE_NOCARE) { + AccessWidth = PciRegisterAccessData->Width; + } + + AccessAddress = Address & ~((1 << AccessWidth) - 1); + + TempBuffer = 0; + Stride = 0; + Pointer = (UINT8 *) &TempBuffer; + + while (1) { + + if (PciRootBridgeIo != NULL) { + Status = PciRootBridgeIo->Pci.Read ( + PciRootBridgeIo, + (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH) AccessWidth, + AccessAddress, + 1, + Pointer + ); + } else if (PciIo != NULL) { + Status = PciIo->Pci.Read ( + PciIo, + (EFI_PCI_IO_PROTOCOL_WIDTH) AccessWidth, + (UINT32) AccessAddress, + 1, + Pointer + ); + } + + if (Status != EFI_SUCCESS) { + return Status; + } + + Stride = 1 << AccessWidth; + AccessAddress += Stride; + if (AccessAddress >= (Address + (1 << Width))) { + // + // if all datas have been read, exist + // + break; + } + + Pointer += Stride; + + if ((AccessAddress & 0xff) < PciRegisterAccessData->EndOffset) { + // + // if current offset doesn't reach the end + // + continue; + } + + FreePool (PciRegisterAccessData); + + // + // continue checking access incompatibility + // + Status = PciRegisterAccessCheck (PciDeviceInfo, PCI_REGISTER_READ, AccessAddress & 0xff, AccessWidth, &PciRegisterAccessData); + if (Status == EFI_SUCCESS) { + if (PciRegisterAccessData->Width != VALUE_NOCARE) { + AccessWidth = PciRegisterAccessData->Width; + } + } + } + + FreePool (PciRegisterAccessData); + + switch (Width) { + case EfiPciWidthUint8: + * (UINT8 *) Buffer = (UINT8) TempBuffer; + break; + case EfiPciWidthUint16: + * (UINT16 *) Buffer = (UINT16) TempBuffer; + break; + case EfiPciWidthUint32: + * (UINT32 *) Buffer = (UINT32) TempBuffer; + break; + default: + return EFI_UNSUPPORTED; + } + + return Status; + } + } + // + // AccessWidth incompatible check not supportted + // or, there doesn't exist incompatibility on this operation + // + if (PciRootBridgeIo != NULL) { + Status = PciRootBridgeIo->Pci.Read ( + PciRootBridgeIo, + (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH) Width, + Address, + 1, + Buffer + ); + + } else { + Status = PciIo->Pci.Read ( + PciIo, + (EFI_PCI_IO_PROTOCOL_WIDTH) Width, + (UINT32) Address, + 1, + Buffer + ); + } + + return Status; +} + +/** + Update register value by checking PCI device incompatibility. + + This function check register value incompatibilites on PCI device. Return the register + value. + + @param PciDeviceInfo A pointer to EFI_PCI_DEVICE_INFO. + @param AccessType Access type, READ or WRITE. + @Param Address The address within the PCI configuration space. + @param Buffer Store the register data. + + @retval EFI_SUCCESS The data has been updated. + +**/ +STATIC +EFI_STATUS +UpdateConfigData ( + IN EFI_PCI_DEVICE_INFO *PciDeviceInfo, + IN UINT64 AccessType, + IN UINT64 Width, + IN UINT64 Address, + IN OUT VOID *Buffer +) +{ + EFI_STATUS Status; + EFI_PCI_REGISTER_VALUE_DATA *PciRegisterData; + UINT64 AndValue; + UINT64 OrValue; + UINT32 TempValue; + + // + // check register value incompatibility + // + Status = PciRegisterUpdateCheck (PciDeviceInfo, AccessType, Address & 0xff, &PciRegisterData); + + if (Status == EFI_SUCCESS) { + + AndValue = (PciRegisterData->AndValue) >> ((Address & 0x3) * 8); + OrValue = (PciRegisterData->OrValue) >> ((Address & 0x3) * 8); + + TempValue = * (UINT32 *) Buffer; + + if (PciRegisterData->AndValue != VALUE_NOCARE) { + TempValue &= (UINT32) AndValue; + } + if (PciRegisterData->OrValue != VALUE_NOCARE) { + TempValue |= (UINT32) OrValue; + } + + switch (Width) { + case EfiPciWidthUint8: + *(UINT32 *)Buffer = *(UINT32 *)Buffer & 0xffffff00 + (UINT8)TempValue; + break; + + case EfiPciWidthUint16: + *(UINT32 *)Buffer = *(UINT32 *)Buffer & 0xffff0000 + (UINT16)TempValue; + break; + case EfiPciWidthUint32: + *(UINT32 *)Buffer = TempValue; + break; + + default: + return EFI_UNSUPPORTED; + } + + FreePool (PciRegisterData); + } + + return Status; +} + +/** + Write PCI device configuration register by specified address. + + This function check the incompatiblilites on PCI device, and write date + into register. + + @param PciRootBridgeIo A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL. + @param PciIo A pointer to EFI_PCI_PROTOCOL. + @param PciDeviceInfo A pointer to EFI_PCI_DEVICE_INFO. + @param Width Signifies the width of the memory operations. + @Param Address The address within the PCI configuration space for the PCI controller. + @param Buffer For read operations, the destination buffer to store the results. For + write operations, the source buffer to write data from. + + @retval EFI_SUCCESS The data was read from or written to the PCI root bridge. + @retval EFI_INVALID_PARAMETER Width is invalid for this PCI root bridge. + @retval EFI_INVALID_PARAMETER Buffer is NULL. + @retval EFI_OUT_OF_RESOURCES The request could not be completed due to a lack of resources. + +**/ +STATIC +EFI_STATUS +WriteConfigData ( + IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL *PciRootBridgeIo, OPTIONAL + IN EFI_PCI_IO_PROTOCOL *PciIo, OPTIONAL + IN EFI_PCI_DEVICE_INFO *PciDeviceInfo, + IN UINT64 Width, + IN UINT64 Address, + IN VOID *Buffer + ) +{ + EFI_STATUS Status; + UINT64 AccessWidth; + EFI_PCI_REGISTER_ACCESS_DATA *PciRegisterAccessData; + UINT64 AccessAddress; + UINTN Stride; + UINT8 *Pointer; + UINT64 Data; + UINTN Shift; + + ASSERT ((PciRootBridgeIo == NULL) ^ (PciIo == NULL)); + + if (PcdGet8 (PcdPciIncompatibleDeviceSupportMask) & PCI_INCOMPATIBLE_ACCESS_WIDTH_SUPPORT) { + // + // check access compatibility at first time + // + Status = PciRegisterAccessCheck (PciDeviceInfo, PCI_REGISTER_WRITE, Address & 0xff, Width, &PciRegisterAccessData); + + if (Status == EFI_SUCCESS) { + // + // there exist incompatibility on this operation + // + AccessWidth = Width; + + if (PciRegisterAccessData->Width != VALUE_NOCARE) { + AccessWidth = PciRegisterAccessData->Width; + } + + AccessAddress = Address & ~((1 << AccessWidth) - 1); + + Stride = 0; + Pointer = (UINT8 *) &Buffer; + Data = * (UINT64 *) Buffer; + + while (1) { + + if (AccessWidth > Width) { + // + // if actual access width is larger than orignal one, additional data need to be read back firstly + // + Status = ReadConfigData (PciRootBridgeIo, PciIo, PciDeviceInfo, AccessWidth, AccessAddress, &Data); + if (Status != EFI_SUCCESS) { + return Status; + } + + // + // check data read incompatibility + // + UpdateConfigData (PciDeviceInfo, PCI_REGISTER_READ, AccessWidth, AccessAddress & 0xff, &Data); + + Shift = (Address - AccessAddress) * 8; + switch (Width) { + case EfiPciWidthUint8: + Data = (* (UINT8 *) Buffer) << Shift | (Data & ~(0xff << Shift)); + break; + + case EfiPciWidthUint16: + Data = (* (UINT16 *) Buffer) << Shift | (Data & ~(0xffff << Shift)); + break; + } + + // + // check data write incompatibility + // + UpdateConfigData (PciDeviceInfo, PCI_REGISTER_WRITE, AccessWidth, AccessAddress * 0xff, &Data); + } + + if (PciRootBridgeIo != NULL) { + Status = PciRootBridgeIo->Pci.Write ( + PciRootBridgeIo, + (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH) AccessWidth, + AccessAddress, + 1, + &Data + ); + } else { + Status = PciIo->Pci.Write ( + PciIo, + (EFI_PCI_IO_PROTOCOL_WIDTH) AccessWidth, + (UINT32) AccessAddress, + 1, + &Data + ); + } + + if (Status != EFI_SUCCESS) { + return Status; + } + + Data = Data >> ((1 << AccessWidth) * 8); + + Stride = 1 << AccessWidth; + AccessAddress += Stride; + if (AccessAddress >= (Address + (1 << Width))) { + // + // if all datas have been written, exist + // + break; + } + + Pointer += Stride; + + if ((AccessAddress & 0xff) < PciRegisterAccessData->EndOffset) { + // + // if current offset doesn't reach the end + // + continue; + } + + FreePool (PciRegisterAccessData); + + // + // continue checking access incompatibility + // + Status = PciRegisterAccessCheck (PciDeviceInfo, PCI_REGISTER_WRITE, AccessAddress & 0xff, AccessWidth, &PciRegisterAccessData); + if (Status == EFI_SUCCESS) { + if (PciRegisterAccessData->Width != VALUE_NOCARE) { + AccessWidth = PciRegisterAccessData->Width; + } + } + }; + + FreePool (PciRegisterAccessData); + + return Status; + } + + } + // + // AccessWidth incompatible check not supportted + // or, there doesn't exist incompatibility on this operation + // + if (PciRootBridgeIo != NULL) { + Status = PciRootBridgeIo->Pci.Write ( + PciRootBridgeIo, + (EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH) Width, + Address, + 1, + Buffer + ); + } else { + Status = PciIo->Pci.Write ( + PciIo, + (EFI_PCI_IO_PROTOCOL_WIDTH) Width, + (UINT32) Address, + 1, + Buffer + ); + } + + return Status; +} + +/** + Abstract PCI device device information. + + @param PciRootBridgeIo A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL. + @param PciIo A pointer to EFI_PCI_PROTOCOL. + @param Pci A pointer to PCI_TYPE00. + @Param Address The address within the PCI configuration space for the PCI controller. + @param PciDeviceInfo A pointer to EFI_PCI_DEVICE_INFO. + + @retval EFI_SUCCESS Pci device device information has been abstracted. + +**/ +STATIC +EFI_STATUS +GetPciDeviceDeviceInfo ( + IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL *PciRootBridgeIo, OPTIONAL + IN EFI_PCI_IO_PROTOCOL *PciIo, OPTIONAL + IN PCI_TYPE00 *Pci, OPTIONAL + IN UINT64 Address, OPTIONAL + OUT EFI_PCI_DEVICE_INFO *PciDeviceInfo +) +{ + EFI_STATUS Status; + UINT64 PciAddress; + UINT32 PciConfigData; + PCI_IO_DEVICE *PciIoDevice; + + ASSERT ((PciRootBridgeIo == NULL) ^ (PciIo == NULL)); + + if (PciIo != NULL) { + PciIoDevice = PCI_IO_DEVICE_FROM_PCI_IO_THIS (PciIo); + + // + // get pointer to PCI_TYPE00 from PciIoDevice + // + Pci = &PciIoDevice->Pci; + } + + if (Pci == NULL) { + // + // while PCI_TYPE00 hasn't been gotten, read PCI device device information directly + // + PciAddress = Address & 0xffffffffffffff00ULL; + Status = PciRootBridgeIo->Pci.Read ( + PciRootBridgeIo, + EfiPciWidthUint32, + PciAddress, + 1, + &PciConfigData + ); + + if (EFI_ERROR (Status)) { + return Status; + } + + if ((PciConfigData & 0xffff) == 0xffff) { + return EFI_NOT_FOUND; + } + + PciDeviceInfo->VendorID = PciConfigData & 0xffff; + PciDeviceInfo->DeviceID = PciConfigData >> 16; + + Status = PciRootBridgeIo->Pci.Read ( + PciRootBridgeIo, + EfiPciWidthUint32, + PciAddress + 8, + 1, + &PciConfigData + ); + if (EFI_ERROR (Status)) { + return Status; + } + + PciDeviceInfo->RevisionID = PciConfigData & 0xf; + + Status = PciRootBridgeIo->Pci.Read ( + PciRootBridgeIo, + EfiPciWidthUint32, + PciAddress + 0x2c, + 1, + &PciConfigData + ); + + if (EFI_ERROR (Status)) { + return Status; + } + + PciDeviceInfo->SubsystemVendorID = PciConfigData & 0xffff; + PciDeviceInfo->SubsystemID = PciConfigData >> 16; + + } else { + PciDeviceInfo->VendorID = Pci->Hdr.VendorId; + PciDeviceInfo->DeviceID = Pci->Hdr.DeviceId; + PciDeviceInfo->RevisionID = Pci->Hdr.RevisionID; + PciDeviceInfo->SubsystemVendorID = Pci->Device.SubsystemVendorID; + PciDeviceInfo->SubsystemID = Pci->Device.SubsystemID; + } + + return EFI_SUCCESS; +} + +/** + Read PCI configuration space with incompatibility check. + + @param PciRootBridgeIo A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL. + @param PciIo A pointer to the EFI_PCI_IO_PROTOCOL. + @param Pci A pointer to PCI_TYPE00. + @param Width Signifies the width of the memory operations. + @Param Address The address within the PCI configuration space for the PCI controller. + @param Buffer For read operations, the destination buffer to store the results. For + write operations, the source buffer to write data from. + + @retval EFI_SUCCESS The data was read from or written to the PCI root bridge. + @retval EFI_INVALID_PARAMETER Width is invalid for this PCI root bridge. + @retval EFI_INVALID_PARAMETER Buffer is NULL. + @retval EFI_OUT_OF_RESOURCES The request could not be completed due to a lack of resources. + +**/ +STATIC +EFI_STATUS +PciIncompatibilityCheckRead ( + IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL *PciRootBridgeIo, OPTIONAL + IN EFI_PCI_IO_PROTOCOL *PciIo, OPTIONAL + IN PCI_TYPE00 *Pci, OPTIONAL + IN UINTN Width, + IN UINT64 Address, + IN UINTN Count, + IN OUT VOID *Buffer +) +{ + EFI_STATUS Status; + EFI_PCI_DEVICE_INFO PciDeviceInfo; + UINT32 Stride; + + ASSERT ((PciRootBridgeIo == NULL) ^ (PciIo == NULL)); + + // + // get PCI device device information + // + Status = GetPciDeviceDeviceInfo (PciRootBridgeIo, PciIo, Pci, Address, &PciDeviceInfo); + if (Status != EFI_SUCCESS) { + return Status; + } + + Stride = 1 << Width; + + for (; Count > 0; Count--, Address += Stride, Buffer = (UINT8 *)Buffer + Stride) { + + // + // read configuration register + // + Status = ReadConfigData (PciRootBridgeIo, PciIo, &PciDeviceInfo, (UINT64) Width, Address, Buffer); + + if (Status != EFI_SUCCESS) { + return Status; + } + + // + // update the data read from configuration register + // + if (PcdGet8 (PcdPciIncompatibleDeviceSupportMask) & PCI_INCOMPATIBLE_REGISTER_UPDATE_SUPPORT) { + UpdateConfigData (&PciDeviceInfo, PCI_REGISTER_READ, Width, Address & 0xff, Buffer); + } + } + + return EFI_SUCCESS; +} + +/** + Write PCI configuration space with incompatibility check. + + @param PciRootBridgeIo A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL. + @param PciIo A pointer to the EFI_PCI_IO_PROTOCOL. + @param Pci A pointer to PCI_TYPE00. + @param Width Signifies the width of the memory operations. + @Param Address The address within the PCI configuration space for the PCI controller. + @param Buffer For read operations, the destination buffer to store the results. For + write operations, the source buffer to write data from. + + @retval EFI_SUCCESS The data was read from or written to the PCI root bridge. + @retval EFI_INVALID_PARAMETER Width is invalid for this PCI root bridge. + @retval EFI_INVALID_PARAMETER Buffer is NULL. + @retval EFI_OUT_OF_RESOURCES The request could not be completed due to a lack of resources. + +**/ +STATIC +EFI_STATUS +PciIncompatibilityCheckWrite ( + IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL *PciRootBridgeIo, OPTIONAL + IN EFI_PCI_IO_PROTOCOL *PciIo, OPTIONAL + IN PCI_TYPE00 *Pci, OPTIONAL + IN UINTN Width, + IN UINT64 Address, + IN UINTN Count, + IN OUT VOID *Buffer +) +{ + EFI_STATUS Status; + EFI_PCI_DEVICE_INFO PciDeviceInfo; + UINT32 Stride; + UINT64 Data; + + ASSERT ((PciRootBridgeIo == NULL) ^ (PciIo == NULL)); + + // + // get PCI device device information + // + Status = GetPciDeviceDeviceInfo (PciRootBridgeIo, PciIo, Pci, Address, &PciDeviceInfo); + if (Status != EFI_SUCCESS) { + return Status; + } + + Stride = 1 << Width; + + for (; Count > 0; Count--, Address += Stride, Buffer = (UINT8 *) Buffer + Stride) { + + Data = 0; + + switch (Width) { + case EfiPciWidthUint8: + Data = * (UINT8 *) Buffer; + break; + case EfiPciWidthUint16: + Data = * (UINT16 *) Buffer; + break; + + case EfiPciWidthUint32: + Data = * (UINT32 *) Buffer; + break; + + default: + return EFI_UNSUPPORTED; + } + + // + // update the data writen into configuration register + // + if (PcdGet8 (PcdPciIncompatibleDeviceSupportMask) & PCI_INCOMPATIBLE_REGISTER_UPDATE_SUPPORT) { + UpdateConfigData (&PciDeviceInfo, PCI_REGISTER_WRITE, Width, Address & 0xff, &Data); + } + + // + // write configuration register + // + Status = WriteConfigData (PciRootBridgeIo, PciIo, &PciDeviceInfo, Width, Address, &Data); + + if (Status != EFI_SUCCESS) { + return Status; + } + } + + return EFI_SUCCESS; +} + +/** + Read PCI configuration space through EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL. + + @param PciRootBridgeIo A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL. + @param Pci A pointer to PCI_TYPE00. + @param Width Signifies the width of the memory operations. + @Param Address The address within the PCI configuration space for the PCI controller. + @param Buffer For read operations, the destination buffer to store the results. For + write operations, the source buffer to write data from. + + @retval EFI_SUCCESS The data was read from or written to the PCI root bridge. + @retval EFI_INVALID_PARAMETER Width is invalid for this PCI root bridge. + @retval EFI_INVALID_PARAMETER Buffer is NULL. + @retval EFI_OUT_OF_RESOURCES The request could not be completed due to a lack of resources. + +**/ +EFI_STATUS +PciRootBridgeIoRead ( + IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL *PciRootBridgeIo, + IN PCI_TYPE00 *Pci, OPTIONAL + IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH Width, + IN UINT64 Address, + IN UINTN Count, + IN OUT VOID *Buffer + ) +{ + if (PcdGet8 (PcdPciIncompatibleDeviceSupportMask) & PCI_INCOMPATIBLE_READ_SUPPORT) { + // + // if PCI incompatibility check enabled + // + return PciIncompatibilityCheckRead ( + PciRootBridgeIo, + NULL, + Pci, + (UINTN) Width, + Address, + Count, + Buffer + ); + } else { + return PciRootBridgeIo->Pci.Read ( + PciRootBridgeIo, + Width, + Address, + Count, + Buffer + ); + } +} + +/** + Write PCI configuration space through EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL. + + @param PciRootBridgeIo A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL. + @param Pci A pointer to PCI_TYPE00. + @param Width Signifies the width of the memory operations. + @Param Address The address within the PCI configuration space for the PCI controller. + @param Buffer For read operations, the destination buffer to store the results. For + write operations, the source buffer to write data from. + + @retval EFI_SUCCESS The data was read from or written to the PCI root bridge. + @retval EFI_INVALID_PARAMETER Width is invalid for this PCI root bridge. + @retval EFI_INVALID_PARAMETER Buffer is NULL. + @retval EFI_OUT_OF_RESOURCES The request could not be completed due to a lack of resources. + +**/ +EFI_STATUS +PciRootBridgeIoWrite ( + IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL *PciRootBridgeIo, + IN PCI_TYPE00 *Pci, + IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH Width, + IN UINT64 Address, + IN UINTN Count, + IN OUT VOID *Buffer + ) +{ + if (PcdGet8 (PcdPciIncompatibleDeviceSupportMask) & PCI_INCOMPATIBLE_WRITE_SUPPORT) { + // + // if PCI incompatibility check enabled + // + return PciIncompatibilityCheckWrite ( + PciRootBridgeIo, + NULL, + Pci, + Width, + Address, + Count, + Buffer + ); + + } else { + return PciRootBridgeIo->Pci.Write ( + PciRootBridgeIo, + Width, + Address, + Count, + Buffer + ); + } +} + +/** + Read PCI configuration space through EFI_PCI_IO_PROTOCOL. + + @param PciIo A pointer to the EFI_PCI_O_PROTOCOL. + @param Width Signifies the width of the memory operations. + @Param Address The address within the PCI configuration space for the PCI controller. + @param Buffer For read operations, the destination buffer to store the results. For + write operations, the source buffer to write data from. + + @retval EFI_SUCCESS The data was read from or written to the PCI root bridge. + @retval EFI_INVALID_PARAMETER Width is invalid for this PCI root bridge. + @retval EFI_INVALID_PARAMETER Buffer is NULL. + @retval EFI_OUT_OF_RESOURCES The request could not be completed due to a lack of resources. + +**/ +EFI_STATUS +PciIoRead ( + IN EFI_PCI_IO_PROTOCOL *PciIo, + IN EFI_PCI_IO_PROTOCOL_WIDTH Width, + IN UINT32 Address, + IN UINTN Count, + IN OUT VOID *Buffer + ) +{ + if (PcdGet8 (PcdPciIncompatibleDeviceSupportMask) & PCI_INCOMPATIBLE_READ_SUPPORT) { + // + // if PCI incompatibility check enabled + // + return PciIncompatibilityCheckRead ( + NULL, + PciIo, + NULL, + (UINTN) Width, + Address, + Count, + Buffer + ); + } else { + return PciIo->Pci.Read ( + PciIo, + Width, + Address, + Count, + Buffer + ); + } +} + +/** + Write PCI configuration space through EFI_PCI_IO_PROTOCOL. + + @param PciIo A pointer to the EFI_PCI_O_PROTOCOL. + @param Width Signifies the width of the memory operations. + @Param Address The address within the PCI configuration space for the PCI controller. + @param Buffer For read operations, the destination buffer to store the results. For + write operations, the source buffer to write data from. + + @retval EFI_SUCCESS The data was read from or written to the PCI root bridge. + @retval EFI_INVALID_PARAMETER Width is invalid for this PCI root bridge. + @retval EFI_INVALID_PARAMETER Buffer is NULL. + @retval EFI_OUT_OF_RESOURCES The request could not be completed due to a lack of resources. + +**/ +EFI_STATUS +PciIoWrite ( + IN EFI_PCI_IO_PROTOCOL *PciIo, + IN EFI_PCI_IO_PROTOCOL_WIDTH Width, + IN UINT32 Address, + IN UINTN Count, + IN OUT VOID *Buffer + ) +{ + if (PcdGet8 (PcdPciIncompatibleDeviceSupportMask) & PCI_INCOMPATIBLE_WRITE_SUPPORT) { + + // + // if PCI incompatibility check enabled + // + return PciIncompatibilityCheckWrite ( + NULL, + PciIo, + NULL, + Width, + Address, + Count, + Buffer + ); + + } else { + return PciIo->Pci.Write ( + PciIo, + Width, + Address, + Count, + Buffer + ); + } +} diff --git a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciLib.h b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciLib.h index f8e88375d7..17e3587cb0 100644 --- a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciLib.h +++ b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciLib.h @@ -1,18 +1,18 @@ /*++ -Copyright (c) 2006, Intel Corporation -All rights reserved. This program and the accompanying materials -are licensed and made available under the terms and conditions of the BSD License -which accompanies this distribution. The full text of the license may be found at -http://opensource.org/licenses/bsd-license.php - -THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, -WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. +Copyright (c) 2006 - 2007, Intel Corporation +All rights reserved. This program and the accompanying materials +are licensed and made available under the terms and conditions of the BSD License +which accompanies this distribution. The full text of the license may be found at +http://opensource.org/licenses/bsd-license.php + +THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, +WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. Module Name: PciLib.h - + Abstract: PCI Bus Driver Lib header file. @@ -26,6 +26,15 @@ Revision History #ifndef _EFI_PCI_LIB_H #define _EFI_PCI_LIB_H +// +// Mask definistions for PCD PcdPciIncompatibleDeviceSupportMask +// +#define PCI_INCOMPATIBLE_ACPI_RESOURCE_SUPPORT 0x01 +#define PCI_INCOMPATIBLE_READ_SUPPORT 0x02 +#define PCI_INCOMPATIBLE_WRITE_SUPPORT 0x04 +#define PCI_INCOMPATIBLE_REGISTER_UPDATE_SUPPORT 0x08 +#define PCI_INCOMPATIBLE_ACCESS_WIDTH_SUPPORT 0x0a + VOID InstallHotPlugRequestProtocol ( IN EFI_STATUS *Status @@ -274,4 +283,103 @@ Returns: --*/ ; +/** + Read PCI configuration space through EFI_PCI_IO_PROTOCOL. + + @param PciIo A pointer to the EFI_PCI_O_PROTOCOL. + @param Width Signifies the width of the memory operations. + @Param Address The address within the PCI configuration space for the PCI controller. + @param Buffer For read operations, the destination buffer to store the results. For + write operations, the source buffer to write data from. + + @retval EFI_SUCCESS The data was read from or written to the PCI root bridge. + @retval EFI_INVALID_PARAMETER Width is invalid for this PCI root bridge. + @retval EFI_INVALID_PARAMETER Buffer is NULL. + @retval EFI_OUT_OF_RESOURCES The request could not be completed due to a lack of resources. + +**/ +EFI_STATUS +PciIoRead ( + IN EFI_PCI_IO_PROTOCOL *PciIo, + IN EFI_PCI_IO_PROTOCOL_WIDTH Width, + IN UINT32 Address, + IN UINTN Count, + IN OUT VOID *Buffer + ); + +/** + Write PCI configuration space through EFI_PCI_IO_PROTOCOL. + + @param PciIo A pointer to the EFI_PCI_O_PROTOCOL. + @param Width Signifies the width of the memory operations. + @Param Address The address within the PCI configuration space for the PCI controller. + @param Buffer For read operations, the destination buffer to store the results. For + write operations, the source buffer to write data from. + + @retval EFI_SUCCESS The data was read from or written to the PCI root bridge. + @retval EFI_INVALID_PARAMETER Width is invalid for this PCI root bridge. + @retval EFI_INVALID_PARAMETER Buffer is NULL. + @retval EFI_OUT_OF_RESOURCES The request could not be completed due to a lack of resources. + +**/ +EFI_STATUS +PciIoWrite ( + IN EFI_PCI_IO_PROTOCOL *PciIo, + IN EFI_PCI_IO_PROTOCOL_WIDTH Width, + IN UINT32 Address, + IN UINTN Count, + IN OUT VOID *Buffer + ); + +/** + Write PCI configuration space through EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL. + + @param PciRootBridgeIo A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL. + @param Pci A pointer to PCI_TYPE00. + @param Width Signifies the width of the memory operations. + @Param Address The address within the PCI configuration space for the PCI controller. + @param Buffer For read operations, the destination buffer to store the results. For + write operations, the source buffer to write data from. + + @retval EFI_SUCCESS The data was read from or written to the PCI root bridge. + @retval EFI_INVALID_PARAMETER Width is invalid for this PCI root bridge. + @retval EFI_INVALID_PARAMETER Buffer is NULL. + @retval EFI_OUT_OF_RESOURCES The request could not be completed due to a lack of resources. + +**/ +EFI_STATUS +PciRootBridgeIoWrite ( + IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL *PciRootBridgeIo, + IN PCI_TYPE00 *Pci, + IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH Width, + IN UINT64 Address, + IN UINTN Count, + IN OUT VOID *Buffer + ); + +/** + Read PCI configuration space through EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL. + + @param PciRootBridgeIo A pointer to the EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL. + @param Pci A pointer to PCI_TYPE00. + @param Width Signifies the width of the memory operations. + @Param Address The address within the PCI configuration space for the PCI controller. + @param Buffer For read operations, the destination buffer to store the results. For + write operations, the source buffer to write data from. + + @retval EFI_SUCCESS The data was read from or written to the PCI root bridge. + @retval EFI_INVALID_PARAMETER Width is invalid for this PCI root bridge. + @retval EFI_INVALID_PARAMETER Buffer is NULL. + @retval EFI_OUT_OF_RESOURCES The request could not be completed due to a lack of resources. + +**/ +EFI_STATUS +PciRootBridgeIoRead ( + IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL *PciRootBridgeIo, + IN PCI_TYPE00 *Pci, + IN EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_WIDTH Width, + IN UINT64 Address, + IN UINTN Count, + IN OUT VOID *Buffer + ); #endif diff --git a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciOptionRomSupport.c b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciOptionRomSupport.c index 2ced4acb0b..f6b371f8a5 100644 --- a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciOptionRomSupport.c +++ b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciOptionRomSupport.c @@ -1,18 +1,18 @@ /*++ -Copyright (c) 2006, Intel Corporation -All rights reserved. This program and the accompanying materials -are licensed and made available under the terms and conditions of the BSD License -which accompanies this distribution. The full text of the license may be found at -http://opensource.org/licenses/bsd-license.php - -THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, -WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. +Copyright (c) 2006 - 2007, Intel Corporation +All rights reserved. This program and the accompanying materials +are licensed and made available under the terms and conditions of the BSD License +which accompanies this distribution. The full text of the license may be found at +http://opensource.org/licenses/bsd-license.php + +THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, +WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. Module Name: PciOptionRomSupport.c - + Abstract: PCI Bus Driver @@ -88,8 +88,9 @@ Returns: AllOnes = 0xfffffffe; Address = EFI_PCI_ADDRESS (Bus, Device, Function, RomBarIndex); - Status = PciRootBridgeIo->Pci.Write ( + Status = PciRootBridgeIoWrite ( PciRootBridgeIo, + &PciIoDevice->Pci, EfiPciWidthUint32, Address, 1, @@ -98,12 +99,13 @@ Returns: if (EFI_ERROR (Status)) { return Status; } - + // // read back // - Status = PciRootBridgeIo->Pci.Read ( + Status = PciRootBridgeIoRead ( PciRootBridgeIo, + &PciIoDevice->Pci, EfiPciWidthUint32, Address, 1, @@ -334,7 +336,7 @@ Returns: // Clear all bars // for (Offset = 0x10; Offset <= 0x24; Offset += sizeof (UINT32)) { - PciIo->Pci.Write (PciIo, EfiPciIoWidthUint32, Offset, 1, &gAllZero); + PciIoWrite (PciIo, EfiPciIoWidthUint32, Offset, 1, &gAllZero); } // @@ -342,7 +344,7 @@ Returns: // enable its decoder // Value32 = RomBar | 0x1; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, (EFI_PCI_IO_PROTOCOL_WIDTH) EfiPciWidthUint32, RomBarIndex, @@ -376,7 +378,7 @@ Returns: // disable rom decode // Value32 = 0xFFFFFFFE; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, (EFI_PCI_IO_PROTOCOL_WIDTH) EfiPciWidthUint32, RomBarIndex, diff --git a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciPowerManagement.c b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciPowerManagement.c index f871c8681c..4584198a9c 100644 --- a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciPowerManagement.c +++ b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciPowerManagement.c @@ -1,18 +1,18 @@ /*++ -Copyright (c) 2006, Intel Corporation -All rights reserved. This program and the accompanying materials -are licensed and made available under the terms and conditions of the BSD License -which accompanies this distribution. The full text of the license may be found at -http://opensource.org/licenses/bsd-license.php - -THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, -WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. +Copyright (c) 2006 - 2007, Intel Corporation +All rights reserved. This program and the accompanying materials +are licensed and made available under the terms and conditions of the BSD License +which accompanies this distribution. The full text of the license may be found at +http://opensource.org/licenses/bsd-license.php + +THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, +WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. Module Name: PciPowerManagement.c - + Abstract: PCI Bus Driver @@ -71,13 +71,13 @@ Returns: // // Write PMCSR // - PciIoDevice->PciIo.Pci.Write ( - &PciIoDevice->PciIo, - EfiPciIoWidthUint16, - PowerManagementRegBlock + 4, - 1, - &PMCSR - ); + PciIoWrite ( + &PciIoDevice->PciIo, + EfiPciIoWidthUint16, + PowerManagementRegBlock + 4, + 1, + &PMCSR + ); return EFI_SUCCESS; } diff --git a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciResourceSupport.c b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciResourceSupport.c index 482ddc9274..cd781841a1 100644 --- a/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciResourceSupport.c +++ b/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciResourceSupport.c @@ -1,18 +1,18 @@ /*++ -Copyright (c) 2006, Intel Corporation -All rights reserved. This program and the accompanying materials -are licensed and made available under the terms and conditions of the BSD License -which accompanies this distribution. The full text of the license may be found at -http://opensource.org/licenses/bsd-license.php - -THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, -WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. +Copyright (c) 2006 - 2007, Intel Corporation +All rights reserved. This program and the accompanying materials +are licensed and made available under the terms and conditions of the BSD License +which accompanies this distribution. The full text of the license may be found at +http://opensource.org/licenses/bsd-license.php + +THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, +WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. Module Name: PciResourceSupport.c - + Abstract: PCI Bus Driver @@ -1222,7 +1222,7 @@ Returns: case PciBarTypeMem32: case PciBarTypePMem32: - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, (Node->PciDev->PciBar[Node->Bar]).Offset, @@ -1239,7 +1239,7 @@ Returns: Address32 = (UINT32) (Address & 0x00000000FFFFFFFF); - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, (Node->PciDev->PciBar[Node->Bar]).Offset, @@ -1249,7 +1249,7 @@ Returns: Address32 = (UINT32) RShiftU64 (Address, 32); - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, (UINT8) ((Node->PciDev->PciBar[Node->Bar]).Offset + 4), @@ -1317,7 +1317,7 @@ Returns: case PPB_BAR_0: case PPB_BAR_1: - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, (Node->PciDev->PciBar[Node->Bar]).Offset, @@ -1333,7 +1333,7 @@ Returns: case PPB_IO_RANGE: Address32 = ((UINT32) (Address)) >> 8; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint8, 0x1C, @@ -1342,7 +1342,7 @@ Returns: ); Address32 >>= 8; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint16, 0x30, @@ -1352,7 +1352,7 @@ Returns: Address32 = (UINT32) (Address + Node->Length - 1); Address32 = ((UINT32) (Address32)) >> 8; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint8, 0x1D, @@ -1361,7 +1361,7 @@ Returns: ); Address32 >>= 8; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint16, 0x32, @@ -1376,7 +1376,7 @@ Returns: case PPB_MEM32_RANGE: Address32 = ((UINT32) (Address)) >> 16; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint16, 0x20, @@ -1386,7 +1386,7 @@ Returns: Address32 = (UINT32) (Address + Node->Length - 1); Address32 = ((UINT32) (Address32)) >> 16; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint16, 0x22, @@ -1402,7 +1402,7 @@ Returns: case PPB_PMEM64_RANGE: Address32 = ((UINT32) (Address)) >> 16; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint16, 0x24, @@ -1412,7 +1412,7 @@ Returns: Address32 = (UINT32) (Address + Node->Length - 1); Address32 = ((UINT32) (Address32)) >> 16; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint16, 0x26, @@ -1421,7 +1421,7 @@ Returns: ); Address32 = (UINT32) RShiftU64 (Address, 32); - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, 0x28, @@ -1430,7 +1430,7 @@ Returns: ); Address32 = (UINT32) RShiftU64 ((Address + Node->Length - 1), 32); - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, 0x2C, @@ -1915,7 +1915,7 @@ Returns: switch (Node->Bar) { case P2C_BAR_0: - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, (Node->PciDev->PciBar[Node->Bar]).Offset, @@ -1928,7 +1928,7 @@ Returns: break; case P2C_MEM_1: - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, 0x1c, @@ -1937,7 +1937,7 @@ Returns: ); TempAddress = Address + Node->Length - 1; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, 0x20, @@ -1950,7 +1950,7 @@ Returns: // // Set non-prefetchable bit // - PciIo->Pci.Read ( + PciIoRead ( PciIo, EfiPciIoWidthUint16, 0x3e, @@ -1959,7 +1959,7 @@ Returns: ); BridgeControl &= 0xfeff; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint16, 0x3e, @@ -1972,7 +1972,7 @@ Returns: // // Set pre-fetchable bit // - PciIo->Pci.Read ( + PciIoRead ( PciIo, EfiPciIoWidthUint16, 0x3e, @@ -1981,7 +1981,7 @@ Returns: ); BridgeControl |= 0x0100; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint16, 0x3e, @@ -1997,7 +1997,7 @@ Returns: break; case P2C_MEM_2: - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, 0x24, @@ -2007,7 +2007,7 @@ Returns: TempAddress = Address + Node->Length - 1; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, 0x28, @@ -2020,7 +2020,7 @@ Returns: // // Set non-prefetchable bit // - PciIo->Pci.Read ( + PciIoRead ( PciIo, EfiPciIoWidthUint16, 0x3e, @@ -2029,7 +2029,7 @@ Returns: ); BridgeControl &= 0xfdff; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint16, 0x3e, @@ -2041,7 +2041,7 @@ Returns: // // Set pre-fetchable bit // - PciIo->Pci.Read ( + PciIoRead ( PciIo, EfiPciIoWidthUint16, 0x3e, @@ -2050,7 +2050,7 @@ Returns: ); BridgeControl |= 0x0200; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint16, 0x3e, @@ -2065,7 +2065,7 @@ Returns: break; case P2C_IO_1: - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, 0x2c, @@ -2073,7 +2073,7 @@ Returns: &Address ); TempAddress = Address + Node->Length - 1; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, 0x30, @@ -2088,7 +2088,7 @@ Returns: break; case P2C_IO_2: - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, 0x34, @@ -2097,7 +2097,7 @@ Returns: ); TempAddress = Address + Node->Length - 1; - PciIo->Pci.Write ( + PciIoWrite ( PciIo, EfiPciIoWidthUint32, 0x38, diff --git a/EdkModulePkg/Bus/Pci/PciBus/Dxe/pcibus.c b/EdkModulePkg/Bus/Pci/PciBus/Dxe/pcibus.c index eb6713f4dd..a874b8a85c 100644 --- a/EdkModulePkg/Bus/Pci/PciBus/Dxe/pcibus.c +++ b/EdkModulePkg/Bus/Pci/PciBus/Dxe/pcibus.c @@ -1,24 +1,24 @@ /*++ -Copyright (c) 2006, Intel Corporation -All rights reserved. This program and the accompanying materials -are licensed and made available under the terms and conditions of the BSD License -which accompanies this distribution. The full text of the license may be found at -http://opensource.org/licenses/bsd-license.php - -THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, -WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. +Copyright (c) 2006 - 2007, Intel Corporation +All rights reserved. This program and the accompanying materials +are licensed and made available under the terms and conditions of the BSD License +which accompanies this distribution. The full text of the license may be found at +http://opensource.org/licenses/bsd-license.php + +THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, +WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. Module Name: PciBus.c - + Abstract: PCI Bus Driver Revision History - + --*/ #include "pcibus.h" @@ -36,7 +36,6 @@ EFI_DRIVER_BINDING_PROTOCOL gPciBusDriverBinding = { NULL }; -EFI_INCOMPATIBLE_PCI_DEVICE_SUPPORT_PROTOCOL *gEfiIncompatiblePciDeviceSupport = NULL; EFI_HANDLE gPciHostBrigeHandles[PCI_MAX_HOST_BRIDGE_NUM]; UINTN gPciHostBridgeNumber; BOOLEAN gFullEnumeration; @@ -68,8 +67,8 @@ Arguments: Returns: - EFI_SUCCESS - EFI_DEVICE_ERROR + EFI_SUCCESS + EFI_DEVICE_ERROR --*/ // TODO: ImageHandle - add argument and description to function comment @@ -101,7 +100,7 @@ Routine Description: Check to see if pci bus driver supports the given controller Arguments: - + IN EFI_DRIVER_BINDING_PROTOCOL *This, IN EFI_HANDLE Controller, IN EFI_DEVICE_PATH_PROTOCOL *RemainingDevicePath @@ -195,13 +194,13 @@ Routine Description: Start to management the controller passed in Arguments: - + IN EFI_DRIVER_BINDING_PROTOCOL *This, IN EFI_HANDLE Controller, IN EFI_DEVICE_PATH_PROTOCOL *RemainingDevicePath Returns: - + --*/ // TODO: This - add argument and description to function comment @@ -211,12 +210,6 @@ Returns: { EFI_STATUS Status; - Status = gBS->LocateProtocol ( - &gEfiIncompatiblePciDeviceSupportProtocolGuid, - NULL, - (VOID **) &gEfiIncompatiblePciDeviceSupport - ); - // // If PCI Platform protocol is available, get it now. // If the platform implements this, it must be installed before BDS phase @@ -240,12 +233,12 @@ Returns: if (EFI_ERROR (Status)) { return Status; } - + // // Enable PCI device specified by remaining device path. BDS or other driver can call the // start more than once. // - + StartPciDevices (Controller, RemainingDevicePath); return EFI_SUCCESS; @@ -267,7 +260,7 @@ Routine Description: if all the the children get closed, close the protocol Arguments: - + IN EFI_DRIVER_BINDING_PROTOCOL *This, IN EFI_HANDLE Controller, IN UINTN NumberOfChildren, @@ -275,7 +268,7 @@ Arguments: Returns: - + --*/ // TODO: This - add argument and description to function comment // TODO: Controller - add argument and description to function comment diff --git a/EdkModulePkg/Bus/Pci/PciBus/Dxe/pcibus.h b/EdkModulePkg/Bus/Pci/PciBus/Dxe/pcibus.h index cd34d8742e..1cc957503b 100644 --- a/EdkModulePkg/Bus/Pci/PciBus/Dxe/pcibus.h +++ b/EdkModulePkg/Bus/Pci/PciBus/Dxe/pcibus.h @@ -1,18 +1,18 @@ /*++ -Copyright (c) 2006, Intel Corporation -All rights reserved. This program and the accompanying materials -are licensed and made available under the terms and conditions of the BSD License -which accompanies this distribution. The full text of the license may be found at -http://opensource.org/licenses/bsd-license.php - -THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, -WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. +Copyright (c) 2006 - 2007, Intel Corporation +All rights reserved. This program and the accompanying materials +are licensed and made available under the terms and conditions of the BSD License +which accompanies this distribution. The full text of the license may be found at +http://opensource.org/licenses/bsd-license.php + +THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, +WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. Module Name: pcibus.h - + Abstract: PCI Bus Driver @@ -216,7 +216,6 @@ typedef struct _PCI_IO_DEVICE { // // Global Variables // -extern EFI_INCOMPATIBLE_PCI_DEVICE_SUPPORT_PROTOCOL *gEfiIncompatiblePciDeviceSupport; extern EFI_DRIVER_BINDING_PROTOCOL gPciBusDriverBinding; extern EFI_COMPONENT_NAME_PROTOCOL gPciBusComponentName; extern LIST_ENTRY gPciDevicePool; diff --git a/EdkModulePkg/EdkModulePkg.fpd b/EdkModulePkg/EdkModulePkg.fpd index 2e7e9344d8..c7b481d792 100644 --- a/EdkModulePkg/EdkModulePkg.fpd +++ b/EdkModulePkg/EdkModulePkg.fpd @@ -3575,6 +3575,8 @@ + + @@ -3689,6 +3691,14 @@ 4 320 + + PcdPciIncompatibleDeviceSupportMask + 0x0001003f + gEfiEdkModulePkgTokenSpaceGuid + UINT8 + 1 + 0x0 + NULL @@ -9019,6 +9029,8 @@ + + @@ -9133,6 +9145,14 @@ 4 320 + + PcdPciIncompatibleDeviceSupportMask + 0x0001003f + gEfiEdkModulePkgTokenSpaceGuid + UINT8 + 1 + 0x0 + NULL @@ -13569,6 +13589,8 @@ + + @@ -13683,6 +13705,14 @@ 4 320 + + PcdPciIncompatibleDeviceSupportMask + 0x0001003f + gEfiEdkModulePkgTokenSpaceGuid + UINT8 + 1 + 0x0 + NULL @@ -17741,6 +17771,8 @@ + + @@ -17855,6 +17887,14 @@ 4 320 + + PcdPciIncompatibleDeviceSupportMask + 0x0001003f + gEfiEdkModulePkgTokenSpaceGuid + UINT8 + 1 + 0x0 + NULL diff --git a/EdkModulePkg/EdkModulePkg.spd b/EdkModulePkg/EdkModulePkg.spd index fe9b696304..6aef854ed5 100644 --- a/EdkModulePkg/EdkModulePkg.spd +++ b/EdkModulePkg/EdkModulePkg.spd @@ -8,7 +8,7 @@ This Module provides standard reference information for EFI/Tiano implementations. Copyright (c) 2006 - 2007, Intel Corporation. All rights reserved. - This program and the accompanying materials are licensed and made available + This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at http://opensource.org/licenses/bsd-license.php THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, WITHOUT WARRANTIES @@ -72,6 +72,10 @@ Include/Library/EdkGenericPlatformBdsLib.h Generic Platform Bds library + + Include/Library/PciIncompatibleDeviceSupportLib.h + This library includes the PCI incompatible devices list. + @@ -404,6 +408,7 @@ Library/EdkOemHookStatusCodeLibNull/EdkOemHookStatusCodeLibNull.msa Library/EdkGenericBdsLib/EdkGenericBdsLib.msa Library/EdkGenericPlatformBdsLib/EdkGenericPlatformBdsLib.msa + Library/EdkPciIncompatibleDeviceSupportLib/EdkPciIncompatibleDeviceSupportLib.msa Universal/Console/ConSplitter/Dxe/ConSplitter.msa Universal/Console/GraphicsConsole/Dxe/GraphicsConsole.msa Universal/Console/Terminal/Dxe/Terminal.msa @@ -919,7 +924,7 @@ UINT32 FIXED_AT_BUILD 0x08 - The maximum number of callback function, which will be triggered when + The maximum number of callback function, which will be triggered when a PCD entry is been set, can be registered for a single PCD entry in PEI phase. @@ -929,7 +934,7 @@ UINT32 FIXED_AT_BUILD 0x0 - The base address of the VPD (Vital Product Data) region. It is + The base address of the VPD (Vital Product Data) region. It is normally a region reserved on flash. @@ -966,7 +971,7 @@ BOOLEAN FEATURE_FLAG TRUE - This feature flag can be used to enable or disable the Pcd PEIM database + This feature flag can be used to enable or disable the Pcd PEIM database traverse capability. Disable it can reduce the size of final image generated. @@ -976,7 +981,7 @@ BOOLEAN FEATURE_FLAG TRUE - This feature flag can be used to enable or disable the Pcd DXE database + This feature flag can be used to enable or disable the Pcd DXE database traverse capability. Disable it can reduce the size of final image generated. @@ -1294,5 +1299,20 @@ FALSE If this PCD is set as TRUE, NT emulator will be endabled. + + PcdPciIncompatibleDeviceSupportMask + 0x0001003f + gEfiEdkModulePkgTokenSpaceGuid + UINT8 + FIXED_AT_BUILD + 0x0 + The PCD masks for PCI incompatible devices support. + Acpi_Resource_Update_Support :1; + Register_Read_Support :1; + Register_Write_Support :1; + Register_Update_Support :1; + Register_Access_Width_Support :1; + Reserved :3; + \ No newline at end of file diff --git a/EdkModulePkg/Include/Library/PciIncompatibleDeviceSupportLib.h b/EdkModulePkg/Include/Library/PciIncompatibleDeviceSupportLib.h new file mode 100644 index 0000000000..527a704042 --- /dev/null +++ b/EdkModulePkg/Include/Library/PciIncompatibleDeviceSupportLib.h @@ -0,0 +1,134 @@ +/** @file + PCI Incompatible device support Libary. + +Copyright (c) 2007 Intel Corporation. All rights reserved.
+This software and associated documentation (if any) is furnished +under a license and may only be used or copied in accordance +with the terms of the license. Except as permitted by such +license, no part of this software or documentation may be +reproduced, stored in a retrieval system, or transmitted in any +form or by any means without the express written consent of +Intel Corporation. + +**/ + +#define PCI_REGISTER_READ 0xfffffffffffffff1ULL +#define PCI_REGISTER_WRITE 0xfffffffffffffff2ULL +#define VALUE_NOCARE 0xffffffffffffffffULL + +// +// PCI device device information +// +typedef struct { + UINT64 VendorID; + UINT64 DeviceID; + UINT64 RevisionID; + UINT64 SubsystemVendorID; + UINT64 SubsystemID; +} EFI_PCI_DEVICE_INFO; + + +// +// store hardcode value of resgister +// +typedef struct { + UINT64 AndValue; + UINT64 OrValue; +} EFI_PCI_REGISTER_VALUE_DATA; + +// +// store access width information +// +typedef struct { + UINT64 StartOffset; + UINT64 EndOffset; + UINT64 Width; +} EFI_PCI_REGISTER_ACCESS_DATA; + + +// +// ACPI resource descriptor +// +typedef struct { + UINT64 ResType; + UINT64 GenFlag; + UINT64 SpecificFlag; + UINT64 AddrSpaceGranularity; + UINT64 AddrRangeMin; + UINT64 AddrRangeMax; + UINT64 AddrTranslationOffset; + UINT64 AddrLen; +} EFI_PCI_RESOUCE_DESCRIPTOR; + +/** + Checks the incompatible device list for ACPI resource update and return + the configuration. + + This function searches the incompatible device list according to request + information. If the PCI device belongs to the devices list, corresponding + configuration informtion will be returned, in the meantime return EFI_SUCCESS. + + @param PciDeviceInfo A pointer to PCI device information. + @param Configuration Returned information. + + @retval returns EFI_SUCCESS if check incompatible device ok. + Otherwise return EFI_UNSUPPORTED. +**/ +RETURN_STATUS +EFIAPI +PciResourceUpdateCheck ( + IN EFI_PCI_DEVICE_INFO *PciDeviceInfo, + OUT VOID *Configuration + ); + +/** + Checks the incompatible device list and return configuration register mask values. + + This function searches the incompatible device list according to request + information. If the PCI device belongs to the devices list, corresponding + configuration informtion will be returned, in the meantime return EFI_SUCCESS. + + @param PciDeviceInfo A pointer to EFI_PCI_DEVICE_INFO. + @param AccessType Access Type, READ or WRITE. + @param Offset The address within the PCI configuration space. + @param Configuration Returned information. + + @retval returns EFI_SUCCESS if check incompatible device ok. + Otherwise return EFI_UNSUPPORTED. +**/ +RETURN_STATUS +EFIAPI +PciRegisterUpdateCheck ( + IN EFI_PCI_DEVICE_INFO *PciDeviceInfo, + IN UINT64 AccessType, + IN UINT64 Offset, + OUT VOID *Configuration + ); + +/** + Checks the incompatible device list for access width incompatibility and + return the configuration + + This function searches the incompatible device list for access width + incompatibility according to request information. If the PCI device + belongs to the devices list, corresponding configuration informtion + will be returned, in the meantime return EFI_SUCCESS. + + @param PciDeviceInfo A pointer to PCI device information. + @param AccessType Access type, READ or WRITE. + @param Offset The address within the PCI configuration space. + @param AccessWidth Access width needs to check incompatibility. + @param Configuration Returned information. + + @retval returns EFI_SUCCESS if check incompatible device ok. + Otherwise return EFI_UNSUPPORTED. +**/ +RETURN_STATUS +EFIAPI +PciRegisterAccessCheck ( + IN EFI_PCI_DEVICE_INFO *PciDeviceInfo, + IN UINT64 AccessType, + IN UINT64 Offset, + IN UINT64 AccessWidth, + OUT VOID *Configuration + ); diff --git a/EdkModulePkg/Library/EdkPciIncompatibleDeviceSupportLib/EdkPciIncompatibleDeviceSupportLib.c b/EdkModulePkg/Library/EdkPciIncompatibleDeviceSupportLib/EdkPciIncompatibleDeviceSupportLib.c new file mode 100644 index 0000000000..f6638c1607 --- /dev/null +++ b/EdkModulePkg/Library/EdkPciIncompatibleDeviceSupportLib/EdkPciIncompatibleDeviceSupportLib.c @@ -0,0 +1,388 @@ +/** @file + The implementation of PCI incompatible device support libary. + +Copyright (c) 2007 Intel Corporation. All rights reserved.
+This software and associated documentation (if any) is furnished +under a license and may only be used or copied in accordance +with the terms of the license. Except as permitted by such +license, no part of this software or documentation may be +reproduced, stored in a retrieval system, or transmitted in any +form or by any means without the express written consent of +Intel Corporation. + +**/ + +#include "IncompatiblePciDeviceList.h" + +/** + Check whether two PCI devices matched + + @param PciDeviceInfo A pointer to EFI_PCI_DEVICE_INFO. + @param Header A pointer to EFI_PCI_DEVICE_INFO. + + @retval returns EFI_SUCCESS if two PCI device matched. +**/ +STATIC +EFI_STATUS +DeviceCheck ( + IN EFI_PCI_DEVICE_INFO *PciDeviceInfo, + IN EFI_PCI_DEVICE_INFO *Header + ) +{ + // + // See if the Header matches the parameters passed in + // + if (Header->VendorID != DEVICE_ID_NOCARE) { + if (PciDeviceInfo->VendorID != Header->VendorID) { + return EFI_UNSUPPORTED; + } + } + + if (Header->DeviceID != DEVICE_ID_NOCARE) { + if (PciDeviceInfo->DeviceID != Header->DeviceID) { + return EFI_UNSUPPORTED; + } + } + + if (Header->RevisionID != DEVICE_ID_NOCARE) { + if (PciDeviceInfo->RevisionID != Header->RevisionID) { + return EFI_UNSUPPORTED; + } + } + + if (Header->SubsystemVendorID != DEVICE_ID_NOCARE) { + if (PciDeviceInfo->SubsystemVendorID != Header->SubsystemVendorID) { + return EFI_UNSUPPORTED; + } + } + + if (Header->SubsystemID != DEVICE_ID_NOCARE) { + if (PciDeviceInfo->SubsystemID != Header->SubsystemID) { + return EFI_UNSUPPORTED; + } + } + + return EFI_SUCCESS; +} + + +/** + Check the incompatible device list for ACPI resource update and return + the configuration + + This function searches the incompatible device list according to request + information. If the PCI device belongs to the devices list, corresponding + configuration informtion will be returned, in the meantime return EFI_SUCCESS. + + @param PciDeviceInfo A pointer to PCI device information. + @param Configuration Returned information. + + @retval returns EFI_SUCCESS if check incompatible device ok. + Otherwise return EFI_UNSUPPORTED. +**/ +RETURN_STATUS +EFIAPI +PciResourceUpdateCheck ( + IN EFI_PCI_DEVICE_INFO *PciDeviceInfo, + OUT VOID *Configuration + ) +{ + UINT64 Tag; + UINT64 *ListPtr; + UINT64 *TempListPtr; + EFI_PCI_DEVICE_INFO *Header; + EFI_ACPI_ADDRESS_SPACE_DESCRIPTOR *AcpiPtr; + EFI_ACPI_ADDRESS_SPACE_DESCRIPTOR *OldAcpiPtr; + EFI_PCI_RESOUCE_DESCRIPTOR *Dsc; + EFI_ACPI_END_TAG_DESCRIPTOR *PtrEnd; + UINTN Index; + + ASSERT (PciDeviceInfo != NULL); + + // + // Initialize the return value to NULL + // + * (VOID **) Configuration = NULL; + + ListPtr = IncompatiblePciDeviceListForResource; + while (*ListPtr != LIST_END_TAG) { + + Tag = *ListPtr; + + switch (Tag) { + case DEVICE_INF_TAG: + Header = (EFI_PCI_DEVICE_INFO *) (ListPtr + 1); + ListPtr = ListPtr + 1 + sizeof (EFI_PCI_DEVICE_INFO) / sizeof (UINT64); + + if (DeviceCheck (PciDeviceInfo, Header) != EFI_SUCCESS) { + continue; + } + + // + // Matched an item, so construct the ACPI descriptor for the resource. + // + // + // Count the resource items so that to allocate space + // + for (Index = 0, TempListPtr = ListPtr; *TempListPtr == DEVICE_RES_TAG; Index++) { + TempListPtr = TempListPtr + 1 + ((sizeof (EFI_PCI_RESOUCE_DESCRIPTOR)) / sizeof (UINT64)); + } + // + // If there is at least one type of resource request, + // allocate a acpi resource node + // + if (Index == 0) { + return EFI_ABORTED; + } + + AcpiPtr = AllocateZeroPool ( + sizeof (EFI_ACPI_ADDRESS_SPACE_DESCRIPTOR) * Index + sizeof (EFI_ACPI_END_TAG_DESCRIPTOR) + ); + + OldAcpiPtr = AcpiPtr; + + // + // Fill the EFI_ACPI_ADDRESS_SPACE_DESCRIPTOR structure + // according to the EFI_PCI_RESOUCE_DESCRIPTOR structure + // + for (; *ListPtr == DEVICE_RES_TAG;) { + + Dsc = (EFI_PCI_RESOUCE_DESCRIPTOR *) (ListPtr + 1); + + AcpiPtr->Desc = ACPI_ADDRESS_SPACE_DESCRIPTOR; + AcpiPtr->Len = sizeof (EFI_ACPI_ADDRESS_SPACE_DESCRIPTOR); + AcpiPtr->ResType = (UINT8) Dsc->ResType; + AcpiPtr->GenFlag = (UINT8) Dsc->GenFlag; + AcpiPtr->SpecificFlag = (UINT8) Dsc->SpecificFlag; + AcpiPtr->AddrSpaceGranularity = Dsc->AddrSpaceGranularity;; + AcpiPtr->AddrRangeMin = Dsc->AddrRangeMin; + AcpiPtr->AddrRangeMax = Dsc->AddrRangeMax; + AcpiPtr->AddrTranslationOffset = Dsc->AddrTranslationOffset; + AcpiPtr->AddrLen = Dsc->AddrLen; + + ListPtr = ListPtr + 1 + ((sizeof (EFI_PCI_RESOUCE_DESCRIPTOR)) / sizeof (UINT64)); + AcpiPtr++; + } + // + // put the checksum + // + PtrEnd = (EFI_ACPI_END_TAG_DESCRIPTOR *) (AcpiPtr); + PtrEnd->Desc = ACPI_END_TAG_DESCRIPTOR; + PtrEnd->Checksum = 0; + + *(VOID **) Configuration = OldAcpiPtr; + + return EFI_SUCCESS; + + case DEVICE_RES_TAG: + // + // Adjust the pointer to the next PCI resource descriptor item + // + ListPtr = ListPtr + 1 + ((sizeof (EFI_PCI_RESOUCE_DESCRIPTOR)) / sizeof (UINT64)); + break; + + default: + return EFI_UNSUPPORTED; + } + } + + return EFI_UNSUPPORTED; + +} + +/** + Check the incompatible device list and return configuraton register mask values. + + This function searches the incompatible device list according to request + information. If the PCI device belongs to the devices list, corresponding + configuration informtion will be returned, in the meantime return EFI_SUCCESS. + + @param PciDeviceInfo A pointer to EFI_PCI_DEVICE_INFO. + @param AccessType Access Type, READ or WRITE. + @param Offset The address within the PCI configuration space. + @param Configuration Returned information. + + @retval returns EFI_SUCCESS if check incompatible device ok. + Otherwise return EFI_UNSUPPORTED. +**/ +RETURN_STATUS +EFIAPI +PciRegisterUpdateCheck ( + IN EFI_PCI_DEVICE_INFO *PciDeviceInfo, + IN UINT64 AccessType, + IN UINT64 Offset, + OUT VOID *Configuration + ) +{ + EFI_PCI_DEVICE_INFO *Header; + UINT64 Tag; + UINT64 *ListPtr; + EFI_PCI_REGISTER_VALUE_DATA *RegisterPtr; + EFI_PCI_REGISTER_VALUE_DATA *Dsc; + + ASSERT (PciDeviceInfo != NULL); + + ListPtr = IncompatiblePciDeviceListForRegister; + + // + // Initialize the return value to NULL + // + * (VOID **) Configuration = NULL; + + while (*ListPtr != LIST_END_TAG) { + + Tag = *ListPtr; + + switch (Tag) { + case DEVICE_INF_TAG: + Header = (EFI_PCI_DEVICE_INFO *) (ListPtr + 1); + ListPtr = ListPtr + 1 + sizeof (EFI_PCI_DEVICE_INFO) / sizeof (UINT64); + + // + // Check whether the PCI device matches the device in the incompatible devices list? + // If not, ship next + // + if (DeviceCheck (PciDeviceInfo, Header) != EFI_SUCCESS) { + continue; + } + + // + // Matched an item, check whether access matches? + // + for (; *ListPtr == DEVICE_RES_TAG;) { + ListPtr ++; + if (((EFI_PCI_REGISTER_VALUE_DESCRIPTOR *)ListPtr)->Offset == (Offset & 0xfc)) { + if (((EFI_PCI_REGISTER_VALUE_DESCRIPTOR *)ListPtr)->AccessType == AccessType) { + + Dsc = (EFI_PCI_REGISTER_VALUE_DATA *) (ListPtr + 2); + RegisterPtr = AllocateZeroPool (sizeof (EFI_PCI_REGISTER_VALUE_DATA)); + + RegisterPtr->AndValue = Dsc->AndValue; + RegisterPtr->OrValue = Dsc->OrValue; + + *(VOID **) Configuration = RegisterPtr; + + return EFI_SUCCESS; + } + } + ListPtr += sizeof (EFI_PCI_REGISTER_VALUE_DESCRIPTOR) / (sizeof (UINT64)); + } + return EFI_UNSUPPORTED; + + case DEVICE_RES_TAG: + // + // Adjust the pointer to the next item + // + ListPtr = ListPtr + 1 + ((sizeof (EFI_PCI_REGISTER_VALUE_DESCRIPTOR)) / sizeof (UINT64)); + break; + + default: + return EFI_UNSUPPORTED; + } + } + + return EFI_UNSUPPORTED; +} + +/** + Check the incompatible device list for access width incompatibility and + return the configuration + + This function searches the incompatible device list for access width + incompatibility according to request information. If the PCI device + belongs to the devices list, corresponding configuration informtion + will be returned, in the meantime return EFI_SUCCESS. + + @param PciDeviceInfo A pointer to PCI device information. + @param AccessType Access type, READ or WRITE. + @param Offset The address within the PCI configuration space. + @param AccessWidth Access width needs to check incompatibility. + @param Configuration Returned information. + + @retval returns EFI_SUCCESS if check incompatible device ok. + Otherwise return EFI_UNSUPPORTED. +**/ +RETURN_STATUS +EFIAPI +PciRegisterAccessCheck ( + IN EFI_PCI_DEVICE_INFO *PciDeviceInfo, + IN UINT64 AccessType, + IN UINT64 Offset, + IN UINT64 AccessWidth, + OUT VOID *Configuration + ) +{ + EFI_PCI_DEVICE_INFO *Header; + UINT64 Tag; + UINT64 *ListPtr; + EFI_PCI_REGISTER_ACCESS_DATA *RegisterPtr; + EFI_PCI_REGISTER_ACCESS_DATA *Dsc; + + ASSERT (PciDeviceInfo != NULL); + + ListPtr = DeviceListForAccessWidth; + + // + // Initialize the return value to NULL + // + * (VOID **) Configuration = NULL; + + while (*ListPtr != LIST_END_TAG) { + + Tag = *ListPtr; + + switch (Tag) { + case DEVICE_INF_TAG: + Header = (EFI_PCI_DEVICE_INFO *) (ListPtr + 1); + ListPtr = ListPtr + 1 + sizeof (EFI_PCI_DEVICE_INFO) / sizeof (UINT64); + + // + // Check whether the PCI device matches the device in the incompatible devices list? + // If not, ship next + // + if (DeviceCheck (PciDeviceInfo, Header) != EFI_SUCCESS) { + continue; + } + + // + // Matched an item, check whether access matches? + // + for (; *ListPtr == DEVICE_RES_TAG;) { + ListPtr ++; + if (((EFI_PCI_REGISTER_ACCESS_DESCRIPTOR *) ListPtr)->AccessType == AccessType && + ((EFI_PCI_REGISTER_ACCESS_DESCRIPTOR *) ListPtr)->AccessWidth == AccessWidth ) { + + Dsc = (EFI_PCI_REGISTER_ACCESS_DATA *) (ListPtr + 2); + + if((Dsc->StartOffset <= Offset) && (Dsc->EndOffset > Offset)) { + + RegisterPtr = AllocateZeroPool (sizeof (EFI_PCI_REGISTER_ACCESS_DATA)); + + RegisterPtr->StartOffset = Dsc->StartOffset; + RegisterPtr->EndOffset = Dsc->EndOffset; + RegisterPtr->Width = Dsc->Width; + + *(VOID **) Configuration = RegisterPtr; + + return EFI_SUCCESS; + } + } + ListPtr += sizeof (EFI_PCI_REGISTER_ACCESS_DESCRIPTOR) / (sizeof (UINT64)); + } + return EFI_UNSUPPORTED; + + case DEVICE_RES_TAG: + // + // Adjust the pointer to the next item + // + ListPtr = ListPtr + 1 + ((sizeof (EFI_PCI_REGISTER_ACCESS_DESCRIPTOR)) / sizeof (UINT64)); + break; + + default: + return EFI_UNSUPPORTED; + } + } + + return EFI_UNSUPPORTED; +} + diff --git a/EdkModulePkg/Library/EdkPciIncompatibleDeviceSupportLib/EdkPciIncompatibleDeviceSupportLib.msa b/EdkModulePkg/Library/EdkPciIncompatibleDeviceSupportLib/EdkPciIncompatibleDeviceSupportLib.msa new file mode 100644 index 0000000000..e7b8529ebd --- /dev/null +++ b/EdkModulePkg/Library/EdkPciIncompatibleDeviceSupportLib/EdkPciIncompatibleDeviceSupportLib.msa @@ -0,0 +1,44 @@ + + + + EdkPciIncompatibleDeviceSuppportLib + DXE_DRIVER + 1ca1c1f9-5baf-4204-b6e5-5e24109a4e4e + 1.0 + PCI Incompatible device support Library + Check PCI incompatible devices and set necessary configuration + Copyright (c) 2007, Intel Corporation. + All rights reserved. This program and the accompanying materials + are licensed and made available under the terms and conditions of the BSD License + which accompanies this distribution. The full text of the license may be found at + http://opensource.org/licenses/bsd-license.php + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. + FRAMEWORK_BUILD_PACKAGING_SPECIFICATION 0x00000052 + + + IA32 X64 IPF EBC + false + EdkPciIncompatibleDeviceSupportLib + + + + PciIncompatibleDeviceSupportLib + + + MemoryAllocationLib + + + + EdkPciIncompatibleDeviceSupportLib.c + IncompatiblePciDeviceList.h + + + + + + + EFI_SPECIFICATION_VERSION 0x00020000 + EDK_RELEASE_VERSION 0x00020000 + + \ No newline at end of file diff --git a/EdkModulePkg/Library/EdkPciIncompatibleDeviceSupportLib/IncompatiblePciDeviceList.h b/EdkModulePkg/Library/EdkPciIncompatibleDeviceSupportLib/IncompatiblePciDeviceList.h new file mode 100644 index 0000000000..a00c614c75 --- /dev/null +++ b/EdkModulePkg/Library/EdkPciIncompatibleDeviceSupportLib/IncompatiblePciDeviceList.h @@ -0,0 +1,209 @@ +/** @file + The incompatible PCI device list + +Copyright (c) 2007 Intel Corporation. All rights reserved.
+This software and associated documentation (if any) is furnished +under a license and may only be used or copied in accordance +with the terms of the license. Except as permitted by such +license, no part of this software or documentation may be +reproduced, stored in a retrieval system, or transmitted in any +form or by any means without the express written consent of +Intel Corporation. + +**/ + +#ifndef _EFI_INCOMPATIBLE_PCI_DEVICE_LIST_H +#define _EFI_INCOMPATIBLE_PCI_DEVICE_LIST_H + +#include +#include + + +#define PCI_DEVICE_ID(VendorId, DeviceId, Revision, SubVendorId, SubDeviceId) \ + VendorId, DeviceId, Revision, SubVendorId, SubDeviceId + +#define PCI_BAR_TYPE_IO ACPI_ADDRESS_SPACE_TYPE_IO +#define PCI_BAR_TYPE_MEM ACPI_ADDRESS_SPACE_TYPE_MEM + +#define DEVICE_INF_TAG 0xFFF2 +#define DEVICE_RES_TAG 0xFFF1 +#define LIST_END_TAG 0x0000 + +// +// descriptor for access width of incompatible PCI device +// +typedef struct { + UINT64 AccessType; + UINT64 AccessWidth; + EFI_PCI_REGISTER_ACCESS_DATA PciRegisterAccessData; +} EFI_PCI_REGISTER_ACCESS_DESCRIPTOR; + +// +// descriptor for register value of incompatible PCI device +// +typedef struct { + UINT64 AccessType; + UINT64 Offset; + EFI_PCI_REGISTER_VALUE_DATA PciRegisterValueData; +} EFI_PCI_REGISTER_VALUE_DESCRIPTOR; + + +// +// the incompatible PCI devices list for ACPI resource +// +GLOBAL_REMOVE_IF_UNREFERENCED UINT64 IncompatiblePciDeviceListForResource[] = { + // + // DEVICE_INF_TAG, + // PCI_DEVICE_ID (VendorID, DeviceID, Revision, SubVendorId, SubDeviceId), + // DEVICE_RES_TAG, + // ResType, GFlag , SFlag, Granularity, RangeMin, + // RangeMax, Offset, AddrLen + // + // + // Device Adaptec 9004 + // + DEVICE_INF_TAG, + PCI_DEVICE_ID(0x9004, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE), + DEVICE_RES_TAG, + PCI_BAR_TYPE_IO, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_BAR_EVEN_ALIGN, + PCI_BAR_ALL, + PCI_BAR_NOCHANGE, + // + // Device Adaptec 9005 + // + DEVICE_INF_TAG, + PCI_DEVICE_ID(0x9005, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE), + DEVICE_RES_TAG, + PCI_BAR_TYPE_IO, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_BAR_EVEN_ALIGN, + PCI_BAR_ALL, + PCI_BAR_NOCHANGE, + // + // Device QLogic 1007 + // + DEVICE_INF_TAG, + PCI_DEVICE_ID(0x1077, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE), + DEVICE_RES_TAG, + PCI_BAR_TYPE_IO, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_BAR_EVEN_ALIGN, + PCI_BAR_ALL, + PCI_BAR_NOCHANGE, + // + // Device Agilent 103C + // + DEVICE_INF_TAG, + PCI_DEVICE_ID(0x103C, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE), + DEVICE_RES_TAG, + PCI_BAR_TYPE_IO, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_BAR_EVEN_ALIGN, + PCI_BAR_ALL, + PCI_BAR_NOCHANGE, + // + // Device Agilent 15BC + // + DEVICE_INF_TAG, + PCI_DEVICE_ID(0x15BC, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE), + DEVICE_RES_TAG, + PCI_BAR_TYPE_IO, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_ACPI_UNUSED, + PCI_BAR_EVEN_ALIGN, + PCI_BAR_ALL, + PCI_BAR_NOCHANGE, + // + // The end of the list + // + LIST_END_TAG +}; + +// +// the incompatible PCI devices list for the values of configuration registers +// +GLOBAL_REMOVE_IF_UNREFERENCED UINT64 IncompatiblePciDeviceListForRegister[] = { + // + // DEVICE_INF_TAG, + // PCI_DEVICE_ID (VendorID, DeviceID, Revision, SubVendorId, SubDeviceId), + // PCI_RES_TAG, + // PCI_ACCESS_TYPE, PCI_CONFIG_ADDRESS, + // AND_VALUE, OR_VALUE + + // + // Device Lava 0x1407, DeviceId 0x0110 + // + DEVICE_INF_TAG, + PCI_DEVICE_ID(0x1407, 0x0110, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE), + DEVICE_RES_TAG, + PCI_REGISTER_READ, + PCI_CAPBILITY_POINTER_OFFSET, + 0xffffff00, + VALUE_NOCARE, + + // + // Device Lava 0x1407, DeviceId 0x0111 + // + DEVICE_INF_TAG, + PCI_DEVICE_ID(0x1407, 0x0111, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE), + DEVICE_RES_TAG, + PCI_REGISTER_READ, + PCI_CAPBILITY_POINTER_OFFSET, + 0xffffff00, + VALUE_NOCARE, + + // + // The end of the list + // + LIST_END_TAG +}; + +// +// the incompatible PCI devices list for the access width of configuration registers +// +GLOBAL_REMOVE_IF_UNREFERENCED UINT64 DeviceListForAccessWidth[] = { + // + // DEVICE_INF_TAG, + // PCI_DEVICE_ID (VendorID, DeviceID, Revision, SubVendorId, SubDeviceId), + // DEVICE_RES_TAG, + // PCI_ACCESS_TYPE, PCI_ACCESS_WIDTH, + // START_ADDRESS, END_ADDRESS, + // ACTUAL_PCI_ACCESS_WIDTH, + // + + // + // Sample Device + // + //DEVICE_INF_TAG, + //PCI_DEVICE_ID(0xXXXX, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE, DEVICE_ID_NOCARE), + //DEVICE_RES_TAG, + //PCI_REGISTER_READ, + //EfiPciWidthUint8, + //0, + //0xFF, + //EfiPciWidthUint32, + // + + // + // The end of the list + // + LIST_END_TAG +}; + +#endif -- 2.39.2