sb/intel/bd82x6x: assign PCH XHCI controller ops in chipset devicetree

Since the XHCI controller in the PCH is always on the same device
function, the device operations can be statically assigned in the
devicetree and there's no need to bind the XHCI device operations to the
PCI device during runtime via a list of PCI IDs.

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I8685bec734415346a53330c9bd1aa82986995f1a
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79170
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Rudolph <patrick.rudolph@9elements.com>
This commit is contained in:
Felix Held 2023-11-18 18:03:40 +01:00
parent 898757fc44
commit afebab1ebe
2 changed files with 2 additions and 10 deletions

View File

@ -19,7 +19,7 @@ chip northbridge/intel/sandybridge
device pci 06.0 alias peg60 off end # PEG60 device pci 06.0 alias peg60 off end # PEG60
chip southbridge/intel/bd82x6x # Intel Series 6/7 PCH chip southbridge/intel/bd82x6x # Intel Series 6/7 PCH
device pci 14.0 alias xhci off end # USB 3.0 Controller (only on 7 series) device pci 14.0 alias xhci off ops bd82x6x_usb_xhci_ops end # XHCI Controller only on 7 series
device pci 16.0 alias mei1 on end # Management Engine Interface 1 device pci 16.0 alias mei1 on end # Management Engine Interface 1
device pci 16.1 alias mei2 off end # Management Engine Interface 2 device pci 16.1 alias mei2 off end # Management Engine Interface 2
device pci 16.2 alias me_ide_r off end # Management Engine IDE-R device pci 16.2 alias me_ide_r off end # Management Engine IDE-R

View File

@ -41,7 +41,7 @@ static const char *xhci_acpi_name(const struct device *dev)
return "XHC"; return "XHC";
} }
static struct device_operations usb_xhci_ops = { struct device_operations bd82x6x_usb_xhci_ops = {
.read_resources = pci_dev_read_resources, .read_resources = pci_dev_read_resources,
.set_resources = pci_dev_set_resources, .set_resources = pci_dev_set_resources,
.enable_resources = pci_dev_enable_resources, .enable_resources = pci_dev_enable_resources,
@ -49,11 +49,3 @@ static struct device_operations usb_xhci_ops = {
.ops_pci = &pci_dev_ops_pci, .ops_pci = &pci_dev_ops_pci,
.acpi_name = xhci_acpi_name, .acpi_name = xhci_acpi_name,
}; };
static const unsigned short pci_device_ids[] = { 0x1e31, 0 };
static const struct pci_driver pch_usb_xhci __pci_driver = {
.ops = &usb_xhci_ops,
.vendor = PCI_VID_INTEL,
.devices = pci_device_ids,
};