summaryrefslogtreecommitdiff
path: root/src/drivers/usb
diff options
context:
space:
mode:
authorFelix Held <felix-coreboot@felixheld.de>2023-06-07 20:50:08 +0200
committerFelix Held <felix-coreboot@felixheld.de>2023-06-09 00:10:00 +0000
commited6c99990439b51a7b3b03dd5d4d122b69ddbc58 (patch)
tree6b2a6db96a4e35c4dad4613859934867ebd32d9f /src/drivers/usb
parent8880baf6bc55a16485f523113e72f119cc9da8c5 (diff)
soc/amd: add ops xhci_pci_ops to XHCI controllers in devicetree
Instead of adding the new PCI IDs of the XHCI controllers in every new chip generation to the pci_xhci driver, bind the driver to the internal PCI devices of the XHCI controllers via the device ops statement in the chipset devicetree. The PCI device function of the XHCI2 controller in Mendocino can be either a dummy device or the XHCI controller, so the device ops are attached to that device in the mainboard devicetree instead. The Glinda code is right now just a copy of the Mendocino code, so it'll change in the future, but for consistency the equivalent changes to those in Mendocino are applied there too. Since the device ops are now attached to the devices via the static devicetree entry, also remove both the xhci_pci_driver struct and the amd_pci_device_ids array from drivers/usb/pci_xhci/pci_xhci.c. TEST=SSDT entries for the XHCI controllers are still generated on Mandolin. Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Change-Id: I9c455002c6d2aac576fe24eee0c31744b4507bb0 Reviewed-on: https://review.coreboot.org/c/coreboot/+/75713 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Jon Murphy <jpmurphy@google.com> Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com> Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
Diffstat (limited to 'src/drivers/usb')
-rw-r--r--src/drivers/usb/pci_xhci/pci_xhci.c17
1 files changed, 0 insertions, 17 deletions
diff --git a/src/drivers/usb/pci_xhci/pci_xhci.c b/src/drivers/usb/pci_xhci/pci_xhci.c
index ca51278e39..6d022d1455 100644
--- a/src/drivers/usb/pci_xhci/pci_xhci.c
+++ b/src/drivers/usb/pci_xhci/pci_xhci.c
@@ -251,20 +251,3 @@ const struct device_operations xhci_pci_ops = {
.acpi_fill_ssdt = xhci_fill_ssdt,
.acpi_name = xhci_acpi_name,
};
-
-static const unsigned short amd_pci_device_ids[] = {
- PCI_DID_AMD_FAM17H_MODEL18H_XHCI0,
- PCI_DID_AMD_FAM17H_MODEL18H_XHCI1,
- PCI_DID_AMD_FAM17H_MODEL20H_XHCI0,
- PCI_DID_AMD_FAM17H_MODEL60H_XHCI,
- PCI_DID_AMD_FAM17H_MODELA0H_XHCI0,
- PCI_DID_AMD_FAM17H_MODELA0H_XHCI1,
- PCI_DID_AMD_FAM17H_MODELA0H_XHCI2,
- 0
-};
-
-static const struct pci_driver xhci_pci_driver __pci_driver = {
- .ops = &xhci_pci_ops,
- .vendor = PCI_VID_AMD,
- .devices = amd_pci_device_ids,
-};