From a56ff9087bc3af55ed47d7bb78bb42c8faa24aa5 Mon Sep 17 00:00:00 2001 From: Felix Held Date: Thu, 16 Nov 2023 14:20:40 +0100 Subject: sb/intel/bd82x6x: assign PCH HDA controller ops in chipset devicetree Since the HD audio controller in the PCH are always on the same device functions, the device operations can be statically assigned in the devicetree and there's no need to bind the host bridge device operations to the PCI device during runtime via a list of PCI IDs. TEST=Lenovo X220 still boots to Linux and audio still works Signed-off-by: Felix Held Tested-by: Patrick Rudolph Change-Id: I9bbbe9f4490dc6fb21174d63d1c8906d69ea3ee0 Reviewed-on: https://review.coreboot.org/c/coreboot/+/79118 Reviewed-by: Patrick Rudolph Tested-by: build bot (Jenkins) --- src/southbridge/intel/bd82x6x/azalia.c | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) (limited to 'src/southbridge') diff --git a/src/southbridge/intel/bd82x6x/azalia.c b/src/southbridge/intel/bd82x6x/azalia.c index 97884c682f..ddaa8a1bb0 100644 --- a/src/southbridge/intel/bd82x6x/azalia.c +++ b/src/southbridge/intel/bd82x6x/azalia.c @@ -122,7 +122,7 @@ static const char *azalia_acpi_name(const struct device *dev) return "HDEF"; } -static struct device_operations azalia_ops = { +struct device_operations bd82x6x_azalia_ops = { .read_resources = pci_dev_read_resources, .set_resources = pci_dev_set_resources, .enable_resources = pci_dev_enable_resources, @@ -130,11 +130,3 @@ static struct device_operations azalia_ops = { .ops_pci = &pci_dev_ops_pci, .acpi_name = azalia_acpi_name, }; - -static const unsigned short pci_device_ids[] = { 0x1c20, 0x1e20, 0 }; - -static const struct pci_driver pch_azalia __pci_driver = { - .ops = &azalia_ops, - .vendor = PCI_VID_INTEL, - .devices = pci_device_ids, -}; -- cgit v1.2.3